Witam. Otóż mam prosty skrypcik do edytowania kont użytkowników. Wiadomo - jeden plik z formularzem a drugi wysyłający to wszystko do bazy. No i mam mały problemik z wysłaniem do bazy bo nie wiem za bardzo jak rozwiązać kwestię edytowania. Podaję mój kod:
kawałek z formularzem z pliku 1:

[PHP] pobierz, plaintext 
<?php
$zapytanie = mysql_query('SELECT * FROM users WHERE ranga = 0');
 
echo '<form name = "educz" action = "educz2.php" method = "post" />';
 
echo '<table border = "1"><tr><td>ID</td><td>Imię</td>
<td>Nazwisko</td><td>Klasa</td><td>Login</td><td>Hasło</td></tr>';
 
while ($wyswietl = mysql_fetch_object($zapytanie)) {
 
echo '<tr>';
 
echo '<td><input type = "text" name = "id" value = "'.$wyswietl -> id.'" /></td>';
echo '<td><input type = "text" name = "imie" value = "'.$wyswietl -> imie.'" /></td>';
echo '<td><input type = "text" name = "nazwisko" value = "'.$wyswietl -> nazwisko.'" /></td>';
echo '<td><input type = "text" name = "klasa" value = "'.$wyswietl -> klasa.'" /></td>';
echo '<td><input type = "text" name = "login" value = "'.$wyswietl -> login.'" /></td>';
echo '<td><input type = "text" name = "haslo" value = "'.$wyswietl -> haslo.'" /></td>';
 
echo '</tr>';
 
}
 
echo '</table>';
echo '<input type="submit" value="wyslij" />';
echo '</form>';
?>
[PHP] pobierz, plaintext 

pliczek 2:

[PHP] pobierz, plaintext 
<?php
$id = ($_POST['id']);
$imie = ($_POST['imie']);
$nazwisko = ($_POST['nazwisko']);
$klasa = ($_POST['klasa']);
$login = ($_POST['login']);
$haslo = ($_POST['haslo']);
 
//------------------
 
if(!empty($id)){
 
	$query = "UPDATE `users` SET id='$id', imie='$imie', nazwisko='$nazwisko', klasa='$klasa', login='$login', haslo='$haslo'" or die(mysql_error());
	$result = mysql_query ($query);	
}
else{
	$query = "DELETE FROM `users` WHERE id='$id' " or die(mysql_error());
	$result = mysql_query ($query);   
}
 
echo 'Dane ucznia/ów zostały zmienione!';
 
header( 'Location: ses.php' );
?>
[PHP] pobierz, plaintext 

Nie wiem czy to przez te wakacje czy co, ale jakoś nie mogę ostatnio za bardzo wymyślić żadnego sensownego rozwiązania... proszę o pomoc

rzucilo mi sie w oczy usuwanie uzytkownika po id jesli $id jest puste - masz empty($id) to delete where id='$id'.

No i ja bym w ogole nie ruszal pola id w update.

Chciałem tylko zauważyć Pani Agnieszko,że zapytanie dla mySQL nie ma żadnych warunków. Otóż akcja update bez warunków ( np. WHERE id=... ) spowoduje zmianę wszystkich wartości w tabeli na te podane w formularzu, chyba,że występują klucze wtedy powinno wyrzucić błąd o próbie duplikowania kluczy w tabeli.

powinno wyglądać tak :

[SQL] pobierz, plaintext 
$query = "UPDATE `users` SET id='$id', imie='$imie', nazwisko='$nazwisko', klasa='$klasa', login='$login', haslo='$haslo' WHERE id=$id
[SQL] pobierz, plaintext 

Ponieważ rozumiem,że id jest unikalne

Co do usuwania to o ile dobrze pamiętam składnia była

[SQL] pobierz, plaintext 
DELETE id FROM users WHERE id='$id'
[SQL] pobierz, plaintext 

Posłuchałam waszych rad i trochę to pomogło, lecz nie do końca. W tej chwili mogę pięknie edytować ale tylko użytkownika, który znajduje się na końcu listy użytkowników, resztę jak pozmieniam to nie aktualizuje sie w bazie. A usuwanie w tej chwili mi w ogóle nie działa.

Teraz zauważyłem,że tobie się wyświetla cała lista i chciałabyś,żeby zmieniało Ci wszystko naraz ? To chyba niemożliwe.

Jeżeli pole id to standardowy autoinkrementowany znacznik, to:

[SQL] pobierz, plaintext 
UPDATE pole = wartosc WHERE id = $_POST['id']
[SQL] pobierz, plaintext 

Natomiast kasowanie:

