tv-forum.info

Fredel hat geschrieben: ↑Sa 18. Apr 2020, 11:17 @sv00010

Schwerer lesbar und umständlich. Was wenn 5 oder gar 10 Settings geprüft werden müssen?

Das mit dem else if hatte ich geschrieben, weil ich nicht weiß, ob eine eventuelle Doppelausführung gewollt ist oder nicht.

Code: Alles auswählen

       
                if (settings.AutoDownloadAwaitAdFree)
                {
                    newTelecasts = newTelecasts.Where(tc => tc.AdFree == true);
                }
                if (settings.AutoDownloadSkipDuplicates)
                {
                    newTelecasts = newTelecasts.Where(tc => tc.Duplicate == false);
                }

Wenn AutoDownloadAwaitAdFree == true ist und auch AutoDownloadSkipDuplicates == true, dann wird zuerst der erste if-Block ausgeführt und danach dann der zweite.
Das kann so gewollt sein oder es wurde einfach Copy & Paste verwendet und vergessen umzuändern.
So etwas ist mir selber schon mal passiert (unabsichtlich und falsch).

thomasfl hat geschrieben: ↑Sa 18. Apr 2020, 11:44 Der ganze STV Manager hat aber mittlerweile tausende Zeilen Code, so dass ich selbst erstmal 1-2 Stunden investieren muss, um überhaupt die Logik wieder zu verstehen.

Dann wäre es jetzt eigentlich angebracht, eine Dokumentation zu erstellen.
Ob nur für dich selber oder auch für andere Programmierer ist dabei eigentlich zweitrangig.

tv-forum.info

STV Manager übertragung an JD Fehler

STV Manager übertragung an JD Fehler

STV Manager übertragung an JD Fehler