Zasada jednej odpowiedzialności jak działa?

Question

Chciałbym spytać czy dobrze rozumiem te zasadę podając dwa przykłady dotyczące wyświetlania oraz sterowania Menu programu. Która praktyka jest lepsza? Uruchomienie funkcji main i program przeskakuje z funkcji do funkcji (przykład 1) czy pisanie całej logiki i co ma się dziać po danym kroku, w funkcji main (przykład 2)? Mam nadzieje że zrozumiale wytłumaczyłem o co mi chodzi. Czy na tym polega zasada jednej odpowiedzialności jak na przykładzie 2, aby dana funkcja zajmowała się tylko jedną rzeczą i nie odwoływała do kolejnej?

public class App {
       public static void main(String[] args) {
            Menu menu = new Menu();
            menu.showMenu();
       }
}

public class Menu {

    public void showMenu() {
        System.out.println("1: Start quizu");
        System.out.println("2: Zakończ program");
        chooseMenuOption();
    }

    private void chooseMenuOption() {

        Scanner scanner = new Scanner(System.in);
        int choice = scanner.nextInt();

        switch(choice) {
            case 1:
                System.out.println("Z ilu pytań ma składać się quiz?");

                Quiz quiz = new Quiz();
                quiz.prepareQuestionList();
                break;
    }
}

public class App {

      public static void main(String[] args) {
          Menu menu = new Menu();
          menu.showMenu();
          int chooseMenuOption = menu.chooseMenuOption();
          if(chooseMenuOption==1) {
               Quiz quiz = new Quiz();
               quiz.prepareQuestionList();
          }
      }
}

public class Menu {

    public void showMenu() {
        System.out.println("1: Start quizu");
        System.out.println("2: Zakończ program");
    }

        public int chooseMenuOption() {

        Scanner scanner = new Scanner(System.in);
        int choice = scanner.nextInt();
        if (choice==1)
        return 1;
}

 

Answer 1

Zdecydowanie opcja 1. jest lepsza - ponieważ masz większą izolacje fragmentów kodu. Jeśli coś się zepsuje to masz mniejsze fragmenty kodu do analizy i szybciej wyłapiesz błędy. Z drugiej strony możesz też łatwiej coś rozbudowywać oraz używać jednej funkcjonalności w kilku miejscach.

Comments

Dziękuję, myślałem że to opcja druga będzie lepsza ze względu na lepszą czytelność kodu ale stosowałem te pierwszą bo jest wygodniejsza

---

@pawi125, nie mogę się z tym zgodzić. Moim zdaniem metoda druga jest o wiele lepsza przy bardziej rozbudowanych programach, zapewnia większą czytelność i jest łatwiejsza do rozbudowy struktury w dalszej przyszłości.

---

Dlaczego tak uwazasz?

To jest strasznie maly przyklad i za wiele na nim nie widac, ale mio wszytsko juz tutaj metoda main wykonuje za duzo operacji.

---

oh, chodziło mi oczywiście o metodę pierwszą (facepalm).

pomyliłem kody, w pełni się z Tobą zgadzam.