elwood

Guten Abend!

Ich habe ein Problem beim Aufruf einer Funktion. Leider ist es sehr umständlich, die Problematik kurz und verständlich wieder zu geben, aber ich werde mich bemühen.

In meinem Online-Shop werden nur 4 Produkte angeboten.
Auf der "Order"- Seite sieht man die Abbildungen der 4 Produkte und entsprechende Eingabefelder für Bestellmenge und "Add to Cart-Buttons".

Sind alle Eingaben korrekt, wird der Kunde auf die "Shipping Detail"- Seite weitergeleitet.

Dort erscheint ein Formular für die üblichen Kundendaten (Name, Lieferadresse, eMail usw.) sowie eine Auflistung der bestellten Waren.

Sind auch hier alle Eingaben korrekt, erscheint nach Klick auf den "Complete Transaction"-Button eine Meldung, dass die Bestellung erfolgreich war. Die gesammelten Daten werden in der Datenbank erfasst.

Das ganze spielt sich auf einer Seite ab, die einzelnen Aktionen habe ich in Funktionen ausgelagert.

Hier der Code in gekürzter Fassung:

Bis hier funktioniert alles tadellos.

Nun das Problem: ich möchte auf der "Shipping Detail"-Seite noch einmal die 4 Produkte mit den zuvor eingegebenen Bestellmengen anzeigen, damit der Kunde seine Bestellung ändern kann. Er korrigiert also ggf. die Bestellmenge oder das Produkt, klickt auf den "Change Order"-Button (welchem die Funktion ChangeOrder() zugwiesen wurde) und es erscheint noch einmal die "Shipping Detail"-Seite jetzt mit den geänderten Eingaben.

Hierzu habe ich folgenden Code eingefügt(vgl. obige Zeilen 21-31):

Ich erhalte zwar keine Fehlermeldung, wenn ich aber den „Change Order“-Button anklicke passiert gar nichts.

Bzw. erscheinen im noch nicht ausgefüllten Kundendatenformular die üblichen Fehlermeldungen (please enter name, please enter Adress usw.) so als ob ich den "Complete Transaction"-Button angeklickt hätte.

Verständlich???

Wenn Ihr noch weitere Informationen benötigt bzw. etwas unklar sein sollte, bitte nachfragen.

Danke im Voraus für Eure Hilfe,

Matthias

Mister Ad

Igäl

Ich nehme mal an die ganzen if($_POST['gesendet']) {...} sind Abfagen, ob ein Submit-Button gedrückt wurde, oder? Hast du dazu eine kleine Legende, welcher gesendet, welchem deiner beschriebenen Buttons entspricht?

__________________
Das Recht auf Dummheit gehört zur Garantie der freien Entfaltung der Persönlichkeit.
Mark Twain

dr.e.

Hallo elwood,

ich würde das Design hier zunächst mal überdenken. So einen Spaghetti-Code kann doch keiner mehr pflegen, wenn der fertig ist. Versuch dir doch mal aufzumalen, wie die 4 Seiten genau aussehen sollen und entwickle dann genau für diese Seiten deine Ausgaben und parametrisier deine URL so, dass diese dynamisch zusammengesetzt werden kann. Alles in den Code zu schreiben ist wahnsinnig unübersichtlich. Zudem kannst du Sessions für die Stati der Bestellung speichern und dort nachfragen, ob einer schon auf die "Shipping-Details" darf.

__________________
Viele Grüße,
Dr.E.

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1. Think about software design before you start to write code!
2. Discuss and review it together with experts!
3. Choose good tools (-> Adventure PHP Framework (APF))!
4. Write clean and reusable software only!
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

elwood

Guten Abend!

