Unübersichtlich und „aufgeblähter" Code?

[frankx]

Ahoi Jnnbo

wozu kopierst du $GET["hersteller"] nach $hersteller? Ist das für bind wichtig? Ich würde es lassen, wenn es nicht sein muss.

Ich dachte zudem, dass Du aus der DB was auslesen willst, was nur mit Herstellerangabe geht. Entweder ist immer ein Hersteller dabei, dann brauchst Du garnicht prüfen, oder Du musst prüfen. Das ist eine Frage der Logik, nicht des Stils.

Meine Kenntnisse sind aber eher marginal, halte Dich lieber an die anderen hier im Thread. Wollte nur mal zeigen, wie man durch logisches Zusammenfassen, konsitentes Einrücken und Weglassen der Umkopiererei schon gleich mehr Übersicht hat. Ob der Code noch korrekt ist, musst Du wissen.

Dank und Gruß,

bob from berlin