Po użyciu wyjątku pętla uruchamia się cały czas.

Rejestracja:prawie 5 lat
Ostatnio:ponad 2 lata
Postów:7

Cześć,

Mam mały problem z kodem. Może ktoś by mi był to w stanie jakoś wyjaśnić.
Miałem problem że przy wprowadzaniu czegoś innego niż dostępne opcje aplikacja przestała działać. Postanowiłem dodać try catch i generalnie działa z jednym małym ale... jeżeli użyje "break;" to wiadomo aplikacja kończy działanie ale w momencie jak nie użyję break; to pętla uruchamia się cały czas i użytkownik nie ma już możliwości ponownego wybrania opcji.

Może ktoś pomoże mi to zrozumieć.

Kopiuj

package com.company;

import java.util.InputMismatchException;
import java.util.Scanner;


public class Main{

        public static void main(String[] args) {
        
        Scanner scanner = new Scanner(System.in);
        Party party = new Party();
        boolean exit = true;


        while (exit){
            System.out.println("Wybierz opcję:");
             System.out.println("1 - Wyświetl gośći");
             System.out.println("2 - Dodaj gości");
             System.out.println("3 - Pokaż posiłki");
             System.out.println("4 - Znajdź po numerze telefonu");
             System.out.println("5 - Usuń gościa");
             System.out.println("6 - Wyjście");


      try {
          int userChoice = scanner.nextInt();
          if (userChoice >= 7) {
          System.out.println("Wybrałeś nieistniejącą opcję!");
        } else


      switch (userChoice) {
            case (1): party.displayGuest();
            break;

            case (2): party.addGuest();
            break;

            case (3): party.showMeals();
            break;

            case (4): party.displayGuestByPhoneNumber();
            break;

            case (5): party.removeGuest();
            break;

            case (6): exit = !exit;
            break;
        }
       } catch (InputMismatchException e){
         System.out.println("Wybrano opcję w nieprawidłowym formacie!");
         break;
      }
    }
  }
}

Java

edytowany 1x, ostatnio: Riddle 2022-08-23 20:58

Productionserver

2022-08-23 16:32

Productionserver2022-08-23 16:32

Rejestracja:prawie 4 lata
Ostatnio:około 8 godzin
Postów:219

A dlaczego ma kończyć działanie, skoro nie zmieniasz wartości zmiennej exit ?
Poszukaj też słowa kluczowego default przy switch'u.

tomepaw

2022-08-23 16:36

tomepaw2022-08-23 16:36

Rejestracja:ponad 14 lat
Ostatnio:ponad rok
Lokalizacja:Lublin
Postów:218

W bloku catch(){}, jeżeli nie masz linijki break; jedyne co się dzieje to wypisanie komunikatu, po czym pętla wykonuje się od nowa. Klasa Scanner w przypadku rzucania wyjątku InputMismatchException nie pomija błędnych danych, zgodnie z dokumentacją

When a scanner throws an InputMismatchException, the scanner will not pass the token that caused the exception, so that it may be retrieved or skipped via some other method.

W kolejnym obiegu pętli następuje ponowna próba parsowania tych samych błędnych danych do wartości liczbowych, co znowu powoduje wyrzucenie wyjątku i tak w kółko. Jeżeli chcesz pominąć wadliwe dane możesz np. wywołać scanner.next() w bloku catch.

_13th_Dragon

2022-08-23 17:31

_13th_Dragon2022-08-23 17:31

Rejestracja:ponad 19 lat
Ostatnio:2 miesiące

Z jakiej racji oszukujesz sam siebie? Czyżby autosabotaż?

Kopiuj

        boolean exit = true;
        while (exit)
        {
          if(...) exit=!exit;
        }

Porównaj do:

Kopiuj

        boolean active=true; //active, more, good
        while(active)
        {
          if(...) active=false;
        }

Lub do:

Kopiuj

        boolean stop=false; //stop, bad, exit, quit
        while(!stop)
        {
          if(...) stop=true;
        }