Danke Euch beiden zunächst für die schnelle Reaktion! Beim Schreiben dieser Zeilen ist mir aufgefallen, warum der "Change Order"-Button die gleiche Aktion ausführt wie der "Complete Transaction"-Button, obwohl ich beiden eine andere Funktion zugewiesen habe!!! Es ist zwar eine Schande, ich gestehe es aber trotzdem. Ich habe den "Complete Transaction"-Button einfach nur kopiert (schreibfaul) und lediglich den image-Namen geändert, NICHT aber name. Natürlich zeigt er mir dann bei beiden Buttons das gleiche Ergebnis an. Und natürlich erhalte ich auch keinen Parse-Error, da ja programmiertechnisch kein Fehler besteht.


@Igäl

Es gibt 3 Buttons. Jedem Button wird eine Funktion zugewiesen. Zusätzlich werden die Formulareingaben geprüft. Ist z.B. die Bestellmenge leer oder höher als 5, wird man erst gar nicht auf die Shipping Detail-Seite weitergeleitet usw.

if $_POST['gesendet']=="1" {ShippingDetails();}
if $_POST['gesendet1']=="1" {ChangeOrder();}
if $_POST['gesendet2']=="1" {CompleteTransaction();}

Add To Cart auf der "Order"-Seite
<input type="image" name="gesendet" border="0" src="images/button_add.gif" width="120" height="19" onFocus="if(this.blur)this.blur()">
<input type="hidden" name="gesendet" value="1">

Change Order auf der "Shipping Detail"-Seite
<input type="image" name="gesendet1" border="0" src="images/button_change.gif" width="120" height="19" onFocus="if(this.blur)this.blur()">
<input type="hidden" name="gesendet1" value="1">

Complete Transaction auf der "Shipping Detail"-Seite
<input type="image" name="gesendet2" border="0" src="images/button_complete.gif" width="120" height="19" onFocus="if(this.blur)this.blur()">
<input type="hidden" name="gesendet2" value="1">


@dr.e.

Ziel dieses Projekts war es, einen kleinen Online-Shop mit geringer Produktzahl zu programmieren. Er sollte so unkompliziert wie möglich, aber dennoch voll funktionsfähig sein.

Als Anregung suchte ich im Netz unter etlichen php-free Skripten nach entsprechendem Material.

Der "kleinste" Shop den ich fand -ausgerichtet für ca. 100 Produkte- besteht aus gezählten 4 Ordnern, 13 Unterordnern, sowie 163 Dokumenten exkl. Bilddateien.

Mein Minishop aus einem Ordner namens MyShop, einem Unterordner namens images, 1 Dokument namens order.php und 1 Dokument namens admin.php = Online-Admin zum ändern der Preise, Ust, Artikelbilder, Anzeige der Monats-u. Jahresstatistik etc.

Dass sich auch bei mir die Anzahl an Ordnern oder Dateien erhöhen kann, z.B. bei Skripterweiterung oder auslagern der admin.php in eine .htaccess-Datei ist selbstredend.

Was meinst Du mit "so einen Spaghetti-Code kann doch keiner mehr pflegen, wenn der fertig ist"?

