PHP - Nachrichtensystem

IngamPlay · 26. Mai 2014

Guten Abend!
ich sitze nun seit rund zwei Stunden daran ein Nachrichtensystem zu schreiben und zu entwickeln. Über diesem YouTube Link wollte ich es lernen, sowie mit meiner Datenbank etc. anwenden. Obwohl ich nach mehrfachem durchschauen alles 1:1 so habe, wie in dem Video, möchte die php einfach keinen Eintrag in meine Datenbank machen. ebenfalls habe ich diese 1:1 so erstellt wie im Video. Da dies auf dauer sehr lange an meinen Nerven zieht, bin ich in der Hoffnung das mir hier eventuell einer weiter helfen könnte!

PHP

<?phpsession_start();
if(isset($_SESSION["username"])) {
?>
<?php
} else {
?>
<?php
}
?>
<?php
$verbindung = mysql_connect("*****", "*****", "*****")
or die ("Fehler im System");




mysql_select_db("*****")
or die ("Verbindung zur Datenbank war nicht möglich");




?>  
<?php
    if(isset($_POST['send']) && str_replace(" ", "", $_POST['messageText']) != "")
    {
        $message = mysql_real_escape_string($_POST['messageText']);
        $sentTo = mysql_real_escape_string($_GET['goesto']);
        $dateTime = date("Y-m-d H:i:s");
        $insertDate = mysql_query("INSERT INTO messageService VALUES('', {$_SESSION['userid']}, {$sentTo}, '{$message}', '{$dateTime}', 0)");
    }
?>







<form name="sendMessageForm" action="<?php echo $_SERVER['PHP_SELF']; ?>?goesto=<?php echo $_GET['goesto']; ?>" method="post">
<textarea name="messageText" style="width:100%; height:80px;"></textarea>
<input type="submit" value="Senden" name="send"/>
</form>
<?php
mysql_close($verbindung); ?>

Alles anzeigen

wolf · 26. Mai 2014

Du solltest deinen Quellcode etwas aufgeräumter haben und verstehen
auserdem hilft php error reporting oft

--ich arbeite gerade an deinem Code

IngamPlay · 26. Mai 2014

Die Ordnung mit dem Quellcode ist da leider nicht so meins! Meine anderen Dateien sehen leider auch nicht besser aus ... müsste ich mich mal ran setzen.
Php error reporting? habe ich noch nie etwas von gehört! Bin auch gerade mal seit rund 3 - 4 Tagen in PHP, also noch ein sehr neuer! Aber danke für die Unterstützung

wolf · 26. Mai 2014

So hier mal dein Code...

Deine Tabelle sollte so aussehen

Code

+------------+------------+----------+------------+-------+
|  MsgId     |  UserName  |  Ziel    | Nachricht  | Zeit  |
+------------+------------+----------+------------+-------+
| Auto Incre |  varchar   |  varchar | varchar    | date  |
| PrimaryKey |  (50)      |  (50)    | (10000)    |       |
+------------+------------+----------+------------+-------+

Und dein PHP (ungetestet aber so ungefähr sollte es )

PHP

<?php




    $DB_Server      = "localhost";
    $DB_Benutzer    = "root";
    $DB_Passwort    = "****";
    $DB_Datenbank   = "nachrichten";




    //Alle Fehler zeigen
    error_reporting(E_ALL);




    //Wenn der Send Button gedrückt wurde
    if (isset($_POST['send'])){
        session_start();
        // Wenn der Username gesetzt ist
        if(isset($_SESSION["username"])) {
            //wenn du keine db verbindung herstellen kannst
            if(!$DB_Verbindung = mysql_connect($DB_Server, $DB_Benutzer, $DB_Passwort)){
                die("Fehler beim Verbinden");
            }
            //Wenn es eine Verbindung gibt (könnte man sich auch das die sparen)
            else{
                //Wenn die Datenbank nicht ausgewählt werden konnte
                if(!mysql_select_db($DB_Datenbank)){
                    die("Fehler mit der Datenbank");
                }
                //Wenn eine Verbindung besteht (könnte man sich auch das die sparen)
                else{
                    $nachricht = mysql_real_escape_string(trim($_POST['nachricht']));
                    $ziel      = mysql_real_escape_string(trim($_POST['empfänger']));
                    $benutzer  = mysql_real_escape_string($_SESSION['username']); // --Sicherheits Bug behoben 
                    $zeit      = date("Y-m-d H:i:s");
                    $sql       = "INSERT INTO `messageService`
                                    VALUES(
                                        '',
                                        `$benutzer`,
                                        `$ziel`,
                                        `$nachricht`,
                                        `zeit`
                                    );
                                ";
                    //Wenn es nicht geschrieben werden konnte
                    if (!mysql_query($sql)) {
                        die("Konnte nicht eingetragen werden");
                    }   
                    //Wenn es drinnsteht             
                    else{
                        echo "Wurde geschrieben";
                    }
                    mysql_close($DB_Verbindung);
                }
            }
        }
        //sonst
        else {
            echo "Nicht angemeldet";
        }
    }