Atsin 2022-08-23 17:43

Faktycznie to ma sens. Wprowadziłem poprawki.

Atsin

2022-08-23 17:50

Atsin2022-08-23 17:50

Rejestracja:prawie 5 lat
Ostatnio:ponad 2 lata
Postów:7

Dziękuję wszystkim za podpowiedzi i rady.
Rozwiązałem problem w trochę inny sposób i chciałbym się dowiedzieć czy to co zrobiłem ma sens. (Bo działać działa ale to nie tylko chyba o to chodzi :P)

Po wprowadzonych zmianach klasa main wygląda tak:

Kopiuj

package com.company;

import java.util.Scanner;

public class Main{

    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);
        Party party = new Party();

        boolean active = true;


        while (active){
            System.out.println("Wybierz opcję:");
            System.out.println("1 - Wyświetl gośći");
            System.out.println("2 - Dodaj gości");
            System.out.println("3 - Pokaż posiłki");
            System.out.println("4 - Znajdź po numerze telefonu");
            System.out.println("5 - Usuń gościa");
            System.out.println("6 - Wyjście");

                    switch (party.selectOption()) {
                        case (1): party.displayGuest();
                            break;

                        case (2): party.addGuest();
                            break;

                        case (3): party.showMeals();
                            break;

                        case (4): party.displayGuestByPhoneNumber();
                            break;

                        case (5): party.removeGuest();
                            break;

                        case (6): active = false;
                            break;
                    }
        }
        
    }
}

a metoda "selectOption" tak:

Kopiuj

        try {
            Integer option = Integer.valueOf(scanner.nextLine());
            if (option >= 7) {
                System.out.println("Brak opcji o takim numerze");
            }return option;
        } catch (NumberFormatException e) {
            System.out.println("Wprowadź właściwy format wyboru");
        } return 0;
    }

edytowany 1x, ostatnio: Riddle 2022-08-23 21:05

_13th_Dragon 2022-08-23 17:52

zapominasz o option <= 0

_13th_Dragon

2022-08-23 17:55

_13th_Dragon2022-08-23 17:55

Rejestracja:ponad 19 lat
Ostatnio:2 miesiące

Atsin napisał(a):

Rozwiązałem problem w trochę inny sposób i chciałbym się dowiedzieć czy to co zrobiłem ma sens.

Zapominasz o option <= 0

edytowany 1x, ostatnio: _13th_Dragon 2022-08-23 17:56

Atsin

2022-08-23 17:58

Atsin2022-08-23 17:58

Rejestracja:prawie 5 lat
Ostatnio:ponad 2 lata
Postów:7

Racja. Wprowadziłem poprawkę:

Kopiuj

