运行时检查失败变量周围的堆栈已损坏

#include  main() { int num[9], i = 0, count = 0; while (i<10) { scanf("%d", &num[i]); if (num[i] % 2 == 0) { count++; } i++; } printf("we have %d double numbers\n", count); } 

运行时检查失败＃2 – 变量周围的堆栈已损坏

我该怎么办？

Answer 1

你的while循环命中i所有值从0到9（包括0和9），但尝试访问num[9]会使你超出范围。 您需要减少while循环范围：

 while (i<9) { ... } 

---

另外，你真的应该给你的main()函数一个返回类型，因为现代编译器不能容忍它丢失：

 int main() { ... return 0; } 

Answer 2

可用于访问具有N个元素的数组的索引的有效范围是[0, N - 1]或者是相同的[0，N]。

因此while语句中的条件

 while (i<10) 

必须重写像

 while (i < 9) 

错误的原因是在整个程序中使用“幻数”。 尝试使用命名常量而不是幻数，在这种情况下，很容易理解在代码的哪个部分使用了什么幻数。

程序看起来像

 #include  #define N 9 int main( void ) { int num[N]; unsigned int count = 0; unsigned int i = 0; while ( i < N ) { scanf( "%d", &num[i] ); if ( num[i] % 2 == 0 ) ++count; i++; } printf( "we have %u double numbers\n", count); } 

而不是while循环，最好使用for循环，因为变量i不在循环外使用。

例如

 #include  #define N 9 int main( void ) { int num[N]; unsigned int count = 0; for ( unsigned int i = 0; i < N; i++ ) { scanf( "%d", &num[i] ); if ( num[i] % 2 == 0 ) ++count; } printf( "we have %u double numbers\n", count); } 

声明数组索引的更正确方法是使用size_t类型。

实际上，该程序中没有使用该数组。 您可以在不使用数组的情况下计算输入的值。

考虑到根据C标准，没有参数的函数main应声明为

 int main( void )