?>
<form method="POST" action="<?php echo($_SERVER['PHP_SELF']); ?>">
    <label>Empfänger</label>
    <select name="empfänger">
        <option>Username1</option>
        <option>Username2</option>
    </select>




    <label>Nachricht</label>
    <textarea name="nachricht"></textarea>




    <input type="submit" name="send" value="Senden">
</form>

Alles anzeigen

cottton · 27. Mai 2014

$benutzer = mysql_real_escape_string($_SESSION['username']);

=)

wolf · 27. Mai 2014

Ich hab es extra noch gelöscht -- aus dem Grund, dass eine Session var nicht veränderbar ist (vom User) und joa..

cottton · 27. Mai 2014

Aber von irgendwo muss ja der Wert her kommen, der in $_SESSION['username'] abgelegt wird.
Sicherlich aus $_POST[...]. Und dann wirds gefährlich =)

wolf · 27. Mai 2014

Ich habe drauf gezählt das der wert aus der db kommt :0

cottton · 27. Mai 2014

Kann man sich aber nicht darauf verlassen =)

IngamPlay
ich habe mir gerade mal paar Ausschnitte des Videos angesehen. Ich empfehle das alles wieder zu vergessen und das hier zu nutzen: http://www.php-kurs.com/
(Auch auf Youtube zu finden: https://www.youtube.com/channel/UC5Sax3MmkSyJXkGqRw2EXtw)

Warum?
Im (von Dir geposteten) Video wird "konischer Kram" gezeigt.
zB
- int(255) - macht keinen Sinn ( => int, bigint, mediumint, ... benötigen und ignorieren schlicht weg diese Werte)
- Tabellenfeld (column) "fromUser" ist eine schlechte Angewohnheit. Sieht evtl jeder anders. Aber man sollte keine Großbuchstaben verwenden.

Da ist noch viel mehr. Fängt bei der Wortwahl and ("... einfach mal ..." - so lern man nix Handfestes) ... aber ich will ja keinen Aufsatz schreiben ... =)
Und alles hier aufzuklären nimmt auch zu viel zeit in Anspruch.

Bei Fragen immer Fragen.
Du bist hier glücklicherweise in einem Forum gelandet in dem kein Quark oder "Gelegenheits-Fakten" (gibts das?) erzählt werden.
=)

IngamPlay · 27. Mai 2014

wolf
Erstmal vielen Dank für deine Bemühungen! Hat auch fast alles geklappt. Meine Daten eingegeben und selber ein wenig angepasst mit Login etc. Nun ist jedoch das Problem das ich ebenfalls nichts in die DB eintragen kann. Womöglich liegt es an dem "i" bei mysql. Wurde im Fehlercode ja beschrieben.
---------------------------------------------------------------
Deprecated: mysql_connect() [function.mysql-connect]: The mysql extension is deprecated and will be removed in the future: use mysqli or PDO instead in /users/metrostorm/www/Messanger/messages.php on line 17
Konnte nicht eingetragen werden

---------------------------------------------------------------

Weißt du eventuell wie man dann mysqli verwendet? Ich habe einfach mal ganz billig ein "i" hinter mysql geschrieben, aber wer hätte es gedacht, es klappt nicht.

wolf · 27. Mai 2014

doch, genau richtig

PHP

