Methoden Methoden anpassen und fehlende Funktionen hinzufügen

[mihe7]

Ich würde mal ganz einfach anfangen:

return switchedTo.equals(passed) ? getStartPoint() : switchedTo;

 

[DagobertDuck]

Danke, ich habe die Funktion der Methode falsch verstanden... Das war jetzt ja wirklich nicht so schwierig.

Leider Gottes funktioniert der besagte Beispielablauf immer noch nicht. Ich blicke hier leider nicht mehr so richtig durch und das Debuggen macht das ganze in diesem Fall leider auch nicht wirklich leichte für mich. Kannst du einen Fehler erkennen...? Hier sind einmal die Durchläufe bis zur Fehlermeldung:

Was läuft hier noch schief...? Es kann durchaus sein, dass es immer noch etwas mit der Länge zu tun hat, die eventuell falsch dekrementiert wird.

 

[mihe7]

Danke, ich habe die Funktion der Methode falsch verstanden... Das war jetzt ja wirklich nicht so schwierig.

Oops, my fault - wir brauchen ja den Richtungsvektor:

return switchedTo.equals(passed) ? getStartPoint().directionTo(switchedTo) : switchedTo.directionTo(getStartPoint());

 

[DagobertDuck]

return switchedTo.equals(passed) ? getStartPoint().directionTo(switchedTo) : switchedTo.directionTo(getStartPoint());

Das kann so nicht stimmen, oder?
Meinst du:

    public Point getDirectionTo(Point passed) {
        return switchedTo.equals(passed) ? toDirection(switchedTo) : toDirection(getStartPoint());
    }

Es geht nur so, da die Methode in der Klasse Track ist. Ist das so denn richtig? Der Beispielablauf funktioniert nämlich leider immer noch nicht und ich bekomme denselben Fehler.

 

[mihe7]

Das kann so nicht stimmen, oder?

Ich meine den Richtungsvektor start -> switch bzw. umgekehrt switch -> start.

 

[DagobertDuck]

Ups. Es funktioniert allerdings trotzdem noch nicht 🙁.

 

[mihe7]

Also, in #178 ist doch der Ablauf, an welcher Stelle unterscheidet sich Deiner denn davon?

 

[Mika34]

Der ist ja das Problem 🙂 Fahr mal um eine Kurve, dann fährt die Lok z. B. nach rechts, während die Waggons noch nach oben fahren. getDirectionTo liefert sozusagen die Fahrtrichtung der Waggons auf dem Gleis.

Damit meinst du genau diese Methode, oder @mihe7 ?

 

[mihe7]

Ja, genau. Wir suchen ja von einem Punkt auf einem Track ausgehend immer den vorherigen Endpunkt (und damit den vorherigen Track), wobei "vorherig" durch die Fahrtrichtung bestimmt ist. Nun ist die Fahrtrichtung auf verschiedenen Abschnitten des Netzes aber nicht identisch mit der Fahrtrichtung der Lok. Daher müssen wir für den jeweiligen Track die Fahrtrichtung bestimmen.

 

[DagobertDuck]

Also, in #178 ist doch der Ablauf, an welcher Stelle unterscheidet sich Deiner denn davon?

Mhm, warum ist meine startTrack (1,1) -> (5,1)? Es müsste doch (5,1) -> (8,1) sein, oder nicht?

 

[DagobertDuck]

Dann hapert es wahrscheinlich an meiner findTrack Methode

 

[DagobertDuck]

/

 

[mihe7]

Könntest du hier bitte einmal drüberschauen

Einfacher ist es, wenn Du mit dem Debugger durchgehst. Für (6,1), (1,0) müsste in den else-Zweig gesprungen werden. Evtl. gibt es noch ein Problem in isPassable.

Für einen Track (1,1,5,1) muss isPassable(6,1) false liefern.

 

[DagobertDuck]

findTrack scheint richtig zu funktionieren und auch dein Ablauf aus #178 ist identisch mit meinem. Allerdings kommt es dann beim zweiten/dritten offenbar zu Problemen die mir noch nicht ersichtlich sind... Ich weiß nicht was ich noch tun soll.

 

[mihe7]

Wenn der Ablauf zu #178 identisch ist, dann bekommst Du "no connection to track found" und dort ist auch angegeben, was zu tun ist (s. #172)

 

[DagobertDuck]

while: length == 2 > 0, also wird der Schleifenrumpf ausgeführt
requiredTracks enthält currentTrack (1,1,5,1) nicht -> requiredTracks = {(5,1,8,1), (1,1,5,1)}
passed = (1,1)
length = length - position.distanceTo(passed) = length - 4 = -2
getConnection(passed, currentTrack) liefert null -> Fehler

