[php/mysql] Problem przy dodawniu do bazy Opcje

[aga323]

Witam. Otóż mój problem wygląda następująco: mam zrobiony dziennik internetowy z ocenami w bazie tabele userzy, oceny itd. w tabeli oceny mam kolumny id_user, id_przedmiot, semestr i ocena. I mam problem przy kolumnie semestr. Gdy przez skrypt dodaje ocenki na pierwszy semestr wszystko ładnie działa bez zastrzeżeń, ale gdy dodaję na drugi semestr to wcześniej mi dodawało na oba semestry tą ocenę, a teraz to mi nawet w ogóle nie dodaje... proszę o pomoc
Oto mój kod:

[PHP] pobierz, plaintext 
<?
session_start();
ob_start();
 
include ("connect.php");
//------------------
// Tablica post
//---------------
$ilosc = count($_POST);
$powtuzen =  $ilosc /2;
//--------------------
$id_przedmiot = $_POST['przedmiot'];
$query = "SELECT id FROM `users`";
$result = mysql_query ($query);
for ($i=$ilosc; $i>=1; $i--)
{
	$t = 'T'.$i;
	$t2 = 'T2'.$i;
	$b = $_POST[$t];
	$b2 = $_POST[$t2];
	$u = 'U'.$i;
	$u2 = 'U2'.$i;
	$user = $_POST[$u];
	$user2 = $_POST[$u2];
	$semestr = 1;
	$semestr2 = 2;
	if(!empty($b))
	{
		$a = explode(',', $b);
		$cos =  count($a);
		for ($j=0; $j<$cos; $j++)
		{
			$ocena =  $a[$j];
			if(!empty($ocena))
			{
				//-------------------------
				//   Zapytanie
				//-------------------------
				$query = "INSERT INTO `oceny` (id_user, id_przedmiot, semestr, ocena) VALUES ('$user', '$id_przedmiot', '1', '$ocena')" or die(mysql_error());
				$result = mysql_query ($query);
				//-------------------------
			}
		}
	}
 
 
 
	else if(!empty($b2))
	{
		$a = explode(',', $b2);
		$cos =  count($a);
		for ($k=0; $k<$cos; $k++)
		{
			$ocena2 =  $a[$k];
			if(!empty($ocena2))
			{
				//-------------------------
				//   Zapytanie
				//-------------------------
				$query = "INSERT INTO `oceny` (id_user, id_przedmiot, semestr, ocena) VALUES ('$user2', '$id_przedmiot', '2', '$ocena2')" or die(mysql_error()) ;
				$result = mysql_query ($query);
 
 
				//-------------------------
			}
		}
	}
}
header( 'Location: ses.php?co=lista_p' );
?>
[PHP] pobierz, plaintext 

[Garrappachc]

A ten semestr to jaki typ? Tinyint, int, char...

[Hazel]

No to tak. Przede wszystkim zapis

[PHP] pobierz, plaintext 
<?php
$query = "INSERT INTO `oceny` (id_user, id_przedmiot, semestr, ocena) VALUES ('$user2', '$id_przedmiot', '2', '$ocena2')" or die(mysql_error()) ;
?>
[PHP] pobierz, plaintext 

bije po oczach, bo błąd MySQL nie może wyskoczyć, dopóki w ogóle nie ma zapytania do bazy. W tym wierszu przypisujesz do zmiennej $query łańcuch znaków, będący nawiasem mówiąc zapytaniem do bazy. Nic się tutaj nie dzieje z MySQL. Dopiero wiersz kolejny, to jest $result = mysql_query($query) wykonuje zapytanie i dopiero on ma prawo wygenerować błąd MySQL. Poprawnym zapisem jest zatem:

[PHP] pobierz, plaintext 
<?php
$query = "INSERT INTO `oceny` (id_user, id_przedmiot, semestr, ocena) VALUES ('$user2', '$id_przedmiot', '2', '$ocena2')";
$result = mysql_query($query) or die(mysql_error());
?>
[PHP] pobierz, plaintext 

Teraz funkcja mysql_error() ma szansę zwrócić cokolwiek. No i nie ma sensu zagłębiac się w kod, dopóki nie uruchomisz tego co jest i nie zobaczysz, co takiego wypluwa zapytanie do bazy. Na oko innych poważnych błędów nie widać, dlatego najlepiej będzie, jak zobaczysz przy odpowiednim wywołaniu mysql_error(), co jest nie tak.
Pozdrawiam.

Ten post edytował Hazel 28.12.2007, 15:02:18

[aga323]

Zrobiłam tak jak mówiłeś Hazel, ale nadal nie wywala mi żadnego błędu a ocen po prostu nie dodaje
Może błąd leży w pliku, który przekazuje wcześniej te dane... W razie co podam ten wcześniejszy:

[PHP] pobierz, plaintext 
<?
 
include ("connect.php");
$id = $_GET['id'];
$klasa = $_SESSION['klasa'];
//-------------------------
//   Zapytanie
//-------------------------
$przedmioty = mysql_query("SELECT * FROM przedmioty WHERE id=$id");
$row = mysql_fetch_assoc($przedmioty);
//-------------------------
 
echo '<form name="ocen" action="up.php" method="POST">';
echo '<input type="hidden" name="przedmiot" value="'.$id.'" />';
echo 'Przedmiot: ';
echo $row['Nazwa'];
 
