Mysql, Dodawanie rekordów do bazy danych. Opcje

[plej]

Witam.
bo mam problem z dodawaniem tematów do forum które sam buduje (w ramach edukacyjnych) chodzi że wszystko ładnie mi pobiera itd. ale jak już chce dodać klikając na przycisk "Dodaj" pojawia mi się błąd "Notice: Undefined index: id_kategorii in /opt/lampp/htdocs/footbool/example3/add_post.php on line 5" i nie mogę tego rozszyfrować proszę o pomoc, Z góry bardzo dziękuje:)

Oto pliki
posts.php:

[PHP] pobierz, plaintext 
<?php
ob_start();
session_start();
$id_kategorii = $_GET["id_kategorii"];
$temat = $_GET["temat"];
$tytul = $temat;
include('naglowek.php');
include('panel-logowania.php');
include('menu-gora.php');
include('menu-lewe.php');
?>
	<div id="srodek">
		<div id="srodekt"></div>
		<div id="srodeks">
<?php
echo '<font color="black" size="3" ><center>'.$temat.'</center></font>';
include('db.php');
$query = ("SELECT * FROM artykuly WHERE id_kategorii = $id_kategorii");
mysql_query('SET NAMES \'utf8\'');
$result = mysql_query($query);
if (!$result) {
	die("Nie można wykonać zapytanai do bazy danych.");
}
while ($row = mysql_fetch_assoc($result)) {
$tytul = $row["tytul"];
$id_artykulu = $row["id_artykulu"];
$uzytkownik = $row["uzytkownik"];
echo '<div class="naglowek"><a href="view_post.php?id_artykulu='.$id_artykulu.'&tytul='.$tytul.'">'.$tytul.'</a> | Dodał: '.$uzytkownik.'</div>';
$query2 = "SELECT * FROM komentarze WHERE id_artykulu = $id_artykulu";
mysql_query('SET NAMES \'utf8\'');
$result2 = mysql_query($query2);
if (!$result2) {
	die("Nie można nawiązać połączenia z bazą danych.");
}
$ile = mysql_num_rows($result2);
echo '<div class="srodek">Odpowiedzi: '.$ile.'</div>';
}
mysql_close($polaczenie);
echo '<br/><br/>';
if (isset($_SESSION['login'])) {
	echo '<a href="add_post.php?id_kategorii='.$id_kategorii.'">Dodaj temat</a> | <a href="modify_post.php?akcja=moje_tematy">Moje tematy</a>';
} else {
	echo 'Musisz się <a href="logowanie.php">zalogować</a> lub <a href="rejestracja.php">zarejestrować</a>, aby dodać nowy temat';
}
?>
		</div>
		<div id="srodekb"></div>
	</div>
 
 
<?php
include('prawa-strona.php');
include('stopka.php');
?> 
[PHP] pobierz, plaintext 

add_post.php

[PHP] pobierz, plaintext 
<?php
ob_start();
session_start();
$tytul = "Dodawanie tematu";
$id2 = $_GET["id_kategorii"];
include('naglowek.php');
include('panel-logowania.php');
include('menu-gora.php');
include('menu-lewe.php');
?>
	<div id="srodek">
		<div id="srodekt"></div>
		<div id="srodeks">
<?php
if (isset($_SESSION['login'])) {
 
 
    include 'db.php'; 
$query = "SELECT * FROM kategorie WHERE id_kategorii = $id2";
mysql_query('SET NAMES \'utf8\'');
$result = mysql_query($query);
if (!$result) {
	die("Nie można wykonać zapytania do bazy danych.");
}
while ($row = mysql_fetch_assoc($result)) {
$id = $row["id_kategorii"];
$temat = $row["temat"];
}
    $uzytkownik = $_SESSION["login"];
          if ($_POST["wyslane"]) {
	    $tytul2 = $_POST["tytul"];
	    $tresc = $_POST["tresc"];
 
	    $query = "INSERT INTO artykuly VALUES('',  '$id', '$uzytkownik', '$tytul2', '$tresc', NOW())";
	    mysql_query('SET NAMES \'utf8\'');
	    $result = mysql_query($query);
	    if (!$result) {
		die("Nie można nawiązać połączenia z bazą danych.");
	    }
 
	   }
 
        mysql_close($polaczenie);
 // tworzenie formularza HTML
echo <<< KONIEC
<center><font color="black">
Dodajesz temat do forum: <b>$temat</b></font>
<br /></center>
<form action="add_post.php" method="post">
<input type="hidden" name="wyslane" value="TRUE" />
<p>Tytuł: <input type="text" name="tytul" /></p>
<div class="label"><label for="tresc">Treść</label></div>
<textarea id="tresc" name="tresc" cols="40" rows="10"></textarea>
<p><input type="submit" value="Dodaj" /></p></form>
KONIEC;
 
} else {
	echo 'Musisz się <a href="logowanie.php">zalogować</a> lub <a href="rejestracja.php">zarejestrować</a>, aby dodawać nowe temat, komentarze.';
}
?>
<br />
		</div>
		<div id="srodekb"></div>
	</div>