Ich -als Anfänger- finde ihn gut strukturiert und dank der Kommentare leicht verständlich. Der komplette Bestellvorgang in order.php (Artikel wählen/ Kundendaten/ Bestellung ändern/ Bestellung absenden umfasst gerade mal 650 Zeilen.

Unkomplizierter geht’s nicht mehr...

Aber ich lerne gern dazu -deshalb holt man sich ja Denkanstöße von Profis in einem Forum- und lasse mich eines Besseren belehren!!!

Einmal fertig, braucht man ja nichts mehr pflegen, oder habe ich da gerade einen Denkfehler? Änderungen der Preise, Artikelbilder usw. werden mit der Admin getätigt, und sollte man sich entscheiden, dass komplette Layout zu wechseln, geschieht das in der header.php bzw. einer externen CSS-Datei.

Übrigens ist der Code schon fertig und funktioniert reibungslos. Die Idee mit dem nochmaligen Anzeigen der Produktpalette auf der "Shipping Detail"-Seite ist als zusätzlicher "Service" gedacht, damit der Kunde bei evt. Änderungen nicht noch mal zurückblättern muss. Zuvor hatte ich einen "Continue Shopping"-Button gehabt, welcher wieder zur "Order"-Seite führte.

Gerne werde ich dem Shop noch ein ansprechendes Layout verpassen und das Skript zum freien Gebrauch bzw. Weiterverarbeitung/ Verbesserung in Eurem Forum posten.

In diesem Sinne,
mfg Matthias

dr.e.

Hallo elwood,

Damit bezeichne ich den Code, den du oben geschrieben hast. Wie du sehr richtig bereits bemerkt hast, sortieren andere Vorbilder ihren Code in mehrere Unterordner und in mehrere darin enthaltenen Dateien. Das mag auf den ersten Blick unübersichtlich erscheinen - ist es aber nicht. Es ist eine konsequente Ordnung von Dingen. Ich würde dir raten die einzelnen Funktionen in kleine Dateien auszulagern ud bei Bedarf einzubinden, sonst wird dein Code - wie ich schon sagte - nach einer Zeit nicht mehr zu pflegen sein.


Wird der Online-Shop einmal produktiv eingesetzt und du lässt ein paar Leute diesen teste, wette ich um jeden Betrag, dass die Amazon & Co. verwöhnten Leute dir 10 Features nennen, die Fehlen, die du aber unbedingt - und das auch aus Marketing- und Absatzsteigerungs-Gründen (X-Selling z.B.) - noch mit aufnehmen solltest. Allein beim ersten Feature wirst du an mich denken...


Dass der Code zum jetzigen Zeitpunkt "as is" funktioniert ist schon klar. Nur von ästhetischem Code ist er weit entfernt. Services? Also doch noch weitere Funktionen? Wenn ja, dann kommst du da ganz schnell an deine Grenzen.


Das ist ein guter Gedanke, jedoch solltest du dir vorher ein paar strukturelle Gedanken machen, sonst bekommst du das Wort "Spaghetti-Code" noch öfter zu hören.


Nimm bitte meine Kritik positiv auf, ich möchte dir nur mit ein paar (vielleicht auch machmal sarkatischen) Beispielen zeigen, dass du hier noch viel strukturelle Gedanken investieren solltest.

__________________
Viele Grüße,
Dr.E.

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1. Think about software design before you start to write code!
2. Discuss and review it together with experts!
3. Choose good tools (-> Adventure PHP Framework (APF))!
4. Write clean and reusable software only!
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

elwood

Hallo, dr.e.!

Hatte bis jetzt eine andere Sichtweise.

Mir ist schon öfters aufgefallen, das es gerade beim Programmieren mehrere Lösungen für ein Problem bzw. verschiedene Wege gibt, ein Projekt zu realisieren.

Habe zum Beispiel einmal den Code eines "freien" Counters aus dem Internet um mehr als 50% gekürzt, bei gleichbleibender Funktionalität!!!

Daher mein Grundgedanke, so wenig Code und so unkomliziert wie möglich bei gleichem Endergebnis...

Verstehe aber Deine Ratschläge und werde sie mir zukünftig zu Herzen nehmen!

Alles Gute weiterhin, wir werden sicher noch voneinander hören.

Von mir aus kann dieser Thread jetzt geschlossen werden.

mfg Matthias

Der_Gerhard

Dass wenige Zeilen nichts mit unkompliziertem Code zu tun haben, hat ja dr.E schon geschrieben.

Zusätzlich solltest Du mal Deine Einrückungen angucken.
Da stehen wild if, elseif und else untereinander, die so rein gar nichts auf der gleichen Ebene zu suchen haben.

Die Einrückung der Bedingung im if ist deutlich besser als Block zu erkennen (was aber hier nicht stimmt) als der bedingt auszuführende Code.
Genau so ein Durcheinander wird ja durch Spaghetticode recht gut beschrieben. Man sieht keinen Anfang und kein Ende.

__________________
**********************************
Nein, ich bin nicht die Signatur.
Ich putze hier nur.
**********************************