0

看来我的 fgets() 实现在这里不正确，非常感谢一些额外的眼睛来查看我所做的事情！

这是代码

int main(int argc, const char* argv[]){
    int numIntegers;
    char buffer[20];
    int intArray[10];
    //if no argument is passed in, terminate
    if (argc == 1){
            printf("no argument given, terminating..\n");
            return EXIT_FAILURE;
    }
    else{
            numIntegers = atoi(argv[1]);
            //we only want numbers greater than 0
            if (numIntegers <= 0){
                    printf("# must be greater than 0\n");
                    return EXIT_FAILURE;
            }
            else{
                    printf("Enter %d integer values to place in array: \n", numIntegers);
                    for (int i = 0; i < numIntegers; i++){
                            fgets(buffer, numIntegers, stdin);
                            intArray[i] = atoi(buffer);
                            printf("Index is = %d \n", i);
                    }
            }
    }

    //for (int i =0; i < numIntegers; i++){
    //      printf("Index[%d] = %d \n", i, intArray[i]);
    //}
}

这是输出，除了整数之外没有其他文本的行是用户输入。注意 i 的值如何重置。仅当我给出的初始参数超过 10 时才会出现此问题。无论出于何种原因，它都会将 for 循环变成无限循环。

$ ./a.out 11
Enter 11 integer values to place in array:
5
Index is = 0
2
Index is = 1
1
Index is = 2
2
Index is = 3
3
Index is = 4
4
Index is = 5
123
Index is = 6
123
Index is = 7 
123
Index is = 8
1
Index is = 9
2
Index is = 2
2
Index is = 3
3
Index is = 4
5
Index is = 5
1
Index is = 6
12
Index is = 7
4

1 回答 1

2

您正在使用

fgets(buffer, numIntegers, stdin);

第二个参数应该是缓冲区的大小——在你的例子中是 20。这至少是一个明显的问题......

下一个问题：您允许numIntegers大于 10 - 因此您将在intArray. 也需要解决...

if(numIntegers > 10) {
  printf("cannot have number greater than 10!\n");
  // abort, retry, ignore...
}

事实上 - 这是你的代码，已经解决了错误：注意使用定义的大小，BUFSIZE这样MAXNUM你就不必在多个地方改变它，如果你改变主意......

#include <stdio.h>
#define BUFSIZE 20
#define MAXNUM 10
#define EXIT_FAILURE 0

int main(int argc, const char* argv[]){
    int i, numIntegers;
    char buffer[BUFSIZE];
    int intArray[MAXNUM];
    //if no argument is passed in, terminate
    if (argc == 1){
            printf("no argument given, terminating..\n");
            return EXIT_FAILURE;
    }
    else{
            numIntegers = atoi(argv[1]);
            //we only want numbers greater than 0
            if (numIntegers <= 0 || numIntegers > MAXNUM){
                    printf("# must be greater than 0 and less than %d!\n", MAXNUM);
                    return EXIT_FAILURE;
            }
            else{
                    printf("Enter %d integer values to place in array: \n", numIntegers);
                    for (i = 0; i < numIntegers; i++){
                            fgets(buffer, BUFSIZE, stdin);
                            intArray[i] = atoi(buffer);
                            printf("Index is = %d \n", i);
                    }
            }
    }
 }

最后——你可能想知道为什么你的整数计数器似乎“重置”了？好吧 - 你intArray是堆栈上的 10 个整数块；并且当您声明循环变量时i，它会占据内存中的下一个int intArray[10];位置（就像在您进入循环之前最后一次声明变量一样） - 当您“索引”到（您所在的内存位置for）时，您碰巧到达了intArray[10]不允许访问，但你还是这样做了）。您碰巧输入了值2- 因此，i被重置为2...

如果您i在程序开始时声明（正如我所做的那样，因为我的编译器默认情况下不会“执行”C99 - 我已经那么老了！），问题会以不同的方式出现 - 或者根本不会出现。

于 2013-10-08T21:32:45.003 回答