<?php
include('prawa-strona.php');
include('stopka.php');
?>
[PHP] pobierz, plaintext 

Proszę nie zwracać uwagi że kody nie są piękne bo to wiem:) nauka czyni mistrza z czasem się poprawie

[mat-bi]

Notice: Undefined index: id_kategorii in /opt/lampp/htdocs/footbool/example3/add_post.php on line 5

Eeee, to naprawdę taie ciężkie do odszyforwania? Jesli znasz angielski, powinno to łatwo pójść.

[plej]

Eeee, to naprawdę taie ciężkie do odszyforwania? Jesli znasz angielski, powinno to łatwo pójść.

Tak ciężko:) proszę o wyrozumiałość nie mam styczności z php nie wiem roku czy dłużej ucze się go zaledwie miesiąć a angielski cieżko mi idzie:/

Ten post edytował plej 19.05.2011, 02:13:54

[mat-bi]

Mam pytanie - czemu startujesz bufor, a nigdzie go nie wysyłąsz?

[plej]

nie wiem
Liczę na waszą pomoc


Pomożesz ?

Ten post edytował plej 19.05.2011, 02:20:17

[nospor]

a angielski cieżko mi idzie

Miałem nie odpowiadać w tym temacie, ale znalazł plej twoją wypowiedź w innym wątku. Pozwól, że ją zacytuję:

jak nie umiesz zbytnio angielskiego to na google chrome włącz i ustaw tłumaczenie stronki:)

Tak więc czegoś tu nie rozumiem. Innych pouczasz jak mają sobie radzić z angielskim a sam tutaj płaczesz, że angielskiego nie znasz.

[Kshyhoo]

Jezu, jak to ludzie wcześnie wstają ;p
Jak dobrze widzę, chcesz przekazać parametr $id_kategorii w linku:

[PHP] pobierz, plaintext 
echo '<a href="add_post.php?id_kategorii='.$id_kategorii.'">Dodaj temat</a>';
[PHP] pobierz, plaintext 

Ale widocznie go nie przekazujesz. Sprawdź dlaczego, np. tak:

[PHP] pobierz, plaintext 
print('<pre>GET: ');
print_r($_GET);
print('</pre>');
[PHP] pobierz, plaintext 

[nospor]

@Kshyhoo przecież w linku on go przekazuje. Nie przekazuje go gdy klika DODAJ w formularzu.

[Kshyhoo]

To ja wiem, ale chciałem, żeby sam poszukał ;p

[plej]

Tak wiem googgle chrome tłumaczy wiem co oznacza błąd ale nie znam tak php jeszcze zęby się domyślić co zrobić :/

Po naciśnieciu Dodaj wyskakuje błąd. wtedy dopisałem pare if'ów i pisze że zmienna $id2 nie istnieje i że nie można wykonać zapytania do bazy danych:/

[Kshyhoo]

Skoro pisze, że zmienna $id2 nie istnieje, to znaczy, że skrypt jej nie widzi.
Sprawdź swój formularz. Zmienną tą możesz przekazać np. poprzez ukryte pole formularza.

[mortus]

No to tak łopatologicznie. Jeśli mamy formularz, który wygląda tak:

[HTML] pobierz, plaintext 
<form action="add_post.php" method="POST">
  <!-- ... -->
  <input type="submit" value="Dodaj" />
</form>
[HTML] pobierz, plaintext 

