PHP dodaje za dużo rekordów do bazy

Question

Witam, jestem bardzo początkujący jeśli chodzi o PHP i napotkałem na taki problem, mianowicie stworzyłem sobie prosty system komentarzy/opinii. Niestety "coś" po dodaniu komentarza dodaje mi wcześniej puste rekordy. Wygląda to tak:

Co najlepsze, czasem dodają się te puste rekordy, a czasem nie.

Kod (head):

<?php

    require_once "connect.php";

    $polaczenie = @new mysqli($host, $db_user, $db_password, $db_name);

    if($polaczenie->connect_errno!=0)
    {
        echo "Error: ".$polaczenie->connect_errno;
		$polaczenie->close();
    }
    else
    {
	
		if(!isset($autor) || !isset($tresc))
		{
		$autor = $_POST['autor'];
		$tresc = $_POST['tresc'];
		
        
        $autor = htmlentities($autor, ENT_QUOTES, "UTF-8");
        $tresc = htmlentities($tresc, ENT_QUOTES, "UTF-8");
        
        $dodaj=mysqli_query($polaczenie, "INSERT INTO opinie SET id_opinii=NULL, autor='$autor', tresc='$tresc'");
        
		}
		
    }

    $wyjmij = mysqli_query($polaczenie, "SELECT * FROM opinie");
    
    $polaczenie->close();
    
?>

oraz w części BODY:

<div class="opinie">
                <?php
                
                if(mysqli_num_rows($wyjmij) > 0)
                {
                    while ($r = mysqli_fetch_assoc($wyjmij))
                    {
                        echo '<div class="opinia">';
                        echo '<p class="autor">'.$r['autor'].'</p>';
                        echo '<p>'.$r['tresc'].'</p>';
                        echo '</div>';
                    }
                }
                    
                ?>
            </div>

Jeśli widzicie też jakiś inny karygodny błąd, chętnie wysłucham (widzę, że coś jest nie tak z kodowaniem znaków w bazie, lecz na stronie wyświetlają się poprawnie :P). Ale pierw proszę, pomóżcie z tymi pustymi rekordami :(

EDIT: jak to jest, że 3 minuty po dodaniu pytania sam się orientujesz co się dzieje? :P Odkryłem, że każde odświeżenie strony dodaje jeden pusty rekord. No dobrze, w takim razie zmieniam pytanie na:
Co i jak zmienić, żeby tego uniknąć?

Comments

Przejdz na PDO bo jest nowsze i łatwiejsze. Usun malpy z kodu. Bede trul tak do znudzenia każdemu.

Answer 1

Dodaje bo.... !isset. A jakbyś tak włączył sobie wyświetlanie błędów to byś zobaczył notice ( linia 17-18)

Comments

Smieszy mnie to jak bierzecie kod Zelenta dodajecie coś z innych kodów do tego (obiektowe mysqli to po kij proceduralne) i dziwicie się że nie działa.

---

nie rozumiem co Cie tak śmieszy? czego oczekujesz od kogoś kto uczy się od paru dni tego języka, zobaczył dwa filmiki na YT i próbuje coś sensownego napisać? każdy kiedyś zaczynał... chyba dobrze, że się próbuje czegoś nowego?

---

Oczekuje, że nie będzie brał gotowych kodów i sklejał kilka z nich w jeden :/

Bardziej się zniechęcisz, niż czegoś nauczysz,
Jeśli chcesz wiedzieć, to ja zaczynałem od czytania dokumentacji, a później zacząłem czytać blogi, gdzie ludzie pisali odnośnie nowych technik i bibliotek.

Jeżeli chcesz się czegoś nauczyć to:

- Wymyśl sobie jakąś prostą apke, np. Lista zadań.  (NIE bierz się za logowanie/rejestracje na początku - to gwóźdź do trumny)
- Przejdź od razu na OOP: https://webmastah.pl/jak-programowac-obiektowo-cz-1-wstep/
- Popatrz na to: https://php-kurs.gitbooks.io/phpkurs/content/
- Tutaj: https://symfony.com/doc/current/book/http_fundamentals.html ( PODSTAWY działania HTTP)
- Tutaj: http://www.phptherightway.com/
- Tutaj: http://phpers.github.io/faq/

Obecnie używa się najczęściej w PHP:
 -> Podejścia obiektowego
 -> PDO
 -> Composera
 -> Twiga

Jak Ci się chce uczyć to googluj te zagadnienia i rozwijaj się.

Answer 2

Wrzuć przed ifa 
       $autor = $_POST['autor'];
        $tresc = $_POST['tresc'];

 

Dodatkowo isset zamień na funkcję empty, która sprawdzi czy zmienna jest pusta, a nie czy zmienna istnieje.