Forum PHP.pl

Witam. Podejrzewam, że temat był tu wałkowany setki razy ale i tak postanowiłem zapytać.
Zaliczam swoje początki z bazami ( wiem, kod będzie okropny, ale zacząć jakoś trzeba ).

Kod aktualnie wygląda tak, jest to wersja robocza.

[PHP] pobierz, plaintext 
$host = 'localhost';
	$user = 'root';
	$pass = 'root';
	$db_select = 'baza';
	$con=mysqli_connect($host,$user,$pass,$db_select);
	if (mysql_connect($host, $user, $pass) and mysql_select_db($db_select)) {
		$edycja_id = $_POST['id_edytuj'];
		$id = $edycja_id;
		$edycja_tytul = $_POST['tytul'];
		$tytul = htmlspecialchars($edycja_tytul, ENT_QUOTES,'UTF-8');
		$edycja_opis = $_POST['opis'];
		$opis = htmlspecialchars($edycja_opis, ENT_QUOTES,'UTF-8');
		mysqli_query($con,"UPDATE galeria SET tutyl=".$tytul." opis=".$opis." WHERE id=".$id." LIMIT 1 SET CHARSET utf8");
		echo mysql_error();
		echo "<br />Dane zmienione pomyślnie!";
	mysql_close();
	}
[PHP] pobierz, plaintext 

Problem jaki mam jest dość banalny. Dane z formularza z wcześniejszego pliku są przekazywane dobrze. Jeżeli wyświetlę treść zapytania przez echo w miejsce zmiennych dostanę przekazane wartości. mysql_error nie zwraca nic.
Wszystko niby działa pięknie i ładnie lecz nie następuje żadna zmiana w tabeli. Dane pozostają te same. Wyszukiwanie po ID ułatwia i wyklucza tu pomyłki. Kombinowałem już z różnym sposobem zapisu wartości, limitem itp. Niestety nie przyniosło skutków. Jak z INSERT, DELETE itp nie miałem większych problemów (kto szuka ten znajdzie zazwyczaj) tak tutaj utknąłem i nie wiem jak to ugryźć ;]

Może ktoś jakoś pomóc?
Albo wskazać błąd albo podesłać linka do jakiejś strony gdzie zobaczę co robię źle?
Samo zapytanie (SET CHARSET utf8 zostało dodane później i nie zmieniło nic a jest dość przydatne przy polskich znakach) opierałem o tutka z w3schools.com o UPDATE właśnie.
Proszęsię nie czepiać tylko o estetykę kodu. Jak tylko całość zacznie funkcjonować będzie poprawiana pod względem czytelności itp ;]

Ten post edytował zearid 21.03.2013, 17:57:11

W SET wypisujesz co zmieniasz PO PRZECINKACH... u ciebie brakuje.

[PHP] pobierz, plaintext 
SET tutyl=".$tytul." opis=".$opis."
[PHP] pobierz, plaintext 

Daj tak

[PHP] pobierz, plaintext 
SET tutyl=".$tytul.", opis=".$opis."
[PHP] pobierz, plaintext 

Powinno pomóc.

@up
Gdyby wykonał moją instrukcję, dostałby błąd i sam powinien na to wpaść (IMG:style_emoticons/default/smile.gif)

1. Co do przecinków. Wiem, że brakowało. Próbowałem i tak i tak (spotkałem się z różnymi opiniami na ten temat w sieci). Po prostu już nie zmieniłem w poście (mój błąd).
2. Poprawki niesione.

Generalnie jeżeli jest przesyłany tekst z polskimi znakami to pojawia się problem, przesyłając z formularza cyfry dostaję:

[PHP] pobierz, plaintext 
error in sql
Unknown column 'tutyl' in 'field list'
[PHP] pobierz, plaintext 

Od razu mówię, że kolumna tytuł istnieje i jest poprawnie nazwana (kolumny to: id, url, tytul, opis, kategoria)

Przesyłając coś z na przykład literą "ł":
(pierwsza treść to: Przykładowy test zapisu)

[PHP] pobierz, plaintext 
error in sql
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 
'?adowy test zapisu, opis=pliku po edycji WHERE id=18 LIMIT 1' at line 1
[PHP] pobierz, plaintext 

Wszystkie dane są przetwarzane pod kątem utf-8

Formularz:

[PHP] pobierz, plaintext 
<?php
	//Formularz do edycji danych obrazka
	$id_edycja = $_POST['id_edycja'];
	$id = $id_edycja;
	require 'db_config.php';
	if (mysql_connect($host, $user, $pass) and mysql_select_db($db_select)) {
 
		$edycja = mysql_query("SELECT * FROM galeria WHERE id=$id_edycja");
		while($r = mysql_fetch_assoc($edycja)) {
		$tytul = htmlspecialchars($r['tytul'], ENT_QUOTES,'UTF-8');
		$opis = htmlspecialchars($r['opis'], ENT_QUOTES,'UTF-8');;
		echo '<form method="POST" action="edycja_zapis.php" id="edytuj" name="edytuj">
			<input name="id_edytuj" value="'.$id.'" type="hidden" />
			Tytuł: <input id="tytul" name="tytul" value="'.$tytul.'"><br />
			Opis: <textarea id="opis" name="opis">'.$opis.'</textarea><br />
			<input type="submit" value="Zapisz" />
			</form>';
		}
 
	mysql_close();
	}
?>
[PHP] pobierz, plaintext 

Najprawdopodobniej ma także problem ze spacjami w wartościach do zapisu.

Ten post edytował zearid 22.03.2013, 08:17:57

Jedyny Twój problem jest taki że nie stosujesz apostrofów i nie odróżniasz stringów od innych wartości...

[PHP] pobierz, plaintext 
"UPDATE galeria SET tutyl='".$tytul."', opis='".$opis."' WHERE id=".$id." LIMIT 1 SET CHARSET utf8";
[PHP] pobierz, plaintext 

Wcześniej próbowałem i z i bez apostrofów.
Po wprowadzeniu poprawek sugerowanych na początku faktycznie o tym nie pomyślałem i mój błąd. Po dodaniu apostrofów rusza lecz ma jeszcze jeden mały problem.
nie potrafi znaleźć kolumny tytul w tabeli, kolumnę opis bez problemu.

Przyznam, że robię błędy podstawowe i debilne także lecz każdy uczy się najlepiej na błędach.
Do baz nie mogłem się nigdy przekonać. Dają kolosalne możliwości ale trochę właśnie takimi problemami odstraszają.

To nie bazy danych odstraszają tylko beznadziejny, nieczytelny sposób kodowania na rozszezeniu które już nie jest wspierane w PHP (IMG:style_emoticons/default/smile.gif) Poczytaj o PDO, pzygotowaniu zapytań, bindowaniu parametrów i uwierz że te problemy znikną.

@viking nawet PDO nie pomoże jak ktoś robi banalne literówki i nie czyta komunikatów błędów.

Mimo wszystko przy zapisie:

[SQL] pobierz, plaintext 
UPDATE galeria SET tytul=?, opis=? WHERE id=?
[SQL] pobierz, plaintext 

trochę łatwiej skupić się na znalezieniu błędu. Zwłaszcza gdy początkujący (już pomijając że nie czytają komunikatów) gubią się strasznie w poprawnym złączeniu stringów a sprintf mało kto używa.

edit. A to drugie z kolei wynika z faktu że w polskiej wersji dokumentacji słowem o tym nie wspominają jak widzę.

Ten post edytował viking 22.03.2013, 11:11:24

Ok. Głupia literówka. Chyba z siebie niepotrzebnie debila zrobiłem. Niestety na starość człowiek ślepnie (IMG:style_emoticons/default/haha.gif)
Teraz już działa jak potrzeba w takim razie dzięki za pomoc i przepraszam za zamieszanie z nie odnalezioną kolumną ;]

Powiem szczerze, że z bazami coś tam kiedyś było ale to edytor generował do nich kod i za cholerę nie nauczyłem się nic z tego, zwłaszcza że program generował dużo śmieciowego kodu.

PDO - będę musiał poczytać. Z bazami staram się jakoś zacząć a przeniesienie galerii z plików do bazy daje spore ułatwienie przy dodawaniu kolejnych rzeczy.
Robię banalne błędy, niestety. Trochę w tym mojej winy a trochę syna (2.5r), który uwielbia kod (IMG:style_emoticons/default/haha.gif) i dopisywanie byle czego więc trochę na szybkiego się pisze.

Generalnie najlepiej jest jak najwięcej czytać, lecz nie zawsze jest kiedy.
Szukałem sporo o update i dla przykładu wszystko wskazywało na użycie mysqli_query zamiast mysql_query i człowiek się dziwi, że brakuje parametrów.

Jeszcze sporo nauki mnie czeka względem baz itp ale każdy wytknięty błąd mimo, że dość do wkurza, jest dobrą lekcją.


Ten post edytował zearid 22.03.2013, 11:18:22