【问题标题】:Warning Conflicting Types警告冲突类型
【发布时间】:2013-09-09 21:11:46
【问题描述】:

我是 C 新手,我不知道为什么我的两个扫描函数都会出现类型冲突的错误。非常感谢改进我的代码的解决方案或任何建议。下面列出了分配方向。

你的程序应该读入一个文件的信息,该文件由 器官等待名单,以及收到的器官序列 捐款。对于收到的每个器官,您的程序应打印出 他们收到的人的姓名和器官。机关应 去找在候补名单上时间最长的人,谁是 器官的匹配。就本作业而言,匹配 当供体器官相同且供体血型相同时发生 与收件人相同。一旦找到一个器官匹配,他们 不应再次匹配。

输入文件格式

输入文件的第一行将包含一个正整数 n (n ≤ 120000),表示等待名单上的器官数量。接下来的 n 行将包含信息 每个器官大约一个。这些行中的每一行都将包含该人的 姓名、需要更换的器官、血型、日期 被添加到器官数据库以及它们被添加到 器官数据库。这些项目中的每一个都将由一个空格分隔。全部 名称将仅由字母和下划线组成,所有器官 名字将由小写字母组成,所有血型都将 前面提到的 8 个字符串之一,所有日期都将是 格式 m/d/y,其中 m、d 和 +y 表示数字月份日期和 患者被添加到器官捐赠名单的年份(为此 特定器官)。最后,时间将采用 hr:min 的形式,其中 hr(0 ≤ hr ≤ 23) 和 min(0 ≤ min ≤ 59) 表示数字小时和 患者被添加到器官捐赠名单的时间。 您可以保证没有两个器官被添加到列表中 相同的日期和时间,并且没有名称或器官名称将包含更多 超过 19 个字符。

输入文件的以下行(行号 n+2),将包含一个 单个正整数,k (k ≤ 1000) 代表个数 在某个固定的时间段内收到的器官。以下k 行将包含有关收到的器官的信息,按顺序 他们收到了。这些行中的每一行都将包含两个字符串 用空格隔开:器官名称和血型 捐赠者。这些都将遵守之前给出的规范。

输出规格

为收到的每个器官输出一行。如果数据库中存在尚未收到器官的匹配收件人,则打印出收件人的姓名,然后 他们收到的器官。如果数据库中不存在匹配项,则打印出来 以下是一行。

No match found

样本输出:

Adam_Smith kidney
Jessica_Arte liver
Shelly_Zenith liver
No match found
No match found
John_Lynch kidney

样本输入:

10
John_Lynch kidney A- 12/13/2007 14:43
Beth_Silva lung O+ 1/23/2009 13:10
Adam_Smith kidney A- 12/13/2007 9:59
Jenn_Gray liver AB+ 9/9/1999 1:23
Sylvia_Maria lung O+ 1/24/2009 0:09
Sean_Stark heart AB- 8/23/2000 9:13
Shelly_Zenith liver AB- 9/10/1999 1:13
Jessica_Arte liver AB+ 8/31/1999 23:38
Bill_Muse heart O+ 3/23/2013 3:12
Samantha_Bogedon intestine B- 6/30/2012 17:08
6
kidney A-
liver AB+
liver AB-
heart O-
intestine A+
kidney A-

代码:

#include <stdio.h>
#include <stdlib.h>

#define SIZE 20
#define BLOODTYPESIZE 4

//Structure to store month day and year
typedef struct {
    int month;
    int day;
    int year;
} dateT;
// Structure to stores hour and minute
typedef struct {
    int hour;
    int minute;
} timeT;
// structure to store name organname bloodtype dateadded time added and whether or not it was received
typedef struct {
    char name[SIZE];
    char organname[SIZE];
    char bloodtype[BLOODTYPESIZE];
    dateT dateAdded;
    timeT timeAdded;
    int received;
} organT;

int main(){
    //Declare variables
    int numberOfOrgans,x,numberOfDonors,y,match,possible;
// Create pointer to structure
organT* organ;
scanf("%d",&numberOfOrgans);
organ = (organT *)malloc(numberOfOrgans*sizeof(organT));
scanOrgans(numberOfOrgans,organ);
scanf("%d",&numberOfDonors);
organT* donors;
donors = (organT *)malloc(numberOfDonors*sizeof(organT));
scanDonors(numberOfDonors, donors);
for(y=0;y<numberOfDonors;y++){
    match=-1;
    possible=-1;
    for(x=0;x<numberOfOrgans;x++){
        if((strcmp(organ[x].organname, donors[y].organname)==0) && (strcmp(organ[x].bloodtype, donors[y].bloodtype)==0) && (organ[x].received!=1)){
           match++;
            if (match>0)
                possible=organCompare(organ, x, possible);
            else if (match==0)
                possible=x;

            }
        else continue;
    }
    if (possible>-1){
        organ[possible].received=1;
        printf("%s %s\n",organ[possible].name, organ[possible].organname);
    }
    else
        printf("No match found\n");
        free(organ);
        free(donors);
    }
}
void scanOrgans(int x,organT* organ){

    int i;
    char temp[256];

    for(i=0;i<x;i++){

        scanf("%s",&organ[i].name);
        scanf("%s",&organ[i].organname);
        scanf("%s",&organ[i].bloodtype);
        scanf("%s",&temp);
        sscanf(temp,"%d/%d/%d",&organ[i].dateAdded.month,&organ[i].dateAdded.day,&organ[i].dateAdded.year);
        scanf("%s",&temp);
        sscanf(temp,"%d:%d",&organ[i].timeAdded.hour,&organ[i].timeAdded.minute);
   }
}
void scanDonors(int x, organT* donors ){
    int i;
    for(i=0;i<x;i++){
    scanf("%s%s", &donors[i].organname, &donors[i].bloodtype);
    }
}

