[Tut/Release] MwSt Rechner

Soo =)
Hab dann mal was zum kritisieren <.<
Als erstes das EVA. Wieso mixt du PHP und HTML durcheinander?^^
Lass doch erstmal PHP ran und tu anschließend das HTML Formular rein.

Die Problematik mit dem Punkt und Komma würde außerdem garnicht auftreten würde man das Komma durch den Punkt von PHP ersetzen lassen.

Außerdem gibts die PHP Funktion is_numeric() um zu prüfen ob dass was du ihm gibst wirklich eine Zahl ist. Das erübrigt sich zwar weil PHP sowieso nicht mit Strings rechnen kann ( bzw. kann es schon aber nicht unter diesen umständen) aber des sauberen bzw. logischen Codes wegen sollte man das schon prüfen lassen.
Vllt sollte man noch überprüfen ob die eingegebene Nummer positiv ist oder nicht und dementsprechend Error Messages ausgeben.
Du kannst die auf-die-zu-rundenen Nachkommastellen als Parameter der Funktion round mitgeben z.B. round($randominteger,2) schon rundet er dir das auf 2 Nachkommastellen :)

Wenn ich das Script mal intensiver durchgehen würde, würden mir sicher noch mehr Sachen auffallen. Aber das is halt jetzt das was mir beim einmaligen drüberlesen aufgefallen ist.

Also da gibts noch einiges zu machen und ich würd mich über eine überarbeitete Version freuen ;-)
Wenn du es als tut released hast dann würde ich doch evtl. noch mehr Dinge kommentieren :P

Lg,
Luna

*gg sieht doch schonmal viel besser aus ^.^

Zitat

elseif(!is_numeric($dblbetrag))
{
echo "Bitte geben Sie einen positiven, numerischen Betrag an!";
}

das elseif kannste eigentlich weglassen und am Ende als else dranhängen. Da du ja schon in der eigentlichen If-Abfrage den check auf numeric hast, ist das Teil automatisch false sobald die EIngabe keine numerische ist und er springt sofort zum nächsten else/elseif block. Deshalb einfach ganz am Ende als else dranhängen. Oder noch besser du separierst es. Also entweder machst du aus dieser obigen elseif nur ein else und aus der nächsten elseif:

Zitat

elseif(empty($dblbetrag))
{
echo "Bitte geben Sie einen Betrag ein und klicken auf Abschicken";
}

nur ein if, so werden beide Fehlermeldungen ausgegeben notfalls, wenn aber eine Bedingung True ist dann wird eben nur eine Ausgegeben.

Hau noch das komplette PHP Script oben hin, also alles was PHP is gnadenlos an den Anfang der Datei, erst danach das HTML. ANsonsten Top umgesetzt wie ich`s gesagt habe :P

Lg,
Lunatic

Soo hier mal ne "leicht" verbesserte Version des Mwst Rechners ^.^
Wer will kann sich da ja mal durchschnacken.

<? 


//Kleine Funktion zur Ersetzung eines Punktes in ein Komma
function replacedot($string){
	$return_val = str_replace(".", ",", $string);
	return $return_val;
}


//Kleine Funktion um die Nachkommastellen zu überprüfen
function checkfloats($var) {
	$parts = explode(".",(string)$var);
	if(count($parts)!=2) return 0;
	else return strlen($parts[1]);
}


//Kleine Funktion um zu übeprüfen ob überhaupt etwas vor den Nachkomma stellen steht
function checkprefloats($var){
	 if(preg_match("/[0-9]+\.[0-9]+/",$var) ) return true;
	 else return false;
}


  //Festlegen des MWst Satzes in &
  $mwst = 0.19;


 
  if(isset($_POST['php_submit'])){ //Überprüfen ob der Submit button gedrückt wurde
	  $usr_input = str_replace(",",".",$_POST['betrag']); //Übertragen der Usereingabe in eine PHP Variable
	  $error = array(); //Erstellen eines Error Arrays
	  
	  if(!isset($usr_input) || empty($usr_input)) 
		$error[] = "Bitte gib einen Wert an";
	  if(!is_numeric($usr_input))
		$error[] = "Der eingegebene Wert ist keine Zahl";
	  if(checkfloats($usr_input) > 2)
		$error[] = "Der eingegebene Wert hat mehr als zwei Nachkommastellen";
	  if($usr_input < 0)
		$error[] = "Der eingegebene Wert ist negativ";
	  if(!checkprefloats($usr_input))
		$error[] = "Der eingegebene Wert hat keinen Wert vor dem Komma";
	
	  if (count($error) != 0){ //Wenn das Array das die Errors zählt nicht 0 ist gebe die Errors aus
		foreach($error as $err_val){ //Foreach Schleife zur Ausgabe der Errors
			echo "<span style=\"color: red;\">".$err_val."</span> <br/>";
		}
	  }
	  else //Ansonsten gebe den MWST Satz aus
	  {
		echo "Hallo";
		$no_mwst = $usr_input * (1 - $mwst);
		$mwst_val = $usr_input - $no_mwst;
		


		//Dann Ausgabe mit , und € (19,95€)
		echo "Betrag ohne MwSt: " . replacedot(round($no_mwst, 2)) . "€<br \>";
		echo "MwSt Anteil: " . replacedot(round($mwst_val, 2)) . "€<br \><br \>";
		
	  }
  
  }


?>


<html>
<head>
<title>MwSt Rechner</title>
</head>


<body>




  <!-- Erläuterung zum Umgang mit dem Programm: -->
  <br>
  Beträge nur in Zahlen mit , getrennt,<br>
  z.B. 19,95<br><br>


  <!-- Formular zur Eingabe des Werts: -->


  <form action='' method='post'>
  <input type='text' name='betrag'>&nbsp;
  <input type='submit' value='abschicken' name='php_submit'><br><br>
  </form>


</body>


</html>

Is btw halbwegs getestet ich das war nur ne 5 min Arbeit deshalb hab ich nich viel dran rumgebastelt

Lg
Luna