print-Anweisung verschluckt

[Jennue]

Guten Morgen,
die unten stehende Methode soll zurückgeben, ob es eine Kante zwischen den Knoten String n1 und String n2 gibt (Graphen). Nun verwirrt es mich ein wenig, dass die print-Anweisungen total fehlerhaft sind, d.h. sie geben nur die Hälfte aus.

So schreibt er:
String1: acht String2: sieben
acht siebenh Iter1: 0 null
acht siebenh Iter1: 1 eins
....
acht siebenh Iter1: 7 sieben
FindEdge nach Iter1: 8 acht acht sieben
test1
siebenieben

und gibt false zurück.

Gefragt habe ich nach der Verbindung "acht" zu "sieben" (diese existiert).

	public boolean findEdge2(String n1, String n2){
		String str1 = n1;
		String str2 = n2;
		System.out.println("String1: " + str1 + " String2: " + str2);
		boolean b = false;			
		for(Iterator iter = nodes.iterator();iter.hasNext(); ){
			Nodes tmp1 = (Nodes)iter.next();
			System.out.println("FindEdge nach Iter1: " + tmp1.getId()  + " "+ tmp1.getName() + " " + str1 + " " + n2);
			if(str1.equals(tmp1.getName())){
				System.out.println("test1");
				for(Iterator itersucc = tmp1.m_Succ.iterator();itersucc.hasNext(); ){
					Nodes tmp2 = (Nodes)itersucc.next();
					System.out.println("wuhu: " + tmp2.getName() + " " + str2);
					if(str2.equals(tmp2.getName())){
						b = true;
					}
				}	
			}	
		}
		return b;
	}

Stimmt was nicht an meinem Code oder wo ist mein Fehler? Würde mich über Hilfe sehr freuen.
Viele Grüße

 

[SlaterB]

grundsätzlich ist dein Code richtig, siehe unten mein Beispiel mit ergänzter Node-Klasse + main-Methode (was du alles hättest posten können..)
merkwürdig sind in deinem Log die Ausgaben, die nicht durch System.out.println()-Methoden zu erklären sind,

was bei dir der Fehler ist ist noch nicht zu erkennen,
die Grundannahme ist schlicht, dass die Kante nicht vorhanden ist,
sieben hat anscheinend nichtmal eine einzige Verknüpfung, sonst gäbe es ja wuhu-Ausgaben,
in der test1-Zeile kannst du noch tmp1.m_Succ.isEmpty() ausgeben

public class Test
{
    List<Nodes> nodes = new ArrayList<Nodes>();

    public static void main(String[] args)
    {
        Nodes n7 = new Nodes(7, "sieben");
        Nodes n8 = new Nodes(8, "acht");
        n7.m_Succ.add(n8);

        Test t = new Test();
        t.nodes.add(n7);
        System.out.println(t.findEdge2("sieben", "acht"));
    }

    public boolean findEdge2(String n1, String n2)
    {
        String str1 = n1;
        String str2 = n2;
        System.out.println("String1: " + str1 + " String2: " + str2);
        boolean b = false;
        for (Iterator iter = nodes.iterator(); iter.hasNext();)
        {
            Nodes tmp1 = (Nodes)iter.next();
            System.out.println("FindEdge nach Iter1: " + tmp1.getId() + " " + tmp1.getName() + " " + str1 + " " + n2);
            if (str1.equals(tmp1.getName()))
            {
                System.out.println("test1");
                for (Iterator itersucc = tmp1.m_Succ.iterator(); itersucc.hasNext();)
                {
                    Nodes tmp2 = (Nodes)itersucc.next();
                    System.out.println("wuhu: " + tmp2.getName() + " " + str2);
                    if (str2.equals(tmp2.getName()))
                    {
                        b = true;
                    }
                }
            }
        }
        return b;
    }
}


class Nodes
{
    List<Nodes> m_Succ = new ArrayList<Nodes>();
    int id;
    String name;

    Nodes(int id, String name)
    {
        this.id = id;
        this.name = name;
    }

    String getName()
    {
        return name;
    }

    int getId()
    {
        return id;
    }
}

String1: sieben String2: acht
FindEdge nach Iter1: 7 sieben sieben acht
test1
wuhu: acht acht
true

 

[MQue]

Also was man sagen kann ist, das print- Anweisungen nie falsch sind, bei dir liegt es am Code, der macht was anderes als du erwartest.
Debug dich mal durch deinen Code und schau, was du erwartest und was tatsächlich passiert.
Wenn du dann noch fragen hast, dann poste ein funktionierendes/lauffähiges Programm, dann kann man mehr sagen.
lg

 

[Jennue]

Vielen Dank für die Antworten.
Die Überprüfung auf isEmpty() hatte mir false zurückgeliefert, das eigentliche Problem habe ich dann an einer anderen Stelle gefunden. Und zwar habe ich txt.Dateien eingelesen und er hat wohl die Enter-Zeichen mit gespeichert, dies hab ich dann mit scan.next().trim() gelöst. Jetzt funktioniert es.

PS: Bei der nächsten Frage stelle ich auch mehr Quellcode, dachte das reicht vllt