Methoden Methoden anpassen und fehlende Funktionen hinzufügen

[DagobertDuck]

Dann wäre es ja nur:

    private Track getConnection(Point point, Track track) {
        for (Track tr : edges.get(point)) {
            if (!tr.equals(track)) {
                return tr;
            }
        }
        return null;
    }

Stimmt das so? Falls ja, kann der Code noch etwas verschönert werden (z. B. mittels Stream o. Ä.)?

 

[Mika34]

Dann wäre es ja nur:

    private Track getConnection(Point point, Track track) {
        for (Track tr : edges.get(point)) {
            if (!tr.equals(track)) {
                return tr;
            }
        }
        return null;
    }

Stimmt das so? Falls ja, kann der Code noch etwas verschönert werden (z. B. mittels Stream o. Ä.)?

In diesem Fall wird nun welcher Punkt der Methode übergeben? Der Verbindungspunkt von zwei Gleisen? Aber welche Gleise sind das im Konkreten?

 

[mihe7]

In diesem Fall wird nun welcher Punkt der Methode übergeben? Der Verbindungspunkt von zwei Gleisen? Aber welche Gleise sind das im Konkreten?

Ist noch irgendwas unklar?

getConnection(point, track): liefert das Anschlussgleis von track, an Punkt point.

und

An einem Punkt laufen max. 2 Tracks zusammen. Der Anschlusstrack kann ja nur der von track verschiedene sein, sofern es ein solches überhaupt gibt.

sollte die Frage doch beantworten.

 

[mihe7]

Stimmt das so? Falls ja, kann der Code noch etwas verschönert werden (z. B. mittels Stream o. Ä.)?

Sollte passen.

Stream: return edges.get(point).stream().findAny(t -> !t.equals(track)).orElse(null); Du könntest natürlich auch ein Optional<Track> zurückgeben und Dir orElse(null) sparen.

 

[DagobertDuck]

Super, danke. Jetzt fehlt also noch

public Point getPassedPoint(Point direction) {
        
}

Hier ist mir leider auch noch unklar, wie ich dies implementieren soll. Da diese Methode in der Klasse Track ist, verstehe ich nicht, wie ich an den letzten Punkt kommen soll, wenn dieser z. B. auf drei Gleisen weiter ist.

 

[mihe7]

Da diese Methode in der Klasse Track ist, verstehe ich nicht, wie ich an den letzten Punkt kommen soll, wenn dieser z. B. auf drei Gleisen weiter ist.

Gar nicht. Die Methode bezieht sich nur auf den aktuellen Track.

 

[Mika34]

Dann wäre es ja nur:

    private Track getConnection(Point point, Track track) {
        for (Track tr : edges.get(point)) {
            if (!tr.equals(track)) {
                return tr;
            }
        }
        return null;
    }

Stimmt das so? Falls ja, kann der Code noch etwas verschönert werden (z. B. mittels Stream o. Ä.)?

Somit ist track hier das Gleis auf welchem sich der Zugkopf befindet und Point der Verbindungspunkt der beiden Gleise.
Den Verbindungspunkt bekommt man dann genauso über das Objekt track, oder?

 

[DagobertDuck]

@mihe7 Also die Funktionsweise der Methode ist ja wie folgt:

• Es muss zuerst geprüft werden, ob das Gleis horizontal oder vertikal ist.
• Dann muss geschaut werden, in welche Richtung der Zug fährt (Richtungsvektor).
• Je nachdem in welche Richtung und ob das Gleis horizontal oder vertikal ist, wird von der x- bzw. y-Koordinate des Zugkopfes die Länge abgezogen.
• Falls dieser Punkt dann nicht auf der Track liegen sollte, wird der letzte Punkt des Gleises zurückgegeben.

Leider habe ich noch nicht ganz verstanden, wie ich den Richtungsvektor nun verwenden soll. Zuerst wird er mit der bereits gegebenen Methode normiert. Anschließend muss eine Fallunterscheidung stattfinden. Wie sieht diese aus?

 

[mihe7]

Pseudo-Code