<?php




    $DB_Server      = "localhost";
    $DB_Benutzer    = "root";
    $DB_Passwort    = "****";
    $DB_Datenbank   = "nachrichten";




    //Alle Fehler zeigen
    error_reporting(E_ALL);




    //Wenn der Send Button gedrückt wurde
    if (isset($_POST['send'])){
        session_start();
        // Wenn der Username gesetzt ist
        if(isset($_SESSION["username"])) {
            //wenn du keine db verbindung herstellen kannst
            if(!$DB_Verbindung = mysqli_connect($DB_Server, $DB_Benutzer, $DB_Passwort)){
                die("Fehler beim Verbinden");
            }
            //Wenn es eine Verbindung gibt (könnte man sich auch das die sparen)
            else{
                //Wenn die Datenbank nicht ausgewählt werden konnte
                if(!mysqli_select_db($DB_Datenbank)){
                    die("Fehler mit der Datenbank");
                }
                //Wenn eine Verbindung besteht (könnte man sich auch das die sparen)
                else{
                    $nachricht = mysqli_real_escape_string(trim($_POST['nachricht']));
                    $ziel      = mysqli_real_escape_string(trim($_POST['empfänger']));
                    $benutzer  = mysqli_real_escape_string($_SESSION['username']); // --Sicherheits Bug behoben 
                    $zeit      = date("Y-m-d H:i:s");
                    $sql       = "INSERT INTO `messageService`
                                    VALUES(
                                        '',
                                        `$benutzer`,
                                        `$ziel`,
                                        `$nachricht`,
                                        `zeit`
                                    );
                                ";
                    //Wenn es nicht geschrieben werden konnte
                    if (!mysqli_query($sql)) {
                        die("Konnte nicht eingetragen werden");
                    }   
                    //Wenn es drinnsteht             
                    else{
                        echo "Wurde geschrieben";
                    }
                    mysqli_close($DB_Verbindung);
                }
            }
        }
        //sonst
        else {
            echo "Nicht angemeldet";
        }
    }
?>

Alles anzeigen

Allerdings solltest du dir überlegen auf eine neue Art von DB verbindung via PDO umzusteigen...

Hier ein PDO verbindungs beispiel ..

PHP

<?php
error_reporting(E_ALL);




$DB['server']   = "localhost";
$DB['user']     = "root";
$DB['password'] = "++++";
$DB['database'] = "Datenbank";




//versuche
try {
    // Eine verbindung aufzubauen, die bestehen bleibt
    $DB['conect'] = new PDO('mysql:host='.$DB['server'].';dbname='.$DB['database'], $DB['user'], $DB['password'],array(PDO::ATTR_PERSISTENT => true));
}
//wenn nicht geht
catch(PDOException $e) {
    //mache eine Fehlermeldung
    generate_connect_ErrorMessage($e,__LINE__,__FILE__);
}#catch




// Das SQL querry in eine Variable schreiben
$SQL['NaSchr']['query'] = "INSERT INTO `loginattemps` (
                                `UserName`,
                                `Ziel`,
                                `Nachricht`,
                                `Zeit`
                            )
                            VALUES (
                                :username,
                                :target,
                                :message,
                                :time
                            );
                        ";
//Das querry preparen
$SQL['NaSchr']['stmt'] = $DB['conect']->prepare($SQL['NaSchr']['query']);
if (
    //Das ganze ausführen und die inhalte übergeben
    !$SQL['NaSchr']['stmt']->execute(
        array(
            ':username' => $_SESSION['username'],
            ':target' => trim($_POST['empfänger']),
            ':message' => trim($_POST['nachricht']),
            ':time' => date("Y-m-d H:i:s")
        )
    )
){
    //Wenn nicht geschrieben werden konnte
    generate_query_ErrorMessage($SQL['NaSchr']['stmt']->errorInfo(),__LINE__,__FILE__,$SQL['NaSchr']['query']);
}
else{
    //Wenn geschrieben wurde
    /* //Weis nicht ob das hier geht.. :D
    while($SQL['NaSchr']['fetch'] = $SQL['NaSchr']['stmt']->fetch(PDO::FETCH_ASSOC)){
        foreach ($SQL['NaSchr']['fetch'] as $key => $value) {
            echo $key." => ".$value."<br>";
        }
    }
    */
    echo "Geschrieben";
}
//am Ende alle evtl vorhandenen Meldungen anzeigen lassen
sqlError();










/*
 * Der vorteil an den langen variablen namen ...
 * Wenn du jetzt ganz viele solche SQL's ausführst
 * Kannst du dir Alle komplett anzeigen lassen:
 */
 /*
 echo "<pre>";
 print_r($SQL);
 echo "</pre>" 
*/
 //und dann siehst du alle Info's













