返回类型时不兼容的类型

时间:2019-05-20 19:10:53

标签: c ansi

我对convertToPoint函数有问题。

int convertToPoint(int argc, char *argv[]) {
  struct point p;
  int x, y;

  p.x = atoi(argv[1]);
  p.y = atoi(argv[2]);

  return p;
}

期望返回类型为point的结构,但是会收到以下错误:

错误:返回类型“结构点”时类型不兼容,但应为“ int”    返回p;

出什么问题了?

2 个答案:

答案 0 :(得分:2)

这是一个非常简单的问题。您说要返回一个struct point,但是您的代码说该函数应返回int

int convertToPoint(
^^^
ups, shall return int

因此只需将其更改为struct point-就像这样:

#include <stdio.h>

struct point 
{
    int x;
    int y;
};

struct point convertToPoint(int argc, char *argv[]) {
    struct point p;
    p.x = atoi(argv[1]);
    p.y = atoi(argv[2]);
    return p;
}


int main(int argc, char *argv[]) {
    struct point p = convertToPoint(argc, argv);
    printf("%d %d\n", p.x, p.y);
}

也就是说-在不使用时传递argc有点奇怪。删除该函数参数或使用它来检查是否已给出足够的参数。喜欢:

    p.x = (argc > 1) ? atoi(argv[1]) : 0;
    p.y = (argc > 2) ? atoi(argv[2]) : 0;

还要注意,我删除了int x, y;,因为这些变量没有被使用。

答案 1 :(得分:0)

问题是您要告诉编译器您将返回intint convertToPoint(...)。您想说struct point convertToPoint(...)

如果您知道如何解析错误消息,则会看到此错误消息

error: incompatible types when returning type ‘struct point’ but ‘int’ was expected return p;

  1. return p;->就编译器而言,这是麻烦的语句。

  2. incompatible types when returning->您返回的是错误的内容,请检查返回的内容以及签名是什么

  3. type ‘struct point’->这就是您要返回的东西

  4. but ‘int’ was expected->这是函数签名中的值。

这是一个完整的例子

// convert.c
#include <stdio.h>
#include <stdlib.h>

struct point {
  int x;
  int y;
};


struct point convertToPoint(int argc, char *argv[]) {
  struct point p;
  int x, y;

  p.x = atoi(argv[1]);
  p.y = atoi(argv[2]);

  return p;
}

int main(int argc, char** argv) {
    struct point p = convertToPoint(argc, argv);
    printf("%d, %d", p.x, p.y);
}

证明有效

~/src ❯❯❯ gcc -ansi convert.c -o convert                                                                                                                                               ✘ 139 
~/src ❯❯❯ ./convert 1 2
1, 2%   

最后,您可以做一些重构来清理

// convert.c
#include <stdio.h>
#include <stdlib.h>

struct point {
  int x;
  int y;
};


struct point convertToPoint(char x[], char y[]) {
  struct point p;

  p.x = atoi(x);
  p.y = atoi(y);

  return p;
}

int main(int argc, char** argv) {
    //TODO: check for 2 args and print a helpful error message
    struct point p = convertToPoint(argv[0], argv[1]);
    printf("%d, %d", p.x, p.y);
}