Verbesserung Gefragt

Mr.Zoey · 03/27/2013, 00:17

Code:

<html>
<head>
<title>Test</title>
</head>
<body>
<h1>Link Adresse(.php)</h1>
<form method="POST" action="">
<input name="a" type="text"><br>
<input name="submit" type="submit" value="Auslesen">
</form>
<?php
$a = $_POST['a'];
$plus = show_source("$a");
$minus = $a-$b;

?>
<h2>Ergebnis:</h2>
<?php echo $plus; ?>

</body>
</html>

Wollte mal wissen ob ich was besser machen kann ?
Ich bin noch neu im Web-Dev und weiß genau das es schrottig geschrieben ist
aber wo raus muss ich achten und so weiter ^^

PseudoPsycho · 03/27/2013, 10:02

Wenn es das bewirken soll, was ich glaube
(nämlich ein Formular anzeigen, wo man eine Zahl eingibt. Und nach dem Versenden das Ergebnis irgendwelcher Berechnungen anzeigen (wo auch immer $b auf einmal herkommt)), dann rate ich dir dazu, dir ein Buch zu kaufen und das durchzuarbeiten oder wenigstens ein Tutorial. Hier bist du jedenfalls falsch, sry.

Und damit du die Tutorials etwas schneller findest:

Mikesch01 · 03/27/2013, 12:44

PHP Code:



<html>
<head>
<title>Test</title>
</head>
<body>
<h1>Link Adresse(.php)</h1>
<form method="POST" action="">
<input name="a" type="text"><br>
<input name="submit" type="submit" value="Auslesen">
</form>
<?php
$a = $_POST['a']; //zur Sicherheit würde ich hier sämtliche HTML-Sonderzeichen escapen -> htmlentities($_POST['a'])
$plus = show_source($a); //Anführungszeichen müssen nicht gemacht werden, wenn du eine Variable verwendest, die einen String speichert.
$minus = $a-$b; //?! Was soll das sein?^^

?>
<h2>Ergebnis:</h2>
<?php echo $plus; ?>

</body>
</html>

Naja, der Zweck ist natürlich sehr beschränkt und sollte eigentlich so auf keinen Fall auf einen Webspace geladen werden (sonst liest dir jeder dein schönes Script aus

).

Im Großen und Ganzen OK, aber da eigentlich kaum Code vorhanden ist, kann man auch schwer sagen, was du besser machen solltest.

MrPuschel · 03/27/2013, 15:13

Wenn wir schon mal dabei sind, gewöhn dir möglichst schnell an deinen Variablen und Elementen möglichst aussagekräftige Namen zu geben. Am besten in CamelCase Notation. Die Länge ist dabei nicht so wichtig. Das sähe Beispielsweise so aus:

HTML Code:

<html>
<head>
<title>Test</title>
</head>
<body>

<h1>Link Adresse(.php)</h1>
<form method="POST" action="">
<input name="ersteZahl" type="text"><br>
<input name="submit" type="submit" value="Auslesen">
</form>

<?php
$ersteZahl = $_POST['ersteZahl']; 
$ergebnisPlus = show_source($myNumber); 
$ergebnisMinus = $ersteZahl - $zweiteZahl; 
?>

<h2>Ergebnis:</h2>
<?php echo $ergebnisPlus; ?>
</body>
</html>

Das hilft 1. Dir beim programmieren. 2. Anderen wenn du mal hier im Forum Hilfe suchst. Überleg dir ausserdem ob du durchgehend Englische oder Deutsche Variablen und Methoden Namen verwenden möchtest. Möglichst nicht mischen. Hat keinen technischen Grund, ist nur eine Stilnote

Synatex · 03/27/2013, 16:16

Da du hier ja eindeutig mit Zahlen rechnest wäre es für PHP cool zu wissen das es auch wirklich zahlen sind. Dann wird nämlich unnötiger Kram, welcher Sicherheitstechnisch relevant ist, gar nicht erst mit benutzt. Das stellst du an in dem du vor die Variablen ein einfaches (int) packst. Beispiel:

PHP Code:



<?php

$a = (int)$_POST['ersteZahl'];

