c - 字谜项目

Question

我在尝试获取“isZero”函数来检测单词是否为字谜时遇到了问题。在 main() 中询问“isZero”是否等于 1，它只会给我“anagram”。如果我将它设置为 0，它只会给我“不是字谜”。对我来说，它没有计算任何东西，它只是打印出目前为真的任何陈述。不确定如何解决这个问题，可以使用一些指导。

 #include <stdio.h>
 #include <ctype.h>

 #define MAX 26

void intialize(char a[], char b[], int c[]);
void setLetters(char newCount[], int newNumber[]);
void checkLetters(char b[], int newNumber[]);
int isZero(int c[]);
void getstring(char a[]);
void getString(char b[]);

int main(void)
{
    char a[MAX], b[MAX];
    int c[MAX];

    intialize( a, b, c);
    getstring(a);
    getString(b);
    setLetters(a, c);
    checkLetters(b, c);

    if (isZero(c) == 1) {
        printf("anagram");
    } else
        printf("not anagram");
    return 0;
}

void intialize(char a[], char b[], int c[])
{
    int i;
    for(i = 0; i < MAX; ++i) {

        a[i] = '\0';
        b[i] = '\0';
        c[i] = 0;
    }
}
void setLetters(char newCount[], int newNumber[])
{
    int i, index = 0;
    for(i = 0; i < MAX; ++i) {
        if(isalpha(newCount[i])) {
            newCount[i] = tolower(newCount[i]);
            index = (int)(newCount[i] - 'a');
            newNumber[index] +=1;
        }
    }
}
void checkLetters(char b[], int newNumber[])
{
    int i, index;
    for(i = 0; i < MAX; ++i) {
        if(isalpha(newNumber[i])) {
            newNumber[i] = tolower(newNumber[i]);
            index = (int)(newNumber[i] - 'a');
            newNumber[index] -= 1;
        }
    }
}
int isZero(int c[])
{
    int i, j = 0;
    for(i = 0; i < MAX; ++i) {
        if(c[i] == 0)
            j = 1;
        else
            return 0;
    }
    return j;
}
void getstring(char a[])
{
    char line[MAX];

    printf("Enter a string: ");
    gets(line);
}
void getString(char b[])
{
    char line[MAX];

    printf("Enter a string: ");
    gets(line);
}

Answer 1

使用 C 语言时，您必须放慢速度并理解每行的每个部分的作用。 C 中没有足够接近 是正确 的部分。话虽这么说，你对如何解决这个问题有一个整体的想法。但是，很明显您刚刚开始使用 C(给出其他答案和评论)。

在开始编写函数之前，确定您需要函数做什么。然后尝试确定如何最好地处理该任务。如果您需要一个函数来获取字符串输入，请编写一个函数来完成。如果您发现自己必须编写一个函数来填充每个字符串 stop，那么您就违背了该函数的目的。为 a[] 编写一个函数来执行相同的操作，而为 b[] 编写另一个相同的函数 是没有意义的。您不需要一个函数来遍历所有数组，将新声明的数组设置为 zero/NULL，这就是数组初始化语法的用途。

在您期望函数起作用之前，请花时间学习如何将值传递给函数(如果需要返回 - 从中​​获取值)。当您将数组传递给函数时，会发生指针衰减。这意味着数组 a[] 在传递给函数时衰减为 *a 。您可以通过声明您的函数接受 *a 作为参数来利用这一点。虽然这对于一个简单的 1-D 数组来说并不是惊天动地的，但对于 2-D 数组及更高级别的数组来说，衰减变得更加复杂。

除了弄清楚哪些代码作为函数有意义之外，您还需要像对待语法一样精确地对待 C 中的逻辑。如果您不确定某行的任何部分，请查找它，查看您正在使用的函数的手册页，或查阅您的编译器 (Gnu/MS) 的语言引用等，以确保您知道正是您的代码所做的。从长远来看，它将为您节省时间。让 C 程序员新手头疼的第一件事是尝试略读手册或略读书籍，然后开始编写代码。学习 C 的关键是慢下来。

从不，从不，从不使用 gets()。如果您正在上课，而老师使用它给您布置了作业，请前往管理部门并要求退款。 gets() 不再是标准 C 库的一部分，因为它很容易被破坏和利用。使用 fgets、getline 或 scanf(阅读有关正确使用 scanf 格式的整个部分string 如果您选择使用它)。这是一个很好的功能，但它有很多很多的陷阱，只是等待部分了解它的使用的人。

话虽如此，您已经掌握了一种处理字谜的整体逻辑。下面我在整理代码时提供了上述要点的示例。花时间通读它并理解我为什么要做出这些改变。此外，我为输入的单词添加了快速 length 检查。如果它们的长度不同，则无需更进一步。如果您有任何疑问，请告诉我。这里有很多乐于助人的好人。

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

#define MAX 26

void setLetters(char *newCount, int *newNumber);
void checkLetters(char *newCount, int *newNumber);
int isZero (int *c);
void getstring (char *a);

int main (void)
{
    char a[MAX] = {0};  /* initialize all to zero/NULL */
    char b[MAX] = {0};
    int  c[MAX] = {0};

    getstring(a);
    getstring(b);
    printf ("\n You entered:\n\n  a: %s\n  b: %s\n\n", a, b);

    /* test lengths - if differ, not anagram */
    if (strlen(a) != strlen(b)) {
        printf (" Lenghts differ, cannot be an anagram.\n\n");
        return 1;
    }

    setLetters (a, c);      /* set key array  (c)   */
    checkLetters (b, c);    /* check key array (c)  */

    if (isZero(c))
        printf(" The words form an anagram.\n\n");
    else
        printf(" The words are not and anagram.\n\n");

    return 0;
}

void setLetters (char *newCount, int *newNumber)
{
    int i = 0;
    int index = 0;

    for (i = 0; i < MAX; ++i) {
        if (isalpha (newCount[i])) {
            newCount[i] = tolower (newCount[i]);
            index = (int)(newCount[i] - 'a');
            newNumber[index] +=1;
        }
    }
}

void checkLetters(char *newCount, int *newNumber)
{
    int i = 0;
    int index = 0;

    for (i = 0; i < MAX; ++i) {
        if (isalpha (newCount[i])) {
            newCount[i] = tolower (newCount[i]);
            index = (int)(newCount[i] - 'a');
            newNumber[index] -= 1;
        }
    }
}

int isZero (int *c)
{
    int i = 0;
    for (i = 0; i < MAX; ++i)
        if (c[i] == 1)
            return 0;

    return 1;
}

void getstring (char *a)
{
    printf ("\n Enter a string: ");
    scanf  ("%[^\n]%*c", a);
}

输出:

$ ./bin/anaproj

 Enter a string: yekcim

 Enter a string: mickey

 You entered:

  a: yekcim
  b: mickey

 The words form an anagram.

$ ./bin/anaproj

 Enter a string: yekcim

 Enter a string: mickez

 You entered:

  a: yekcim
  b: mickez

 The words are not and anagram.