[PHP] Błąd w pliku wtf? Opcje

[muk4]

Oto mój skrypt:

[PHP] pobierz, plaintext 
<?
include 'config.php';
echo" Aby dodać rekord wypeij formularz!";
 
if($_POST['wyslij']{
 
$query = "INSERT INTO records (kill, dead, exp, kasa, godz_start, min_start, godz_kon, min_kon) VALUES ($_POST['kill'], $_POST['dead'], $_POST['exp'], $_POST['kasa'], $_POST['godz_start'], $_POST['min_start'], $_POST['godz_kon'], $_POST['min_kon'])";
 
$wykonaj = mysql_query($query);
 
} else {
 
echo"Wypełnij formularz:";
echo"
<form action="dodaj.php" method="post">
<table><tr><td>
Zabojstwa: <INPUT NAME="kill" TYPE="text" SIZE=10><BR />
Zginiecia: <INPUT NAME="dead" TYPE="text" SIZE=10><BR />
Doswiadczenie: <INPUT NAME="exp" TYPE="text" SIZE=10><BR />
Dinary: <INPUT NAME="kasa" TYPE="text" SIZE=10><BR />
Godzina startu: <INPUT NAME="godz_start" TYPE="text" SIZE=10> :
 <INPUT NAME="min_start" TYPE="text" SIZE=10><BR />
Godzina konca: <INPUT NAME="godz_kon" TYPE="text" SIZE=10> :
 <INPUT NAME="min_kon" TYPE="text" SIZE=10><BR />
 
</td></tr>
<tr><td align=center>
<INPUT TYPE="submit" name="wyslij" value="Wyslij">
</td></tr></table>
</FORM>";
 
}
?>
[PHP] pobierz, plaintext 

A oto błąd:

Kod

Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in C:\xampp\htdocs\warrock\dodaj.php on line 7

Ten post edytował muk4 30.07.2007, 13:58:51

[Cienki1980]

W 5tej linii brakuje Ci zamykającego nawiasu od if.

I nie wiem czy to przez forum ale wyświetlenie formularza też może być błędne .. polecałbym echo 'tutaj cala zawartosc formularza'; a nie tak jak jest teraz echo "cala zawartosc formularza";

[muk4]

Poprawiłem ten nawias i zmieniłem " na ' ale dalej to samo :<

Moim zdaniem to coś w zapytaniu jest nei tak ale nie wiem co.

Ten post edytował muk4 30.07.2007, 13:55:25

[loonger]

zamien:
in_kon) VALUES ($_POST['kill'], $_POST['dead'], ...
na:
in_kon) VALUES (" . $_POST['kill'] . ", " . $_POST['dead'] . ", ...
i wszystkie te $_POST-y poowyrzucaj miedzy kropki poza "

[Cienki1980]

[PHP] pobierz, plaintext 
<?php
//zmien
$query = "INSERT INTO records (kill, dead, exp, kasa, godz_start, min_start, godz_kon, min_kon) 
VALUES ($_POST['kill'], $_POST['dead'], $_POST['exp'], $_POST['kasa'], $_POST['godz_start'],
 $_POST['min_start'], $_POST['godz_kon'], $_POST['min_kon'])";
//na
$query = "INSERT INTO records (kill, dead, exp, kasa, godz_start, min_start, godz_kon, min_kon) 
VALUES ('".$_POST['kill']."', '".$_POST['dead']."', '".$_POST['exp']."', '".$_POST['kasa']."',
 '".$_POST['godz_start']."', '".$_POST['min_start']."', '".$_POST['godz_kon']."', '".$_POST['min_kon']."')";
?>
[PHP] pobierz, plaintext 

Wszędzie używam składni '".$_POST['']."' a nie samego ".$_POST['']." bo nie wiem jakiego typu są dane wkładane do tej tabeli.

Sprawdź to czy będzie lepiej.

[muk4]

Zrobione jak pokazałeś
Teraz wyskakuje inny błąd związany ze składnią MYSQL :/

Kod

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 ''kill', 'dead', 'exp', 'kasa', 'godz_start', 'min_start', 'godz_kon', 'min_kon')' at line 1

[Cienki1980]

Zrobione jak pokazałeś
Teraz wyskakuje inny błąd związany ze składnią MYSQL :/

Kod

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 ''kill', 'dead', 'exp', 'kasa', 'godz_start', 'min_start', 'godz_kon', 'min_kon')' at line 1

Pokaż jak poprawiłeś bo coś tam chyba namieszałeś.

[muk4]

[PHP] pobierz, plaintext 
<?php
include 'config.php';
 
 
if($_POST['wyslij']){
 
$query = "INSERT INTO records (kill, dead, exp, kasa, godz_start, min_start, godz_kon, min_kon) VALUES ('".$_POST['kill']."', '".$_POST['dead']."', '".$_POST['exp']."', '".$_POST['kasa']."', '".$_POST['godz_start']."', '".$_POST['min_start']."', '".$_POST['godz_kon']."', '".$_POST['min_kon']."')";
 
$wykonaj = mysql_query($query);
[...]
?>
[PHP] pobierz, plaintext 

Ten post edytował muk4 30.07.2007, 16:05:06

[Cienki1980]

Zapytanie wygląda poprawnie, może nie zgadza się coś z nazwami kolumn ... pokaż jeszcze może strukturę tabeli SQL.

[muk4]

CREATE TABLE `records` (
`id` int(4) NOT NULL auto_increment,
`kill` varchar(4) collate latin1_general_ci NOT NULL,
`dead` varchar(4) collate latin1_general_ci NOT NULL,
`exp` varchar(6) collate latin1_general_ci NOT NULL,
`kasa` varchar(4) collate latin1_general_ci NOT NULL,
`godz_start` varchar(3) collate latin1_general_ci NOT NULL,
`min_start` varchar(3) collate latin1_general_ci NOT NULL,
`godz_kon` varchar(3) collate latin1_general_ci NOT NULL,
`min_kon` varchar(3) collate latin1_general_ci NOT NULL,
PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci AUTO_INCREMENT=1 ;

[Cienki1980]

Tu też wygląda wszystko poprawnie ... ostatnia myśl to zrób echo $query po tym jak stworzysz zapytanie i zobacz czy tworzy się poprawnie.

[muk4]

echo $query?

Kod

INSERT INTO records (kill, dead, exp, kasa, godz_start, min_start, godz_kon, min_kon) VALUES ('32', '32', '33', '2', '3', '24', '24', '2')

Czy powinny być apostrofy?

Ten post edytował muk4 30.07.2007, 16:15:20

[Cienki1980]

Typy pól w bazie masz varchar więc na moje oko być powinny.

Sprawdź to zapytanie bezpośrednio w bazie ( np w phpMyAdminie ) i zobacz czy tam zadziała ... bo ja nie widzę nic niepoprawnego w tym zapytaniu.

[muk4]

PhpMyAdmin wywala ten sam błąd. Pewnie dlatego że wcześniej używałem mysql_error().

Zmieniłem zapytanie. Teraz wygląda tak:

"INSERT INTO records VALUES (tutaj te zmenne wszystkie)"

i działa ^^. Dzięki że chciałeś mi pomóc.

[plurr]

proponuje tworzyc krotkie zmienne, mniej problemow $imie = $_POST['imie'] itd. pozniej w insert into robisz values ('$imie') zamiast (..."'.$_POST['imie'].'"...

[nospor]

problem byl poniewaz slowo kill jest slowem zastrzezonym:
http://dev.mysql.com/doc/refman/5.0/en/reserved-words.html
Powinienes byl wiec pisac `kill`