if (option >= 7 || option <= 0) {
                System.out.println("[Brak opcji o takim numerze]");

A rozwiązanie żeby złapać NumberFormatException w metodzie jest akceptowalne czy nie powinno się tak robić?

_13th_Dragon 2022-08-23 18:00

Za dużo liczb magicznych skoro masz opcje 1..6 to zapisz to po ludzku if((1>option)||(option>6)) nie wspominając o tym że warto menu "zautomatyzować".

Atsin 2022-08-23 18:11

W jakim sensie „zautomatyzować”. Z góry przepraszam za głupie pytania ale jestem świeży. 😉

_13th_Dragon

2022-08-23 18:53

_13th_Dragon2022-08-23 18:53

Rejestracja:ponad 19 lat
Ostatnio:2 miesiące

Atsin napisał(a):

W jakim sensie „zautomatyzować”

Coś w ten deseń:

Kopiuj

import java.util.Scanner;
public class SMenu
{
   public static void main(String[] args)
   {
      Party party=new Party();
      MenuItem.Run("Menu główne",Party.menu,party);
   }
}

class Party
{
   public void displayGuest() { System.out.println("displayGuest\n"); }
   public void addGuest() { System.out.println("addGuest\n"); }
   public void showMeals() { System.out.println("showMeals\n"); }
   public void displayGuestBySurname() { System.out.println("displayGuestBySurname\n"); }
   public void displayGuestByPhoneNumber() { System.out.println("displayGuestByPhoneNumber\n"); }
   public void removeGuest() { System.out.println("removeGuest\n"); }

   public static MenuItem[] menu=new MenuItem[]
   {
      new MenuItem("Wyświetl gośći",(party) -> party.displayGuest()),
      new MenuItem("Dodaj gości",(party) -> party.addGuest()),
      new MenuItem("Pokaż posiłki",(party) -> party.showMeals()),
      new MenuItem
      (
         "Znajdź gościa",
         new MenuItem[]
         {
             new MenuItem("Wg nazwiska",(party) -> party.displayGuestBySurname()),
             new MenuItem("Wg telefonu",(party) -> party.displayGuestByPhoneNumber()),
             new MenuItem("Powrót"),
         }
      ),
      new MenuItem("Usuń gościa",(party) -> party.removeGuest()),
      new MenuItem("Wyjście"),
   };
}

class MenuItem
{
   public static interface MenuExecuter { public void execute(Party party); }

   private static Scanner scanner = new Scanner(System.in);
   private String name;
   private MenuExecuter executer;
   private MenuItem[] submenu;

   private MenuItem(String name,MenuExecuter executer,MenuItem[] submenu)
   {
      this.name=name;
      this.executer=executer;
      this.submenu=submenu;
   }

   public MenuItem(String name,MenuExecuter executer) { this(name,executer,null); }
   public MenuItem(String name,MenuItem[] submenu) { this(name,null,submenu); }
   public MenuItem(String name) { this(name,null,null); }

   public static void Run(String name,MenuItem[] submenu,Party party)
   {
      while(true)
      {
         System.out.println("-- "+name+":");
         for(int i=0;i<submenu.length;++i) System.out.println((i+1)+". "+submenu[i].name);
         System.out.print("Wybierz opcję: ");
         try
         {
            int pos=Integer.valueOf(scanner.nextLine());
            if((1<=pos)&&(pos<=submenu.length))
            {
               MenuItem sub=submenu[pos-1];
               if(sub.submenu!=null)
               {
                  String subname=name+" -> "+sub.name;
                  Run(name,sub.submenu,party);
               }
               else if(sub.executer!=null)
               {
                  sub.executer.execute(party);
               }
               else break;
            }
            else System.out.println("Nie ma takiej opcji");
         }
         catch(Exception e)
         {
            System.out.println("Wprowadź właściwy format opcji");
         }
         System.out.println();
     }
   }
}

Uwaga pisano na kolanie.

Zauważ że rozwinięcie na kolejne submenu - bezproblemowo -> wystarczy dodać tabelkę.
Jakieś inne menu w tej samej aplikacji - bezproblemowo -> wystarczy zrobić z MenuItem generic'a.
Przejście na GUI - bezproblemowo -> wystarczy napisać funkcję tworzącą menu wg tabelki i spinającą słownikiem menu z MenuItem

Zobacz pozostałe 3 komentarze

_13th_Dragon 2022-08-23 19:46

No to się przypomni i wróci aby obejrzeć to po lambdach.

Atsin 2022-08-23 19:59

Próbuję sobie to przeanalizować ale jeszcze nie ten poziom. Wielu rzeczy nie rozumiem. :P

_13th_Dragon 2022-08-23 20:00

Wg mnie jedynie co tu jest ponad poziom podstawowy to lambdy

Atsin 2022-08-23 20:05

W takim razie mój poziom jest poniżej podstawowego. :P Uczę się w zasadzie od kilku dni w chwilach wolnych od pracy. (co prawda kiedyś tam jakieś proste rzeczy widziałem)

_13th_Dragon 2022-08-23 20:10

To powiedź czego jeszcze nie rozumiesz oprócz lambd?

piotrpo

2022-08-23 19:39

piotrpo2022-08-23 19:39

Rejestracja:ponad 7 lat
Ostatnio:4 dni
Postów:3277

Problem masz z tym, że jak Scanner dostanie na wejście coś co nie jest intem, to będzie próbował to przetworzyć za każdym razem, kiedy wywołujesz np. nextInt().

Kopiuj

package com.sandbox;

import java.util.InputMismatchException;
import java.util.Scanner;


public class Main {

    public static void main(String[] args) {

        Scanner scanner = new Scanner(System.in);
        Party party = new Party();
        boolean run = true;


        while (run) {
            System.out.println("Wybierz opcję:");
            System.out.println("1 - Wyświetl gośći");
            System.out.println("2 - Dodaj gości");
            System.out.println("3 - Pokaż posiłki");
            System.out.println("4 - Znajdź po numerze telefonu");
            System.out.println("5 - Usuń gościa");
            System.out.println("6 - Wyjście");


            try {
                switch (scanner.nextInt()) {
                    case (1):
                        party.displayGuest();
                        break;

                    case (2):
                        party.addGuest();
                        break;

                    case (3):
                        party.showMeals();
                        break;

                    case (4):
                        party.displayGuestByPhoneNumber();
                        break;

                    case (5):
                        party.removeGuest();
                        break;

                    case (6):
                        run = false;
                        break;
                    default:
                        System.out.println("Wybrałeś nieistniejącą opcję!");//pozbyłem się if'a - to też był błąd np. można było wprowadzić -1
                }
            } catch (InputMismatchException e) {
                System.out.println("Wybrano opcję w nieprawidłowym formacie!");
                scanner.nextLine(); //tego brakowało
            }
        }
    }
}

edytowany 1x, ostatnio: piotrpo 2022-08-23 19:48

Zobacz pozostałe 2 komentarze

_13th_Dragon 2022-08-23 19:44

Nie zupełnie, co dostaniesz jak podasz: Ala ma kota? Rozwiązanie od OT, które zacytowałem wyżej zadziała poprawnie, zaś twoje - nie.

piotrpo 2022-08-23 19:47

Aj, faktycznie, wtopa

_13th_Dragon 2022-08-23 19:55

Eee tam wtopa, podałeś alternatywną wersję, może akurat nie w tym przypadku okazują się jej zalety ale zawsze początkującemu się przyda.

Atsin 2022-08-23 19:58

Próbowałem w tym catchu dodać scanner ale chyba robiłem coś źle bo dalej nie ruszało.

piotrpo 2022-08-23 20:03

Naucz się używać debuggera, będzie prościej zrozumieć.

Liczba odpowiedzi na stronę

Treść *

Treść

Podgląd

Kliknij, aby dodać treść...

Kliknij, aby dodać załącznik lub wklej ze schowka.

Instrukcja obsługi Markdown

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj underline.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, , , <kbd>, <del>, , <dfn>, <pre>, <blockquote>, <hr/>, ,  oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H2O i m2 otrzymasz: H₂O i m².

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript document.write('Hello World'); ```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

Alt+L - zaznaczenie całej linii
Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

Ctrl+Enter - dodaj post
⌘+Enter - dodaj post (MacOS)

Po użyciu wyjątku pętla uruchamia się cały czas.

Atsin

Productionserver

tomepaw

_13th_Dragon

Atsin

_13th_Dragon

Atsin napisał(a):

Atsin

_13th_Dragon

Atsin napisał(a):

piotrpo

Smileys & People

Animals & Nature

Food & Drink

Travel & Places

Activities

Objects

Symbols

Typografia

Linki

Wewnętrzne odnośniki

Wspomnienia użytkowników

Znaczniki HTML

Skróty klawiszowe

Indeks górny oraz dolny

Składnia Tex

Kod źródłowy

Krótkie fragmenty kodu

Kod wielolinijkowy

Tabelki

Lista uporządkowana i nieuporządkowana

Składnia Markdown

Skróty klawiszowe

Praca dla programistów

Forum dyskusyjne

Sprawy administracyjne

O nas

Skontaktuj się z nami