Dodawanie plików w formularzu Opcje

[radziowie]

Przekształciłem jeden ze swoich plików i tak powstało dodawanie plików. Teraz mam problem w linijce 72 i 78. Zmienna $opis jest nieznana. Dlaczego?

[PHP] pobierz, plaintext 
<?php
//stosować tylko do projektu do wyszukania błędów
      error_reporting(E_ALL);
      ini_set('display_errors','1');
// usunąć po zakończeniu projektowania	  
//załączenie plików
include ('../meta.php');
include ('../gora_logo.php');
include ('./menu_admin.php');
include ('../dodatki/bonus.php');
polacz_baza();
$data = date('Y-m-j'); 
function add_file () {
echo "<h1>INSTRUKCJE - DODAWANIE I WYŚWIETLANIE</H1>";
echo '<form enctype="multipart/form-data" action="instrukcje.php" method="post">
Tytuł instrukcji: <input type="text" name="tytul" /><br />
Opis: <textarea name="opis" cols="50" rows="10"></textarea><br />
Autor: <input type="text" name="wykladowca" /><br />
<input type="hidden" name="MAX_FILE_SIZE" value="1240000" />
Plik (max 1 MB):<input name="plik" type="file" />
 
 
<input type="submit" name="submit" value="wyślij plik" />
</form>';
} 
 
?>
<table bgcolor="white" width="900" cellspacing="0" cellpadding="10" border="1" align="center">
<tr>
           <td colspan="2" align="center" valign="middle">
 
<?php
/////////////////////////////////////////
/////////////////////////////////////////
//SPRAWDZIĆ DANE WYSYŁANE PRZEZ FORMULARZ
/////////////////////////////////////////
/////////////////////////////////////////
 
//sprawdzenie czy istnieją zmienne
if (isset($_POST['tytul']) && isset($_POST['opis']) && isset($_POST['wykladowca']) && isset($_POST['plik'])) {
//mysql_real_escape_string Dodaje znaki unikowe w łańcuchu znaków do użycia w instrukcji SQL, trim wycina spacje
$tytul = mysql_real_escape_string(trim($_POST['tytul']));
$opis = mysql_real_escape_string(trim($_POST['opis']));
$wykladowca = mysql_real_escape_string(trim($_POST['wykladowca']));
$plik = $_POST['plik'];
 
 
 
 
 }
  //sprawdza czy istnieje zmienna$_POST['submit'], czyli czy nastąpiło wysłanie danych
