Identical scoring 2, undokumentiert #2#6
Open
lieblb wants to merge 10000 commits intobheyser:trunkfrom
Open
Conversation
Loading
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
16 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Der Begriff "identischer Lösungen" im Lückentext geht momentan immer von exakter Identität aus. Das ist aber mglw. nicht das, was man intuitiv erwartet.
Man nehme einen Test, wo bei den Lösungen nicht zwischen Groß- und Kleinschreibung unterschieden wird, aber nur identische Bewertung aus ist:
Jetzt gibt jemand zweimal dieselbe korrekte Lösung ein:
Die momentane Implementierung vergibt hier zwei Punkte. Aus meiner Sicht wäre es aber sinnvoller, da bei der Lösungslogik die Einstellung "nicht zwischen Groß- und Kleinschreibung unterscheiden" zieht, beide Lösungen tatsächlich als identisch zu werten, und daher nur die erste zu werten.
Relevante Änderung:
c1fbb84
Die vorgeschlagene Lösung ist unvollständig, da bei Levensthein-Distanzen der Vergleich von zuvor gegebenen Lösungen mit späteren Lösungen wohl kein guter Ansatz ist.