Sorry, das war etwas falsch ausgedrückt. Für length == 1 stimmt der Ablauf überein aber ab length == 2 ist er anders.


Die zweite Iteration macht für mich noch Sinn und es scheint alles zu stimmen.


Darf diese Iteration überhaupt noch stattfinden...? Ich meine jetzt ist currentTrack Element von requiredTracks und daher kommt es zur Exception.

 

[mihe7]

Die zweite Iteration macht für mich noch Sinn und es scheint alles zu stimmen.

Für mich macht da eigentlich gar nichts Sinn: getConnection müsste in der zweiten Iteration null liefern, die Richtung (1,1) ist totaler Quatsch. Da stimmt hinten und vorne nichts. Du musst Zeile für Zeile genau schauen, welche Werte Du erwartest und was tatsächlich rauskommt.

 

[DagobertDuck]

Danke, dann werde ich das morgen noch mal in Ruhe durchgehen. Aber wieso sollte getConnection in der zweiten iteration null liefern? Dann würde ja eine exception geworfen werden. Die Eingabe sollte aber valide sein und der Zug aufgegleist werden.

 

[mihe7]

Aber wieso sollte getConnection in der zweiten iteration null liefern?

Weil es keinen Track mehr gibt?!?

Es gibt zwei Tracks (1,1,5,1) und (5,1,8,1), Du beginnst beim Track (5,1,8,1), dann liefert die erste Iteration den "Vorgängertrack", also (1,1,5,1). In der zweiten Iteration gibt es keinen Vorgängertrack, also muss getConnection null liefern.

Dann würde ja eine exception geworfen werden.

Ja, s. #215 und #172.

 

[DagobertDuck]

OK, das macht Sinn. Hier ist der letzte Durchlauf von getConnection.

Woran scheitert es denn hier...?

 

[mihe7]

point müsste im letzten Durchlauf (1,1) sein.

 

[DagobertDuck]

Danke, jetzt funktioniert zumindest dieser Ablauf. Bei folgendem Ablauf sieht das allerdings noch anders aus (wenn der Zug auf ein Eckpunkt gesetzt wird):

add track (1,1) -> (5,1) add track (1,-3) -> (1,1) create engine steam T3 Emma 3 false true add train 1 T3-Emma put train 1 at (1,1) in direction 1,0 Error, directions do not match

Hier wird ein Fehler ausgegeben, obwohl der Zug aufgegleist werden soll.

Der letzte Durchgang von getPassedPoint


Stimmt mit dir hier alles...?

 

[Mika34]

Danke, jetzt funktioniert zumindest dieser Ablauf. Bei folgendem Ablauf sieht das allerdings noch anders aus (wenn der Zug auf ein Eckpunkt gesetzt wird):

add track (1,1) -> (5,1) add track (1,-3) -> (1,1) create engine steam T3 Emma 3 false true add train 1 T3-Emma put train 1 at (1,1) in direction 1,0 Error, directions do not match

Hier wird ein Fehler ausgegeben, obwohl der Zug aufgegleist werden soll.

Der letzte Durchgang von getPassedPoint
Anhang anzeigen 13096

Stimmt mit dir hier alles...?
Anhang anzeigen 13097

Was hast du bei der getConnection abgeändert, das es bei dir geklappt hat?

 

[mihe7]

Stimmt mit dir hier alles...?

Nein. passed liegt auf dem Track, also muss eine Komponente von dir 0 sein.

 

[Mika34]

Edit: Es war pure Dummheit...

 

[mihe7]

@mihe7 Siehst du weshalb bei mir im ersten Durchgang schon null zurückgegeben wird? Das macht doch gar keinen Sinn

Was hier keinen Sinn macht, ist Dein Vergleich: if (!tr.equals(tr)) wird niemals true sein.

 

[DagobertDuck]

Also, passed wird als (1,1) übergeben


und dann wird mittels currentTrack.getDirectionTo(passed) (1,-1) berechnet und dir zugewiesen


Dann hapert es wohl an currentTrack und somit an getConnection, oder?? Ich blicke hier langsam nicht mehr durch ^^

 

[Mika34]

Also, passed wird als (1,1) übergeben
Anhang anzeigen 13098

und dann wird mittels currentTrack.getDirectionTo(passed) (1,-1) berechnet und dir zugewiesen
Anhang anzeigen 13099

