2013-10-10 51 views
-1

所以我试图找到pat.txt最低的一年,最高年don.txt流量不正确:逻辑是循环和IFS的

Pat.txt

Androf O kidney 24 2012 
Blaren B kidney 35 2010 
Cosmer A kidney 35 2000 
Eralod O heart 53 2009 
Forend B kidney 31 2003 

唐.TXT

Zerk B kidney 20 2009 
Rampe A kidney 31 2005 
Darech B kidney 34 2008 
Seo A kidney 26 2010 
Yuio B kidney 26 2013 

代码如下:

struct Person { 

    string surname; 
    string BType; 
    string organ; 
    int age; 
    int year, ID, IDp; 
} Patient[50], Donor[50]; 

然后感兴趣的代码:

int Date = 5000; 
     int Datel = 1000; 
      for (i = 0; i < 6; i ++){ 
        for (i1 = 0; i1 < 6; i1++){ 

          if ((Patient[i].BType == Donor[i1].BType) && (Patient[i].organ == Donor[i1].organ)){ 

            if (Patient[i].year < Date){ 
             Date = Patient[i].year; 

            //} 
             if ((Patient[i].year == Date) && (Donor[i1].year > Datel)){ 
              Date = Patient[i].year; 
              Datel = Donor[i1].year; 
              cout << Date << " " << Datel << "\n"; 

             } 

            } 
          } 
         } 
       } 

目前我得到的最高的病人和捐赠者最低(2010年至2009年),我需要获得(2003-2013)

我觉得逻辑是不正确在一个正在做平等的if语句中。目前它正在找到最高的患者和最低的献血者。我必须翻转它们。花了几个小时试图弄清楚这一点,如果有人能看到我的错误,我会非常感激。我敢肯定它的一个愚蠢的小错误,但对于我的生活,我只是无法找到它

+0

你试过调试它吗? –

+1

请使用更好的变量名称..它使人们试图阅读你的代码能够更容易地遵循它。什么是“日期”和“日期1”?一个应该是最古老的病人的一年?为什么不称他们为“newestPatientDate”和“oldestDonerDate”? – PherricOxide

+0

转到您觉得自己最没信心的代码,提出一个关于它的问题,然后尝试回答该问题。 – Leonardo

回答

2

我很困惑,为什么不是简单的代码?

int oldestPatientDate = 5000; 
int newestDonorDate = 1000; 
for (i = 0; i < 6; i++) 
    if (oldestPatientDate > Patient[i].year) 
     oldestPatientDate = Patient[i].year; 
for (i = 0; i < 6; i++) 
    if (newestDonorDate < Donor[i].year) 
     newestDonorDate = Donor[i].year; 
cout << oldestPatientDate << " " << newestDonorDate << "\n"; 

但是,也许还有比您所描述的更多的任务。

完全同意PherricOxide关于更好的变量名称。如果你选择好的变量名称,编程是更容易