CS50 Pset 2：可读性

Question

我在通过printf作为整数返回字符串时遇到问题。这是我一直遇到的错误，但是如果我只是％i到％s，则它会编译并打印文本。我需要在字符串中打印字母的数量，而不是实际的文本本身。

可读性。C：20：20：错误：格式指定类型'int'，但该参数具有type'string'（aka'char *'）[-werror，-wormat] printf（“％i \ n”，文本）; ~~ ^~~~ ％s

我的count_letters函数似乎可以正常工作，但我敢肯定我在我缺少的地方有一个错误。代码在这里：

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

int count_letters(string text);

int main(void)
{
    // provides text: statement
    printf("Text:");

    // gets string from user
    string text = get_string(" ");

    // should count amount of letters and skip spaces and punctutation
    int count_letters(string text);

    // prints number of characters in int form
    printf("%i\n", text);
}
int count_letters(string text)
{
    int letters = 0;
    int i;

    for (i = 0; letters < strlen(text); i++)
    {
        if isalpha(text[i])
        {
            letters++;
        }
    }
    return letters;
}

Answer 1

我不知道CS50标头来自哪里，但我认为这是一门课程。
我可以看到两个问题：

1. int count_letters（字符串text）; in main：

nery：

// should count amount of letters and skip spaces and punctutation
    int count_letters(string text);

you redeclare函数而不是使用它并获取其返回值，请使用int result = count_letters（ text）;而不是。

2. printf本身

在这里：

// prints number of characters in int form
    printf("%i\n", text);

您通过text是字符串（类型char*），但是％i（或％d）需要一个整数，您可以（并且应该）使用上一个函数的返回值在这种情况下：因此，printf（“％i \ n”，结果）;（如果您在1中建议进行修改）。

否则在标准C中（请记住，我保持简单，并尝试做类似的代码的事情，因此缺少错误检查）：

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

int count_letters(char* text);

int main(void)
{
    // provides text: statement
    printf("Text:");

    // gets string from user with size 14 (+1 for \0)
    char text[15];
    scanf("%14s", text); 

    // should count amount of letters and skip spaces and punctutation
    int result=count_letters(text);

    // prints number of characters in int form
    printf("%i\n", result);
}

int count_letters(char* text)
{
    int letters = 0;
    int i;

    for (i = 0; letters < strlen(text); i++)
    {
        if isalpha(text[i])
        {
            letters++;
        }
    }
    return letters;
}