0

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

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();

    }
}

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

4

4 回答 4

5

您明确地说将下一个值放在第一个位置。

查看您的代码：

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

i总是0

要解决此问题，只需将 i 值保存到某个全局变量中，或者将下一个索引像函数中的参数一样接受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
   }
}
于 2013-01-29T07:59:59.160 回答
2
  1. 你的 for 循环中有一个错误，我假设你想使用变量Length 的buffer
  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);
于 2013-01-29T08:02:17.823 回答
1

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

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();

    }
}
于 2013-01-29T08:04:21.743 回答
-1

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

 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++;
于 2013-01-29T08:02:06.490 回答