//Hier noch die Funktionen fürs SQL error reporting
/**
 * @param   $e    System Error
 * @param   $l    Konstant Line of catch
 * @param   $f    Konstant File of catch
 * @param   $q    Failed query
 */
function generate_query_ErrorMessage($e,$l,$f,$q){
    global $DB;
    //Wenn keine System Error Message gegeben ist, sondern nur der error code
    if(is_numeric($e[2])){
        $e[2] = error_get_last_str($e[2]);
    }
    $DB['status'] = 1;
    $DB['message'][] = $e[2]." -- Code : ".$e[0]." -- in ".$_SERVER['SERVER_NAME']." $f on line $l. -- Query: $q";




}




/**
 * @param   $e    System Error
 * @param   $l    Konstant Line of catch
 * @param   $f    Konstant File of catch
 */
function generate_connect_ErrorMessage($e,$l,$f){
    global $DB;
    $DB['status'] = 1;
    $DB['message'][] = $e->getMessage()." in ".$_SERVER['SERVER_NAME']." $f on Line $l";
}
function sqlError(){
    global $SYS;
    if(isset($DB['status']) && $DB['status'] == 1) {
        $i = 0;
        foreach ($DB['message'] as $DB['message']['single']) {
            $i++;
            $DB['message']['single'] = str_replace(" for user ", ' ', $DB['message']['single']);
            $DB['message']['single'] = str_replace("'root'", '', $DB['message']['single']);
            echo "<pre>".$i." -> ".$DB['message']['single']."</pre>";
        }
    }
}
/**
 * @param   $type   is a Error Code
 */ 
function error_get_last_str($type){
    switch($type){
        case E_ERROR: // 1 //
            return 'E_ERROR';
        case E_WARNING: // 2 //
            return 'E_WARNING';
        case E_PARSE: // 4 //
            return 'E_PARSE';
        case E_NOTICE: // 8 //
            return 'E_NOTICE';
        case E_CORE_ERROR: // 16 //
            return 'E_CORE_ERROR';
        case E_CORE_WARNING: // 32 //
            return 'E_CORE_WARNING';
        case E_CORE_ERROR: // 64 //
            return 'E_COMPILE_ERROR';
        case E_CORE_WARNING: // 128 //
            return 'E_COMPILE_WARNING';
        case E_USER_ERROR: // 256 //
            return 'E_USER_ERROR';
        case E_USER_WARNING: // 512 //
            return 'E_USER_WARNING';
        case E_USER_NOTICE: // 1024 //
            return 'E_USER_NOTICE';
        case E_STRICT: // 2048 //
            return 'E_STRICT';
        case E_RECOVERABLE_ERROR: // 4096 //
            return 'E_RECOVERABLE_ERROR';
        case E_DEPRECATED: // 8192 //
            return 'E_DEPRECATED';
        case E_USER_DEPRECATED: // 16384 //
            return 'E_USER_DEPRECATED';
        default:
            return 'Undefined Error';
    }#switch
}
?>

Alles anzeigen

IngamPlay · 29. Mai 2014

wolf
Danke für den Ratschlag, ich werde mir mal überlegen um zu steigen. Vorerst wollte ich jedoch meine eigenen Fehler korregieren und auch verstehen.

Nun habe ich mit mysqli nur ein kleines Problem, Die Fehlermeldung ist nur das die Variable $DB_Datenbank nicht ausreicht. Wenn ich die Daten extrern ohne Variable eingebe wird mir beispielsweise gesagt das ich zu viele Daten angegeben habe.

--------------------------------------------------------------------

Hier der Fehlercode:
Warning: mysqli_select_db() expects exactly 2 parameters, 1 given in /users/metrostorm/www/Messanger/messages.php on line 23
Fehler mit der Datenbank

--------------------------------------------------------------------

Wüsstest du was man noch angeben muss? Ich dachte nochmal den Datenbank namen oder die Tabelle nochmals angeben. Aber bisher war keins davon richtig. Ebenfalls bringt es leider nichts "null" zu schreiben.

cottton · 29. Mai 2014

PHP

mysqli_select_db($link, "world"); # also $link ist die verbindung, und "world" die db

bei "Example#2 Procedural style" => http://php.undmedlibrary.org/m…tion.mysqli-select-db.php

PHP - Nachrichtensystem

Jetzt mitmachen!

Teilen