if (isset($_POST['submit'])) {
//dodawanie danych do bazy
$plik_tmp = $_FILES['plik']['tmp_name'];
$plik_nazwa = $_FILES['plik']['name'];
$plik_rozmiar = $_FILES['plik']['size'];
$plik = $plik_nazwa;
	if(is_uploaded_file($plik_tmp)) {
     move_uploaded_file($plik_tmp, "../upload/$plik_nazwa");
    echo "Plik: <strong>$plik_nazwa</strong> o rozmiarze 
    <strong>$plik_rozmiar bajtów</strong> został przesłany na serwer!";
}
$zapytanie = "INSERT INTO instrukcje VALUES (NULL, '".$tytul."', '".$opis."', '".$wykladowca."', '".$plik."')";
$result = mysql_query($zapytanie);
}
//sprawdzenie czy istnieją zmienne
if (isset($_GET['co']) && isset($_GET['id']) && $_GET['co'] == 'skasuj') {
$wynik = mysql_query("DELETE FROM instrukcje WHERE id = '".$_GET['id']."'");
add_file();
//sprawdzenie czy istnieją zmienne
} elseif (isset($_POST['co']) && $_POST['co'] == 'popraw') {
	if ($_POST['tytul'] && $_POST['opis'] && $_POST['wykladowca'] && $_POST['plik']) {
	$popraw = mysql_query("UPDATE instrukcje SET tytul = '".$_POST['tytul']."', opis = '".$_POST['opis']."', wykladowca = '".$_POST['wykladowca']."'
	, plik = '".$_POST['plik']."'	
	WHERE id = '".$_POST['id']."';")
	or die(mysql_error());
	$wynik = mysql_query($popraw);
	} elseif (!$tytul || !$opis || !$wykladowca  || !$plik) {
	echo '<b>Wpis nie został zaktualizowany. Przynajmniej jedno z pól nie zostało wypełnione.<br /> Popraw to!</b><br />';
	}
	add_file();
 
	}	
//sprawdzenie czy istnieją zmienne
elseif (isset($_GET['co']) && $_GET['co'] == 'edytuj') {
$edycja = mysql_query("SELECT * FROM instrukcje WHERE id = '".$_GET['id']."';");
$rekord = mysql_fetch_assoc($edycja);
$nrid = $rekord['id'];
$tytul = $rekord['tytul'];
$opis = $rekord['opis'];
$wykladowca = $rekord['wykladowca'];
$plik = $rekord['plik'];
echo "<table border=\"1\">
<tr>
    <td width=\"120\"><form action=\"instrukcje.php\" method=\"post\">
	nr id: ".$nrid."<input type=\"hidden\" name=\"co\" value=\"popraw\">
	<input type=\"hidden\" name=\"id\" value=\"".$nrid."\"><br /></td>
</tr><tr>	
    Tytuł:<input type=\"text\" name=\"tytul\" value=\"".htmlspecialchars($tytul)."\"><br /></td>
</tr><tr>
         <td width=\"120\">Treść wiadomości:<textarea name=\"text\" cols=\"50\" rows=\"10\">".htmlspecialchars($opis)."</textarea><br /></td>
</tr><tr>
         <td width=\"120\">Autor:<input type=\"number\" name=\"autor\" min=\"0\" max=\"7\" value=\"".htmlspecialchars($wykladowca)."\"><br /></td>
</tr><tr>
		<td>Data: ".$plik."</td>
</tr><tr>
         <td align=\"center\"><input type=\"submit\" name =\"popraw\" value=\"popraw\"><br />
         </form></td>
</tr>
</table>";
//sprawdza czy nie istnieje zmienna $_GET['co'], jeśli nie istnieje wtedy wyświetlany jest funkcja add_news(), czyli formularz
} elseif (!isset($_GET['co'])) {
add_file();
}
 
$wynik = mysql_query("SELECT * FROM instrukcje ORDER BY id DESC;") or 
	die ("zle polecenie");
//liczba rekordów
 
$ile = mysql_num_rows($wynik);
echo "Znaleziono: ".$ile." rekordów";
echo "<table border=\"1\" width=\"750\"><tr><td width=\"30\">Nr. ID</td><td width=\"100\">Tytuł</td><td width=\"400\">Opis</td>
	<td width=\"10\">Wykladowca</td>
	<td>Plik</td>
	<td width=\"50\">Usuwanie
	</td><td>Edytowanie</td></tr>";
	//wyświetlanie danych z bazy komórka po komórce
 
	WHILE ($rekord = mysql_fetch_assoc ($wynik)) {
$nrid = $rekord['id'];
$tytul = $rekord['tytul'];
$opis = $rekord['opis'];
$wykladowca = $rekord['wykladowca'];
$plik = $rekord['plik'];
 
	echo "<tr><td width=\"30\">".$nrid."</td><td width=\"100\">".stripslashes($tytul)."</td>
	<td width=\"40\">".stripslashes($opis)."</td>
	<td width=\"10\">".stripslashes($wykladowca)."</td>
	<td width=\"120\">".$plik."</td>
	<td><a href=\"instrukcje.php?co=skasuj&amp;id=$nrid\">usuń</a></td><td>
	<a href=\"instrukcje.php?co=edytuj&amp;id=$nrid\">edytuj</a></td></tr>";
	}
echo "</table>";
 
 
?>
 </tr>
</table>
<?php
include ('../stopka.php');
?>
</body>
</html>
[PHP] pobierz, plaintext 

[HTML] pobierz, plaintext 
Notice: Undefined index: opis in C:\Program Files (x86)\WebServ\httpd-users\praca\admin\instrukcje.php on line 72
 
Notice: Undefined variable: opis in C:\Program Files (x86)\WebServ\httpd-users\praca\admin\instrukcje.php on line 78
[HTML] pobierz, plaintext 

Ten post edytował radziowie 8.11.2010, 22:35:03

[ciekawskiii]

tak szczerze, to patrze na to juz 5 minut i nie moge sie polapac

[radziowie]

Postaram się poprawić kod, ale w tym celu muszę dowiedzieć się gdzie tkwi problem w kodzie.
Może ktoś mi pomóc? Dodam tylko, że jak usunę zmienną opis w tych linijkach to podobnie jest ze zmienną $wykladowca. Dlaczego kod nie "czepia" się zmiennej tytul a do reszty ma "pretensje", że tak powiem.

Ten post edytował radziowie 9.11.2010, 19:39:10

[modern-web]

Nie mieszaj kodu HTML z PHP...
To po pierwsze. Lepiej jest zrobić stronę w HTML i po prostu użyć include do skryptu, który powinien znajdować się nie dość, że w osobnym pliku to jeszcze katalogu. Jest to znacznie bezpieczniejsza metoda.

Co do samego kodu PHP - faktycznie; nie idzie się połapać.
Zrób to co napisałem powyżej, a skrypt będzie znacznie czytelniejszy.

Pozdrawiam.

[dantes]

Zgadzam się z kolegami, że brakuje czytelności i danych, na których testujesz.

BTW. W liniach 73-75 masz `update` do bazy, ale z powrotem odwołujesz się do pól z $_POST (np. $_POST['opis'] zamiast $opis). Więc nie chronisz przy update przed np. SQL Injection.

pozdrawiam

[radziowie]

Kod poprawiłem troszkę. Nie poprawiłem tylko kodu html. Nie rozumiem do końca. Mam stworzyć osobny html i w nim formularz (go z kolei wkleić do podfolderu jakiegoś?)?

[PHP] pobierz, plaintext 
<?php
//stosować tylko do projektu do wyszukania błędów
      error_reporting(E_ALL);
      ini_set('display_errors','1');
// usunąć po zakończeniu projektowania	  
//załączenie plików
include ('../meta.php');
include ('../gora_logo.php');
include ('./menu_admin.php');
include ('../dodatki/bonus.php');
polacz_baza();
$data = date('Y-m-j'); 
function add_file () {
echo "<h1>INSTRUKCJE - DODAWANIE I WYŚWIETLANIE</H1>";
echo '<form enctype="multipart/form-data" action="instrukcje.php" method="post">
Tytuł instrukcji: <input type="text" name="tytul" /><br />
Opis: <textarea name="opis" cols="50" rows="10"></textarea><br />
Autor: <input type="text" name="wykladowca" /><br />
<input type="hidden" name="MAX_FILE_SIZE" value="1240000" />
Plik (max 1 MB):<input name="plik" type="file" />
 
 
<input type="submit" name="submit" value="wyślij plik" />
</form>';
} 
echo '<table bgcolor="white" width="900" cellspacing="0" cellpadding="10" border="1" align="center">
<tr>
           <td colspan="2" align="center" valign="middle">';
 
/////////////////////////////////////////
/////////////////////////////////////////
//SPRAWDZIĆ DANE WYSYŁANE PRZEZ FORMULARZ
/////////////////////////////////////////
/////////////////////////////////////////
 
//sprawdzenie czy istnieją zmienne
if (isset($_POST['tytul']) && isset($_POST['opis']) && isset($_POST['wykladowca']) && isset($_POST['plik'])) {
	//mysql_real_escape_string Dodaje znaki unikowe w łańcuchu znaków do użycia w instrukcji SQL, trim wycina spacje
	$tytul = mysql_real_escape_string(trim($_POST['tytul']));
	$opis = mysql_real_escape_string(trim($_POST['opis']));
	$wykladowca = mysql_real_escape_string(trim($_POST['wykladowca']));
	$plik = $_POST['plik'];
 }
  //sprawdza czy istnieje zmienna$_POST['submit'], czyli czy nastąpiło wysłanie danych
if (isset($_POST['submit'])) {
//dodawanie danych do bazy
	$plik_tmp = $_FILES['plik']['tmp_name'];
	$plik_nazwa = $_FILES['plik']['name'];
	$plik_rozmiar = $_FILES['plik']['size'];
		if(is_uploaded_file($plik_tmp)) {
		move_uploaded_file($plik_tmp, "../upload/$plik_nazwa");
		echo "Plik: <strong>$plik_nazwa</strong> o rozmiarze 
		<strong>$plik_rozmiar bajtów</strong> został przesłany na serwer!";
		} else {
		echo "Nie udało się wysłać pliku <strong>$plik_nazwa</strong> na serwer"; 
		}
$zapytanie = "INSERT INTO instrukcje VALUES (NULL, '".$tytul."', '".$opis."', '".$wykladowca."', '".$plik_nazwa."')";
$result = mysql_query($zapytanie);
}
 
//sprawdzenie czy istnieją zmienne
if (isset($_GET['co']) && isset($_GET['id']) && $_GET['co'] == 'skasuj') {
$wynik = mysql_query("DELETE FROM instrukcje WHERE id = '".$_GET['id']."'");
add_file();
 
//sprawdzenie czy istnieją zmienne
} elseif (isset($_POST['co']) && $_POST['co'] == 'popraw') {
	if ($_POST['tytul'] && $_POST['opis'] && $_POST['wykladowca']) {
	$popraw = mysql_query("UPDATE instrukcje SET tytul = '".$_POST['tytul']."', opis = '".$_POST['opis']."', wykladowca = '".$_POST['wykladowca']."'
	WHERE id = '".$_POST['id']."';") or
	die(mysql_error());
	$wynik = mysql_query($popraw);
	} elseif (!$tytul || !$opis || !$wykladowca) {
	echo '<b>Wpis nie został zaktualizowany. Przynajmniej jedno z pól nie zostało wypełnione.<br /> Popraw to!</b><br />';
	}
	add_file();
 
}	
//sprawdzenie czy istnieją zmienne
elseif (isset($_GET['co']) && $_GET['co'] == 'edytuj') {
$edycja = mysql_query("SELECT * FROM instrukcje WHERE id = '".$_GET['id']."';");
$rekord = mysql_fetch_assoc($edycja);
$nrid = $rekord['id'];
$tytul = $rekord['tytul'];
$opis = $rekord['opis'];
$wykladowca = $rekord['wykladowca'];
$plik = $rekord['plik'];
echo "<table border=\"1\">
<tr>
    <td width=\"120\"><form action=\"instrukcje.php\" method=\"post\">
	nr id: ".$nrid."<input type=\"hidden\" name=\"co\" value=\"popraw\">
	<input type=\"hidden\" name=\"id\" value=\"".$nrid."\"><br /></td>
</tr><tr>	
    Tytuł:<input type=\"text\" name=\"tytul\" value=\"".htmlspecialchars($tytul)."\"><br /></td>
</tr><tr>
         <td width=\"120\">Treść wiadomości:<textarea name=\"text\" cols=\"50\" rows=\"10\">".htmlspecialchars($opis)."</textarea><br /></td>
</tr><tr>
         <td width=\"120\">Autor:<input type=\"number\" name=\"autor\" min=\"0\" max=\"7\" value=\"".htmlspecialchars($wykladowca)."\"><br /></td>
</tr><tr>
		<td>Nazwa pliku: ".$plik."</td>
</tr><tr>
         <td align=\"center\"><input type=\"submit\" name =\"popraw\" value=\"popraw\"><br />
         </form></td>
</tr>
</table>";
//sprawdza czy nie istnieje zmienna $_GET['co'], jeśli nie istnieje wtedy wyświetlany jest funkcja add_news(), czyli formularz
} elseif (!isset($_GET['co'])) {
add_file();
}
 
$wynik = mysql_query("SELECT * FROM instrukcje ORDER BY id DESC;") or 
	die ("zle polecenie");
//liczba rekordów
 
$ile = mysql_num_rows($wynik);
echo "Znaleziono: ".$ile." rekordów";
echo "<table border=\"1\" width=\"750\"><tr><td width=\"30\">Nr. ID</td><td width=\"100\">Tytuł</td><td width=\"400\">Opis</td>
	<td width=\"10\">Wykladowca</td>
	<td>Plik</td>
	<td width=\"50\">Usuwanie
	</td><td>Edytowanie</td></tr>";
	//wyświetlanie danych z bazy komórka po komórce
 
	WHILE ($rekord = mysql_fetch_assoc ($wynik)) {
$nrid = $rekord['id'];
$tytul = $rekord['tytul'];
$opis = $rekord['opis'];
$wykladowca = $rekord['wykladowca'];
$plik = $rekord['plik'];
 
	echo "<tr><td width=\"30\">".$nrid."</td><td width=\"100\">".stripslashes($tytul)."</td>
	<td width=\"40\">".stripslashes($opis)."</td>
	<td width=\"10\">".stripslashes($wykladowca)."</td>
	<td width=\"120\">".$plik."</td>
	<td><a href=\"instrukcje.php?co=skasuj&amp;id=$nrid\">usuń</a></td><td>
	<a href=\"instrukcje.php?co=edytuj&amp;id=$nrid\">edytuj</a></td></tr>";
	}
echo "</table>
 </tr>
</table>";
 
include ('../stopka.php');
echo "</body>
</html>";
?>
[PHP] pobierz, plaintext 

[daros17]

W jednym pliku trzymasz kod html, a w drugim kod php Następnie w pliku gdzie masz html dajesz include('nazwapliku.php'); i kod staje się o wiele czytelniejszy

[CuteOne]

daros17: chyba na odwrót ;p

[PHP] pobierz, plaintext 
 
<?php
 
if(isset($_POST['submit']) {
// jakiś kod
   include 'templates/plik_html_po_submicie.php';
}
else {
 
   include 'templates/plik_html_przed_submitem.php';
}
?>
[PHP] pobierz, plaintext 

To najprostszy sposób na oddzielenie html od php [bardziej wyrafinowane to np. szablony SMARTY]

[daros17]

@CuteOne

fakt na odwrót powinno być , późna godzina można się pomylić

[radziowie]

Może toś pokazać chociaż jeden przykład pliku html, bo dokładnie nie wiem co tam umieścić np. w funkcji warunkowej "submit". Tak jest ledwo kilka linijek html'a, więc nie wiem czy jest potrzeba umieszczania tego w osobnym pliku.
Nie mam pojęcia co w tym kodzie jest nie tak.

Ten post edytował radziowie 11.11.2010, 21:27:04