[SQL] pobierz, plaintext 
DELETE FROM tabela WHERE id = $_POST['id']
[SQL] pobierz, plaintext 

Albo samo id, skoro wcześniej przepisujesz do zmiennych
Tyle, że u Ciebie id jest podawane z formularza, jeżeli nie jest określone to skąd system ma wiedzieć, który rekord skasować?
A jeśli pole id to u Ciebie jakieś inne pole, niezwiązane a autoinkrementem, to musisz (powinnaś) dodać jakieś tego właśnie typu pole...

Tylko w sumie za bardzo nie wiem jak dodać, w jakiej formie zrobić to pole, może ktoś miałby jakiś pomysł jak to zrobić?

mozesz teraz zmodyfikowac tabele

i najlepiej zeby to id bylo kluczem glownym w twojej tabeli

Dodasz jako nowe pole:

[SQL] pobierz, plaintext 
ALTER TABLE `tabela` ADD `uid` INT UNSIGNED NOT NULL AUTO_INCREMENT PRIMARY KEY;
[SQL] pobierz, plaintext 

Zmienisz istniejące:

[SQL] pobierz, plaintext 
ALTER TABLE `tabela` CHANGE `id` `id` INT UNSIGNED NOT NULL AUTO_INCREMENT
[SQL] pobierz, plaintext 

[SQL] pobierz, plaintext 
ALTER TABLE `tabela` DROP PRIMARY KEY, ADD PRIMARY KEY(`id`)
[SQL] pobierz, plaintext 

e.. bo już się trochę gubię... Mam tak tabela users i w niej pole id tym integer (10) Usigned, not null, auto_increment. Później mam jeszcze jakieś tam dane to są varchary (imię, nazwisko itd.). I mam indeks na id typ unique.
I teraz nie wiem czy mam to dobrze czy coś zmienić...?

Prawie dobrze, tylko, że nie UNIQUE tylko PRIMARY KEY (chociaż u Ciebie to nie będzie różnicy, to tylko przy relacjach), więc SQL'a zostaw jak jest.
Ale skoro masz autoinkrementowane pole id w tabeli, to ono samo będzie się uzupełniać, wywal to pole z formularza!
I wtedy:

[SQL] pobierz, plaintext 
UPDATE tabela SET pole1='wartosc1', pole2='wartosc2' WHERE id = {$_POST['id']}
[SQL] pobierz, plaintext 

A id w formularzu daj najlepiej w polu typu hidden. Czyli:

[PHP] pobierz, plaintext 
<?php
while ($wyswietl = mysql_fetch_object($zapytanie)) {
 
echo '<tr>';
 
echo '<td><input type = "hidden" name = "id" value = "'.$wyswietl -> id.'" /></td>';
 
(...)
 
echo '</tr>';
 
}
?>
[PHP] pobierz, plaintext 

Tylko, że:
1. Jeśli chcesz edytować tylko jednego użytkownika, to formularz musisz umieścić w pętli razem z tymi polami
2. Jeśli chcesz wszystkich, to napisz, bo wtedy będzie nieco inaczej

No interesuje mnie raczej żeby edytować wszystkich. Bo to by było najwygodniejsze, chyba, że byłoby to zbyt trudne do wykonania, bo doświadczenia zbyt dużego to w pisaniu skryptów jeszcze nie mam. Wtedy trzeba by wymyślić inne rozwiązanie

Wtedy nazwy pól (name="nazwa") muszą być tabelaryczne, czyli np <input type="text" name="nazwa[]" (...) />
a w skrypcie php po tych polach "jedziesz" pętlą for, czyli np.:

[PHP] pobierz, plaintext 
<?php
$j=sizeof($_POST['id']);
for ($i=0;  $i<$j; $i++) {
	query("UPDATE tabela SET imie = {$_POST['imie'][$i]}, nazwisko = {$_POST['nazwisko'][$i]} WHERE id = {$_POST['id'][$i]}");
}
?>
[PHP] pobierz, plaintext 

Kurcze, zrobiłam tak jak mówiłeś, ale w tej chwili to już nic mi nie edytuje ani nie usuwa, nawet tego ostatniego...

To co napisałem to idea, nie sprawdzałem jej. Teraz nie mam czasu, jak będę miał a w międzyczasie nie napisze tego nikt inny to wtedy się tym zajmę, sorry.

Dam ci przykład:

[PHP] pobierz, plaintext 
<?php
if ($co == 'popraw') {
if ($login && $haslo) {
$query = "UPDATE admin SET login='$login', haslo='$haslo' WHERE id='$id';";
$wynik = mysql_query ($query);
}
} elseif ($co == 'edytuj') {
$query = "SELECT * FROM admin WHERE id='$id';";
$wynik = mysql_query ($query);
$row = mysql_fetch_array ($wynik);
$id = $row[0]; $login = $row[1]; $haslo = $row[2];
echo '<form method="post">
<input type="hidden" name="co" value="popraw">
<input type="hidden" name="id" value="'.$id.'">
Login: <input type="text" name="login" value="'.$login.'"><br>
Hasło: <input type="password" name="haslo" value="'.$haslo.'"><br>
<input type="submit" value="Popraw"></form>';
} elseif ($co == 'skasuj') {
$wynik = mysql_query ("DELETE FROM admin WHERE id='$id';");
}
$wynik = mysql_query ("SELECT * FROM admin;");
echo "<table border=1 cellspacing=0 cellpadding=2><tr><td><b>Login:</b></td><td></td><td></td>n";
while ($row = mysql_fetch_array ($wynik)) {
$id = $row[0]; $login = $row[1]; $haslo = $row[2];
echo "<tr><td>$login</td><td><a href="?co=skasuj&id=$id">Skasuj</a></td><td><a href="?co=edytuj&id=$id">Edytuj</a></td></tr>";
}
echo '</table>';
?>
[PHP] pobierz, plaintext 

Tutaj jest, jak używam.
Masz tutaj edycje (dwa pola, login i hasło) oraz usuwanie.

A może lepiej byłoby to rozwiązać w inny sposób, bo ten na razie nie działa Skoro edytowanie wszystkich naraz jest raczej mało możliwe może lepiej zrobić listę użytkowników i po kliknięciu na dana osobę zmieniać tam jej dane... Co o tym myślicie?

Jest możliwe ;]

[PHP] pobierz, plaintext 
<form name = "educz" action = "educz2.php" method = "post" />
 
  <table border = "1">
	<tr>
	  <td>ID</td>
	  <td>Imię</td>
	  <td>Nazwisko</td>
	  <td>Klasa</td>
	  <td>Login</td>
	  <td>Hasło</td>
	</tr>
 
<?php
 
	$zapytanie = mysql_query( 'SELECT * FROM users WHERE ranga = 0' );
	while( $wyswietl = mysql_fetch_object( $zapytanie ) ):
 
?>
 
	<tr>
	  <td><?php echo $wyswietl->id; ?></td>
	  <td><input type="text" name="imie[<?php echo $wyswietl->id; ?>]" value="<?php echo htmlspecialchars( $wyswietl->imie ); ?>" /></td>
	  <td><input type="text" name="nazwisko[<?php echo $wyswietl->id; ?>]" value="<?php echo htmlspecialchars( $wyswietl->nazwisko ); ?>" /></td>
	  <td><input type="text" name="klasa[<?php echo $wyswietl->id; ?>]" value="<?php echo htmlspecialchars( $wyswietl->klasa ); ?>" /></td>
	  <td><input type="text" name="login[<?php echo $wyswietl->id; ?>]" value="<?php echo htmlspecialchars( $wyswietl->login ); ?>" /></td>
	  <td><input type="text" name="haslo[<?php echo $wyswietl->id; ?>]" value="<?php echo htmlspecialchars( $wyswietl->haslo ); ?>" /></td>
	</tr>
 
<?php
 
	endwhile;
 
?>
 
  </table>
 
  <input type="submit" name="submit" value="Wyślij" />
</form>
[PHP] pobierz, plaintext 

A później w drugim pliku zrób sobie:

[PHP] pobierz, plaintext 
<?php
 
	if( isset( $_POST['submit'] ) )
	{
		echo '<pre>' . print_r( $_POST, true ) . '</pre>';
		exit;
	}
 
?>
[PHP] pobierz, plaintext 

Wyświetlone wyniki powinny wskazać ci kierunek myślenia.

Aha, teraz to rzeczywiście trochę jaśniej Ale mam pytanie teraz czy ten drugi plik może zostać mniej więcej tak samo jak miałam go wcześniej, czy tez pisać zupełnie od nowa i czy to edytowanie i usuwani musi być w pętli czy inaczej to rozwiązać?

Trzeba trochę przebudować ten plik, tak żeby w pętli aktualizował dane.


Rozwiązanie pierwsze: Aktualizuj wszystkie. Wszystkie dane zostaną zaktualizowane, nawet te, które się nie zmieniły.

Rozwiązanie drugie: JS. Do każdego wiersza dodać jeszcze jedno pole hidden:

[HTML] pobierz, plaintext 
<input type="hidden" name="zmodyfikowane[<?php echo $wyswietl->id; ?>]" value="0" />
[HTML] pobierz, plaintext 