public Point getPassedPoint(Point direction) {
    Point v = new Point(switchedTo.x - startPoint.x, switchedTo.y - startPoint.y).toDirection(); 
    if (v.x != direction.x && v.y != direction.y) {
        throw ... // Richtungen passen nicht zusammen
    }
    return v.equals(direction) ? startPoint : switchedTo;
}

 

[mihe7]

Somit ist track hier das Gleis auf welchem sich der Zugkopf befindet und Point der Verbindungspunkt der beiden Gleise.
Den Verbindungspunkt bekommt man dann genauso über das Objekt track, oder?

Ja, so in etwa: track ist halt das Gleis, zu dem man das anliegende Gleis haben möchte und point ist der Verbindungspunkt der Gleise.

Den Verbindungspunkt bekommt man mit Hilfe der Methode getPassedPoint vom track-Objekt, richtig - unter Angabe der Richtung, in die der Zug fährt.

 

[Mika34]

Pseudo-Code

public Point getPassedPoint(Point direction) {
    Point v = new Point(switchedTo.x - startPoint.x, switchedTo.y - startPoint.y).toDirection();
    if (v.x != direction.x && v.y != direction.y) {
        throw ... // Richtungen passen nicht zusammen
    }
    return v.equals(direction) ? startPoint : switchedTo;
}

Und die Methode gibt das Ende des Zuges an, wenn ich es richtig verstanden habe, oder?

 

[DagobertDuck]

public void putTrain(Train train, Point point, Point directionVector) throws LogicException {
    ...
    Track track = findTrack(point).get();
    List<Track> requiredTracks = getRequiredTracks(track, point, directionVector, trainLength);
    // For loop to save the train to all required tracks
}

Wie geht es nun weiter? Reicht es bereits die Methode getRequiredTracks einer Liste zuzuweisen? Falls ein benötigtes Gleis nicht frei sein sollte wird dadurch doch auch eine Exception geworfen, oder nicht?
Jetzt muss der Zug gespeichert werden. Reicht dazu eine Map<Point, Train>, um den Richtungsvektor und den Zug abzuspeichern?

 

[mihe7]

Und die Methode gibt das Ende des Zuges an, wenn ich es richtig verstanden habe, oder?

Nein, das macht die Methode, die ich #88 skizziert habe. Aber die Methode braucht getPassedPoint und getConnection.

Ganz einfach

   WWW-WWWL
+-----+------+----+
a     b      c    d
 Tr1    Tr2   Tr3

Du siehst hier drei Tracks Tr1, Tr2 und Tr3 mit den Endpunkten a, b, c und d gezeichnet als schöne Linien und + als Endpunkte Außerdem sieht man den Zug, dessen Waggons mit W und dessen Lok mit L dargestellt sind. Das "-" zwischen den Waggons soll nur verdeutlichen, dass an der Stelle nur ein Punkt ist.

findTrack() liefert zum Punkt "L" (Lok) erst einmal den Track Tr2. Als nächstes müssen wir wissen, wo Tr2 aufhört. Uns interessiert aber nicht der Endpunkt von Tr2, den die Lok erst noch erreicht (c), sondern der Endpunkt, an dem die Lok bereits vorbeigefahren ist (b). Das ist die Aufgabe von getPassedPoint().

Jetzt brauchen wir den Track, der an Punkt "b" an Tr2 angeschlossen ist. Das ist Aufgabe von getConnection().

 

[mihe7]

Falls ein benötigtes Gleis nicht frei sein sollte wird dadurch doch auch eine Exception geworfen, oder nicht?

Richtig.

Zug gespeichert werden. Reicht dazu eine Map<Point, Train>, um den Richtungsvektor und den Zug abzuspeichern?

Ich würde den Richtungsvektor einfach mal dem Zug mitgeben.

 

[Mika34]

Nein, das macht die Methode, die ich #88 skizziert habe. Aber die Methode braucht getPassedPoint und getConnection.

Ganz einfach

   WWW-WWWL
+-----+------+----+
a     b      c    d
Tr1    Tr2   Tr3

