[MySQL][PHP] insert Opcje

[-wabanek-]

Witam
może ktoś rzucic na to okiem, nie mogę nic już wymyślec

[PHP] pobierz, plaintext 
<?php
// odbieramy dane z formularza
$imie = $_POST['imie'];
$nazwisko = $_POST['nazwisko'];
$wynik = $_POST['wynik'];
 
if($imie and $nazwisko and $wynik) { 
$con = mysql_connect("localhost","root","");
if (!$con)
  {
  die('Could not connect: ' . mysql_error());
  }
 
 $ins = @mysql_query("INSERT INTO wynik_konkursu SET imie='$imie', nazwisko='$nazwisko', wynik= '$wynik'");
 
    if($ins) echo "Rekord został dodany poprawnie";
    else echo "Błąd nie udało się dodać nowego rekordu"; 
 
mysql_close($con);
}
?> 
[PHP] pobierz, plaintext 

wywala błędy
Notice: Undefined index: imie in C:\xampp\htdocs\insert.php on line 3

Notice: Undefined index: nazwisko in C:\xampp\htdocs\insert.php on line 4

Notice: Undefined index: wynik in C:\xampp\htdocs\insert.php on line 5

[r4xz]

ale coś nie działa...?

btw. notice != error

[szym011]

Nie istnieją twoje zmienne globalne. Zrób tak:

[PHP] pobierz, plaintext 
$imie = isset($_POST['imie']);
$nazwisko = isset($_POST['nazwisko']);
$wynik = isset($_POST['wynik']);
[PHP] pobierz, plaintext 

[-wabanek-]

Nie istnieją twoje zmienne globalne. Zrób tak:

[PHP] pobierz, plaintext 
$imie = isset($_POST['imie']);
$nazwisko = isset($_POST['nazwisko']);
$wynik = isset($_POST['wynik']);
[PHP] pobierz, plaintext 

Nie działa, nic nie pokazuje, nie ma błędu ale też nie dodaje wyniku do bazy

[Niktoś]

Nie znam się na tyle na php ,ale czy ten warunek ,można nazwać warunkiem??
if($imie and $nazwisko and $wynik)

Poza tym popatrz najpierw jak wygląda instrukcja sql -insert,raczej nie ma w niej set ,za to set jest w update.

[Dominis]

INSERT wyglada tak:

[SQL] pobierz, plaintext 
INSERT INTO table_name (column1, column2, column3,...)
VALUES (value1, value2, value3,...)
[SQL] pobierz, plaintext 

Notice, Ci daje znaki że zmienne:

[PHP] pobierz, plaintext 
$imie = $_POST['imie'];
$nazwisko = $_POST['nazwisko'];
$wynik = $_POST['wynik'];
[PHP] pobierz, plaintext 

$_POST nie istnieja(nie sa wczesniej zadeklarowane).

Następnie z tego co widze, twoj skrypt łączy sie z mysql, ale nie wybiera żadnej bazy.

Popraw to wszystko i powinno działac.

[szym011]

No tak. Nie spojrzałem na zapytanie.
Poprawnie powinno być tak:

[PHP] pobierz, plaintext 
$ins = @mysql_query("INSERT INTO `wynik_konkursu` (`imie`, `nazwisko`, `wynik`) VALUES ('$imie', '$nazwisko', '$wynik')");
[PHP] pobierz, plaintext 

Radzę przy pisaniu takich rzeczy nie używać znaku małpy przed funkcją, oraz użyć or die(mysql_error()) co pozwoli trafnie sprecyzować błąd w zapytaniu.

A wysyłasz coś do tego skryptu za pomocą metody POST?
Nie pokazuje (a wcześniej pokazywało błędy), ponieważ nie zostały nadesłane formularzem takie dane.

Ten post edytował szym011 27.11.2011, 23:13:20

[Sephirus]

Ok i dodatkowo

[PHP] pobierz, plaintext 
$imie = $_POST['imie'];
$nazwisko = $_POST['nazwisko'];
$wynik = $_POST['wynik'];
 
(...)
 
$ins = @mysql_query("INSERT INTO wynik_konkursu SET imie='$imie', nazwisko='$nazwisko', wynik= '$wynik'");
 
[PHP] pobierz, plaintext 