W każdym z pozostałych pól tego wiersza dodać zdarzenie onKeyPress, które zmieni wartość tego pola na "1"
Po wysłaniu formularza będzie wiadomo które pola zostały zmodyfikowane (i należy je uaktualnić w bazie), a które nie. Wady: wymaga włączonego JS.

Mój plik w tej chwili wygląda tak:

[PHP] pobierz, plaintext 
<?php
session_start();
 
include ("connect.php");
//-------------------
 
	if(empty($_POST['login'])){
	$query = "DELETE FROM `users` WHERE id = {$_POST['id']} " or die(mysql_error());
	$result = mysql_query($query);
 
   }
   else{
	$query ="UPDATE `users` SET imie = {$_POST['imie']}, nazwisko = {$_POST['nazwisko']}, klasa = {$_POST['klasa']}, login = {$_POST['login']}, haslo = {$_POST['haslo']}  WHERE id = {$_POST['id']}" or die (mysql_error());
	$result = mysql_query ($query);
	 
   }
header( 'Location: ses.php' );
 
?>
[PHP] pobierz, plaintext 

Wiem, że teraz muszę to wszystko dać do pętli, tylko nie wiem w którym momencie ją wcisną i jakie dane do niej wstawić, więc proszę o małą pomoc

1. Co do kasowania wierszy to ja dodałbym jeszcze jednego checkboxa:

[HTML] pobierz, plaintext 
<input type="checkbox" name="kasuj[<?php echo $wyswietl->id; ?>]" value="1" />
[HTML] pobierz, plaintext 

2. Teraz zauważyłem, że można było lepiej nazwać te pola, a mianowicie zrobić coś w rodzaju:

[HTML] pobierz, plaintext 
<input type="text" name="id[<?php echo $wyswietl->id; ?>][imie]" />
<input type="text" name="id[<?php echo $wyswietl->id; ?>][nazwisko]" />
<input type="text" name="id[<?php echo $wyswietl->id; ?>][klasa]" />
...
<input type="checkbox" name="id[<?php echo $wyswietl->id; ?>][kasuj]" value="1" />
[HTML] pobierz, plaintext 

ale niech już zostanie tak jak jest. A dane z formularza odbierasz w ten sposób:

[PHP] pobierz, plaintext 
<?php
 
	if( isset( $_POST['submit'] ) )
	{
		foreach( $_POST['imie'] as $id => $item )
		{
			if( !empty( $_POST['kasuj'][$id] ) )
			{
				// Kasujemy wiersz o ID = $id
 
				continue;
			}
 
 
			// Aktualizujemy wiersz o ID = $id
			$imie = $_POST['imie'][$id];
			$nazwisko = $_POST['nazwisko'][$id];
			// ...
		} 
	}
 
 ?>
[PHP] pobierz, plaintext 

W tej chwili skrypt elegancko usuwa użytowników Ale nadal mam problem z edytowaniem. Oto mój kod:

[PHP] pobierz, plaintext 
<?php
session_start();
 
include ("connect.php");
//-------------------
 
	if( isset( $_POST['submit'] ) )
	{
		foreach( $_POST['imie'] as $id => $item )
		{
			if( !empty( $_POST['kasuj'][$id] ) )
			{
				// Kasujemy wiersz o ID = $id
			 $query = "DELETE FROM `users` WHERE id = $id " or die(mysql_error());
			 $result = mysql_query ($query);
 
				continue;
			}
			else{
 
			$imie = $_POST['imie'][$id];
			$nazwisko = $_POST['nazwisko'][$id];
			$klasa = $_POST['klasa'][$id];
			$login = $_POST['login'][$id];
			$haslo = $_POST['haslo'][$id];
 
			  $query = ("UPDATE `users` SET imie = $imie, nazwisko = $nazwisko, klasa = $klasa, logi
n = $login, haslo = $haslo  WHERE id = $id") or die (mysql_error());
			  $result = mysql_query ($query);
		   }
		} 
	}
 
 ?>
[PHP] pobierz, plaintext 

tekst trzeba brac w ciapki ''
czyli nie:
imie = $imie
a:
imie = '$imie'
analogicznie inne pola tekstowe.

pozatym nie:

[PHP] pobierz, plaintext 
<?php
$zapytanie = '....' or die(mysql_error());
mysql_query($zapytanie);
?>
[PHP] pobierz, plaintext 

a:

[PHP] pobierz, plaintext 
<?php
$zapytanie = '....';
mysql_query($zapytanie)  or die(mysql_error());
?>
[PHP] pobierz, plaintext 

Wielkie dzięki, teraz wszystko śmiga, że aż miło I sorry, że pisze z takimi głupimi błędami, postaram się już ich nie popełniać