Du siehst hier drei Tracks Tr1, Tr2 und Tr3 mit den Endpunkten a, b, c und d gezeichnet als schöne Linien und + als Endpunkte Außerdem sieht man den Zug, dessen Waggons mit W und dessen Lok mit L dargestellt sind. Das "-" zwischen den Waggons soll nur verdeutlichen, dass an der Stelle nur ein Punkt ist.

findTrack() liefert zum Punkt "L" (Lok) erst einmal den Track Tr2. Als nächstes müssen wir wissen, wo Tr2 aufhört. Uns interessiert aber nicht der Endpunkt von Tr2, den die Lok erst noch erreicht (c), sondern der Endpunkt, an dem die Lok bereits vorbeigefahren ist (b). Das ist die Aufgabe von getPassedPoint().

Jetzt brauchen wir den Track, der an Punkt "b" an Tr2 angeschlossen ist. Das ist Aufgabe von getConnection().

Jetzt habe ich es nun gerafft. Somit ist es totaler Quatsch in diesem Stadium den "Popo" von dem Zug herauszufinden, da dies gerade nicht zu berwerkstelligen ist, wenn ich das richtig verstanden habe

 

[DagobertDuck]

Diese If-Abfrage ist hier nicht ganz richtig, oder? Wo kommt sie stattdessen hin?

    private List<Track> getRequiredTracks(...)
            throws ...{
        ...
        while (length > 0) {
            ...
            if (!getConnection(passed, currentTrack).isPresent()) {
                throw new ...("no connection track found");
            }
            currentTrack = getConnection(passed, currentTrack).get();
        }
        return requiredTracks;
    }

 

[mihe7]

Diese If-Abfrage ist hier nicht ganz richtig, oder?

Warum nicht?

 

[mihe7]

Jetzt habe ich es nun gerafft. Somit ist es totaler Quatsch in diesem Stadium den "Popo" von dem Zug herauszufinden, da dies gerade nicht zu berwerkstelligen ist, wenn ich das richtig verstanden habe

Exakt, darum gehen wir Schritt für Schritt (ok, Track für Track) zurück, bis wir das Ende gefunden haben

 

[DagobertDuck]

Das hat sich erledigt, sorry. Meine erstelltes Gleisnetz war inkorrekt.

Allerdings habe ich noch folgendes Problem:

add track (1,1) -> (5,1) add track (1,1) -> (1,-3) create engine steam T3 Emma 1 true true create coach passenger 1 true true create coach passenger 1 true true add train 1 T3-Emma add train 1 W1 add train 1 W2

Soweit so gut. Wenn ich nun allerdings:

put train 1 at (1,1) in direction 1,0

eingebe, kommt folgende Fehlermeldung:

Error, directions do not match

Was stimmt mit der Methode also noch nicht?

 

[DagobertDuck]

Der Richtungsvektor ist ja valide.

 

[Mika34]

Ja, wie Dein inTrack halt auch. Die findTrack-Methode muss natürlich Track zurückgeben.

Die Methode dürfte in Track etwa so aussehen (Fehler nicht ausgeschlossen):

public boolean canNavigateTo(Point p) {
    if (switchedTo == null) { return false; }
    Point p1 = getStartPoint();
    Point p2 = switchedTo;
    if (p1.equals(p) || p2.equals(p)) { return true; }

    return p1.x == p.x && p.x == p2.x && (p.y-p1.y)/(p2.y-p1.y) == 0 && (p2.y-p1.y)/(p.y-p1.y) > 0 ||
        p1.y == p.y && p.y == p2.y && (p.x-p1.x)/(p2.x-p1.x) == 0 && (p2.x-p1.x)/(p.x-p1.x) > 0;
}

Und findTrack in RailNetwork (oder wo auch immer die Tracks verwaltet werden):

    private Track findTrack(Point point) {
        for (Track track : tracks.values()) {
            if (track.canNavigateTo(point)) {
                return track;
            }
        }
        return null;
    }

@mihe7 welche Funktion hat hier getStartPoint() als Funktion und switchedTo jeweils als Punkte?

 

[Mika34]

Nein, dazu brauchst Du doch keine Tiefensuche. Ganz einfach: edges liefert Dir zu einem gegebenen Punkt alle Gleise, die dort starten oder enden.

