Forum PHP.pl

Witam, oto mój problem. Po zaznaczeniu rekordu i wciśnięciu EDYTUJ zadany rekord wyświetla się i jest gotowy do edycji. Po wprowadzeniu zmian i kliknięciu ZAPISZ, zmiany zostają zachowane w zadanym rekordzie jednak tworzy się kolejny z tymi samymi danymi. Jak temu zapobiec? Kod w linku ponieważ ograniczenia długości/wielkości postów nie pozwoliły wkleić go tutaj.

[PHP][MySQL]Skąd się bierze dodatkowy rekord?

nie wykonujesz podczas edycji zarówno UPDATE jak i INSERT na bazie danych ?

Trochę bardziej czytelne: http://wklej.org/hash/c0555ed23a5/

w 172 linii dodaj warunek :

[PHP] pobierz, plaintext 
 && !isset($_POST['edytuj'])
[PHP] pobierz, plaintext 

Wkleiłem w 171 aby w ogóle nie wchodzić do sekcji dodawania nowego wpisu. Sądzę, że rozwiązało problem ale przetestuję jeszcze.

Okazało się, że podczas kiedy użytkownik wciska EDIT dodaje mu automatycznie nowy pusty rekord (wyświetlając też ten do edycji) ale dzieje się to tylko w Operze. Jak temu zaradzić?

Nie bardzo potrafie się w tym kodzie odnaleźć, być może dlatego, że już późno. Niemniej jednak spróbuj zamiast kolejnych if-ów dawać elseif, na zasadzie:

[PHP] pobierz, plaintext 
if(isset($POST['edytuj'])) {
     //cośtam
}
elseif(isset($_POST['edit'])) {
     //cośtam cośtam
}
else {
     //cośtam cośtam cośtam
}
[PHP] pobierz, plaintext 

Tym sposobem wykona się tylko jeden warunek, w przeciwnym razie wykonasz jeden i przejdziesz do następnego - i wykonasz go, jeśli będzie prawdziwy, a chyba nie to było Twoim zamierzeniem (jeśli się mylę, to sorki).

Ten post edytował Michasko 18.01.2013, 01:18:55

Nie obraź się kolego, ale ten Twój kod to zabójstwo dla serwera baz danych. Co chwila otwierasz nowe połączenia i ustawiasz kodowanie:

[PHP] pobierz, plaintext 
connection();
$ins  = mysql_query("SET NAMES 'utf8'"); 
$ins  = mysql_query("SET CHARACTER SET 'utf8_general_ci'"); 
[PHP] pobierz, plaintext 

podczas gdy powinieneś to zrobić tylko raz, na samym początku skryptu. Poza tym w kilku miejscach dwukrotnie wykonujesz takie same zapytania niemal jedno pod drugim, jak np.:

[PHP] pobierz, plaintext 
$sql = "SELECT * FROM delegacja WHERE id='$opcja'";
$sql = "SELECT DISTINCT nazwa FROM miasta";
[PHP] pobierz, plaintext 

Gubisz niektóre tagi HTML, co powoduje, że nie można kodu prawidłowo sformatować w edytorze z podświetlaniem składni.

Kod zdecydowanie wymaga reorganizacji, głównie ze względu na to, że otwartych połączeń z serwerem baz danych masz w optymalnym ustawieniu chyba 5, a niepotrzebnych zapytań chyba jeszcze więcej (choć tego nie liczę).

Ten post edytował mortus 18.01.2013, 09:02:39

Dzięki za konstruktywną krytykę, usunąłem już zbędne linie z połączeniem i kodowaniem ale jeżeli chodzi o takie same zapytania... to nie bardzo rozumiem. Te dwa, które podałeś jako przykład pobierają dane z dwóch różnych tabel. Co do tagów HTML to raczej wszystkie są i mój edytor podświetla mi bez zarzutu. Może kwestia nie przeklejenia tam całego kodu spowodowała problem z tagami. Jeżeli mógłbyś jeszcze coś doradzić to chętnie się zastosuję, jeżeli miało by to skrócić mój kod lub zoptymalizować go w jakimś sensie.