Repository navigation
Wartungs-Update 1.2026.1009.1 - #1
Merged
Merged
Conversation
In `Circle.vb` wurden die `ArgumentException`-Meldungen mehrerer `Shared`-Methoden sprachlich vereinheitlicht: „Radius darf nicht negativ sein.“ wurde zu „Der Radius darf nicht negativ sein.“ geändert.
Neue Ressource `RadiusIsNegative` und `Messages.vb` in `Geometry.vbproj` ergänzt; negative Radiusprüfungen in `Circle`, `Cone`, `Cylinder` und `Sphere` auf die zentrale lokalisierbare Fehlermeldung umgestellt; kleines Formatdetail in `Sphere` angepasst.
AssemblyInfo.vb: System.Runtime.InteropServices importiert und AssemblyVersion/AssemblyFileVersion auf 1.2026.1002.1 erhöht. Circle.vb: Parameter r konsistent in radius umbenannt, XML-Kommentare und abhängige Hilfsmethoden angepasst; RadiusFromDiameter meldet nun präziser, dass der Durchmesser nicht negativ sein darf.
Messages.vb auf Module umgestellt und um DiameterIsNegative erweitert. Die Durchmesser-Prüfungen in Circle, Cone, Cylinder und Sphere werfen nun bei negativen Werten die zentrale Ressourcennachricht. In Circle wurden Parameter und XML-Dokumentation vereinheitlicht. Assembly- und File-Version auf 1.2026.1003.0 angehoben.
Neue Ressourcenmeldung `RadiusIfZeroOrLess` samt `Messages.RadiusIsZeroOrLess(...)` hinzugefügt und in `Circle.AngleFromArcLength` sowie `Circle.AngleFromChordLength` als gemeinsamer Guard für `radius <= 0` verwendet.
Neue lokalisierte Fehlermeldung `InvalidTriangleSides` für `Triangle.vb` verwendet und Ressourcen sowie Assembly-Version auf `1.2026.1003.1` aktualisiert.
Messages.vb entfernt; ArgumentException nutzt nun direkt My.Resources. Guard-Clauses für negative Radien, ungültige Durchmesser sowie Dreiecksseiten und Cosinuswerte vereinheitlicht. Assembly-Version auf 1.2026.1003.3 angehoben.
Neue lokalisierte Ressourcen für negative Höhe, Länge und Breite ergänzt und in Cone.vb sowie Cuboid.vb für die Validierung verwendet. Hardcodierte Fehlermeldungen wurden durch My.Resources.* ersetzt.
Guard-Clauses in 2D-/3D-Geometrieklassen vereinheitlicht, Einzeiler meist auf blockierte If-Prüfungen umgestellt und negative Höhen auf My.Resources.HeightIsNegative umgestellt.
Guard-Clauses in mehreren 2D- und 3D-Geometrieklassen auf konsistente Mehrzeilen-If-Then-End If-Blöcke umgestellt und harte Textmeldungen durch My.Resources-Meldungen ersetzt.
Ressourcenmeldungen ergänzt, Geometrie-Validierungen auf My.Resources umgestellt, XML-Doku in Cone vereinheitlicht und Projektdatei inkl. Assembly-Version auf 1.2026.1004.1 aktualisiert.
Ressourcen ergänzt, fehlerhafte Resource-Properties bereinigt, Exception-Texte in Circle, Ellipse, Kite und Rhombus auf konsistente Ressourcenstrings umgestellt und Assembly-Version auf 1.2026.1004.2 angehoben.
Neue lokalisierte Ressourcen für IsPolarAngleInBounds und CheckCapHeightLimit ergänzt und die hart codierten Fehlermeldungen in Sphere.vb durch My.Resources-Texte ersetzt.
`Cylinder.vb` verwendet nun `My.Resources` für die Texte zu `innerRadius >= outerRadius` und `heightOffset`; die neuen lokalisierbaren Zeichenfolgen wurden in `Resources.resx` und `Resources.Designer.vb` ergänzt. Zusätzlich wurde die Zeilenaufteilung bei der Koordinatenberechnung bereinigt.
Neue Ressourcenschlüssel für Umfangs- und Seitenprüfungen ergänzt und Validierungen in Frustum, Prism, Parallelogram und RegularPolygon auf zentralisierte Ressourcentexte umgestellt.
Neue Ressource `CheckNonNegativeArea` ergänzt und die bisherigen Inline-Fehlermeldungen für negative Flächenwerte in den Geometrieklassen vereinheitlicht. Exceptions werden jetzt konsistent über `My.Resources.CheckNonNegativeArea` behandelt.
Lokalisierte Ressourcen ergänzt, harte Fehlermeldungen in Cube, Cuboid, Frustum, Prism und RegularPolygon auf Ressourcen umgestellt, Koordinatenberechnung in Cylinder leicht refaktoriert und Assembly-/Dateiversion auf 1.2026.1006.0 angehoben.
…sser gleich Null eingeführt, bestehende ArgumentException-Meldungen in 2D-/3D-Klassen darauf umgestellt und Ressourcentexte sprachlich sowie fachlich präzisiert.
Neue Lokalisierungseinträge für CheckNonNegativeSide, CheckVerticesNotNull und RequireExactlyFourVertices ergänzt und die Geometrieklassen Cube, Kite, Parallelogram, Polygon, Rectangle, Rhombus, Square, Trapezoid und Triangle auf die zentralen Ressourcen umgestellt.
Neue Ressource `IncludedAngleMustBeBetweenZeroAndPi` für die Prüfung eingeschlossener Winkel in `Parallelogram`, `Rhombus` und weiteren Geometrie-Methoden verwendet; in `RegularPolygon` wurde die direkte Fehlermeldung für `Nothing`-Eckpunkte durch `CheckVerticesNotNull` ersetzt.
Lokalisierte Fehlermeldungen für Parallelogramm- und Eckpunkt-Validierungen in Parallelogram.vb eingebunden; harte Texte entfernt und Winkelmeldung in My.Resources ergänzt.
Versionsnummer auf 1.2026.1009.1 angehoben, fehlende Ressourcen ergänzt und harte Fehlermeldungs-Strings in Geometry-Klassen durch My.Resources ersetzt; ResXManager-Konfiguration für de-DE ergänzt und sprachliche Korrekturen vorgenommen.
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Public parameter renames break named callers, while several exception names, messages, and resource entries are incorrect.
6 open findings
What changed in this PR
Zentralisiert Validierungsfehler in Ressourcen und vereinheitlicht Guard-Clauses sowie Projektmetadaten.
Changes:
- Ersetzt fest codierte Fehlermeldungen durch Ressourcen.
- Überarbeitet Parameter, Validierungen und XML-Dokumentation.
- Aktualisiert Ressourcen-, Versions- und ResXManager-Konfiguration.
| File | Description |
|---|---|
ResXManager.config.xml |
Konfiguriert Ressourcenverwaltung. |
Geometry/TwoDimensional/Triangle.vb |
Lokalisiert Validierungsfehler. |
Geometry/TwoDimensional/Trapezoid.vb |
Lokalisiert Guard-Clauses. |
Geometry/TwoDimensional/Square.vb |
Zentralisiert Fehlermeldungen. |
Geometry/TwoDimensional/Rhombus.vb |
Vereinheitlicht Validierungen. |
Geometry/TwoDimensional/RegularPolygon.vb |
Lokalisiert Polygonprüfungen. |
Geometry/TwoDimensional/Rectangle.vb |
Aktualisiert Fehlermeldungen. |
Geometry/TwoDimensional/Polygon.vb |
Zentralisiert Vertex-Prüfungen. |
Geometry/TwoDimensional/Parallelogram.vb |
Lokalisiert Geometriefehler. |
Geometry/TwoDimensional/Kite.vb |
Vereinheitlicht Guard-Clauses. |
Geometry/TwoDimensional/Ellipse.vb |
Lokalisiert Achsenprüfung. |
Geometry/TwoDimensional/Circle.vb |
Überarbeitet Parameter und Ressourcen. |
Geometry/ThreeDimensional/Sphere.vb |
Lokalisiert Kugelvalidierungen. |
Geometry/ThreeDimensional/Pyramid.vb |
Vereinheitlicht Eingabeprüfungen. |
Geometry/ThreeDimensional/Prism.vb |
Zentralisiert Prismafehler. |
Geometry/ThreeDimensional/Frustum.vb |
Lokalisiert Stumpfvalidierungen. |
Geometry/ThreeDimensional/Cylinder.vb |
Vereinheitlicht Radiusprüfungen. |
Geometry/ThreeDimensional/Cuboid.vb |
Zentralisiert Kantenprüfungen. |
Geometry/ThreeDimensional/Cube.vb |
Lokalisiert Würfelfehler. |
Geometry/ThreeDimensional/Cone.vb |
Aktualisiert Kegelvalidierungen. |
Geometry/My Project/Resources.resx |
Fügt Fehlerressourcen hinzu. |
Geometry/My Project/Resources.Designer.vb |
Generiert Ressourcenzugriffe. |
Geometry/My Project/AssemblyInfo.vb |
Erhöht Assembly-Version. |
Geometry/Geometry.vbproj |
Aktualisiert Ressourcenmetadaten. |
Files not reviewed (1)
- Geometry/My Project/Resources.Designer.vb: Generated file
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
PR Classification
Refactoring und Wartungs-Update zur Zentralisierung von Fehlermeldungen, Vereinheitlichung von Validierungen und Aktualisierung von Projektmetadaten.
PR Summary
Die Pull Request ersetzt harte Fehlermeldungen durch lokalisierte Ressourcen und vereinheitlicht Guard-Clauses sowie Parameterprüfungen in den Geometrieklassen. Zusätzlich wurden XML-Kommentare bereinigt und Projekt-/Versionsmetadaten aktualisiert.
Circle.vb,Ellipse.vb,Kite.vb,Parallelogram.vb,Polygon.vb,Rectangle.vb,RegularPolygon.vb,Rhombus.vb,Square.vb,Trapezoid.vb,Triangle.vb: Fehlertexte aufMy.Resources.*umgestellt und Validierungen für Vertices, Seitenanzahl, Toleranzen und Geometriebedingungen vereinheitlicht.Cone.vb,Cylinder.vb,Cube.vb,Cuboid.vb,Sphere.vb,Prism.vb,Pyramid.vb,Frustum.vb: Guard-Clauses auf Ressourcen umgestellt und Prüfungen für negative bzw. ungültige Werte konsistenter gemacht.Resources.resx,Resources.Designer.vb: Neue lokalisierte Fehlermeldungen ergänzt und ResX-Metadaten aktualisiert.Geometry.vbproj,AssemblyInfo.vb: Ressourcen-Metadaten korrigiert und Assembly-Version auf1.2026.1009.1angehoben.ResXManager.config.xml: Konfiguration für Ressourcen- und Sprachverwaltung hinzugefügt.