Upewnij się, że zmienne wrzucane do zapytania są bezpieczne...

[-wabanek-]

Dalej nie działa (IMG:style_emoticons/default/sciana.gif)

[PHP] pobierz, plaintext 
<?php
       $imie = isset($_POST['imie']);
    $nazwisko = isset($_POST['nazwisko']);
    $wynik = isset($_POST['wynik']);
if($imie and $nazwisko and $wynik) { 
$adres_ip_serwera_mysql_z_baza_danych = 'localhost';
$nazwa_bazy_danych = 'zawody';
$login_bazy_danych = 'root';
$haslo_bazy_danych = '';
if ( !mysql_connect($adres_ip_serwera_mysql_z_baza_danych,$login_bazy_danych,$haslo_bazy_danych) ) {
   echo 'Nie moge polaczyc sie z baza danych';
	 exit (0);
}
 
if ( !mysql_select_db($nazwa_bazy_danych) ) {
   echo 'Blad otwarcia bazy danych';
	 exit (0);
}
 
     $ins = @mysql_query("INSERT INTO `wynik_konkursu` (`imie`, `nazwisko`, `wynik`) VALUES ('$imie', '$nazwisko', '$wynik')");
 
    if($ins) echo "Rekord zostal dodany poprawnie";
    else echo "Blad nie udalo sie dodac nowego rekordu"; 
 
mysql_close($con);
}
?> 
[PHP] pobierz, plaintext 

[pyro]

[PHP] pobierz, plaintext 
$imie = isset($_POST['imie']);
    $nazwisko = isset($_POST['nazwisko']);
    $wynik = isset($_POST['wynik']);
[PHP] pobierz, plaintext 

To zawsze zwróci albo true albo false. Do zmiennej przekazujesz tylko wynik isset(), a nie wartości POST.

Życzę powodzenia, jeżeli wszędzie będziesz korzystał z tak długich nazw zmiennych (IMG:style_emoticons/default/tongue.gif)

[Prym]

Dalej nie działa (IMG:style_emoticons/default/sciana.gif)

[PHP] pobierz, plaintext 
<?php
       $imie = isset($_POST['imie']);
    $nazwisko = isset($_POST['nazwisko']);
    $wynik = isset($_POST['wynik']);
if($imie and $nazwisko and $wynik) { 
$adres_ip_serwera_mysql_z_baza_danych = 'localhost';
$nazwa_bazy_danych = 'zawody';
$login_bazy_danych = 'root';
$haslo_bazy_danych = '';
if ( !mysql_connect($adres_ip_serwera_mysql_z_baza_danych,$login_bazy_danych,$haslo_bazy_danych) ) {
   echo 'Nie moge polaczyc sie z baza danych';
	 exit (0);
}
 
if ( !mysql_select_db($nazwa_bazy_danych) ) {
   echo 'Blad otwarcia bazy danych';
	 exit (0);
}
 
     $ins = @mysql_query("INSERT INTO `wynik_konkursu` (`imie`, `nazwisko`, `wynik`) VALUES ('$imie', '$nazwisko', '$wynik')");
 
    if($ins) echo "Rekord zostal dodany poprawnie";
    else echo "Blad nie udalo sie dodac nowego rekordu"; 
 
mysql_close($con);
}
?> 
[PHP] pobierz, plaintext 

Ale co ci zwraca za błąd? nie dodaje do bazy? (np. pustych rekordów) czy co bo tak to wygląda raczej poprawnie zmieniłbym warunek na

Kod

if(  isset($imie) && isset($nazwisko) && isset($wynik))

[-wabanek-]

Ale co ci zwraca za błąd? nie dodaje do bazy? (np. pustych rekordów) czy co bo tak to wygląda raczej poprawnie zmieniłbym warunek na

Kod

if(  isset($imie) && isset($nazwisko) && isset($wynik))

dodaje rekord, ale w wyświetla tylko wartośc 1 albo nic a w wyniku 0! o co tu chodzi?(IMG:style_emoticons/default/questionmark.gif)

[Prym]

dodaje rekord, ale w wyświetla tylko wartośc 1 albo nic a w wyniku 0! o co tu chodzi?(IMG:style_emoticons/default/questionmark.gif)

