fehlerfreies formular - aber wie schneller, besser, schlanker?

fripon77 · 22.12.2009, 21:35

ich hoffe, mein wunsch ist nicht unverschämt: ich würde mich freuen, wenn ein erfahrener php´ler mein folgendes script auf effizienz & sicherheit überprüft - quasi als mentor. das script selbst funktioniert fehlerfrei. ich habe mir das die letzten tage selbst angelesen & erarbeitet - auch dank dieses beitrags (Logarithmus, Zins, Runden...) und eurer unterstützung.

um aber zu erkennen, was man besser, schneller, schlanker machen könnte, bedarf es sicherlich viel erfahrung - und das geht in ein paar tagen bestimmt nicht.

also wer lust hat, mir tipps zu geben, der kann das script vielleicht mal überfliegen...

PHP-Code:

  <?php

if ( $_GET['darlehensbetrag'] == "" OR $_GET['zinssatz'] == "" OR $_GET['mtl_rate_inkl_tilgung'] == "")

{

    echo '<form id="laufzeitrechner" action="rechner.php" method="GET">';

 
    if ( $_GET['darlehensbetrag'] == "" )

    {

        if ( $_GET['schon_mal_abgesendet'] == "ja" )

        {

            echo "<p class='error'>Hinweis: Bitte Feld Darlehensbetrag ausfüllen</p>";

        }

    }

    echo 'Darlehensbetrag (EUR): ';

    echo '<input type="text" name="darlehensbetrag" value="';

 
    echo $_GET['darlehensbetrag'];

 
    echo '" size="12" maxlength="12" />';

 
    if ( $_GET['zinssatz'] == "" )

    {

        if ( $_GET['schon_mal_abgesendet'] == "ja" )

        {

            echo "<p class='error'>Hinweis: Bitte Feld Zinssatz ausfüllen</p>";

        }

    }

    echo '<p>Zinssatz (%):';

    echo '<input type="text" name="zinssatz" value="';

 
    echo $_GET['zinssatz'];

 
    echo '" size="5" maxlength="5" />';

    

    if ( $_GET['mtl_rate_inkl_tilgung'] == "" )

    {

        if ( $_GET['schon_mal_abgesendet'] == "ja" )

        {

            echo "<p class='error'>Hinweis: Bitte Feld Rate inkl. Tilgung ausfüllen</p>";

        }

    }

    echo '<p>Rate inkl. Tilgung (EUR): ';

    echo '<input type="text" name="mtl_rate_inkl_tilgung" value="';

 
    echo $_GET['mtl_rate_inkl_tilgung'];

 
    echo '" size="9" maxlength="9" />';

    

 
    echo '<p>Verrechnung: '; 

    echo '<select name="verrechnung">    

    <option value="12">monatlich</option>

    <option value="3">vierteljährlich</option>

    <option value="2">halbjährlich</option>

    <option value="1">jährlich</option>

    </select></p>';

 
    echo '<input type="hidden" name="schon_mal_abgesendet" 

          value="ja" />';

 
    echo '<br />';

    echo '<input type="Submit" name="" value="Laufzeit berechnen" />';

 
    echo '</form>';

 
    echo "<p>Hinweise: nachschüssige Verrechnung, keine Berücksichtigung von Sondertilgungen</p>";

    exit;

 
}

else

