c - C 向数组中添加数据

Question

我有几个数组想要更新。问题是当数据被添加到数组时，一些变量被更新为 0。

我添加数据的代码:

void addStockItem(){
    system(CLS_NAME);
    printf("New Item\n\n");
    printf("New Item Name   : ");
    fflush(stdin);
    scanf(" %[^\n]s",&*itemName[totalItem]);
    //itemName[totalItem][30] = strupr(itemName[totalItem]);
    fflush(stdin);
    printf("New Item Price  : ");
    scanf("%f",&item_Price_Profit[totalItem][0]);
    printf("New Item Profit : ");   
    scanf("%f",&item_Price_Profit[i][1]);
    printf("New Item Qty    : ");   
    scanf("%d",&itemQuantity[totalItem][0]);
    itemQuantity[totalItem][1]=0;
    itemQuantity[totalItem][2]=0;
    ++totalItem;
    allStocks();
}

数据，

int totalItem=13,itemQuantity[][3]={8,0,0,9,0,0,11,0,0,0,0,0,20,0,0,22,0,\
  0,16,0,0,18,0,0,9,0,0,7,0,0,5,0,0,12,0,0,0,0,0},sessionQuantity;

float item_Price_Profit[][2]={1,0.5,2,0.2,3,0.2,4,0.2,5,0.5,6,0.8,7,0.5,8,0.2,9,\
0.2,10,0.2,11,0.5,12,0.8,13,0.9};

char itemName[][30]={"STABILO PENCIL 2B","STABILO PEN 0.5",\
"STABILO ERASER","STABILO RULER","STABILO TEST PAD","STABILO BOOK","STABILO SCISSORS","STABILO SHARPENER","STABILO GLUE","STABILO CHALK","STABILO MARKER PEN","OXFORD DICTIONARY","STABILO HIGHLIGHTER"};

完整代码:http://pastebin.com/jjuCCrjz

[编辑] 在我将 itemQuantity[][3] 更改为 itemQuantity[100][3]、将 item_Price_Profit[][2] 更改为 item_Price_Profit[100][2] 并将 itemName[][30] 更改为 itemName[100] 之后，一切都按预期进行[30]。除了 scanf，还有什么可能是我的错误？

Answer 1

我无法从我的工作计算机访问 pastebin，所以我必须按照发布的内容进行操作。

几个问题:

1. fflush 仅定义为对输出流起作用，对输入流不起作用； fflush(stdin) 的行为是未定义(它会做一些事情，但可能不是你想要的)。如果您需要清除输入流中的垃圾，您需要使用 getchar() 或 fgets() 或类似方法来使用它，直到您看到换行符或一些其他表明您已清除垃圾的指标。 %f 和 %d 转换说明符将跳过任何前导空格，并且在 %[ 转换说明符之前有空格也会导致要跳过的任何前导空格。因此，完全放弃 fflush 调用。

2. scanf("%[^\n]s",&*itemName[totalItem]); - 这看起来很困惑。除非您希望输入始终有一个尾随 s 字符，否则转换说明符应该只是 %[^\n]。 itemName 前面的&* 是多余的；你只需要写

   scanf(" %[^\n]", itemName[totalItem]);

   尽管您可能应该在其中放置一个字段宽度说明符:

   scanf(" %30[^\n]", itemName[titalItem]);

   以避免缓冲区溢出。

3. 您正在访问所有数据项(totalItem、itemName、item_Price_Profit 等)作为全局变量。这通常是胃灼热的秘方。理想情况下，函数及其调用者不应通过全局变量共享状态；相反，它们应该通过参数、返回值和异常(如果支持)进行通信。更像是

    void addStockItem(char *name, float *price, float *profit, float *quantity)
    {
      ...
      scanf(" %30[^\n]", name);
      ...
      scanf("%f", price);
      ...
      scanf("%f", profit);
      ...
      scanf("%d", quantity); 
    }

   这将被称为

    addStockItem(itemName[totalItem], 
                 &item_Price_Profit[totalItem][0], 
                 &item_Price_Profit[i][1], 
                 &itemQuantity[totalItem][0]);

4. 我觉得你的数据结构很古怪，但这可能是因为我看不到你的整个程序。