Dann hapert es wohl an currentTrack und somit an getConnection, oder?? Ich blicke hier langsam nicht mehr durch ^^

Es liegt an getPassedPoint...

 

[mihe7]

Dann hapert es wohl an currentTrack, oder??

Nein, es hapert an getDirectionTo.

 

[DagobertDuck]

Du hast den Code doch so bestätigt, oder nicht:

    public Point getDirectionTo(Point passed) {
        return switchedTo.equals(passed) ? toDirection(getStartPoint()) : toDirection(switchedTo);
    }

 

[DagobertDuck]

Es liegt an getPassedPoint...

Ich glaube nicht. Mit der Methode sollte bei mir alles stimmen.

 

[Mika34]

Die ergibt aber auch keinen Sinn, dann ist im letzten Durchlauf der Richtungsvektor (0,1) und nach oben fahren wird der Zug bestimmt nicht ...

    public CartesianPoint getDirectionTo(CartesianPoint passed) {
        CartesianPoint directionVectorOfTracks = null;
        if(startXCoordiante == passed.getXCoordinate()) {
            if(startXCoordiante < passed.getXCoordinate()) {
                directionVectorOfTracks = new CartesianPoint(-1, 0);
            }
            else {
            
            directionVectorOfTracks = new CartesianPoint(1, 0);
            }
        }
        else if(startYCoordiante == passed.getYCoordinate()) {
            if(startYCoordiante < passed.getYCoordinate()) {
                directionVectorOfTracks = new CartesianPoint(0, -1);

            }
            else {
                directionVectorOfTracks = new CartesianPoint(0, 1);

            }
            

        }
        return directionVectorOfTracks;
    }

 

[mihe7]

Du hast den Code doch so bestätigt, oder nicht:

Nein, ich habe in #205 geschrieben, dass Du den Richtungsvektor von Startpunkt -> switchTo brauchst oder umgekehrt. Du normierst ja einfach nur den StartPunkt oder switchTo.

 

[Mika34]

Aber ein Startpunkt kann ein Endpunkt sein und genauso gut auch umgekehrt. Wie soll man da auf eindeutige Richtungsvektoren kommen

 

[mihe7]

Aber ein Startpunkt kann ein Endpunkt sein und genauso gut auch umgekehrt. Wie soll man da auf eindeutige Richtungsvektoren kommen

Startpunkt = (1,1)
switchedTo = (5,1)

passed = (5,1) => Zug fährt von (1,1) nach (5,1), Vektor (5-1,1-1)=(4,0), normiert (1,0)
passed = (1,1) => Zug fährt Zug (5,1) nach (1,1), Vektor (1-5,1-1) = (-4,0), normiert (-1,0)

Nachtrag:

Allgemein:
passed = switchedTo => Zug fährt von Startpunkt zu switchedTo => Vektor bilden => normieren
passed != switchedTo => Zug fährt von switchedTo zu Startpunkt => Vektor bilden => normieren.

 

[DagobertDuck]

Nein, ich habe in #205 geschrieben, dass Du den Richtungsvektor von Startpunkt -> switchTo brauchst oder umgekehrt. Du normierst ja einfach nur den StartPunkt oder switchTo.

Und wie mache ich das dann am besten? Irgendwie hiermit?

        int firstDirComp = getStartPoint().getFirstComponent() - switchedTo.getFirstComponent();
        int secondDirComp = getStartPoint().getSecondComponent() - switchedTo.getSecondComponent();
        ...

 

[Mika34]

Oder doch in die Richtung?

    public CartesianPoint getDirectionTo(CartesianPoint passed) {
        CartesianPoint directionVectorOfTracks = null;
        if(startXCoordiante == passed.getXCoordinate()) {
            if(startXCoordiante < passed.getXCoordinate()) {
                directionVectorOfTracks = new CartesianPoint(-1, 0);
            }
            else {
            
            directionVectorOfTracks = new CartesianPoint(1, 0);
            }
        }
        else if(startYCoordiante == passed.getYCoordinate()) {
            if(startYCoordiante < passed.getYCoordinate()) {
                directionVectorOfTracks = new CartesianPoint(0, -1);

            }
            else {
                directionVectorOfTracks = new CartesianPoint(0, 1);

            }
            

        }
        return directionVectorOfTracks;
    }

 

[mihe7]

Und wie mache ich das dann am besten?

Am besten spendierst Du Point ein paar Methoden, damit Du den Käse nicht an jeder Ecke neu schreiben musst:

public Point vectorTo(Point p) {
    return new Point(p.x - x, p.y - y);
}