//erstmal eine Abkürzung einführen
private boolean hasConnections(Point p) {
    return !edges.computeIfAbsent(p, x -> new ArrayList<>()).isEmpty();
}

public void addTrack(Track track) throws LogicException {
    if (edges.isEmpty() || hasConnections(track.getStartPoint()) ||
            track.getEndPoints().stream().anyMatch(this::hasConnections)) {
        // track hinzufügen
    } else {
        // Kein Anschlusspunkt vorhanden
    }
}

Das ergibt wenig Sinn. Ich wiederhole mich: "edges liefert Dir zu einem gegebenen Punkt alle Gleise, die dort starten oder enden. "


Ja, computeIfAbsent sorgt nur dafür, dass Du nicht null sondern einen bestimmten Wert (hier: eine leere Liste) erhältst. Aus einer Liste etwas entfernen, das in der Liste nicht vorhanden ist, ist kein Problem.

Du kannst natürlich auch explizit vorher prüfen, ob der Punkt in der Map vorhanden ist:

    public void removeTrack(int trackId) {
        Track track = tracks.get(trackId);
        if (track == null) {
            return; // oder Exception, da Track nicht existiert
        }
        tracks.remove(trackId);
        for (Point p : track.getPoints()) {
            if (edges.containsKey(p)) {
                edges.get(p).remove(track);
            }
        }
    }

Und @mihe7 welche Datenstruktur wurde hier für edges gewählt, denn wenn ich versuche meinen CartesianPoint in eine HasMap zu speichern, bekomme ich einen Error um die Ohren gehauen. Ich schätze mal aufbauend auf den letztem 100 Beiträge das der Key hier nicht die ID ist, sondern der Punkt, aber welcher genau?

    private HashMap<CartesianPoint, RawTrack> tracksReadyForDriving = new HashMap<CartesianPoint, RawTrack>();
    private boolean hasConnections(CartesianPoint p) {
        return !tracksReadyForDriving.computeIfAbsent(p, x -> new ArrayList<>()).isEmpty();
    }

So sieht es aktuell bei mir aus

 

[mihe7]

Und @mihe7 welche Datenstruktur wurde hier für edges gewählt

Map<Point, List<Track>>

sondern der Punkt, aber welcher genau?

Der in den Tracks vorkommt. Wenn Du einen Track A-B hast, wird der Track in der Map sowohl unter A als auch unter B eingetragen. Bei einer Weiche A,B,C wir der Track in der Map unter allen drei Punkten eingetragen. edges liefert also für einen Punkt alle Tracks, die an diesem Punkt starten oder enden.

 

[mihe7]

Soweit so gut. Wenn ich nun allerdings:

put train 1 at (1,1) in direction 1,0

eingebe, kommt folgende Fehlermeldung:

Error, directions do not match

Was stimmt mit der Methode also noch nicht?

Guter Testfall... bei der Suche nach dem Track wird noch nicht berücksichtigt dass es Punkte gibt, die Tracks miteinander verbinden (also in edges als Schlüssel dienen). In dem Fall muss nicht über die values gesucht aber der Track über die Richtung bestimmt werden.

 

[mihe7]

@mihe7 welche Funktion hat hier getStartPoint() als Funktion und switchedTo jeweils als Punkte?

Der Track speichert eine Liste von Punkten, dabei ist der erste Punkt der Startpunkt, der Rest stellt mögliche Endpunkte dar. Bei einem normalen Gleis werden also zwei Punkte gespeichert, bei einer Weiche gibt es mehr Endpunkte.

Das ist sozusagen die statische Sicht auf ein Schienenelement, die den grundsätzlichen Aufbau des Netzes beschreibt: es gibt eine Weiche A,B,C, d. h. an Punkt A gehen zwei Gleise ab, eines nach B und eines nach C.

Aus dynamischer Sicht (also im Betrieb) ist es aber so, dass zu jedem Zeitpunkt jeder Track einen Startpunkt mit genau einem Endpunkt verbinden muss - und dieser Endpunkt wird mit switchedTo angegeben. Bei normalen Gleisen steht der fest, bei Weichen kann er eingestellt werden.