Boże no tak przepraszam że napisałem że kod wygląda ok - nie wygląda. jak przekazujesz dane z POST do zmiennych to przez ten ISSET dodajesz nie wartości ale czy dane istnieja (isset sprawdza czy isntnieje post więc zwraca wynik warunku a nie wartość czyli MUSISZ zrobić w takim wypadku coś takiego:

[PHP] pobierz, plaintext 
$imie = (isset($_POST['imie'])) ? $_POST['imie'] : false;
$nazwisko = (isset($_POST['nazwisko'])) ? $_POST['nazwisko'] : false;
$wynik = (isset($_POST['wynik'])) ? $_POST['wynik'] : false;
[PHP] pobierz, plaintext 

i musi działać (IMG:style_emoticons/default/smile.gif)

[Niktoś]

A po co takie kombinacje ,nie lepiej sprawdzić czy formularz został wysłany i potem robić zwykłe $_POST['imie'];

[PHP] pobierz, plaintext 
<?php
if ($_POST){       
$imie = $_POST['imie'];
    $nazwisko = $_POST['nazwisko'];
    $wynik = i$_POST['wynik'];
if($imie and $nazwisko and $wynik) { 
$adres_ip_serwera_mysql_z_baza_danych = 'localhost';
$nazwa_bazy_danych = 'zawody';
$login_bazy_danych = 'root';
$haslo_bazy_danych = '';
if ( !mysql_connect($adres_ip_serwera_mysql_z_baza_danych,$login_bazy_danych,$haslo_bazy_danych) ) {
   echo 'Nie moge polaczyc sie z baza danych';
	 exit (0);
}
 
if ( !mysql_select_db($nazwa_bazy_danych) ) {
   echo 'Blad otwarcia bazy danych';
	 exit (0);
}
 
     $ins = @mysql_query("INSERT INTO `wynik_konkursu` (`imie`, `nazwisko`, `wynik`) VALUES ('$imie', '$nazwisko', '$wynik')");
 
    if($ins) echo "Rekord zostal dodany poprawnie";
    else echo "Blad nie udalo sie dodac nowego rekordu"; 
 
mysql_close($con);
}
}
?> 
[PHP] pobierz, plaintext 

niż sprawdzać istnienie każdej zmiennej?

[-wabanek-]

Boże no tak przepraszam że napisałem że kod wygląda ok - nie wygląda. jak przekazujesz dane z POST do zmiennych to przez ten ISSET dodajesz nie wartości ale czy dane istnieja (isset sprawdza czy isntnieje post więc zwraca wynik warunku a nie wartość czyli MUSISZ zrobić w takim wypadku coś takiego:

[PHP] pobierz, plaintext 
$imie = (isset($_POST['imie'])) ? $_POST['imie'] : false;
$nazwisko = (isset($_POST['nazwisko'])) ? $_POST['nazwisko'] : false;
$wynik = (isset($_POST['wynik'])) ? $_POST['wynik'] : false;
[PHP] pobierz, plaintext 

i musi działać (IMG:style_emoticons/default/smile.gif)

imie nazwisko działa (IMG:style_emoticons/default/smile.gif) , wynik wpisuje wartośc "0" (IMG:style_emoticons/default/sad.gif) , jest blisko

[Prym]

imie nazwisko działa (IMG:style_emoticons/default/smile.gif) , wynik wpisuje wartośc "0" (IMG:style_emoticons/default/sad.gif) , jest blisko

hm to wygląda że post nie przekazuje danych (czyli może w formularzu nie do końca masz nazwę wartosc? (może jakiś program dał ci ść w nazwę zmiennej - mój pająk czasem mi takie psikusy robi (IMG:style_emoticons/default/smile.gif) )
A ogólnie to ja mam zawsze dodaną taką funkcję do kodu:

[PHP] pobierz, plaintext 
function testArray($co) {echo '<pre>';print_r($co);echo '</pre>';}
[PHP] pobierz, plaintext 

i sobie testuje co jest przekazywane w tablicach (równiez posty i gety) łącznie z nazwą indexu:

[PHP] pobierz, plaintext 
testArray($_POST);
[PHP] pobierz, plaintext 

powyższy kod wyświetli ci co jest przekazywane. i Ponownie Jak nie masz danych w wartosc to znaczy błąd w budowie formularza (błędne name="")