public Point negate() {
     return new Point(-x, -y);
}

Zusammen mit der hoffentlich bereits vorhandenen Methode aus #72 (Point#toDirection):

public Point toDirection() {
    return new Point((int)Math.signum(x), (int)Math.signum(y));
}

kannst Du dann einfach schreiben:

Point dir = getStartPoint().vectorTo(switchedTo).toDirection();
return passed.equals(switchedTo) ? dir : dir.negate();

Bearbeitung in Farbe

 

[mihe7]

Oder doch in die Richtung?

    public CartesianPoint getDirectionTo(CartesianPoint passed) {
        CartesianPoint directionVectorOfTracks = null;
        if(startXCoordiante == passed.getXCoordinate()) {
            if(startXCoordiante < passed.getXCoordinate()) {

Wie soll die zweite Bedingung jemals true werden?!? (weiter habe ich mir die Methode nicht angesehen)

 

[Mika34]

Wie soll die zweite Bedingung jemals true werden?!? (weiter habe ich mir die Methode nicht angesehen)

Hast Recht. Ich habe mich dazu verleiten lassen einen Beispiel case auf die Allgemeinheit anzuwenden und das ging richtig brutal nach hinten los

 

[DagobertDuck]

Danke @mihe7. Jetzt bekomme ich jedenfalls eine NPE:
Exception in thread "main" java.lang.NullPointerException
...getRequiredTracks(RailNetwork.java:253)
...putTrain(RailNetwork.java:277)
...



Wo liegt denn jetzt das Problem 😕 .

 

[Mika34]

Welchen Fall prüfst du? Also welche Tracks, welchen Richtungsvektor, Punkt und welchen Zug mit welcher Länge

 

[Mika34]

Am besten spendierst Du Point ein paar Methoden, damit Du den Käse nicht an jeder Ecke neu schreiben musst:

public Point vectorTo(Point p) {
    return new Point(p.x - x, p.y - y);
}

public Point negate() {
     return new Point(-x, -y);
}

Zusammen mit der hoffentlich bereits vorhandenen Methode aus #72 (Point#toDirection):

public Point toDirection() {
    return new Point((int)Math.signum(x), (int)Math.signum(y));
}

kannst Du dann einfach schreiben:

Point dir = getStartPoint().vectorTo(switchedTo).toDirection();
return passed.equals(switchedTo) ? dir : dir.negate();

Bearbeitung in Farbe

Auch nochmal vielen Dank von mir

 

[DagobertDuck]

Welchen Fall prüfst du? Also welche Tracks, welchen Richtungsvektor, Punkt und welchen Zug mit welcher Länge

Immer noch denselben wie vorher.
Z. B.

add track (1,1) -> (5,1)
add track (1,-3) -> (1,1)
create engine steam T3 Emma 3 false true
add train 1 T3-Emma
put train 1 at (1,1) in direction 1,0

 

[DagobertDuck]

@mihe7

            if (currentTrack != null) {
                dir = currentTrack.getDirectionTo(passed);
            }

Die If-Abfrage hat das Problem behoben. Ist das so in Ordnung oder wird dadurch etwas anderes "zerstört"?

 

[Mika34]

@mihe7

            if (currentTrack != null) {
                dir = currentTrack.getDirectionTo(passed);
            }

Die If-Abfrage hat das Problem behoben. Ist das so in Ordnung oder wird dadurch etwas anderes "zerstört"?

Es Läuft auf jeden Fall etwas mit dem getDirectionTo schief, da es auf Horizontalen und Vertikalen Strecken funktioniert, also das Aufgleisen. Wird jedoch über eine Ecke versucht aufzugleisen, dann schmeißt er einen Error

 

[mihe7]

Die If-Abfrage hat das Problem behoben. Ist das so in Ordnung oder wird dadurch etwas anderes "zerstört"?

Das kann durchaus funktionieren, zeigt aber nicht die Idee dahinter. Es geht darum, dass die Länge an der Stelle 0 sein müsste, d. h. der ganze Block ab positioned=passed nicht mehr ausgeführt werden sollte (s. #172).

 

[mihe7]

Läuft jetzt alles?

 

[Mika34]

Edit: Lag an der AddTrack Implementierung

 

[Mika34]

Denn, desto weiter ich den Zug weiter nach rechts positioniere, also weg von der Kreuzung, dann klappt das. getPassedPoint() und den passedPoint() ist auch richtig, der gibt (1,1) zurück. Und dir = currentTrack.getDirectionTo(passed); stimmt auch, weil der mir (0,-1) zurückgibt