Anmerkungen Lab1

- Cities.cs:42: Meistens ist double.Parse/double.TryParse gegenüber
Convert.ToDouble vorzuziehen. Convert.ToDouble nimmt einige automatische
Konversionen vor (z.B. null à 0), was meistens nicht gewünscht ist.
- Cities.cs:67: Sie sollten Stringvergleiche ohne Berücksichtung der
Gross-/Kleinschreibung mit
string.Equals(a,b,StringComparison.InvariantCultureIgnoreCase)
implementieren. Ansonsten laufen Sie in Probleme. So wird ß (bzw. die
Grosse Variante davon) je nach Spracheinstellung und Programmiersprache
in „SS“ bzw. „ss“ umgewandelt. Weiter sollte string.Equals(…) schneller
sein.
- RouteRequestEventArgs.cs: Üblicherweise werden diese Klassen immutable
designt. Sprich, sie sollten die setter entfernen (oder „private“
deklarieren).
- RouteRequestWatcher.cs:44 Das try/catch sollten Sie nach Möglichkeit
durch ein if ersetzen. Try/catch ist langsam und nur für
aussergewöhnliche Fehlerfälle gedacht.