{

 
        $_darlehensbetrag = str_replace(".", "",$_GET["darlehensbetrag"]); // punkte löschen

        $_darlehensbetrag = str_replace(",", ".", $_darlehensbetrag); // komma durch punkt ersetzen

 
        $_zinssatz = str_replace(".", "", $_GET['zinssatz']); // punkte löschen

        $_zinssatz = str_replace(",", ".", $_zinssatz); // komma durch punkte ersetzen

        

        $_verrechnung = $_GET["verrechnung"];

        

        $_mtl_rate_inkl_tilgung = str_replace(".", "",$_GET["mtl_rate_inkl_tilgung"]); // punkte löschen

        $_mtl_rate_inkl_tilgung = str_replace(",", ".", $_mtl_rate_inkl_tilgung); // komma durch punkte ersetzen

 
        $wert1 = log(($_zinssatz/100-$_mtl_rate_inkl_tilgung*$_verrechnung/$_darlehensbetrag)/

(0-$_mtl_rate_inkl_tilgung*$_verrechnung/$_darlehensbetrag));

        $wert2 = log(1/(1+$_zinssatz/100/$_verrechnung));

        $geslfzt = $wert1/$wert2/$_verrechnung; 

        

        if ($geslfzt > 0) 

        {

            echo "Gesamtlaufzeit: ".round($geslfzt,2); // auf zwei nachkommastellen runden - wie ausgabe mit komma???

        } 

        else 

        {

            echo "Ihre Eingaben sind unlogisch und führen zu entsprechend unlogischen Ergebnissen";

        }

 
}

?>

md5() · 22.12.2009, 21:40

PHP-Code:

      if ( $_GET['darlehensbetrag'] == "" )
    {
        if ( $_GET['schon_mal_abgesendet'] == "ja" )
        {
            echo "<p class='error'>Hinweis: Bitte Feld Darlehensbetrag ausfüllen</p>";
        }
    }

darlehensbetrag brauchst du nicht noch einmal uberprufen da du es schon vorher gemacht hast

christophk · 22.12.2009, 21:49

Ich würde gar nicht so einen Aufwand betreiben, alle Felder einzeln zu überprüfen. Ein einfacher Hinweis "mindestens ein Feld wurde nicht ausgefüllt" sollte in diesem Fall reichen, da du ja sowieso alle brauchst und bei drei Feldern noch ein überschaubares Formular hast.
Außerdem würde ich das Formular über Post laufen lassen, da du dann kein extra hidden-Feld brauchst, sondern die REQUEST_METHOD aus dem $_SERVER-Array überprüfen kannst.

Vielleicht könnte für dich auch der Artikel Affenformular (Standardverfahren) interessant sein.

Was md5() da von sich gibt, ist nicht korrekt, da du ja jedes Feld nochmal überprüfen musst, wenn du für jedes Feld einzeln eine Fehlermeldung ausgeben möchtest.

Gruß

fripon77 · 22.12.2009, 21:58

okay, danke euch. euer feedback beschäftigt mich dann schon mal wieder ein zwei tage

@christoph: wenn ich die bereits ausgefüllten felder jedoch ausgefüllt lassen will, muss ich dann die felder nicht einzeln überprüfen? --> du siehts, ich bin einsteiger...
aber im prinzip hast du schon recht: der besucher, der interesse an der berechnung hat, wird sowieso alles ausfüllen.

Wolla · 23.12.2009, 14:30

Der Code ist sehr unstrukturiert. Ein wenig PHP wechselt sich mit HTML-Teilen ab.

Versuch doch mal, zuerst mit PHP alles zu ermitteln und zu nberechnen, was du brauchst, und dann erst HTML auszugeben. Wenn du das hier mal in 6 Wochen ändern willst, suchst du dich blöd.

Nur eine allgemeine Fehlermeldung vom Stil "Sie haben irgendwas vergessen einzugeben" ist Quatsch. Wenn du schon ermittelst, was fehlst, dann kannst du das deinem Besucher auch exakt mitteilen:

PHP-Code:

  $err = array();
if ( $_POST['feld1'] == '' ) $err[] = 'Feld1 nicht ausgefüllt';
if ( $_POST['feld2'] == '' ) $err[] = 'Feld2 nicht ausgefüllt';
if ( $_POST['feld3'] == '' ) $err[] = 'Feld3 nicht ausgefüllt';

if ( count($err) > 0 ) {
  // Fehler festgestellt
  $fehler = explode('<br>',err);
} else {
  $fehler = '';
}

if ($fehler == '') {

  // Speichern
  include 'dankeseite.php';

} else {

  // formular nochmal anzeigen
  include 'formular.php';
}

Das ist Teil des Affenformulars.

fripon77 · 23.12.2009, 17:31