getStartPoint() liefert also den Startpunkt (erster Punkt der Liste) und switchedTo den aktuellen(!) Endpunkt.

 

[Mika34]

Map<Point, List<Track>>


Der in den Tracks vorkommt. Wenn Du einen Track A-B hast, wird der Track in der Map sowohl unter A als auch unter B eingetragen. Bei einer Weiche A,B,C wir der Track in der Map unter allen drei Punkten eingetragen. edges liefert also für einen Punkt alle Tracks, die an diesem Punkt starten oder enden.

Aber der Key der Map ist jedoch der eindeutig definierte Startpunkt eines Gleis bzw. einer Weiche, wenn ich es richtig verstanden habe.

Könntest du mir bitte noch sagen, ob getStartPoint() und switchedTo() einfache Getter einer Map sind und wie man diese geschickt modellieren kann? Denn in meiner jetzigen Modellierung sieht das doch sehr schrecklich aus

 

[mihe7]

Aber der Key der Map ist jedoch der eindeutig definierte Startpunkt eines Gleis bzw. einer Weiche, wenn ich es richtig verstanden habe.

Nein, in der Map sind alle Punkte der Tracks Schlüssel.

Könntest du mir bitte noch sagen, ob getStartPoint() und switchedTo() einfache Getter einer Map sind und wie man diese geschickt modellieren kann? Denn in meiner jetzigen Modellierung sieht das doch sehr schrecklich aus

Nein, das sind Getter der Klasse Track. Sollte auf Seite 1 dieses Threads erklärt sein.

 

[Mika34]

Es wurde konkret nichts zur Methode geschrieben, sondern angenommen das diese existiert, deshalb hatte ich gefragt...

 

[DagobertDuck]

Guter Testfall... bei der Suche nach dem Track wird noch nicht berücksichtigt dass es Punkte gibt, die Tracks miteinander verbinden (also in edges als Schlüssel dienen). In dem Fall muss nicht über die values gesucht aber der Track über die Richtung bestimmt werden.

Ach so, OK. Wie kann ich das jetzt noch am besten geschickt implementieren?

 

[mihe7]

Du nimmst edges.get(trainStartPoint) und suchst in der Liste nach Tracks, deren Richtungsvektor in wenigstens einer Komponente mit dem gegebenen Richtungsvektor übereinstimmen.

 

[DagobertDuck]

@mihe7 Wie komme ich an den Richtungsvektor der Track? Und soll Point passed = currentTrack.getPassedPoint(direction); regulär danach zugewiesen werden, oder soll es in die For Schleife rein?

 

[Mika34]

Nein, dazu brauchst Du doch keine Tiefensuche. Ganz einfach: edges liefert Dir zu einem gegebenen Punkt alle Gleise, die dort starten oder enden.

//erstmal eine Abkürzung einführen
private boolean hasConnections(Point p) {
    return !edges.computeIfAbsent(p, x -> new ArrayList<>()).isEmpty();
}

public void addTrack(Track track) throws LogicException {
    if (edges.isEmpty() || hasConnections(track.getStartPoint()) ||
            track.getEndPoints().stream().anyMatch(this::hasConnections)) {
        // track hinzufügen
    } else {
        // Kein Anschlusspunkt vorhanden
    }
}

Das ergibt wenig Sinn. Ich wiederhole mich: "edges liefert Dir zu einem gegebenen Punkt alle Gleise, die dort starten oder enden. "


Ja, computeIfAbsent sorgt nur dafür, dass Du nicht null sondern einen bestimmten Wert (hier: eine leere Liste) erhältst. Aus einer Liste etwas entfernen, das in der Liste nicht vorhanden ist, ist kein Problem.

Du kannst natürlich auch explizit vorher prüfen, ob der Punkt in der Map vorhanden ist:

    public void removeTrack(int trackId) {
        Track track = tracks.get(trackId);
        if (track == null) {
            return; // oder Exception, da Track nicht existiert
        }
        tracks.remove(trackId);
        for (Point p : track.getPoints()) {
            if (edges.containsKey(p)) {
                edges.get(p).remove(track);
            }
        }
    }

