if-else-else si

Question

Je suis un débutant en Java et je fais quelques questions de pratique pour rafraîchir mes compétences java.

Ecrivez une méthode nommée season qui prend deux entiers comme paramètres représentant un mois et un jour et qui renvoie une chaîne indiquant la saison pour ce mois et ce jour. Supposons que les mois sont spécifiés sous la forme d'un nombre entier compris entre 1 et 12 (1 pour janvier, 2 pour février, etc.) et que le jour du mois est un nombre entre 1 et 31.

Si la date est comprise entre 12 et 12/16 et 3/15, vous devriez retourner "Winter". Si la date tombe entre 3/16 et 6/15, vous devriez retourner "Spring". Si la date est comprise entre le 16/06 et le 15/09, vous devez retourner "Summer". Et si la date tombe entre le 16/09 et le 12/15, vous devriez retourner "Fall". Mais je ne reçois pas la sortie. Mais cela me semble juste. N'importe qui peut me dire où je me suis trompé?

Answer 1

|| has lower precendence than && et vos conditions ne semblent corrects - que vous vouliez sans doute écrire quelque chose comme:

if((month == 9 && day >= 16) //September, on or after the 16th 
     || month == 10   //or October 
     || month == 11   //or November 
     || (month == 12 && day <=15)) { //or December, but before or on the 15th 
    return "Fall"; 
} 

(même commentaire pour les autres conditions)

---

Vous pouvez le rendre plus court par en utilisant un petit hack, mais la lisibilité n'est peut-être pas aussi bonne (discutable):

int mdd = month * 100 + day; //date in MDD format, for example 507 for May 7th 

if (mdd >= 916 && mdd <= 1215) { 
    return "Fall"; 
} 

Answer 2

Vous devriez essayer d'écrire quelque chose comme ceci:

if((month>9 && month<12) || (month==9 && day>=16) || (month==12 && day<=15)){ 
    return "Fall"; 
} else if 
    ... 
} 

Les préséances sont importants, aussi, mais vous devez réfléchir à ce que vous voulez vraiment atteindre, et de construire votre expression selon que.

Answer 3

if(month>=9 && month<=12 && (day==15||day==16)){ 
    return "Fall"; 
} 

Ce problème est que seuls les 15e et 16e mois sont considérés comme l'hiver.

Solution

Si cela ne tenait qu'à moi, je re-facteur if-else arbre vérifier d'abord pendant des mois, puis vérifiez les jours. Ainsi par exemple:

if(month>=9 && month<=12) 
{ 
    // Some special cases. 
    if((month == 9 && day < 15) || (month == 12 && day > 16)) 
    { 
     // It isn't fall. 
    } 
    else 
    { 
     // It is fall. 
    } 
} 

Answer 4

Cela fonctionne très bien. Vous avez eu quelques erreurs de Minow je corrige

public static String season(int month,int day){ 

    if((month>=9 && month<=12) && (day==15||day==16)) 
     return "Fall";  
    else{ 
     if ((month>=4 && month<=6) && (day==16||day==15)) 
      return "Spring";   
     else{ 
      if ((month>=6 && month<=9) && (day==16||day==15)) 
       return "Summer";  
      else 
       return"Winter"; 
     } 
    } 

}