如何在下一个位置向数组添加值

Question

我正在构建一个像数码相机一样的小型应用，并且在我的照相功能上，我想向我的数组中插入一个随机数（我成功做到了），我的问题是当我拍摄多张照片时，该值总是进入第一个位置。 我的代码：

public override void TakePhoto()
{
    Random rnd = new Random();
    int photo = rnd.Next(1, 10);
    for (int i = 0; i < 1; i++)
    {

        MemoryCard.buffer[i] = photo;
    }
}

class Program
{
    static void Main(string[] args)
    {

        DigitalCamera digitalCamera = 
            new DigitalCamera("kodak", 43, newMemoryCard, 3);

        digitalCamera.TakePhoto();
        digitalCamera.TakePhoto();
        digitalCamera.TakePhoto();

    }
}

每张照片后我如何跳到下一个位置？

Answer 1

您毫不客气地说要将下一个值放在第一位。

查看您的代码：

for (int i = 0; i < 1; i++)
{
   MemoryCard.buffer[i] = photo;
}

i总是0

要解决此问题，只需将i值保存到某个全局变量中，或将下一个索引接受为TakePhoto(...)函数中的参数TakePhoto(...) 。

例：

int curindex = 0; //GLOBAL VARIABLE 

public override void TakePhoto()
{
   Random rnd = new Random();
   int photo = rnd.Next(1, 10);
   if(curindex < MemoryCard.buffer.Length) //IF CURINDEX LESS THE TOTAL ARRAY LENGTH
   {
      MemoryCard.buffer[curindex] = photo;  //ASSIGN PHOTO IN CURRENTINDEX
      curindex ++;                          //INCEREMENT CURRENTINDEX
   }
}

Answer 2

1. 您的for循环中有一个错误，我假设您想使用buffer变量的Length来代替
2. 您调用TakePhoto速度太快，因此始终使用相同的种子创建Random ，因此始终生成相同的“随机数”。

而是将Random实例作为参数传递给方法或使用字段变量。

public override void TakePhoto(Random rnd)
{
    int photo = rnd.Next(1, 10);
    for (int i = 0; i < MemoryCard.buffer.Length; i++)
    {

        MemoryCard.buffer[i] = photo;
    }
}

现在始终使用相同的实例：

Random r = new Random();
DigitalCamera digitalCamera = new DigitalCamera("kodak", 43, newMemoryCard, 3);
digitalCamera.TakePhoto(r);
digitalCamera.TakePhoto(r);
digitalCamera.TakePhoto(r);

Answer 3

我认为对代码进行以下修改可以解决您的问题：

class DigitalCamera
{
    static int currentPhotoNumber = 0;
    private Random rnd = new Random();

    public override void TakePhoto()
    {
        int photo = rnd.Next(1, 10);
        MemoryCard.buffer[currentPhotoNumber++] = photo;
    }
}
class Program
{
    static void Main(string[] args)
    {

        DigitalCamera digitalCamera = new DigitalCamera("kodak", 43, newMemoryCard, 3);

        digitalCamera.TakePhoto();
        digitalCamera.TakePhoto();
        digitalCamera.TakePhoto();

    }
}

Answer 4

您的代码中存在逻辑错误：

 for (int i = 0; i < 1; i++)
    {

        MemoryCard.buffer[i] = photo;
    }

如果查看for循环变量，则循环从i = 0到i <1。 这基本上意味着该循环仅运行一次（对于i = 0）。

您可能需要每次增加i

 MemoryCard.buffer[MemoryCard.numberofphotos] = photo;
 MemoryCard.numberofphotos++;