[Erledigt] Das Script funktioniert irgendwie nicht.

[Forumsgast]

Hallo,

ich habe folgendes Problem:

Wenn sich in dem Datenbank kein Parameter D befindet, dann die Parametern A, B, C, D eintragen und gleichzeitig alle Parametern D zurückgeben, bei denen A=A und B=B und C nicht=C
Außerdem wenn E angeklickt ist, dann muss Ergebnis rauskommen, wo C=C.

P.S: mich würde außerdem interessieren, wie man anstatt von ECHO, HTML-Seiten ausgibt.

<?php
$verbindung = mysql_connect("localhost", "user" , "passwort") 
or die("There are no conection to database"); 

mysql_select_db("db") or die ("Database couldn't be find"); 

$A = $_POST ["A"];
$B = $_POST ["B"];
$C = $_POST ["C"];
$D = $_POST ["D"];
$E = $_POST ["E"];

if($A == "" OR $B== "" OR $C == "" OR $D== "")
{ 
    echo "Please, fill in all fields"; 
    exit; 
}

$result = "SELECT D FROM tb WHERE D LIKE '$D'";
$menge = mysql_query($result); 

if($menge == 0) 
{
$input  = "INSERT INTO tb (A, B, C, D, E) VALUES
('$A', '$B', '$C', '$D' '$E')";
$eintragen = mysql_query($input); 

    if($eintragen == true) 
    {
$result = "SELECT D FROM tb WHERE A LIKE '$A' B LIKE '$B' ";
if($E==ja) 
{
$result .= "AND C LIKE '$C' "
}
else
{
$result .= "AND C NOT LIKE '$C'"
}
$output = mysql_query($result);
echo "$output";
    }
    if($output== 0);
    {
    echo "Sorry, nothing was found";
    }
}
else
{
echo "Sorry, parameter already exist";
}
?>

Danke im Voraus.

[Forumsgast]

Das Script funktioniert irgendwie nicht.

Das Betiteln deines Threads irgendwie auch nicht...

echo ist eine geeignete Direktive um Ausgaben zu tätigen - was genau möchtest du wissen? Es gibt Template-Engines, geht die Frage in diese Richtung?

[Forumsgast]

Dir fehlen eine ganze Menge Basics und dein Quellcode ist voller Sicherheitslücken, außerdem verwendet dein Script eine Veraltete MySQL API.

Insofern du dir das nicht alles selbst beibringen willst Rate ich dazu das ganze in die Scriptbörse als Auftragsarbeit zu stellen.

[Forumsgast]

Zunächst sei Dir gesagt, daß mysql_* in nächsten Updates nicht mehr verfügbar sein wird.
Die Alternative stellen MySQLi und PDO dar.

Dann sind Deine Abfragen fehlerhaft:

$result = "SELECT D FROM tb WHERE A LIKE '$A' B LIKE '$B' "; 

Da fehlt mind. ein AND!

Und dann muß ich zugeben, weiß ich nicht genau, was Du erreichen willst?!
Vielleicht habe ich es einfach nur nicht verstanden und Du erklärst es mir evtl. nochmal?

Und:

P.S: mich würde außerdem interessieren, wie man anstatt von ECHO, HTML-Seiten ausgibt.

Einfach Das HTML ins echo ballern...

[Forumsgast]

Warum prüfst du nicht auf Fehler die von der mysql_ Schnittstelle zurpckkommen? Kein einziges mal fragst du ab, ob es zB einen mysql_error gegeben hat etc...

Das die bald nicht mehr funktioniert und du grobe Sicherheitslücken hast wurde schon gesagt. Hier noch Lesestoff zu deinen Sicherheitslücken:
http://php-de.github.io/jumpto/sql-injection/

Und dann noch:

$A = $_POST ["A"];
$B = $_POST ["B"];
$C = $_POST ["C"];
$D = $_POST ["D"];
$E = $_POST ["E"]; 

da hagelt es noch Notices... aktiviere mal error_reporting, schreib das ganz zu beginn nach dem <?php in dein Script, und behebe auch diese Meldungen.

error_reporting(-1);
ini_set('display_errors', 1); 

Dann noch:

http://php-de.github.io/jumpto/code-...n-doppelquotes

echo "$output"; 

Und INfo am Rande:

if($eintragen == true) 

macht / ist das selbe wie das:

if($eintragen) 

.. denn der Ausdruck in der Klammern wird immer zu true oder false evaluiert - siehe auch: http://php-de.github.io/jumpto/if/


Darf man fragen wo du den schrecklichen Code her hast? Und bitte die PHP Code Tags verwenden für PHP-Code statt der CODE.

[Forumsgast]

Das Betiteln deines Threads irgendwie auch nicht...

echo ist eine geeignete Direktive um Ausgaben zu tätigen - was genau möchtest du wissen? Es gibt Template-Engines, geht die Frage in diese Richtung?

Ja, mich würde schon interessieren die Ausgabe in Templates, wenn das nicht zu kompliziert ist.

[Forumsgast]

Insofern du dir das nicht alles selbst beibringen willst Rate ich dazu das ganze in die Scriptbörse als Auftragsarbeit zu stellen.

