Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LinkedDocumentCollectionTest #26

Closed
YannickWiest opened this issue Dec 12, 2021 · 5 comments
Closed

LinkedDocumentCollectionTest #26

YannickWiest opened this issue Dec 12, 2021 · 5 comments

Comments

@YannickWiest
Copy link

YannickWiest commented Dec 12, 2021

Homework Number W0701

line 109.
Meiner Meinung nach sollte size 5 und nicht 4 sein. Wir haben vor line 108 3 LinkedDocuments und 1 DummyLinked(Adresse abc) in der Sammlung. Allerdings entfernt die add Methode keine Dummy Elemente? Zumindest kann ich das der Aufgabenstellung nicht entnehmen. Folglich sollte also in der Collection einfach ein LinkedDocument eingefügt werden, welches die gleiche Addresse wie das DummyElement hat.

@LadnerJonas
Copy link
Owner

Zwei Dokumente mit derselben Adresse sind soweit ich informiert bin nicht erlaubt. Meines wissens testet das sogar ein public Test.

@YannickWiest
Copy link
Author

Okay danke. Ich habe nochmal die gesamte Aufgabenstellung gelesen und habe diesen Satz hier in der Beschreibung von updateOutgoing gefunden:
" Das Dummy-Element soll am Ende nicht mehr in der Collection enthalten sein. "
Ich hatte das falsch interpretiert und nur für die Unterkollektions, nicht aber für die Hauptcollection implementiert. Durch den fix ist dann natürlich doc0 in line 108 schon enthalten und darf nicht mehr geaddet werde.

@LadnerJonas
Copy link
Owner

Also passt alles?

@YannickWiest
Copy link
Author

Ja jetzt passt alles. Sorry wegen der kurzzeitigen Verwirrung

@YannickWiest
Copy link
Author

Und Danke für das Repository <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants