search

MakeMagazinDE / Smart-Dimmer

2 stars 1 forks source link

Dead Code #2

Open gdampf opened 4 years ago

gdampf commented 4 years ago

Line 203 ist toter Code! Die in Line 203 deklarierte Variabel ti-n2 hat außerhalb des if-Statements keine Gültigkeit und verändert die in Zeile 200 deklarierte Variabel ti-n2 nicht in ihrem Wert. Was will der Autor uns also damit sagen?

gdampf commented 4 years ago

Und wozu dient die Variabel last_connection? Die kann nie einen anderen Wert als 0 haben. Dann wird aber der Reconnect viel zu häufig versucht. Oder verstehe ich da etwas falsch?

gdampf commented 4 years ago

Was meint eigentlich die Zeile pinMode(an_oder_abschnitt,INPUT_PULLUP); Da an_oder_abschnitt vom Typ Boolean ist, kann dies nur ein Überbleibsel aus einer Phase sein, in der der Autor noch einen PIN dafür verwenden wollte, zwischen Phasenanschnitt und Phasenabschnitt zu wechseln. Jetzt ist das nur noch Code-Müll, der nur deshalb nicht zu Fehlern führt, weil C so lässig ist. Tatsächlich wird hier - abhängig von den Einstellungen mal der PIN GPIO0 und mal der PIN GPIO1 auf Eingang geschaltet - ohne jeden Sinn!