Forum PHP.pl

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

Ten post edytował aga323 2.08.2007, 12:03:00

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.

Ten post edytował plurr 2.08.2007, 12:14:27

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...

Ten post edytował drPayton 2.08.2007, 14:15:47

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 

Ten post edytował drPayton 2.08.2007, 15:50:40

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

Ten post edytował drPayton 2.08.2007, 18:29:03

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 

Ten post edytował drPayton 2.08.2007, 20:56:00

Kurcze, zrobiłam tak jak mówiłeś, ale w tej chwili to już nic mi nie edytuje ani nie usuwa, nawet tego ostatniego... (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

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 (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) 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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) 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.