错误：- 从不兼容的指针类型 [-Wincompatible-pointer-types] 传递“get_string”的参数 1

Question

伙计们请告诉我的代码有什么问题。 我已经添加了 cs50 库和 header 文件，但似乎无法正确执行。 我是一个初学者，想知道你的建议。

代码：-

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

int main(void)
{
    string s = get_string("Input:   ");
    printf("Output: ");
    int n = strlen(s);
    for( int i = 0; i < n; i++)
    {
        printf("%c", s[i]);
    }
    printf("\n");
}

错误：-

3.c: In function 'main':
3.c:7:27: warning: passing argument 1 of 'get_string' from incompatible pointer type [-Wincompatible-pointer-types]
    7 |     string s = get_string("Input:   ");
      |                           ^~~~~~~~~~~
      |                           |
      |                           char *
In file included from 3.c:2:
C:/msys64/mingw64/x86_64-w64-mingw32/include/cs50.c:78:28: note: expected 'char **' but argument is of type 'char *'        
   78 | string get_string(va_list *args, const char *format, ...)
      |                   ~~~~~~~~~^~~~
3.c:7:16: error: too few arguments to function 'get_string'
    7 |     string s = get_string("Input:   ");
      |                ^~~~~~~~~~
In file included from 3.c:2:
C:/msys64/mingw64/x86_64-w64-mingw32/include/cs50.c:78:8: note: declared here
   78 | string get_string(va_list *args, const char *format, ...)

Answer 1

cs50 文档声明您应该包括cs50.h ，而不是cs50.c 。 https://cs50.readthedocs.io/libraries/cs50/c/#c.get_char https://cs50.readthedocs.io/libraries/cs50/c/#c.get_string

您需要在包含路径中添加 header 文件。

Answer 2

在评论和之前的答案中已经说明，您不应包含cs50.c文件，而应仅包含cs50.h文件并将cs50.c文件链接到您的项目。

这通常是正确的，除非您有非常具体的理由采取不同的做法。

但是......通常这不应该导致我们在问题中看到的错误。 这是因为cs50.c文件本身包含cs50.h ，我们应该让所有定义和声明可见。

在这个特定的案例中，我们遇到了一些 CS50 库的特定实现细节，这有点令人惊讶。 而且也没有必要...

让我们仔细看看header：

// cs50.h
string get_string(va_list *args, const char *format, ...) __attribute__((format(printf, 2, 3)));
#define get_string(...) get_string(NULL, __VA_ARGS__)

在这两行之后，我们可以按照这个问题的作者的意图使用get_string ： string s = get_string("Input: ");

并不是很明显为什么有人会认为将 function 隐藏在具有相同名称但参数不同的宏后面是一个好主意。 在大多数其他 API 中，function 的名称与宏不同。 但是没关系...

现在让我们看一下 C 文件：

// cs50.c
#include "cs50.h"
...
#undef get_string
string get_string(va_list *args, const char *format, ...)
{
...
}

如果你自己编译这个文件，一切都很好。 .c文件不需要宏，可以在定义 function 之前去掉它。

但是，如果您将其包含在您自己的文件中，而您应该在其中使用宏，则不再可能。 如果直接包含文件， undef会破坏 API。

这强调了您应该只包含标题的事实。 它们应该被包括在内，并且是相应地制作的。 正如我们所见，保存实现的.c文件不一定是这样制作的……

附带说明：这个#undef根本没有必要。 你可以简单地这样做：

// cs50.c
#include "cs50.h"
...
string (get_string)(va_list *args, const char *format, ...)
{
...
}

使用封闭()标识符get_string不再匹配宏get_string()并且不进行替换。 这甚至允许直接包含.c文件。

也许他们故意选择这种方式是为了防止包含 c 文件，但我不会打赌。