?>

PseudoPsycho · 03/27/2013, 17:35

Quote:

Originally Posted by Synatex

Das stellst du an in dem du vor die Variablen ein einfaches (int) packst.

Kann zu Problemen führen... Ich persönlich würd' eher $a=intval($_POST['ersteZahl']); schreiben. Bzw. floatval(), wenn's 'ne Fließkommazahl ist.

Synatex · 03/27/2013, 19:39

Welches Problem denn Beispielsweise? Intval() ist ne Funktion zum typecasten und int() ist das direkte Typecasten. Abgesehn davon das man bei Intval() eine Basis angeben kann sind keine Unterschiede vorhanden und es wird in 99,99% der Ressourcen im Web als schneller und effizienter empfohlen. Was spricht also dagegen?

Mikesch01 · 03/27/2013, 19:48

Quote:

Originally Posted by Synatex

... es wird in 99,99% der Ressourcen im Web als schneller und effizienter empfohlen. Was spricht also dagegen?

*zustimm*

Sorry für Off-Topic, musste aber mal geklärt werden^^

MrPuschel · 03/29/2013, 05:11

Ich muss mal die Spaßbremse spielen.

PHP ist eine Sprache die

beherscht. Zwangsweise, denn PHP kennt überhaupt keine explizite Definition. In PHP passieren andauernd on-the-fly-conversions, wenn dies Notwendig ist. PHP prüft anhand der Operanden und der verwendeten Operatoren wie die Operanden evaluiert werden und welcher Typ zurückgegeben wird. Beispiel:integer + float gibt float zurück. Mit dem Cast nehmt ihr PHP diese Arbeit ab.

(Ein cast ist keine Umwandlung, das würdet ihr mit settype() machen. Mit einem cast teilt ihr PHP nur mit wie der Operand behandelt werden soll).

integer + float ist teurer als (float)a + float. Um ein paar Mikrosekunden. Und jetzt fragt euch selbst wie oft ihr schon Anwendungen mit ein paar Millionen Iterationen zur Laufzeit hattet.

Ich kenne kein praktisches Beispiel in dem ein expliziter cast mit PHP notwendig wäre. Es ist richtig das PHP manchmal unerwartete Ergebnisse produziert, der Fehler liegt aber i.d.R. im impliziten Code. Der cast ist also ein workaround. Auch die Lesbarkeit wird von einem (int) cast nicht erhöht. Der Typ einer Variable sollte sich (für den Leser) aus Namen, Kontext und dem PHPDoc ergeben.

Mir ist bewusst das Ihr wisst was ihr tut. Das tut ihr sogar aus den richtigen Gründen, aber bedenkt das PHP nicht C ist und das ein bischen gesunder Menschenverstand nie schadet bevor man sich mit Millisekunden bewirft. Und am wichtigsten, nicht mit casts auf Neulinge zielen ;3

Synatex · 03/29/2013, 09:51

Quote:

Ich kenne kein praktisches Beispiel in dem ein expliziter cast mit PHP notwendig wäre.

Ich hatte es im oberen Beispiel vorgeschlagen um den Wert zu escapen. Wenn man weiß das die Anwendung so oder so mit einem Integer arbeiten soll brauch man sich doch keine unnötige Arbeit machen und Funktionen wie is_int() oder htmlspecialchars() drauf anwenden wenn einem das typecasting die Arbeit abnimmt.

Das mit den Millisekunden und der Verarbeitung kam nur nach der Info von PseudoPsycho hinzu.

PseudoPsycho · 03/29/2013, 12:38

Quote:

Originally Posted by Synatex

Das mit den Millisekunden und der Verarbeitung kam nur nach der Info von PseudoPsycho hinzu.

Jup. Meine, da irgendwo mal gelesen zu haben, dass (int) $x recht fehleranfällig sei. Wahrscheinlich hab' ich das einfach mit $x+=0, oder so, verwechselt.
Naja, auf die Art wieder 'was dazugelernt (auch wenn's auf die Millisekunden nur im Großen und Ganzen überhaupt ankommt).

Jetzt wieder back to topic?