Co jest nie tak z tym kodem ?

pytanie zadane 30 maja 2017 w PHP przez Kamil Gąsior Obywatel (1,430 p.)
edycja 30 maja 2017 przez HaKIM

Hej mam takie pytanie co jest nie tak z tym kodem ? Szukam i szukam i nie mogę znaleźć :/ Proszę o wyjaśnienie.


        $zapytanie2=$polaczenie->query("UPDATE users SET Zalozyciel='$zalozyciel'WHERE id='$id'");
        if ($zapytanie = $polaczenie->query("INSERT INTO blogi(Krotka_nazwa,Dluga_nazwa,Temat,Zalozyciel)
        VALUES('$short','$full','$temat','$zalozyciel')")) {
            echo '<span style="color:green">Blog załozony pomyślnie.</span>';
        } else echo '<span style="color:red">Blog nie został założony.</span>';

$_SESSION['zalozyciel']=$short;
    mkdir('../blogi/'.$short);
    header('Location: blog.php');

komentarz 30 maja 2017 przez Vento Pasjonat (17,120 p.)

Co dokładnie się dzieje wyskakuje jakiś błąd czy co? Nie jestem wróżbitą, który w magiczny sposób sam znajdzie problem..

komentarz 30 maja 2017 przez Kamil Gąsior Obywatel (1,430 p.)

Takie błedy wyskakuja :D Wybacz :D

komentarz 30 maja 2017 przez Vento Pasjonat (17,120 p.)

Nie masz zdeklarowanej zmiennej $polaczenie dlatego wyskakuja te bledy..

komentarz 30 maja 2017 przez Kamil Gąsior Obywatel (1,430 p.)

Zmienna jest zdeklarowana, ale wyżej.

1 odpowiedź

odpowiedź 30 maja 2017 przez marcin99b Szeryf (82,080 p.)

Dlaczego w instrukcji if masz przypisanie, zamiast porównania?
+masz słabą czytelność kodu, wcięcia są po to, żeby było czytelniej, a nie żeby tylko były

komentarz 30 maja 2017 przez Kamil Gąsior Obywatel (1,430 p.)

Kurde wrzuciłem wam nie ten kod :/ Wybaczcie, ale zaspany jestem.

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

        $user_login=$_SESSION['login'];
        if ($zapytanie = $polaczenie->query("SELECT * from blogi WHERE Zalozyciel='$login'"))
        {
            echo '<span style="color:green">Blog  został założony.</span>';

                 $zapytanie3=$polaczenie->fetch_assoc();

                     $_SESSION['krotka_nazwa']=$zapytanie3['Krotka_nazwa'];

           echo "<h3>Należysz do bloga:".$_SESSION['krotka_nazwa']."</h3>";

        } else echo '<span style="color:red">Blog nie został założony.</span>';

        if(file_exists("../blogi".$_SESSION['krotka_nazwa']."/config.xml"))
        {
            echo "Plik konfiguracyjny istnieje.";
        }else
        {
            echo "Plik konfiguracyjny nie istnieje.";
        }
    }

komentarz 30 maja 2017 przez efiku Szeryf (75,160 p.)

Jeśli to ujrzy światło dzienne, albo będziesz to używał gdzieś na produkcji, to położą Ci ten skrypt i wykradną dane z bazy w mniej niż minutę. ;)

Nie lepiej już teraz przejść na system szablonów ? Twig?

komentarz 31 maja 2017 przez Kamil Gąsior Obywatel (1,430 p.)

Czemu tak myślisz ?:D

Co masz na myślisz z tym systemem szablonów ?

komentarz 31 maja 2017 przez efiku Szeryf (75,160 p.)

No... Myślałem że wpiszesz w google : Twig.

;)

Chociażby po to żeby nie robić takiego spaghetti co masz teraz.

Czemu tak myślę ? Bo nie filtrujesz danych. Proste ;)