int organCompare(organT* organ, int x, int y){
    if (organ[x].dateAdded.year<organ[y].dateAdded.year)
        return x;
    else if(organ[x].dateAdded.year==organ[y].dateAdded.year){
        if (organ[x].dateAdded.month<organ[y].dateAdded.month)
            return x;
        else if(organ[x].dateAdded.month==organ[y].dateAdded.month){
            if (organ[x].dateAdded.day<organ[y].dateAdded.day)
            return x;
            else if (organ[x].dateAdded.day==organ[y].dateAdded.day){
                if (organ[x].timeAdded.hour<organ[y].timeAdded.hour)
                return x;
                else if (organ[x].timeAdded.hour==organ[y].timeAdded.hour){
                    if (organ[x].timeAdded.minute<organ[y].timeAdded.minute)
                    return x;
                    else return y;
                    }
                else return y;
            }
            else return y;

        }
        else return y;
        }
    else return y;

    }

【问题讨论】:

  • 你能把你的编译器给你的确切错误包括在内吗?
  • 去掉代码中不相关的部分和赋值描述,并添加实际的错误消息(及其所有详细信息)。
  • 欢迎来到 Stack Overflow。请尽快阅读About 页面。请用 C 和 C++ 做 not 标记问题,除非它真的是关于这两种语言的互通。在这种情况下,由于您正在学习 C,因此 C++ 的答案会让您感到困惑——所以不要用 C++ 标记。
  • 有时我希望人们至少可以假装这不是作业问题。
  • 另见Segmentation Error in C Program When Run。这是同一个作业问题......虽然可能不是同一个OP。

标签: c


【解决方案1】:

int main()前加上这三行:

 #include <string.h>
 void scanOrgans(int x, organT* organ);  
 void scanDonors(int x, organT* donors );
 int organCompare(organT* organ, int x, int y);

【讨论】:

  • 另外,这听起来不对:scanf("%s",&amp;organ[i].name);scanOrgan 函数中。
  • 谢谢施瓦茨,就是这样。
【解决方案2】:

当使用scanfchar * 时,您通常不需要&amp;,因为您已经有了一个指针。所以改变:

    scanf("%s",&organ[i].name);
    scanf("%s",&organ[i].organname);
    scanf("%s",&organ[i].bloodtype);

到:

    scanf("%s",organ[i].name);
    scanf("%s",organ[i].organname);
    scanf("%s",organ[i].bloodtype);

另外,这不是一个错误,但在 C 中你不应该转换 malloc 的结果,所以改变例如

donors = (organT *)malloc(numberOfDonors*sizeof(organT));

到:

donors = malloc(numberOfDonors*sizeof(organT));

【讨论】:

  • 感谢您的修复。至于选角,我是这样教的,他没有解释为什么我们应该这样做,但我还没有足够的知识与他争论。
  • 您的老师可能混淆了 C 和 C++ - 在 C++ 中,如果您使用 malloc,那么您确实需要转换结果(尽管您通常不应该使用 malloc在 C++ 中)。然而,在 C 语言中,这不是必需的,而且它也具有潜在的危险,因为它在某些情况下会掩盖一个令人讨厌的错误,否则编译器会报告一个警告。
【解决方案3】:

这不是您的问题,但请避免比较器中的结构。试试这个:

int organCompare(organT* organ, int x, int y)
{
    if (organ[x].dateAdded.year < organ[y].dateAdded.year)
        return x;
    else if (organ[x].dateAdded.year > organ[y].dateAdded.year)
        return y;
    else if (organ[x].dateAdded.month < organ[y].dateAdded.month)
        return x;
    else if (organ[x].dateAdded.month < organ[y].dateAdded.month)
        return y;
    else if (organ[x].dateAdded.day < organ[y].dateAdded.day)
        return x;
    else if (organ[x].dateAdded.day > organ[y].dateAdded.day)
        return y;
    else if (organ[x].timeAdded.hour < organ[y].timeAdded.hour)
        return x;
    else if (organ[x].timeAdded.hour > organ[y].timeAdded.hour)
        return y;
    else if (organ[x].timeAdded.minute < organ[y].timeAdded.minute)
        return x;
    else
        return y;
}

【讨论】:

  • 这看起来更清晰,更易于理解。感谢您的改进。
【解决方案4】:

编译器会报错,因为您正在使用未通过函数原型明确定义的函数。我在工作中留下了我的 C 参考,所以我找不到确切的段落,但编译器会猜测函数原型,这会导致您看到的“冲突类型”警告或错误。如果您未指定任何内容,C 会根据内存假定函数返回 int

这是我最好的猜测。如果 David Schwartz 的修复方法有效,我会非常确定这是原因。

【讨论】:

  • 感谢您解释问题所在。 Schwartz 的修复方法奏效了,但现在我知道它为什么奏效了。
猜你喜欢
  • 1970-01-01
  • 2022-11-02
  • 1970-01-01
  • 2014-08-05
  • 1970-01-01
  • 2016-01-05
  • 1970-01-01
  • 2011-12-25
  • 1970-01-01
相关资源
最近更新 更多