逻辑是循环和IFS的

-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语句中。目前它正在找到最高的患者和最低的献血者。我必须翻转它们。花了几个小时试图弄清楚这一点，如果有人能看到我的错误，我会非常感激。我敢肯定它的一个愚蠢的小错误，但对于我的生活，我只是无法找到它

来源

2013-10-10 Borys Ostapienko

你试过调试它吗？ –

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

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

我很困惑，为什么不是简单的代码？

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关于更好的变量名称。如果你选择好的变量名称，编程是更容易。

来源

2013-10-10 23:08:23 john

逻辑是循环和IFS的

回答

相关问题