Új hozzászólás Aktív témák

• Sirpi senior tag

  #10854

  #10853 Sirpi

  2020-01-03 11:02:29

  Új Válasz

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  

  Sirpi

  senior tag

  #10853 Sirpi

  #10853 Sirpi

  Ez a rész háromszor is szerepel: gyum.toString().contains(nev)

  Egyrészt a harmadik automatikusan teljesülni fog, elhagyható, egyetlen vizsgálat is elég. Másrészt gondolom az elso gyum helyett az ertekek változót akartad használni, így az értékek.contains(nev) pont az, amire szükséged van, sőt, a toString-ezés miatt ha akár egyetlen betűt írsz be, ami a felsorolt gyümölcsök valamelyikében szerepel, azt is el fogja fogadni, nem néz teljes egyezést. Kicsit egyszerűsítve:

  public static void kaloriaKiir(List<Gyumolcs> ertekek) {

        Scanner scan = new Scanner(System.in, "ISO-8859-2");
        System.out.println("Kérem a gyümölcs nevét: ");
       String nev;
       while (!ertekek.contains(nev = scan.nextLine())) {
               System.out.println("Nem megfelelő név, újat kérek!");
       }

         System.out.println("Kérem a mennyiséget grammban: ");
         int menny = scan.nextInt();

         for (Gyumolcs gyum : ertekek) {
             if (nev.equals(gyum.Megnevezes)) {
                 System.out.println(menny + " gramm " + gyum.Megnevezes + " " + gyum.kcal / 100 * menny + " gramm kalóriát tartalmaz.");

             }
       }

   }

  Miután elküldtem, rájöttem, hogy az ertekek.contains(nev) nem lesz jó, mert egy List<Gyumolcs> sosem tartalmazhat String-et, de ettől még áll, amit írtam, hogy a toString()-ezés nem a jó irány. Például:

  private static boolean isNameValid(List<Gyumolcs> ertekek, String nev) {
  for (Gyumolcs gyum : ertekek) {
  if (gyum.getName().equals(nev)) {
  return true;
  }
  }
  return false;
  }

  És akkor ezt a metódust lehet hívni a while ciklusból. A gyum.Megnevezes-t pedig engedelmeddel getter-re cseréltem, nem használunk field-et közvetlenül (és nem is kezdjük a nevét nagybetűvel).

Új hozzászólás Aktív témák