echo '<table border="1"><tr><td>Przedmioty</td><td>Semestr 1</td><td>Semestr 2</td></tr>';
 
 
//-------------------------
//   Zapytanie
//-------------------------
 
$query = "SELECT * FROM `users` WHERE klasa = '$klasa' AND ranga ='0'";
$result = mysql_query ($query);
$uczen = $wynik['id'];
$klasa = $_GET['klasa'];
//-------------------------
 
while($wynik=mysql_fetch_array($result))
{
 
	if($wynik['id'] != 1)
	{
 
		echo "\n";
		echo '<tr>';
		echo '<td>';
 
		echo $wynik['imie']. ' ' .$wynik['nazwisko'];
		echo '</td>';
		echo '<td>';
		$uczen = $wynik['id'];
		$klasa = $_GET['klasa'];
		//-------------------------
		//   Zapytanie
		//-------------------------
		$oceny = mysql_query("SELECT * FROM `oceny` WHERE id_user=$uczen AND id_przedmiot=$id AND semestr
=1");
		$ilosc = mysql_num_rows($oceny);
		//-------------------------
 
		if($ilosc != 0)
		{
		   
			while($wiersz = mysql_fetch_assoc($oceny))
			{
 
				echo $wiersz['ocena']. ', ';
			}
			echo '<br>';
			echo '<input type="hidden" name="U'.$cos.'" value="' .$wynik['id']. '" />';
 
			echo "\n";
			echo '<input type="text" name="T' .$cos. '" value="" />';
		}
		else
		{
			echo 'Brak ocen.';
 
			echo "\n";
			echo '<input type="hidden" name="U'.$cos.'" value="' .$wynik['id']. '" />';
			echo '<br>';
 
			echo "\n";
			echo '<input type="text" name="T' .$cos. '" value="" />';
 
			echo "\n";
		}
		$cos++;
		echo '</td>';
		echo '<td>';
		$uczen = $wynik['id'];
		$klasa = $_GET['klasa'];
		$cos2=1;
		//-------------------------
		//   Zapytanie
		//-------------------------
		$oceny2 = mysql_query("SELECT * FROM `oceny` WHERE id_user=$uczen AND id_przedmiot=$id AND semestr
=2");
		$ilosc2 = mysql_num_rows($oceny2);
		//-------------------------
 
		if($ilosc2 != 0)
		{
		   
			while($wiersz2 = mysql_fetch_assoc($oceny2))
			{
 
				echo $wiersz2['ocena']. ', ';
			}
			echo '<br>';
			echo '<input type="hidden" name="U2'.$cos2.'" value="' .$wynik['id']. '" />';
 
		   echo "\n";
			echo '<input type="text" name="T2' .$cos2. '" value="" />';
		}
		else
		{
			echo 'Brak ocen.';
 
			echo "\n";
			echo '<input type="hidden" name="U2'.$cos2.'" value="' .$wynik['id']. '" />';
			echo '<br>';
 
			echo "\n";
			echo '<input type="text" name="T2' .$cos2. '" value="" />';
 
			echo "\n";
		}
	   $cos++;
		echo '</td>';
		echo '</tr>';
 
			echo "\n";
	}
}
echo '</table>';
echo '<input type="submit" value="wyslij" />';
echo '</form>';
?>
[PHP] pobierz, plaintext 

Jakby co to typ tego semestru to tinyint...

Pomimo tego, że nie wywala mi żadnego błędu doszłam do fragmentu, w którym może siedzieć przyczyna. Zaraz podam ten kawałek kodu tylko wyjasnię o co mi chodzi. Więc, gdy w drugim semestrze mam zmienną $cos to wtedy dodaje mi ocenę na pierwszy i drugi semestr, a gdy ta zmienna zmienie na $cos2 to wtedy mi w ogóle nie dodaje nigdzie oceny :/ to teraz podaje kod:

[PHP] pobierz, plaintext 
<?php
$cos=1;
		$oceny2 = mysql_query("SELECT * FROM `oceny` WHERE id_user=$uczen AND id_przedmiot=$id AND semestr
=2");
		$ilosc2 = mysql_num_rows($oceny2);
		//-------------------------
 
		if($ilosc2 != 0)
		{
		   
			while($wiersz2 = mysql_fetch_assoc($oceny2))
			{
 
				echo $wiersz2['ocena']. ', ';
			}
			echo '<br>';
			echo '<input type="hidden" name="U2'.$cos.'" value="' .$wynik['id']. '" />';
 
		   echo "\n";
			echo '<input type="text" name="T2' .$cos. '" value="" />';
		}
		else
		{
			echo 'Brak ocen.';
 
			echo "\n";
			echo '<input type="hidden" name="U2'.$cos.'" value="' .$wynik['id']. '" />';
			echo '<br>';
 
			echo "\n";
			echo '<input type="text" name="T2' .$cos. '" value="" />';
 
			echo "\n";
		}
	   $cos++;
		echo '</td>';
		echo '</tr>';
 
			echo "\n";
	}
?>
[PHP] pobierz, plaintext 

Jakby co to w pierwszym semestrze jest zmienna $cos zrobiona i tam wszystko działa.
Na prawdę proszę o pomoc, bo zależy mi na tym, żeby to szybko zrobić.