Wieso tut man den Track dann in eine Liste speichern?
Ich dachte man speichert einen Track als value und als Key dann jeweils die Endpunkte A,B bzw. A,B,C
tracksReadyForDriving.put(originPoint, newlyBuildTrack);

 

[Mika34]

Edit: Das Model sieht es so vor... Ok.
@DagobertDuck was tut diese Methode? getTrackConnections(). Die gibt nicht die Start und Endpunkte zurück, das macht die getPoints Methode. ie gibt das Anschlussgleis zurück, kann das sein?

 

[DagobertDuck]

Die Methode gibt alle Gleise zurück, die an dem angegebenem Punkt anliegen.

 

[mihe7]

Wie komme ich an den Richtungsvektor der Track?

s. getPassedPoint:
Point v = new Point(switchedTo.x - startPoint.x, switchedTo.y - startPoint.y).toDirection();

Und soll Point passed = currentTrack.getPassedPoint(direction); regulär danach zugewiesen werden, oder soll es in die For Schleife rein?

In die Schleife.

 

[DagobertDuck]

Danke.

Hier ist mein derzeitiger Code:

            Point passed;
            for (Track trackConnection : getTrackConnections(startPoint)) {
                Point switchedTo = trackConnection.getSwitchedTo();
                Point trackDirection = trackConnection.toDirection(new Point(switchedTo.x - startPoint.x,
                        switchedTo.y - startPoint.y));
                if (trackDirection.x == direction.x
                        || trackDirection.y == direction.y) {
                    passed = trackConnection.getPassedPoint(direction);
                } else {
                    // Throw exception
                }
            }
            // Error: variable 'passed' might not have been initialised
            length -= position.distanceTo(passed);

Irgendwas stimmt damit ja noch nicht. Was fehlt noch bzw. was habe ich falsch gemacht?

 

[mihe7]

Äh, wo ist denn der Rest der Methode? Außerdem musst Du passed initialisieren (darum der Compilerfehler), das else rausnehmen und durch ein if nach der Schleife ersetzen:

Point passed = null;
for (...) {
}
if (passed == null) {
    Throw exception
}
...

 

[DagobertDuck]

Leider bekomme ich beim Testen gültiger Eingaben immer noch die Fehlermeldung. Es muss doch sicher etwas an der Methode getPassedPoint geändert werden, oder nicht?

 

[Mika34]

trackConnection.getSwitchedTo(); Gibt an welcher Endpunkt gewählt wurde. Hierbei ist es egal, ob es eine Weiche oder ein normaler Track ist, da man immer setSwitchedTo() ausführt. Vertue ich mich oder stimmt dies?

 

[mihe7]

trackConnection.getSwitchedTo(); Gibt an welcher Endpunkt gewählt wurde. Hierbei ist es egal, ob es eine Weiche oder ein normaler Track ist, da man immer setSwitchedTo() ausführt. Vertue ich mich oder stimmt dies?

So ist es.

 

[mihe7]

Leider bekomme ich beim Testen gültiger Eingaben immer noch die Fehlermeldung. Es muss doch sicher etwas an der Methode getPassedPoint geändert werden, oder nicht?

Poste mal findTracks(), getPassedPoint() und getRequiredTracks()

 

[mihe7]

Ihr müsst Euch das mal etwas visualisieren. Sucht Euch z. B. Bilder von Modelleisenbahnen. Da gibt es Schienenelemente wie normale Gleise und Weichen. Nichts anderes beschreibt Track, nur dass Track davon abstrahiert, indem es ein normales Gleis wie eine Weiche behandelt, die nur 2 Enden hat und somit nicht verstellbar ist.

 

[DagobertDuck]

getPassedPoint()

public Point getPassedPoint(Point direction) {
    if (v.x != direction.x && v.y != direction.y) {
        throw ...("directions do not match");
    }
    return v.equals(direction) ? startPoint : switchedTo;
}

findTracks()

    private Track findTrack(Point point) {
        for (Track track : tracks.values()) {
            if (track.isPassable(point)) {
                return track;
            }
        }
        return null;
    }