@wolla: danke dir. aber du verwirrst mich etwas, denn ich geb doch exakt an, was fehlt...

PHP-Code:

  echo "<p class='error'>Hinweis: Bitte Feld Darlehensbetrag ausfüllen</p>";

fripon77 · 24.12.2009, 10:41

wollas kritik leuchtet mir ein. ich hab mich jetzt mal an diese vorlage gehalten und alles für mich umgeschrieben...

wie gesagt, ich bin anfänger... ich gehe davon aus, dass ich an dieser stelle meinen berechnungscode einfügen muss:

PHP-Code:

  if (empty($error)) {

         

          /*

           * Hier z.B. alle Daten in der Datenbank abspeichern und darauf hin

           * den Benutzer an eine andere Stelle weiterleiten.

           */

          echo 'STATT ECHO MEINE BERECHNUNG HIER REIN... - ABER WIE???';

         

          // Scriptablauf abbrechen, da alles wichtige erledigt ist

          exit();

Mein Berechnungscode (siehe start-posting) funktioniert an sich fehlerfrei.

Wenn ich ihn aber an die o.g. Stelle einfüge, kommen eine Menge Fehlermeldungen:

Code:

Notice: Undefined index: darlehensbetrag in /Applications/XAMPP/xamppfiles/htdocs/xampp/rechner5.php on line 79

Notice: Undefined index: zinssatz in /Applications/XAMPP/xamppfiles/htdocs/xampp/rechner5.php on line 82

Notice: Undefined index: verrechnung in /Applications/XAMPP/xamppfiles/htdocs/xampp/rechner5.php on line 86

Notice: Undefined index: mtl_rate_inkl_tilgung in /Applications/XAMPP/xamppfiles/htdocs/xampp/rechner5.php on line 88

Warning: Division by zero in /Applications/XAMPP/xamppfiles/htdocs/xampp/rechner5.php on line 92
usw.

--> mir ist einfach nicht klar, wie ich die Berechnung RICHTIG einbinden muss? ausgelagert (include) wäre mir unrecht...

robo47 · 24.12.2009, 11:09

crosspost :P
script besser machen

fripon77 · 24.12.2009, 11:48

@robo47: stimmt, mitglied md5 hatte mich darum gebeten, die frage auch in seinem forum zu stellen... können admins in meinem postfach sicher überprüfen. da frage ich mich dann schon, ob es nicht eher "böse" ist, mitglieder in andere foren zu verleiten... aber okay, ich will hier nicht gegen forenregeln verstoßen und hab den thread dort gelöscht.

wie auch immer - meine letzte frage / antwort auf walla bezieht sich mittlerweile auf eine andere problemstellung.

ragtek · 24.12.2009, 12:04

[offtopic]

Zitat:

Zitat von fripon77

@robo47: stimmt, mitglied md5 hatte mich darum gebeten, die frage auch in seinem forum zu stellen... k

Lol, auch eine Möglichkeit Benutzer in ein Forum zu kriegen, wobei ich echt sagen muss, dass es nicht wirklich nötig ist, ein weiteres PHP Forum zu haben, vorallem wenn es dann noch von einem Amateur geführt wird, der 1. den Inhalt von anderen Seiten besorgt und 1:1 kopiert (ohne Quellenangabe, ist zwar laut Quelle erlaubt, aber ich finde das gehört sich nicht) und 2. keinen blossen Schimmer von php hat (siehe youtube video

)
Jeder hat mal klein angefangen, aber muss man dann gleich ein Forum starten, wo sich benutzer wirklich Hilfe erwarten, aber eher nur falsche Antworten kriegen.
[/offtopic]

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
PHP: Formular Generator	Zergling-new	Tutorials	16	06.07.2010 03:14
Mehrseitiges verzweigtes Formular	RobbyNaish	PHP Tipps 2009	1	19.12.2009 01:51
Wert aus Textfeld in neues Formular übernehmen (kein Submit)	PsychoEagle	HTML, Usability und Barrierefreiheit	9	02.01.2007 14:51
[Erledigt] Formular in Formular		HTML, Usability und Barrierefreiheit	4	19.10.2005 12:18
[Erledigt] PHP Formular ~~wichtig~~		PHP Tipps 2005	27	29.05.2005 19:38
[Erledigt] Im Formular user abfragen?		Datenbanken	3	06.01.2005 10:51
[Erledigt] Vokabelabfrage per Formular / Besser in PHP oder DB ?		PHP Tipps 2004	5	21.08.2004 20:21

22.12.2009, 21:35
fripon77 Neuer Benutzer Registriert seit: 19.12.2009 Beiträge: 18 PHP-Kenntnisse: Anfänger	fehlerfreies formular - aber wie schneller, besser, schlanker? ich hoffe, mein wunsch ist nicht unverschämt: ich würde mich freuen, wenn ein erfahrener php´ler mein folgendes script auf effizienz & sicherheit überprüft - quasi als mentor. das script selbst funktioniert fehlerfrei. ich habe mir das die letzten tage selbst angelesen & erarbeitet - auch dank dieses beitrags (Logarithmus, Zins, Runden...) und eurer unterstützung. um aber zu erkennen, was man besser, schneller, schlanker machen könnte, bedarf es sicherlich viel erfahrung - und das geht in ein paar tagen bestimmt nicht. also wer lust hat, mir tipps zu geben, der kann das script vielleicht mal überfliegen... PHP-Code: <?php if ( $_GET['darlehensbetrag'] == "" OR $_GET['zinssatz'] == "" OR $_GET['mtl_rate_inkl_tilgung'] == "") { echo '<form id="laufzeitrechner" action="rechner.php" method="GET">'; if ( $_GET['darlehensbetrag'] == "" ) { if ( $_GET['schon_mal_abgesendet'] == "ja" ) { echo "<p class='error'>Hinweis: Bitte Feld Darlehensbetrag ausfüllen</p>"; } } echo 'Darlehensbetrag (EUR): '; echo '<input type="text" name="darlehensbetrag" value="'; echo $_GET['darlehensbetrag']; echo '" size="12" maxlength="12" />'; if ( $_GET['zinssatz'] == "" ) { if ( $_GET['schon_mal_abgesendet'] == "ja" ) { echo "<p class='error'>Hinweis: Bitte Feld Zinssatz ausfüllen</p>"; } } echo '<p>Zinssatz (%):'; echo '<input type="text" name="zinssatz" value="'; echo $_GET['zinssatz']; echo '" size="5" maxlength="5" />'; if ( $_GET['mtl_rate_inkl_tilgung'] == "" ) { if ( $_GET['schon_mal_abgesendet'] == "ja" ) { echo "<p class='error'>Hinweis: Bitte Feld Rate inkl. Tilgung ausfüllen</p>"; } } echo '<p>Rate inkl. Tilgung (EUR): '; echo '<input type="text" name="mtl_rate_inkl_tilgung" value="'; echo $_GET['mtl_rate_inkl_tilgung']; echo '" size="9" maxlength="9" />'; echo '<p>Verrechnung: '; echo '<select name="verrechnung"> <option value="12">monatlich</option> <option value="3">vierteljährlich</option> <option value="2">halbjährlich</option> <option value="1">jährlich</option> </select></p>'; echo '<input type="hidden" name="schon_mal_abgesendet" value="ja" />'; echo '<br />'; echo '<input type="Submit" name="" value="Laufzeit berechnen" />'; echo '</form>'; echo "<p>Hinweise: nachschüssige Verrechnung, keine Berücksichtigung von Sondertilgungen</p>"; exit; } else { $_darlehensbetrag = str_replace(".", "",$_GET["darlehensbetrag"]); // punkte löschen $_darlehensbetrag = str_replace(",", ".", $_darlehensbetrag); // komma durch punkt ersetzen $_zinssatz = str_replace(".", "", $_GET['zinssatz']); // punkte löschen $_zinssatz = str_replace(",", ".", $_zinssatz); // komma durch punkte ersetzen $_verrechnung = $_GET["verrechnung"]; $_mtl_rate_inkl_tilgung = str_replace(".", "",$_GET["mtl_rate_inkl_tilgung"]); // punkte löschen $_mtl_rate_inkl_tilgung = str_replace(",", ".", $_mtl_rate_inkl_tilgung); // komma durch punkte ersetzen $wert1 = log(($_zinssatz/100-$_mtl_rate_inkl_tilgung$_verrechnung/$_darlehensbetrag)/ (0-$_mtl_rate_inkl_tilgung$_verrechnung/$_darlehensbetrag)); $wert2 = log(1/(1+$_zinssatz/100/$_verrechnung)); $geslfzt = $wert1/$wert2/$_verrechnung; if ($geslfzt > 0) { echo "Gesamtlaufzeit: ".round($geslfzt,2); // auf zwei nachkommastellen runden - wie ausgabe mit komma??? } else { echo "Ihre Eingaben sind unlogisch und führen zu entsprechend unlogischen Ergebnissen"; } } ?>


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

22.12.2009, 21:40
md5() Erfahrener Benutzer Registriert seit: 10.12.2009 Beiträge: 122 PHP-Kenntnisse: Anfänger	PHP-Code: `if ( $_GET['darlehensbetrag'] == "" ) { if ( $_GET['schon_mal_abgesendet'] == "ja" ) { echo "<p class='error'>Hinweis: Bitte Feld Darlehensbetrag ausfüllen</p>"; } }` darlehensbetrag brauchst du nicht noch einmal uberprufen da du es schon vorher gemacht hast

22.12.2009, 21:49
christophk Benutzer Registriert seit: 22.11.2009 Beiträge: 78 PHP-Kenntnisse: Anfänger	Ich würde gar nicht so einen Aufwand betreiben, alle Felder einzeln zu überprüfen. Ein einfacher Hinweis "mindestens ein Feld wurde nicht ausgefüllt" sollte in diesem Fall reichen, da du ja sowieso alle brauchst und bei drei Feldern noch ein überschaubares Formular hast. Außerdem würde ich das Formular über Post laufen lassen, da du dann kein extra hidden-Feld brauchst, sondern die REQUEST_METHOD aus dem $_SERVER-Array überprüfen kannst. Vielleicht könnte für dich auch der Artikel Affenformular (Standardverfahren) interessant sein. Was md5() da von sich gibt, ist nicht korrekt, da du ja jedes Feld nochmal überprüfen musst, wenn du für jedes Feld einzeln eine Fehlermeldung ausgeben möchtest. Gruß

22.12.2009, 21:58
fripon77 Neuer Benutzer Registriert seit: 19.12.2009 Beiträge: 18 PHP-Kenntnisse: Anfänger	okay, danke euch. euer feedback beschäftigt mich dann schon mal wieder ein zwei tage @christoph: wenn ich die bereits ausgefüllten felder jedoch ausgefüllt lassen will, muss ich dann die felder nicht einzeln überprüfen? --> du siehts, ich bin einsteiger... aber im prinzip hast du schon recht: der besucher, der interesse an der berechnung hat, wird sowieso alles ausfüllen. Geändert von fripon77 (22.12.2009 um 22:02 Uhr).

23.12.2009, 14:30
Wolla Moderator Registriert seit: 06.06.2008 Beiträge: 4.849 PHP-Kenntnisse: Fortgeschritten	Der Code ist sehr unstrukturiert. Ein wenig PHP wechselt sich mit HTML-Teilen ab. Versuch doch mal, zuerst mit PHP alles zu ermitteln und zu nberechnen, was du brauchst, und dann erst HTML auszugeben. Wenn du das hier mal in 6 Wochen ändern willst, suchst du dich blöd. Nur eine allgemeine Fehlermeldung vom Stil "Sie haben irgendwas vergessen einzugeben" ist Quatsch. Wenn du schon ermittelst, was fehlst, dann kannst du das deinem Besucher auch exakt mitteilen: PHP-Code: $err = array(); if ( $_POST['feld1'] == '' ) $err[] = 'Feld1 nicht ausgefüllt'; if ( $_POST['feld2'] == '' ) $err[] = 'Feld2 nicht ausgefüllt'; if ( $_POST['feld3'] == '' ) $err[] = 'Feld3 nicht ausgefüllt'; if ( count($err) > 0 ) { // Fehler festgestellt $fehler = explode('<br>',err); } else { $fehler = ''; } if ($fehler == '') { // Speichern include 'dankeseite.php'; } else { // formular nochmal anzeigen include 'formular.php'; } Das ist Teil des Affenformulars. __________________ Warum denkt mein Hund eigentlich immer dann, wenn es an der Tür klingelt, es sei für ihn?

23.12.2009, 17:31
fripon77 Neuer Benutzer Registriert seit: 19.12.2009 Beiträge: 18 PHP-Kenntnisse: Anfänger	@wolla: danke dir. aber du verwirrst mich etwas, denn ich geb doch exakt an, was fehlt... PHP-Code: `echo "<p class='error'>Hinweis: Bitte Feld Darlehensbetrag ausfüllen</p>";`

24.12.2009, 10:41
fripon77 Neuer Benutzer Registriert seit: 19.12.2009 Beiträge: 18 PHP-Kenntnisse: Anfänger	wollas kritik leuchtet mir ein. ich hab mich jetzt mal an diese vorlage gehalten und alles für mich umgeschrieben... wie gesagt, ich bin anfänger... ich gehe davon aus, dass ich an dieser stelle meinen berechnungscode einfügen muss: PHP-Code: `if (empty($error)) { /* * Hier z.B. alle Daten in der Datenbank abspeichern und darauf hin * den Benutzer an eine andere Stelle weiterleiten. / echo 'STATT ECHO MEINE BERECHNUNG HIER REIN... - ABER WIE???'; // Scriptablauf abbrechen, da alles wichtige erledigt ist exit();` Mein Berechnungscode (siehe start-posting) funktioniert an sich fehlerfrei. Wenn ich ihn aber an die o.g. Stelle einfüge, kommen eine Menge Fehlermeldungen: Code: Notice: Undefined index: darlehensbetrag in /Applications/XAMPP/xamppfiles/htdocs/xampp/rechner5.php on line 79 Notice: Undefined index: zinssatz in /Applications/XAMPP/xamppfiles/htdocs/xampp/rechner5.php on line 82 Notice: Undefined index: verrechnung in /Applications/XAMPP/xamppfiles/htdocs/xampp/rechner5.php on line 86 Notice: Undefined index: mtl_rate_inkl_tilgung in /Applications/XAMPP/xamppfiles/htdocs/xampp/rechner5.php on line 88 Warning: Division by zero in /Applications/XAMPP/xamppfiles/htdocs/xampp/rechner5.php on line 92 usw. --> mir ist einfach nicht klar, wie ich die Berechnung RICHTIG einbinden muss?* ausgelagert (include) wäre mir unrecht...

24.12.2009, 11:09
robo47 Moderator Registriert seit: 03.09.2004 Beiträge: 11.798 PHP-Kenntnisse: Fortgeschritten	crosspost :P script besser machen __________________ robo47.net - Blog, Codeschnipsel und mehr \| Caching-Klassen und Opcode Caches in php \| Robo47 Components - PHP Library extending Zend Framework

24.12.2009, 11:48
fripon77 Neuer Benutzer Registriert seit: 19.12.2009 Beiträge: 18 PHP-Kenntnisse: Anfänger	@robo47: stimmt, mitglied md5 hatte mich darum gebeten, die frage auch in seinem forum zu stellen... können admins in meinem postfach sicher überprüfen. da frage ich mich dann schon, ob es nicht eher "böse" ist, mitglieder in andere foren zu verleiten... aber okay, ich will hier nicht gegen forenregeln verstoßen und hab den thread dort gelöscht. wie auch immer - meine letzte frage / antwort auf walla bezieht sich mittlerweile auf eine andere problemstellung.