使用数组确定最小值

Question

为什么这个简单的代码不起作用？ 一切似乎都很好，我没有收到任何错误（IDE：Visual Studio 2013） 有人可以帮忙吗？

int a[30];
int b;
int min = 0;

do {
    cin >> b;
    if (b != 0) {   
        int i = 0;
        a[i] = b;
        i++;
        if (min>a[i])
            min = a[i];
    }
} while (b != 0);

cout << endl << "THE MIN IS : "<<min<<endl;

Answer 1

程序具有未定义的行为，因为您使用了未初始化的数组元素。

在这段代码中sn -p

if (b != 0)
    {   
        int i = 0;
        a[i] = b;
        i++;
        if (min>a[i])
            min = a[i];

    }

您总是将 min 与未分配任何值的 a[1] 进行比较。在 if 语句的块语句中，变量 i 总是被重新创建并初始化为 0。然后它会增加，并将 min 与 a[1] 进行比较。但它只是分配了一个[0]。

事实上你不需要任何数组。代码可以这样写

bool empty = true;
int min;
int value;

while ( std::cin >> value && value != 0 )
{
   if ( empty || value < min )
   {
      empty = false;
      min = value;
   }
}

if ( !empty ) std::cout << "\nThe minimum is equal to " << min << std::endl;
else std::cout << "\nYou did not enter any number" << std::endl;

如果您需要将值存储在数组中，则代码如下所示

const size_t N = 30;
int a[N];
int value;

size_t i = 0;

while ( i < N && std::cin >> value && value != 0 ) a[i++] = value;

if ( i != 0 )
{
   int min = a[0];

   for ( size_t j = 1; j < i; j++ )
   {
      if ( a[j] < min ) min = a[j];
   }

   std::cout << "\nThe minimum is equal to " << min << std::endl;
}
else
{
    std::cout << "\nYou did not enter any number" << std::endl;
} 

考虑到在 C++ 的标头 <algorithm> 中声明了标准算法 std::min_element。