Forum PHP.pl > [PHP][MySQL] Połączenie

Pomoc - Szukaj - Użytkownicy - Kalendarz

-Baltazar-

9.05.2011, 18:48:38

Witam. Mógł by ktoś sprawdzić ten kod? Co jest w nim źle ?

[PHP] pobierz, plaintext 
 
<html>
<head>
  <meta http-equiv="Content-type" content="text/html; charset=utf-8">
 
  <title> Dodaj </title>
 
</head>
<body>
 
</br>
</br>
<form action="#" method="POST">
<table>
	<tr><td>Imie:</td> <td><input type="text" id="imie"></td></tr>
	<tr><td>Nazwisko:</td> <td><input type="text" id="nazwisko"></td></tr>
	<tr><td>Login:</td> <td><input type="text" id="login"></td></tr>
	<tr><td>Hasło:</td> <td><input type="password" id="haslo"></td></tr>
	<tr><td>Klasa:</td> <td><input type="text" id="klasa"></td></tr>
	<tr><td></td><td><input type="submit" value="Dodaj"></td></tr>
</table>
</form>
 
<?php
$imie = $_POST['imie'];
$nazwisko = $_POST['nazwisko'];
$login = $_POST['login'];
$haslo = $_POST['haslo'];
$klasa = $_POST['klasa'];
 
if($imie and $nazwisko and $login and $haslo and $klasa)	{
 
$connection = @mysql_connect('localhost', 'root', '')
or die('Brak połączenia z serwerem MySQL.<br />Błąd: '.mysql_error());
$db = @mysql_select_db('uczniowie', $connection)
or die('Nie mogę połączyć się z bazą danych<br />Błąd: '.mysql_error());
 
$ins = @mysql_query("INSERT INTO test SET imie = 'imie', nazwisko ='nazwisko', login = 'login', haslo ='haslo', klasa = 'klasa' ");
}
 
if($ins) echo("Dodano!");
mysql_close($connection);
 
 
}
?>
</body>
</html>
[PHP] pobierz, plaintext

kadlub

9.05.2011, 19:01:14

a jakie masz błędy

jak ich nie masz to usuń te wszystkie małpy

kill15

9.05.2011, 19:18:57

samych notice będzie miał z 5.
Nie jestem zbyt doświadczony ale ten kod wg mnie jest karygodny ;>
Skąd się bierze klamra w 45 linii ? gdzie się rozpoczyna

Jakaś rada:

Notice będziesz miał z linii 25-29(na początku nie są zmienne przypisane).
Dalej warunek pierwszy no to
sprawdzasz czy istnieje np $_POST['imie'];
a robisz to tak:

[PHP] pobierz, plaintext 
if(isset($_POST['imie']) && isset($_POST['nazwisko']) ...)
{
// kod
}
[PHP] pobierz, plaintext

Dopiero w klamrach przypisujesz zmienne z $_POST.
Dalej to:

[PHP] pobierz, plaintext 
$ins = @mysql_query("INSERT INTO test SET imie = 'imie', nazwisko ='nazwisko', login = 'login', haslo ='haslo', klasa = 'klasa' ");
[PHP] pobierz, plaintext

What the fuck ? imie = 'imie' ? Równie dobrze mogę ci powiedzieć że masło równe jest masłu.
Jeżeli chcesz dodać rekord to robisz to tak:

[PHP] pobierz, plaintext 
$ins = "INSERT INTO test (imie,nazwisko ...) VALUES ('$imie', '$nazwisko'...)";
$ins = mysql_query($ins);
[PHP] pobierz, plaintext

Może zróbcie dział żłobek.

tolomei

10.05.2011, 01:39:38

@kill15 od dziś masz ksywkę parser

Mnie już mianowali

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.