hmm.. was kann sowas kosten?

Dir fehlen eine ganze Menge Basics und dein Quellcode ist voller Sicherheitslücken, außerdem verwendet dein Script eine Veraltete MySQL API.

was sind das Sicherheitslücken z.B.?

[Forumsgast]

Und dann muß ich zugeben, weiß ich nicht genau, was Du erreichen willst?!
Vielleicht habe ich es einfach nur nicht verstanden und Du erklärst es mir evtl. nochmal?

Im Grunde genommen, soll das Script ein Parameter abgleichen, wenn keine Redundanz vorhanden, dann die Daten, die bei diesem Parameter dabei sind speichern und die andere Daten ausgeben, die den gespeicherten Daten gleich sind?

ist es so verständlich?

[Forumsgast]

Darf man fragen wo du den schrecklichen Code her hast?

Ich habe den Code aus verschiedenen Skripten irgendwie selber zusammengebastelt.

[Forumsgast]

was sind das Sicherheitslücken z.B.?

Hier noch Lesestoff zu deinen Sicherheitslücken:
http://php-de.github.io/jumpto/sql-injection/

[Forumsgast]

Ich verkürzte das Script um auf den Fehler zu kommen, funktioniert trotzdem nicht. Und zwar anstatt die Daten in die Datenbank einzutragen, gibt es echo Ausgabe: "Sorry, this already exists in the system."

Ich denke, dass Jemand, der sich mit PHP auskennt, findet den Fehler sofort. Für mich wäre das aber supergroße Hilfe.

P.S. Mit Sicherheitslücken beschäftige ich mich, wenn grundsätzliches gelöst ist. Also vielen Dank im Voraus.

<?php
$verbindung = mysql_connect("localhost", "project" , "pass") 
or die("There are no conection to database"); 

mysql_select_db("database") or die ("Database couldn't be find"); 

$a = $_POST ["a"];
$b = $_POST ["b"];
$c= $_POST ["c"];
$d = $_POST ["d"];
$e = $_POST ["e"];

if($a == "" OR $b == "" OR $e== "")
{ 
    echo "Please, fill in all fields"; 
    exit; 
}

$result = "SELECT e FROM tabele WHERE e LIKE '$e'";
$menge = mysql_query($result); 

if($menge == 0) 
             {
$input  = "INSERT INTO tabele (a, b, c, d, e) VALUES
('$a', '$b', '$c', '$d', '$e)";
$eintragen = mysql_query($input); 

}
else
{
echo "Sorry, this already exists in the system.";
}
?>

[Forumsgast]

Ja, mich würde schon interessieren die Ausgabe in Templates, wenn das nicht zu kompliziert ist.

Es gibt verschiedene fertige Template-Engines. Sehr große und mächtige wie z. B. Twig aber auch kleinere wie Plates uvm. Plates in einer Minute: http://platesphp.com/simple-example/

[Forumsgast]

Ich denke, dass Jemand, der sich mit PHP auskennt, findet den Fehler sofort. Für mich wäre das aber supergroße Hilfe.

Eine große Hilfe wäre, wenn Du anfangen würdest mit debuggen:

1. Du bekommst die Meldung aus dem ELSE-Zweig
2. Dann wird wohl die Bedingung für IF fehlschlagen
3. Prüfen, was $menge beinhaltet ( var_dump )

und immer so weiter, bis Du den Fehler eingegrenzt bzw. gefunden hast.
Wir können nur mit Sicherheit sagen, daß in $menge nicht das drin steht, was Du erwartest, denn sonst käme die Meldung nicht!


ps: zum debugging gehört übrigens auch eine gescheite Fehler-"Analyse":

error_reporting ( -1 );
ini_set ( 'display_errors', true ); 

gepaart mit mysql_error, wenn Du dies beratungsresitent weiter nutzen möchtest.

[Forumsgast]

Bitte sag doch kurz, was $_POST["e"] für ein wert enthaltet/erwartet.

(obwohl hier noch nichts passiert, solange $menge nicht 0 ist) Prüfe bitte noch die Hochkommas in den VALUES.

..und lerne einzurücken http://tut.php-quake.net/de/indent.html

<?php
$verbindung = mysql_connect("localhost", "project" , "pass") 
or die("There are no conection to database"); 
mysql_select_db("database") or die ("Database couldn't be find"); 

$a = $_POST["a"];
$b = $_POST["b"];
$c = $_POST["c"];
$d = $_POST["d"];
$e = $_POST["e"];

if ($a == "" OR $b == "" OR $e== "") { 
    echo "Please, fill in all fields"; 
    exit; 
}

$result = "SELECT e FROM tabele WHERE e LIKE '$e'";
$menge = mysql_query($result); 

if ($menge == 0) {
    $input  = "INSERT INTO tabele (a, b, c, d, e) VALUES ('$a', '$b', '$c', '$d', '$e)";
    $eintragen = mysql_query($input); 
} else {
    echo "Sorry, this already exists in the system.";
}
?>

[Forumsgast]

Ich denke, das Problem liegt in MySQL. Was soll ich da beachten, wenn es sich um ein einfaches Textfeld handelt?