【问题标题】:Passing char array to a struct array within a function将 char 数组传递给函数内的结构数组
【发布时间】:2020-04-01 04:16:36
【问题描述】:

正如标题中提到的,我不知道如何将 char 数组传递给函数内的 struct 数组。

任务是编写一个小仓库系统,您可以在其中添加/删除/搜索项目。但是为了添加/删除一个项目,应该检查项目是否已经存在。

我必须使用一个结构和这个结构的数组。我目前的方法确实有问题,我尝试了几个小时来解决这个问题,但此时我什至不知道要修复什么..

重要提示:我不允许使用 stdio.h 和 string.h 以外的任何其他库,并且我不允许使用除了 scanf 输入和 printf 输出以外的任何东西。我必须将这两个字符串与 strcmp我不认为 我被允许使用 strncpy 进行比较。

也许我的整个方法都是错误的..

这是我的代码

#include <stdio.h>
#include <string.h>

#define LENGTHITEM 100
#define NUMBEROFITEMS 100

typedef struct WareHouse_ {
    char item[LENGTHITEM];
    int number;
} WareHouse;

void newItem(char*, WareHouse*, int);

int main() {

    int end = 0; int inputNumber = 0;
    char decision; char inputItem[NUMBEROFITEMS];
    WareHouse wHouse[NUMBEROFITEMS];

    printf("\nWelcome!");
    printf("\n\nYou can choose the following:");

    while (end != 1) {
        printf("\n\nNew Item (H) / Delete Item (E) | Search Item (S) | Table (A) | End (B): ");
        scanf("%c", &decision);

        switch(decision) {
            case 'h':
            case 'H': printf("\nName: ");
                      scanf("%s, inputItem);
                      printf("Number: "); 
                      scanf("%i", &inputNumber); <-- Here it says: scanf used to convert a string to an integer value (CLion)
                      newItem(inputItem, wHouse, inputNumber);
                      break;
            case 'e':
            case 'E': printf("\nTest E");
                      break;
            case 's':
            case 'S': printf("\nTest S");
                      break;
            case 'a':
            case 'A': printf("\nTest A");
                      break;
            case 'b':
            case 'B': printf("\nGood bye!");
                      end++;
                      break;
        }
    }
    return 0;
}

void newItem(char *inputItem, WareHouse *wHouse , int inputNumber) {

    for (int i = 0; i < NUMBEROFITEMS; i++) {
        if (strcmp(inputItem, wHouse[i].item) == 0) {
            printf("\nItem already exists, number increased %i", inputNumber);
            wHouse[i].number+= inputNumber;

        } else if (strcmp(inputItem, wHouse[i].item) < 0 || strcmp(inputItem, wHouse[i].item) > 0) {
            wHouse[i].number+= inputNumber;

            <-- Here I want to pass the "inputItem" into the "wHouse" array but I don't know how.. -->
        }
    }
}


Program:

    Welcome

    You can choose the following:

    New Item (H) / Delete Item (E) | Search Item (S) | Table (A) | End (B): H <-- My input

    Name and number (Name Number): fish 10 <-- My input


    New Item (H) / Delete Item (E) | Search Item (S) | Table (A) | End (B): <-- BUG IT APPEARS TWO TIMES

    New Item (H) / Delete Item (E) | Search Item (S) | Table (A) | End (B): H

    Name and number (Name Number): fish 10 <-- My input

    <-- Here it should output: "Item already exists, number increased 10" but it does nothing -->


    New Item (H) / Delete Item (E) | Search Item (S) | Table (A) | End (B): <-- BUG IT APPEARS TWO TIMES

    New Item (H) / Delete Item (E) | Search Item (S) | Table (A) | End (B): B

    Good bye

【问题讨论】:

  • 你的 else 子句没有意义,是吗?如果 strcmp 不匹配,如何确保不会覆盖现有项目?另外,为什么不单独阅读字符串和金额?更经常推荐使用 fgets 来读取字符串。另外,fflush 有什么作用?
  • 仅供参考,无论书籍、网站、教程或熟人告诉你fflush(stdin); 是清除stdin 流的标准方法,他们都错了。 See here.
  • @user0800 如前所述,我不允许使用 fgets.. 我的 else 应该说“如果项目不存在”。
  • @WhozCraig 我的教授做了。
  • 对,我错过了那个。你如何防止覆盖?

标签: c arrays struct


【解决方案1】:

要将字符串放入另一个没有strncpy() 的数组中,可以使用strcpy()

strcpy(wHouse[i].item, inputItem);

另一种方法是计算长度以通过strlen() 复制并通过memcpy() 复制。

size_t len = strlen(inputItem);
memcpy(wHouse[i].item, inputItem, len + 1); /* +1 for terminating null character */

如果这两种方式都不允许,你可以自己复制每个字符。

for (size_t p = 0; ; p++) {
    wHouse[i].item[p] = inputItem[p];
    /* break here (not via loop condition) instead of loop condition for copying terminating null character */
    if (inputItem[p] == '\0') break;
}

我在你的代码中发现了另外三个问题。

首先,使用数组wHouse,无需初始化, 所以您使用不确定的值进行计算和调用未定义的行为

可以这样初始化:

WareHouse wHouse[NUMBEROFITEMS] = {{"", 0}};

您可以仅指定第一个元素的值, 并且左边的元素会自动初始化为“零”。

其次,选择会显示两次,因为换行符是通过%c 读取的。 为避免这种情况,您可以在%c 之前添加一个空格字符,以告知scanf 忽略空格字符。

scanf(" %c", &decision);

第三,如果你像你说的那样在newItem()函数中添加“传递”, 所有元素都将被修改,因为元素已针对strcmp() 返回的所有值进行了修改。 相反,我猜您想在未找到指定项目时修改第一个空元素。

要实现这一点,你可以这样写:

void newItem(char *inputItem, WareHouse *wHouse , int inputNumber) {

    int itemFound = 0; /* set to 1 when specified item is found */
    int firstEmptyItem = -1; /* set to the index of the first empty item */

    for (int i = 0; i < NUMBEROFITEMS; i++) {
        if (strcmp(inputItem, wHouse[i].item) == 0) {
            printf("\nItem already exists, number increased %i", inputNumber);
            wHouse[i].number+= inputNumber;

            itemFound = 1; /* the item is found! */

        } else if (strcmp("", wHouse[i].item) == 0) { /* check if this item is empty */
            /* if empty and index is not written yet, write the index */
            if (firstEmptyItem < 0) firstEmptyItem = i;

        }
    }
    /* if the item is not found, and a room to add new item exists */
    if (!itemFound && firstEmptyItem >= 0) {
        int i = firstEmptyItem; /* set index (or you can use firstEmptyItem directly in the following code) */

        wHouse[i].number+= inputNumber;

        /* Here you should pass the "inputItem" into the "wHouse" array. */
    }
}

【讨论】:

  • “item i = firstEmptyItem;”是什么意思。我在那里只得到一个错误。
  • @Andrew 对不起,这是我的错误,item 应该是 int
  • 它有效。我爱你。目前,我不完全知道它的作用,但我认为这是一个我会解决的小问题。现在我必须做同样的事情来删除一个项目:D
  • “size_t”是什么意思?
  • size_t 是用于存储对象大小的无符号整数类型。
猜你喜欢
  • 2023-03-25
  • 2015-02-24
  • 1970-01-01
  • 2012-05-10
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多