Wenn sonst Anweisung innerhalb for Schleife Probleme mit Hashmaps

Question

Grundsätzlich habe ich Probleme mit meiner if-Anweisung. Ich möchte es „keine CDs für“ + val sagen, aber dies ist die Ausgabe erhalte ich:

Welcome to the CD Database 
    Enter search, add, name, list, or quit: 
    name 
    Enter the full or partial name: 
    asdfa 
    Enter search, add, name, list, or quit: 

wie Sie sehen können, ist es nichts zurück.

mein Code:

public void printByName(String val) { 

    int i = 0; 
    for (CompactDisc values : database.values()) { 

     if (values.getArtist().contains(val)) { 

      System.out.println(values); 

     } else if (i > database.size() && !values.getArtist().contains(val)) { 

      System.out.println("No CDs found for " + val); 

     } 

     i++; 
} 
} 

aus irgendeinem Grund mein Zähler nicht iterieren und diese verrückt wurde mir fahren für hours.I es keine CDs gefunden sagen wollen, nachdem es durch die hashmap Schleifen und findet keine eine teilweise Übereinstimmung der Zeichenfolge eingegeben der Benutzer

zusätzliche Ausgabe:

Welcome to the CD Database 
Enter search, add, name, list, or quit: 
name 
Enter the full or partial name: 
Mo 
Artist:Modest Mouse Title:We Were Dead Before the Ship Even Sank price:5.99 
Artist:Thelonious Monk Title:Monk's Dream price:5.99 
Enter search, add, name, list, or quit: 
list 
Artist:Isley Brothers Title:Funky Family price:5.99 
Artist:Muddy Waters Title:At Newport price:6.99 
Artist:Sly & The Family Stone Title:Greatest Hits price:6.99 
Artist:Modest Mouse Title:We Were Dead Before the Ship Even Sank price:5.99 
Artist:St. Germain Title:Tourist price:5.99 
Artist:Bob Dylan Title:Desire price:6.99 
Artist:The Beatles Title:Abbey Road price:6.99 
Artist:Los Straitjackets Title:The Velvet Touch of... price:5.99 
Artist:The Velvet Underground Title:Peel Slowly and See price:6.99 
Artist:Thelonious Monk Title:Monk's Dream price:5.99 
Enter search, add, name, list, or quit: 
quit 
Program ending. 

Answer 1

Es gibt ein Problem in Ihrer logischen Struktur. Ihr 'if' befindet sich in Ihrem "loop" -Block. Es muss draußen sein. Werfen Sie einen Blick:

public void printByName(String val) { 
    int i = 0; 
    for (CompactDisc values : database.values()) { 
     if (values.getArtist().contains(val)) { 
      System.out.println(values); 
      i++; 
     } 
    } 
    if (i==0) { 
     System.out.println("No CDs found for " + val); 
    } 
} 

Answer 2

die Bedingung i > database.size() wird nie wahr becaus sein e Die Schleife wird beendet, bevor i die Datenbankgröße erreicht. Versuchen Sie, Ihren Code zu ändern:

public void printByName(String val) { 
    for (CompactDisc values : database.values()) { 
     if (values.getArtist().contains(val)) { 
      System.out.println(values); 
     } else { 
      System.out.println("No CDs found for " + val); 
     } 

    } 
} 

Answer 3

Es else if (i >= database.size() && !values.getArtist().contains(val)) für die oben genannten Grund sein sollte.