getRequiredTracks()

    private List<Track> getRequiredTracks(Track startTrack, Point startPoint, Point direction, long trainLength) {
        Track currentTrack = startTrack;
        Point position = startPoint;
        List<Track> requiredTracks = new ArrayList<>();
        long length = trainLength;
        while (length > 0) {
            if (currentTrack == null || requiredTracks.contains(currentTrack)) {
                throw new ...("train cannot be positioned");
            }
            requiredTracks.add(currentTrack);
            Point passed = null;
            for (Track trackConnection : getTrackConnections(startPoint)) {
                 Point v = trackConnection.toDirection(new Point(switchedTo.x - startPoint.x, switchedTo.y - startPoint.y));
                if (v.x == direction.x || v.y == direction.y) {
                    passed = trackConnection.getPassedPoint(direction);
                }
            }
            if (passed == null) {
                throw new ...("direction vector not valid");
            }
            length -= position.distanceTo(passed);
            position = passed;
            if (!getConnection(passed, currentTrack).isPresent()) {
                throw new ...("no connection track found");
            }
            currentTrack = getConnection(passed, currentTrack).get();
        }
        return requiredTracks;
    }

 

[mihe7]

Bitte noch die putTrain-Methode (oder wie sie auch immer heißen mag).

 

[DagobertDuck]

/

 

[mihe7]

Der Code

            Point passed = null;
            for (Track trackConnection : getTrackConnections(startPoint)) {
                 Point v = trackConnection.toDirection(new Point(switchedTo.x - startPoint.x, switchedTo.y - startPoint.y));
                if (v.x == direction.x || v.y == direction.y) {
                    passed = trackConnection.getPassedPoint(direction);
                }
            }
            if (passed == null) {
                throw new ...("direction vector not valid");
            }

gehört nicht in getRequiredTracks(). Ändere die Methode wieder zurück.

Es geht darum, dass die Methode

private Track findTrack(Point point) {

nicht berücksichtigt, dass point ein Punkt sein kann, der zwei Gleise miteinander verbindet. Die Methode braucht also noch die Richtung des Zuges, um ggf. das richtige Gleis auswählen zu können - und natürlich den Code, der sich darum kümmert.

D. h. WENN point Schlüssel von edges ist, DANN muss das richtige Gleis anhand der Richtung gewählt werden, SONST muss der Punkt über die Werte von edges (also alle Tracks) gesucht werden.

 

[Mika34]

Point passed = null;
            for (Track trackConnection : getTrackConnections(startPoint)) {
                 Point v = trackConnection.toDirection(new Point(switchedTo.x - startPoint.x, switchedTo.y - startPoint.y));
                if (v.x == direction.x || v.y == direction.y) {
                    passed = trackConnection.getPassedPoint(direction);
                }
            }
            if (passed == null) {
                throw new ...("direction vector not valid");
            }

In welcher Methode ist dies dann sonst zu prüfen. Es bietet sich theoretisch ja nur hier die Möglichkeit dafür, oder nicht?

 

[DagobertDuck]

Hat sich erledigt

 

[mihe7]

Leute, bitte überlegt doch mal ein wenig selber.

In welcher Methode ist dies dann sonst zu prüfen. Es bietet sich theoretisch ja nur hier die Möglichkeit dafür, oder nicht?

In der Form nirgends.

Ich verstehe aber nicht, was das mit meiner Fehlermeldung zu tun hat? Die wird doch durch getPassedPoint() ausgelöst (?).

getPassedPoint stellt fest, dass der Spaß mit dem gegebenen Track nicht funktioniert. Der Punkt ist, dass von einem falschen Gleis ausgegangen wird, das von findTrack geliefert wird.

 

[DagobertDuck]

Leute, bitte überlegt doch mal ein wenig selber.


In der Form nirgends.


getPassedPoint stellt fest, dass der Spaß mit dem gegebenen Track nicht funktioniert. Der Punkt ist, dass von einem falschen Gleis ausgegangen wird, das von findTrack geliefert wird.

Da hast du recht, das habe ich gar nicht beachtet. Leider funktioniert es nur immer noch nicht.