Kommentare einfügen, p_id = NULL

selection · 28. Oktober 2017

Hallo,

Unter jedem Post-Eintrag kann man Kommentare schreiben. In der Datenbank soll die k_id(Kommentar-ID), p_id(Post-Id), u_id(User-ID), Comment, Datum eingetragen werden.

Jeder Post hat eine p_id. Wenn ein Nutzer ein Kommentar schreibt dann muss man auch wissen unter welchen POST. Jetzt wollte ich die p_id in die Datenbank speichern sobald der Nutzer ein Kommentar eingibt jedoch bekomm ich diese Fehlermeldung: Array ( [0] => 23000 [1] => 1048 [2] => Column 'p_id' cannot be null ) .

Bei paar Post-Einträge klappt es und es steht (string(1) "6") jedoch wird das Kommentar nicht gespeichert obwohl erfolg steht. Bei manchen steht das da oben mit Column p_id cannot be Null.

Bin total verwirrt. Den Code werde ich aufjedenfall noch sauberer schreiben jedoch stört mich grad das Problem mit dem Kommentar.

Hier ein Bild von der Situation:

Spoiler anzeigen

_______________________

Bild von der DB:

Spoiler anzeigen

__________________

Code:

Spoiler anzeigen

PHP

<?php
    if($stmt = $pdo->prepare("SELECT id, username, image, im_type, image_ordner,
    u_post.p_id, u_post.u_id, u_post.p_like, u_post.p_heart, u_post.post, u_post.date,
    comments.p_id, comments.u_id, comments.comment, comments.date_comment
    FROM user JOIN u_post ON u_post.u_id = user.id LEFT JOIN comments ON comments.p_id = u_post.p_id ORDER BY date DESC")) {
        if(!$stmt->execute()) {
            print_r($stmt->errorInfo());
        }
        while($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
    ?>
    <section class="user_post_top_left">
        <ul>
        <li class="imgbox"><?php echo '<img src="data:image/' . $row['im_type'] . ';base64,' . base64_encode($row['image']) . '"/>'; ?></li>
        <li><h2><?php echo $row['username']; ?></h2></li>
        </ul>
    </section>
        
        <section class="user_post_time">
        <p><time><small><?php echo $row['date']; ?></small></time></p>
        </section>
    </section>
    
    <section class="user_post_index">
    <p><?php echo $row['post']; ?></p>
    <p><?php echo '<img src="'.$row['image_ordner'].'">'; ?></p>
    <p>
    <p><?php echo $row['comment']; ?>
    
    <span class="user_set_value_right">

    <button type="button" class="like_set"><i class="fa fa-thumbs-o-up" aria-hidden="true"></i></button><small><a href="#"><?php echo $row['p_like']; ?></a></small> <button type="button" class="heart_set"><i class="fa fa-heart-o" aria-hidden="true"></i></button><small><a href="#"><?php echo $row['p_heart']; ?></a></small>
    </span>
    
    <span class="user_set_value_left">
    <form action="" method="POST">
    <?php
    if(isset($_POST['show'])) {
        $comment = $_POST['comment_msg'];
        if($insert = $pdo->prepare("INSERT INTO comments (p_id, u_id, comment) VALUES (?, ?, ?)")) {
            $insert->BindValue(1, $row['p_id']);
            $insert->BindValue(2, $_SESSION['id']);
            $insert->BindValue(3, $comment);
            
            if(!$insert->execute()) {
                print_r($insert->errorInfo());
            } else {
                echo 'erfolg';
            }
            
        }
        
    }
    ?>
    <input type="text" name="comment_msg">
    <input type="submit" name="show" value="Show">
    </span>
    </section>
    <?php
        }
    }
    ?>

Alles anzeigen

cottton · 29. Oktober 2017

Du hast keine p_id in der $row.

Aber Du hast eine u_post.p_id.

PHP

$pdo->prepare("

  SELECT 
    id, 
    username, 
    image, 
    im_type, 
    image_ordner, 
    u_post.p_id,  -- <-- HIER kannst du einen alias verwenden: u_post.p_id AS p_id,
    u_post.u_id, 
    u_post.p_like, 
    u_post.p_heart, 
    u_post.post, 
    u_post.date,

   comments.p_id, 
   comments.u_id, 
   comments.comment, 
   comments.date_comment

   FROM user 
   JOIN u_post ON u_post.u_id = user.id 
   LEFT JOIN comments ON comments.p_id = u_post.p_id 
   ORDER BY date DESC")

)

Alles anzeigen

BTW:

Daten die ausgegeben werden immer maskieren.

Auch, wenn sie aus Deiner db sind.

zB

PHP

<li><h2><?php echo htmlspecialchars($row['username']); ?></h2></li>

Auch ids und was nicht alles.

Was ausgegeben wird, wird maskiert.

selection · 29. Oktober 2017

cottton Aber was muss ich den jetzt verbessern bzw ändern am Code das es funktoniert? Und wieso steht bei POST eine Fehlermeldung? Eig müsste nur die Fehlermeldung kommen von einem Post. Bild ist oben vorhanden wenn du weißt was ich meine. Danke erstmal für die schnelle Antwort

cottton · 29. Oktober 2017

Siehe Code - in der SQL Query der Kommentar: u_post.p_id AS p_id

selection · 30. Oktober 2017

Habe ich ja nur zeigt er mir das gleiche an.

Bei einem zeigt er mir p_id cant null , bei dem andern u_id. Bin total verwirrt.

Spoiler anzeigen

cottton · 30. Oktober 2017

Die values sind nicht vorhanden, die Du binden willst.

Mach mal var_dump() auf die row p_id und die session id:

PHP

if(isset($_POST['show'])) {
        $comment = $_POST['comment_msg'];
        if($insert = $pdo->prepare("INSERT INTO comments (p_id, u_id, comment) VALUES (?, ?, ?)")) {
            var_dump($row['p_id']);
            var_dump($_SESSION['id']);
            $insert->BindValue(1, $row['p_id']);
            $insert->BindValue(2, $_SESSION['id']);
            $insert->BindValue(3, $comment);
            
            if(!$insert->execute()) {
                print_r($insert->errorInfo());
            } else {
                echo 'erfolg';
            }
            
        }
        
    }

Alles anzeigen

Ich verstehe allerdings auch nicht, warum es ein INSERT INTO bei einem Button "Show" gibt.

Evtl wolltest Du dort ein SELECT ?

selection · 30. Oktober 2017

Aber wieso zeigt er mir bei session_id = NULL an? Ich habe ebenfalls bei einem anderen Skript in der gleichen Datei eine INSERT abfrage wo auch $_SESION['id'] eingefügt werden soll in der DB da klappt es.

Um die Kommentare anzeigen zu lassen habe ich schon eine SELECT abfrage. Jetzt muss ich halt nur noch erstellen das jemand ein Kommentar schreiben kann. Keine Ahnung wieso der Button Show heißt

Stef · 1. November 2017

Hey,

warum willst du die Session "Id" abspeichern ?

Ein User hat bei mir in der Datenbank die Spalten: id, username, password, birthday, age, right, isOnline.

Wenn der User sich registriert wird eine neue Zeile erstellt, die alle Spalten befüllt. Da hast du doch die id des jenigen schon. Da selektierst du dann einfach die id, username, age von dem User welcher sich grad anmeldet. Bei der Anmeldung erstellst du dann 2 Sessions welche dann die, aus der DB gezogenen Daten, erhalten:

ID , -> Id aus der db des users
Access, -> der gibts du dann den boolean Wert true

Die Id nimmst du dann um zu erkennen wer was gepostet hat und für deren ihre Profilseite und Access verwendest du zur überprüfung ob dies vorhanden ist und zum zerstören der Session beim logout.

selection · 1. November 2017

Da hast du recht. Das wird geändert. Ich verstehe aber nicht wieso ich nichts kommentieren kann. Wenn ich etwas kommentiere möchte dann gibt er mir string(0) aus. .

Stef · 2. November 2017

Ok.

Zeige mal bitte nochmals deinen aktuell kompletten Code her.

Wir können es auch wenn du willst per Teamviewer mal anschauen. Wenn du es willst einfach ne PN senden.

Kommentare einfügen, p_id = NULL

Jetzt mitmachen!

Teilen