Pobieranie wartości z textarea do zmiennej w PHP

Question

<form method="post" class="todo-form" id="todoForm">
	<div class="todo-form-row">
		<label class="todo-form-label" for="todoMessage">Podaj treść zadania</label>
		<textarea form="todoForm" class="todo-form-message" name="todoMessage" id="todoMessage"></textarea>
	</div>

	<div class="todo-form-row">
		<input type="submit" class="button todo-form-button" value="Dodaj" />
	</div>
</form>
<?php
	$db = new Mysqli ('localhost', 'root', '', 'baza');
	
	$notatka = $_POST['todoMessage'];
	$user_id = $_SESSION['id'];
	
	$sql = "INSERT INTO notatki (id, user_id, notes) VALUES (NULL, '$user_id', '$notatka')";
	
	$wynik = $db -> query($sql);
?>

Próbując dodać do bazy treść wpisaną w textarea dostaje taki błąd.

Notice: Undefined index: todoMessage in D:\xampp\htdocs\projekty\projekt\notatnik.php on line 45

Ktoś ma jakiś pomysł? 

Answer 1

Umieść kod PHP w kondycji, która sprawdza czy formularz został wysłany. W poniższym przykładzie sprawdzamy czy komunikacja odbywa się POSTem, ale w Twoim wypadku powinno na tę chwilę wystarczyć. :)

if ($_SERVER['REQUEST_METHOD'] === 'POST') {
    $db = new Mysqli ('localhost', 'root', '', 'baza');
     
    $notatka = $_POST['todoMessage'];
    $user_id = $_SESSION['id'];
     
    $sql = "INSERT INTO notatki (id, user_id, notes) VALUES (NULL, '$user_id', '$notatka')";
     
    $wynik = $db -> query($sql);
}

Po co ta kondycja? Nie wysłałeś jeszcze formularza z danymi, a próbujesz odnieść się do właśnie tych danych... trochę nielogiczne musisz przyznać.

Comments

Przed dodaniem tego if'a gdy w $notatka sam przysypiałem jakąś wartość np.

$notatka = "testowa notatka";

to dodawało mi to do bazy.

Teraz co prawda zniknął błąd o którym pisałem w poprzednim poście ale nie działa ten if - tak jakby za każdym razem wynikiem tego if'a był false.

---

Oczywiście, że powyższy kod z danymi "testowa notatka" dodawało Ci do bazy, gdyż nie odnosiłeś się do danych z formularza, nim te mogły zaistnieć.

Spróbuj w takim wypadku czegoś mniej eleganckiego:

if ($_POST['todoMessage']) {

}

Zamiast ifa z POSTem i daj znać jaki rezultat.

---

if(!empty($_POST['todoMessage']))

dopiero to zadziałało

Answer 2

Mam jeszcze pytanie jak usunąć zawartość ze zmiennej notatka:

$notatka = $_POST['todoMessage'];

próbowałem unset ale nie działa - po odświeżeniu strony zapisuje do bazy drugi raz tą samą wartość

Comments

Jeśli to windows/linux z chrome:

ctrl + f5 / Otworzyć nowego taba.

To normalne zachowanie przeglądarki co opisujesz.

---

Korzystam z opery ale z tego co pamiętam to na chrome również skrót ctrl + f5 to odświeżenie strony a nie otworzenie nowej karty.

Wracają do problemu można jakoś usunąć ten problem, wyczyścić zmienną?

---

ctrl + f5 to refresh + czyszczenie ciasteczek, które o ile dobrze pamiętam pomagało mi aby pozbyć się przedawnionego posta.

Najprościej to by było rozdzielić front od backendu fizycznie t.j. plikami, przez co uniknąłbyś trzymaniem tego samego zapytania w przeglądarce dla strony X.

---

Jestem początkujący :) więc wiesz... nie umiem takich rzeczy.

Mógłbyś wytłumaczyć dlaczego nie działa mi unset

---

Może podczas odświeżania strony znowu jest wysyłany ten sam formularz (wykonywana jest metoda POST).

Jeżeli tak to po dodaniu wpisu do bazy zrób przekierowanie na GET.

---

Sugeruję zobaczyć jak przeglądarka (tutaj opera) przetrzymuje dane z posta. Gdybym już to zrozumiał, zastosowałbym odpowiednie rozwiązanie.

 

Jestem początkujący :) więc wiesz... nie umiem takich rzeczy.

Tworzysz x1 plik:

form.php

oraz x2:

handle-form.php

W 1 pakujesz kod html a w 2 wszystko pomiedzy <?php ?> i dodajesz do formluarza:

action="/handle-form.php" aby ten przekierował ruch na ten drugi plik.

---

Mógłbyś wytłumaczyć jak zrobić to przekierowanie??

---

Podejrzewam, że wystarczy dodać header w obsłudze posta

<?php
header("Location: moj_plik.php");
exit;
?>

 

---

Chyba działa. Dziękuję bardzo <3