To naciśnięcie przycisku Dodaj będzie równoważne z przekierowaniem do adresu, który określa właściwość action formularza, czyli w tym przypadku do pliku add_post.php. Czegoś chyba brakuje, prawda?

Kolega wyżej wspomniał, że zmienną można przekazać poprzez ukryte pole formularza. Owszem, ale w tym przypadku nie spowoduje to zniknięcia błędu, bowiem należałoby wtedy nieco inaczej obsłużyć cały skrypt (dopisać odpowiednie warunki). Poza tym w komunikacie błędu nie było żadnej wzmianki o zmiennej $id2.

[plej]

Dzięki błędu już nie ma choć do bazy danych przekazuje i tak id_kategorii = 0 , a powinno 1 ale ważne że błędu nie ma:)

Ten post edytował plej 19.05.2011, 12:29:27

[mortus]

Przekazywanie niewłaściwego parametru to również błąd, wklej tutaj kod i sprawdzimy, czy problem tkwi w przekazywaniu parametru, czy może w zapytaniu do bazy danych.

[plej]

[PHP] pobierz, plaintext 
<?php
session_start();
$id2 = $_GET["id_kategorii"];
$tytul = "Dodawanie tematu";
include('naglowek.php');
include('panel-logowania.php');
include('menu-gora.php');
include('menu-lewe.php');
?>
	<div id="srodek">
		<div id="srodekt"></div>
		<div id="srodeks">
<?php
 
	  if (isset($_SESSION['login'])) {
 
 
	  include 'db.php'; 
	  $query = "SELECT * FROM kategorie WHERE id_kategorii = $id2";
	  mysql_query('SET NAMES \'utf8\'');
	  $result = mysql_query($query);
	  if (!$result) {
	  die("Nie można wykonać zapytania do bazy danych.");
	  }
	  while ($row = mysql_fetch_assoc($result)) {
	  $id = $row["id_kategorii"];
	  $temat = $row["temat"];
	  }
 
	  $uzytkownik = $_SESSION["login"];
          if ($_POST["wyslane"]) {
	    $tytul2 = $_POST["tytul"];
	    $tresc = $_POST["tresc"];
 
	    $query = "INSERT INTO artykuly VALUES('',  '$id', '$uzytkownik', '$tytul2', '$tresc', NOW())";
	    mysql_query('SET NAMES \'utf8\'');
	    $result = mysql_query($query);
	    if (!$result) {
		die("Nie można nawiązać połączenia z bazą danych.");
	    }
	  }
 
 
        mysql_close($polaczenie);
 
 // tworzenie formularza HTML
echo <<< KONIEC
<center><font color="black">
Dodajesz temat do forum: <b>$temat</b></font>
<br /></center>
<form action="add_post.php?id_kategorii=$id" method="post">
<input type="hidden" name="wyslane" value="TRUE" />
<p>Tytuł: <input type="text" name="tytul" /></p>
<div class="label"><label for="tresc">Treść</label></div>
<textarea id="tresc" name="tresc" cols="40" rows="10"></textarea>
<p><input type="submit" value="Dodaj" /></p></form>
KONIEC;
 
} else {
	echo 'Musisz się <a href="logowanie.php">zalogować</a> lub <a href="rejestracja.php">zarejestrować</a>, aby dodawać nowe temat, komentarze.';
}
?>
<br />
		</div>
		<div id="srodekb"></div>
	</div>
<?php
// mysql_query('SET NAMES \'utf8\'');
include('prawa-strona.php');
include('stopka.php');
?>
[PHP] pobierz, plaintext 

Linia '3' przypisuje id_kategorii do $id2
linia '19-27' pobieram dane jeśli $id2jest w bazie zeby pobrać nazwe tematu:)
linia '30-40' dodaje rekord do bazy i nizej jest formularze:)

tak wiem że nie filtruje danych:) dopisze to:)


a tak to dobrze jest??

Ten post edytował plej 19.05.2011, 12:38:04

[mortus]

Linia '3' przypisuje id_kategorii do $id2

To dlaczego nie użyjesz zmiennej $id2 w parametrze action formularza?
W liniach 25-28 pętla while jest niepotrzebna, bo jeśli wcześniejsze zapytanie zwróci jakieś dane, to będzie to pojedynczy wiersz, zatem wystarczy:

[PHP] pobierz, plaintext 
// ...
$row = mysql_fetch_assoc($result);
$id = $row['id_kategorii'];
$temat = $row['temat'];
// ...
// opcjonalnie możesz tutaj sprawdzić, czy pobrało właściwe id
// po prostu je wyświetlając
echo $id.'<br />';
// ...
[PHP] pobierz, plaintext 

Poza tym troszkę źle masz wszystko zorganizowane. Nie za bardzo rozumiem, po co są linie 18-28, skoro id kategorii i tak przekazujesz w adresie. Jeśli ma to służyć sprawdzeniu, czy kategoria o takim id istnieje, to takie sprawdzenie powinno się odbyć wcześniej, a nie w tym skrypcie.

[plej]

sprawdzam id czy jest takie w bazie i pobieram temat choć mogę getem pobrać z innego skrypt tzn. z posts.php wiem poprawie to:)
jak wspominałem wcześniej uczę się miesiąc php. Proszę o wyrozumiałość

[mortus]

Nie już rozumiem i tak jest dobrze, tylko warto by było odseparować w kodzie część odpowiedzialną za obsługę formularza, od części odpowiedzialnej za jego wyświetlanie, ja bym to zrobił tak:

[PHP] pobierz, plaintext 
<?php
session_start();
 
// ustalamy tytuł strony i includujemy potrzebne pliki
$tytul = "Dodawanie tematu";
include 'naglowek.php';
include 'panel-logowania.php';
include 'menu-gora.php';
include 'menu-lewe.php';
include 'db.php';
?>
<div id="srodek">
    <div id="srodekt">
    </div>
    <div id="srodeks">
        <?php
		$id2 = $_GET["id_kategorii"];
		// sprawdzamy, czy użytkownik jest zalogowany 
        if (isset($_SESSION['login'])) {
 
			// sprawdzamy, czy użytkownik wysłał formularz
            if ($_POST["wyslane"]) {
 
				// pobieramy potrzebne dane
				$uzytkownik = $_SESSION["login"];
 
				// pobieramy dane z formularza
                $tytul2 = $_POST["tytul"];
                $tresc = $_POST["tresc"];
                // i dodajemy do bazy danych
                $query = "INSERT INTO artykuly VALUES('',  $id2, '$uzytkownik', '$tytul2', '$tresc', NOW())";
                mysql_query('SET NAMES \'utf8\'');
                $result = mysql_query($query);
                if (!$result) {
                    die("Nie można nawiązać połączenia z bazą danych.");
                }
				// tutaj można przekierowanie zrobić
            }
 
			// pobieramy informację na temat określonej kategorii, czyli jej temat
            $query = "SELECT temat FROM kategorie WHERE id_kategorii = $id2";
            mysql_query('SET NAMES \'utf8\'');
            $result = mysql_query($query);
            if (!$result) {
                die("Nie można wykonać zapytania do bazy danych.");
            }
            $row = mysql_fetch_assoc($result);
			$temat = $row["temat"];
            mysql_close($polaczenie);
 
            // tworzenie formularza HTML
			echo <<< KONIEC
			<center><font color="black">
			Dodajesz temat do forum: <b>$temat</b></font>
			<br /></center>
			<form action="add_post.php?id_kategorii=$id" method="post">
			<input type="hidden" name="wyslane" value="TRUE" />
			<p>Tytuł: <input type="text" name="tytul" /></p>
			<div class="label"><label for="tresc">Treść</label></div>
			<textarea id="tresc" name="tresc" cols="40" rows="10"></textarea>
			<p><input type="submit" value="Dodaj" /></p></form>
			KONIEC;
        } else {
            echo 'Musisz się <a href="logowanie.php">zalogować</a> lub <a href="rejestracja.php">zarejestrować</a>, aby dodawać nowe temat, komentarze.';
        }
        ?>
        <br/>
    </div>
    <div id="srodekb">
    </div>
</div>
<?php 
// mysql_query('SET NAMES \'utf8\'');
include ('prawa-strona.php');
include ('stopka.php');
?>
[PHP] pobierz, plaintext 

W sumie to chodzi tylko o lepszą organizację kodu, ale poza tym jest dobrze.

[plej]

No tak:) z czasem poprawie kod:) ale i tak użyje twój sposób hehe:)