felder vergleichen

[habkeinen]

 public static boolean enthaeltDoppelt (String [] namensliste){
        
        
        
        int i=0;
        int j=0;
        for(int b=0;b<namensliste.length-1;b++){
            i++;
        }
            for(int v =0;v<namensliste.length-1;v++){
                j++;
            }
                if(namensliste[i].equals(namensliste[j])){  // Vergleich auf Inhaltsgleichheit der Felder
                    return true;
                }else{
                    return false;
                }    
    }
  }

Das ist nun mein Programmcode.
und nun kommt das Testprogramm meines Professors:

 import org.junit.*;
import static org.junit.Assert.*;


public class Aufgabe2Test {
    
    @Test
    public void testDoppelt() {
        String[] liste1 = {"Anna", "Berta", "Caesar"};  //false
        assertFalse(Aufgabe2.enthaeltDoppelt(liste1));
                
        String[] liste2 = {"Berta"};                    //false
        assertFalse(Aufgabe2.enthaeltDoppelt(liste2));
        
        String[] liste3 = {"Caesar", "Anna", "Caesar", "Berta"};   // true
        assertTrue(Aufgabe2.enthaeltDoppelt(liste3));
        
        String[] liste4 = {"Berta", "Berta", "Anna"};    //true
        assertTrue(Aufgabe2.enthaeltDoppelt(liste4));
        
        String[] liste5 = {"Anna", "Berta", "Caesar", "Caesar"};  // True
        assertTrue(Aufgabe2.enthaeltDoppelt(liste5));
        
        String[] liste6 = {"Anna", "Berta", "Caesar", "Dora", "Anna"}; // true
        assertTrue(Aufgabe2.enthaeltDoppelt(liste6));
        
    }
}

und nun meine Frage: was hab ich in meinem Programmcode falsch gemacht?
weil es nicht funktioniert und ich mein das es so funktionieren sollte.

 

[nrg]

du musst deine schleifen verschachteln.

- erste for-schleife iteriert über das komplette array
- zweite innere for-schleife iteriert auch über das komplette array und prüft bei jeder iteration ob der aktuelle index mit dem aktuellen index der äußeren for-schleife übereinstimmt. wenn er eine übereinstimmung findet, wird true zurück gegeben

edit: ausserdem solltest du v<namensliste.length-1 nochmal überdenken

 

[SlaterB]

was unterscheidet i von b und j von v?

die for-Schleifen sollen sicher verschachtelt sein und das if ganz drin, bei dir steht aber alles hintereinander,
die Einrückung ist verwirrend fehl am Platz

 

[ARadauer]

if(namensliste[i].equals(namensliste[j])){  // Vergleich auf Inhaltsgleichheit der Felder
                    return true;
                }else{
                    return false;
                }

nur weil die ersten nicht gleich sind... heißt das nicht, das die zweiten nicht gleich sein können!


gleich = return true; das stimmt
return false erst, wenn du mit dem über alle drüber iterierene fertig bist..


ach ja und du darfst in der zweiten schleife nicht bei 0 anfagen sonder 1+ der aktuellen stelle der ersten schleife, sonst vergleichst du ja auch 0 mit 0 und zb 1 mit 1...

 

[nrg]

ach ja und du darfst in der zweiten schleife nicht bei 0 anfagen sonder 1+ der aktuellen stelle der ersten schleife, sonst vergleichst du ja auch 0 mit 0 und zb 1 mit 1...

oder die Bedingung erweitern:

if(i != j && namensliste.equals(namensliste[j])

 

[noobadix]

oder die Bedingung erweitern:

if(i != j && namensliste.equals(namensliste[j])

Wir haben doch keine Zeit! Dir fehlt wohl ne Klammer! ^^

 

[nrg]

naja bissle soll der TO ja noch selber machen