😭 Manchmal möchte ich weinen.. - Code Reviews Teil 2
12In diesem Video der Code-Reviews-Reihe haben wir wieder einmal viele der typischen Online-VB.NET/.NET-Programmierer. Es wird sich nicht an Konventionen gehalten, DI ist ein Fremd-Akronym und Sicherheit wird mit Füßen getreten.
Vorwort:
Auch hier möchte ich nochmal darauf hinweisen, dass ich den anderen Kollegen mit diesem Video in keiner Weise angreifen oder beleidigen möchte. Dennoch sehe ich es als sinnvoll an, einmal zu zeigen, worauf man achten sollte/könnte, denn Neulinge, die diesen Code tatsächlich finden, sollten Ihn nicht unbedingt benutzen, bzw. zumindest erfahren, unter welchen Gesichtspunkten man sich weiterentwickeln könnte.
English text for the original video creator:
It's important, that I don't/didn't want to offend you with this video at all. We're all developer colleagues and we all do mistakes, etc., but I wanted to use your example as a discussion base, to argue about what could've been done better or maybe shouldn't be done at all. So I hope you're good with it - best regards, Rob.
📖 Kapitelübersicht:
00:00 Intro
00:23 Dim
00:41 Instanziierung von Membern
01:30 Vorschlaghammer TryCatch
02:12 SQL Command-Parameter
03:18 Kein asynchrone Programmierung
04:00 Call - würg
04:45 KeyDown, KeyPress, KeyUp?
05:33 Enter abfangen
06:15 Convert.ToChar statt Chr
06:39 Pyramid of Doom
07:08 Nur Notwendiges in TryCatch
07:40 Kontexte trennen
09:14 Inkonsistenz
09:38 Dummy Form-Instanzen
10:32 Sicherheitslücken!!
12:21 Keine Hashes verwendet
12:53 Outro
HASHTAGS: #programmieren #codereview #code #optimierung #verbesserung #diskussion