Jak można poprawić ten kod ?

Pytania dotyczące zadania

Pory Roku na Terze

Składnia Java, Poziom 4, Lekcja 4

Rozwiązane

Jak poprawić kod żeby był bardziej schludny ?

Napisz metodę sprawdzPoreRoku. Bazując na numerze danego miesiąca, metoda powinna określać i wyświetlać porę roku (zima, wiosna, lato, jesień).

Przykład dla liczby 2:
zima

Przykład dla liczby 5:
wiosna

Wskazówka: liczby 12, 1 i 2 są miesiącami zimowymi; 3, 4 i 5 wiosennymi itd.

Wymagania:

Program powinien wyświetlać tekst na ekranie.
Metoda main nie powinna wywoływać System.out.println ani System.out.print.
Metoda main musi wywoływać metodę sprawdzPoreRoku.
Metoda sprawdzPoreRoku musi być statyczna i void oraz mieć tylko jeden parametr int.
Metoda sprawdzPoreRoku powinna wyświetlać na tekst na ekranie zgodnie z warunkami zadania.

package pl.codegym.task.task04.task0411; /* Pory Roku na Terze */ public class Solution { public static void main(String[] args) { sprawdzPoreRoku(12); sprawdzPoreRoku(4); sprawdzPoreRoku(7); sprawdzPoreRoku(10); } public static void sprawdzPoreRoku(int miesiac) { if ((miesiac==12)||(miesiac==1)||(miesiac==2)) { System.out.println("zima"); } else if ((miesiac==3)||(miesiac==4)||(miesiac==5)) { System.out.println("wiosna"); } else if ((miesiac==6)||(miesiac==7)||(miesiac==8)) { System.out.println("lato"); } else { System.out.println("jesień"); } } }

Komentarze (2)

Popularne
Najnowsze
Najstarsze

Musisz się zalogować, aby dodać komentarz

Guadalupe Gagnon

Poziom 37 , Tampa, United States

4 stycznia 2023, 21:30

rozwiązanie

To neaten it up consider that you don't need to check every single possible number. You could use greater than or less than instead and check for ranges instead of individual numbers. Of course, 12, 1, and 2 won't work, but we can put that at the bottom. So, shortened, it would look like this:

if (miesiac>=3 && miesiac<=5) {
   // for 3, 4, 5
} else if (miesiac>=6 && miesiac<=8) {
   // for 6, 7, 8
} else if (miesiac>=9 && miesiac==11) {
   // for 9, 10, 11
} else {
  // for 12, 1 and 2
}

Further, because "if-else if" statements will only process up to the point that the very first if statement evaluates to true, you can shorten it by not doing ranges. Instead you can just use all numbers less than:

if (miesiac == 12 || miesiac < 3) {
  // for 12, 1, 2
} else if (miesiac < 6) {
  // for 3, 4, 5
} else if(miesiac < 9) {
  // for 6, 7, 8
} else {
  // for 9, 10, 11
}

Lastly you don't need 4 different println() statements. You could use an intermediate variable and change it to the appropriate value then output that once. Setting a default value means you don't even need the last "else" statement:

String x = "jesień";
if (miesiac == 12 || miesiac < 3){
   x = "zima";
}else if (miesiac < 6) {
   x = "wiosna";
} else if(miesiac < 9) {
   x = "lato";
}

System.out.println(x);

Michał

Poziom 0 , Chojnice, Poland

Expert

5 stycznia 2023, 18:57

Thank you very much 😄