Forum PHP.pl > [PHP]SQL nie wpisuje danych do bazy :(

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP]SQL nie wpisuje danych do bazy :(

porywacz

29.03.2012, 18:45:35

Witam, uczę się php i napisałem skrypt, który wpisuje (a raczej powinien) pola z formularza do tabeli w bazie danych. Niestety nic się nie dodaje, mimo tego, że skrypt wykonuje się poprawnie.
Proszę o pomoc

HTML:

[HTML] pobierz, plaintext 
<html>
<head>
<title>xCheats</title>
<charset type="utf-8"/>
</head>
<body>
<h4><u>Formularz dodawania czitów</u></h4>
<form method="post" action="add_cheats_do.php">
 
	<table>
	<tr>
	<td>Dodał:	</td>	<td><input type="text" name="dodal"></td>
	</tr>
	<tr>
	<td>Tytuł gry:</td>	<td><input type="text" name="tytul"></td>
	</tr>
	<tr>
	<td>Czity:	</td>	<td><input type="text" name="cheats"></td>
	</tr>
	<tr>
	<td></td><td><input type="submit" value="Dodaj czity"></td>
	</tr>
	</table>
	</form>
</body>
</html>
[HTML] pobierz, plaintext

PHP:

[PHP] pobierz, plaintext 
<?php
$dodal = $_POST['dodal'];
$tytul = $_POST['tytul'];
$cheats = $_POST['cheats'];
 
$db = mysql_connect('localhost','root','') or die('Nie udało się połączyć z serwerem');
$db = mysql_select_db('xcheats') or die('Nie udało się połączyć z bazą danych');
 
$query = mysql_query("INSERT INTO `cheats` ('id,dodal,tytul,cheats') VALUES('','$dodal','$tytul','$cheats') or die('Nie udało się zapisać danych do tabeli')")
?>
[PHP] pobierz, plaintext

nospor

29.03.2012, 19:07:41

W ostatnim zapytaniu za daleko zamknąłeś cudzysłów ".... Nawet kolorowanie składni dokladnie to pokazuje innym kolorem.

porywacz

29.03.2012, 19:10:20

Zrobiłem tak, ale nadal nie działa. Tym razem wywala błąd z die() : /

[PHP] pobierz, plaintext 
$query = mysql_query("INSERT INTO `cheats` ('id,dodal,tytul,cheats') VALUES('','$dodal','$tytul','$cheats')") or die('Nie udało się zapisać danych do tabeli')
[PHP] pobierz, plaintext

nospor

29.03.2012, 19:31:02

No bo szastasz apostrofami na lewo i praw... co to niby ma byc:
cheats'
?
Po co ten apostrof tam?

A na przyszłość wyświetlaj błąd zapytania. Tu masz napisane jak:
Temat: Jak poprawnie zada pytanie

Samokodziarz

29.03.2012, 20:13:55

A spróbuj tak:

[PHP] pobierz, plaintext 
$query = mysql_query("INSERT INTO `cheats` (`id`, `dodal`, `tytul`, `cheats`) VALUES('','$dodal','$tytul','$cheats')") or die('Nie udało się zapisać danych do tabeli')
[PHP] pobierz, plaintext

nospor

29.03.2012, 20:16:08

@Samokodziarz zrobiłeś jeszcze gorzej..... Przecież podałem rozwiązanie w poprzednim poscie. Czemu na siłe kombinujesz i to jeszcze o wiele gorzej?

g0ther

29.03.2012, 20:34:29

@nospor a co Ci da usunięcie jednego apostrofu zaraz za cheats ?

porywacz

29.03.2012, 20:40:33

gother ma racje

przed id też jest apostrof, którego nie powinno być ^^ Poprawiłem już ten błąd i wszystko już działa jak należy, więc dzięki za pomoc, chociaż nospor, powinieneś być trochę bardziej wyrozumiały dla POCZĄTKUJĄCEGO

Samokodziarz

29.03.2012, 21:13:28

nospor - ja tam jednak dla pewności zawsze wolę nazwy baz, tabel i kolumn otaczać apostrofami. Fakt, użyłem złych (' '), ale już je zamieniłem na prawidłowe (` `).

nospor

30.03.2012, 07:17:21

Cytat

@nospor a co Ci da usunięcie jednego apostrofu zaraz za cheats ?

A to, że nie widziałem tego przed ID. Go oczywiście też trzeba było usunąć. No ale kurka, naprawdę wszystko palcem trzeba pokazywać?

Cytat

chociaż nospor, powinieneś być trochę bardziej wyrozumiały dla POCZĄTKUJĄCEGO

A ja mogę powiedzieć: początkujący patrz co piszesz. To nie jest powieść sienkiewicza tylo jedna prosta linijka kodu. Poza tym ani ja na ciebie nie krzyczałem, ani ja ciebie od głupków nie wyzywałem więc nie wiem o co ci chodzi z tą wyrozumiałością...

Cytat

nospor - ja tam jednak dla pewności zawsze wolę nazwy baz, tabel i kolumn otaczać apostrofami. Fakt, użyłem złych (' '), ale już je zamieniłem na prawidłowe (` `).

No tak, użyłeś złych. Bardzo złych i o to mi właśnie chodziło.

g0ther

30.03.2012, 08:20:50

Cytat(nospor @ 30.03.2012, 08:17:21 )

A to, że nie widziałem tego przed ID. Go oczywiście też trzeba było usunąć. No ale kurka, naprawdę wszystko palcem trzeba pokazywać?

Jak ktoś jest początkujący to ... tak, trzeba palcem pokazywać.

nospor

30.03.2012, 08:24:47

A potem się przyzwyczai jeden z drugim ze z każdą pierdułą na forum będzie leciał. Mówię to z wieloletniej praktyki. Nawet na początkującym należy wymuszać, by pewne rzeczy próbował sam patrzeć, a już szczególnie przy AŻ jednej linijce kodu.

g0ther

30.03.2012, 08:29:01

Nie mówię, że nie masz racji - też jestem za dawaniem wędki, a nie rybki, ale niektórych trzeba czasami 'popchnąć do przodu' chociażby po to, żeby kiedyś stwierdzili, że się do tego nie nadają

(bez aluzji).

nospor

30.03.2012, 08:32:43

No i ja przecież podałem autorowi rozwiązanie. Pokazałem palcem - "o tu ten apostrof jest niepotrzebny". Zrobiłem tak? Zrobiłem.
Fakt, przegapiłem jeszcze jeden apostrof wcześniej. No ale to już mógł autor sam sobie domyśleć.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.