0

我有一些想要更新的数组。问题是当数据添加到数组中时，一些变量会更新为 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，我还有什么可能犯的错误？

4

2 回答 2

3

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

几个问题：

  1. fflush仅定义为处理输出流，而不是输入流；的行为fflush(stdin)是未定义的（它会做一些事情，但可能不是你想要的）。如果您需要从输入流中清除垃圾，则需要使用getchar()orfgets()或类似方法来使用它，直到看到换行符或其他指示您已清除垃圾的指示符。%fand%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]", 名称);
   ...
   scanf("%f", 价格);
   ...
   scanf("%f", 利润);
   ...
   scanf("%d", 数量);
 }
    这将被称为
    addStockItem(itemName[totalItem],
              &item_Price_Profit[totalItem][0],
              &item_Price_Profit[i][1],
              &itemQuantity[totalItem][0]);

  4. 你的数据结构对我来说真的很奇怪，但这很可能是因为我看不到你的整个程序。

于 2012-09-28T14:27:25.943 回答
2

这：

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

不可能是对的。这是传递一个转换为指针的字符，其中scanf()需要一个指向字符的指针。你可能的意思是：

scanf(" %[^\n]s", itemName[totalItem]);
于 2012-09-28T14:05:46.203 回答