Ein zusätzlicher Sql-Datensatz wird jedes Mal hinzugefügt

Question

Ich arbeite an einer Chat-Web-Anwendung. Das Problem ist, dass jedes Mal, wenn die Seite geladen wird, eine leere Nachricht in die Datenbank eingefügt wird.

Der Code ist wie unten

  <!-- Insert MySQL datbase into HTML  --> 
<?php 
$connection = mysqli_connect("localhost", "root", "", "tru"); 
$query = "SELECT * FROM shouts ORDER BY id Desc LIMIT 8"; 
$shouts = mysqli_query($connection, $query); 
?> 



        <!-- Insert MySQL datbase into HTML  --> 
    <?php while ($row = mysqli_fetch_assoc($shouts)) : ?> 
     <li> <?php echo $row['shout']; ?> <b>&nbspSent at&nbsp</b><?php echo $row['Time']; ?></li> 
      <?php endwhile; ?> 

    </ul> 
    </div> 
    <footer> 
     <form action="index.php" method="post"> 
     <label>Shout Text: </label> 
     <input type="text" name="shout" placeholder="Enter your message here"> 
     <input type="submit" id="submit" value="SHOUT!" > 
    </form> 
     <?php 
        <!-- Insert into MySQL datbase  --> 

$link = mysqli_connect("localhost", "root", "", "tru"); 
$sql = "INSERT INTO shouts (name,shout) VALUES ('$_POST[name]','$_POST[shout]')"; 
if(mysqli_query($link, $sql)){ 
echo "Records added successfully."; 
} else 
{ 
    echo "ERROR: Could not able to execute $sql. " . mysqli_error($link); 
} 

// close connection 
mysqli_close($link); 
?> 
</footer> 
</div> 
</body> 
</html> 

Answer 1

<?php 

if(isset($_POST["name"])){ 
    $link = mysqli_connect("localhost", "root", "", "tru"); 
    $sql = "INSERT INTO shouts (name,shout) VALUES ('$_POST[name]','$_POST[shout]')"; 
    if(mysqli_query($link, $sql)){ 
     echo "Records added successfully."; 
    } 
    else 
    { 
     echo "ERROR: Could not able to execute $sql. " . mysqli_error($link); 
    } 

    // close connection 
    mysqli_close($link); 
} 

Mit einer if sentece Sie haben es gelöst. Es wird nur einen Datensatz hinzufügen, wenn die POST-Name-Variable gesetzt ist.

Answer 2

Sie bedingungslos insert eine Reihe, so wird dies erwartet. Also, was sollte die Bedingung sein? Natürlich, beim Laden der Seite wollen Sie nicht insert einen Schrei, nur auf POST. Sie müssen also prüfen, ob die Anforderung eine Post ist:

if ($_SERVER['REQUEST_METHOD'] === 'POST') { 
    // insert the row 
} 

Aber selbst wenn es sich um eine POST ist, müssen Sie prüfen, ob es gültig ist, so Namen überprüfen und rufen:

if ($_SERVER['REQUEST_METHOD'] === 'POST') { 
    if ((isset($_POST["name"])) && (isset($_POST["shout"])) && ($_POST["name"]) && ($_POST["shout"])) { 
     //insert the row 
    } 
} 

Uhr out für SQL injection obwohl.