Dlaczego Nie Dziala? Opcje

[MarcinJarek]

[PHP] pobierz, plaintext 
<?
$sql = mysql_connect (localhost, root, krasnal);
mysql_select_db(&#092;"php\");
$zapytanie = &#092;"SELECT * FROM uzytkownicy\";
$wykonaj = mysql_query($zapytanie);
 
$q =&#092;"INSERT INTO uzytkownicy (`id`, `nick`, `haslo`, `data`) VALUES ('' , 'nick' , 'haslo' , 'data')\";
$w=mysql_query($q);
mysql_close($sql);
?>
[PHP] pobierz, plaintext 

Dlaczego mi to nie dziala? Zawsze dzialalo a teraz nagle nie pojawia sie blad ale nie wpisuje nic do bazy...

Ten post edytował MarcinJarek 23.08.2004, 13:38:38

[mhs]

1) jezeli pole jest id jest kluczem podstawowym tabeli (primary key auto_increment) to przy dodawaniu rekordu do bazy danych mozesz opuscic to pole i zapytanie napisac tak:

[SQL] pobierz, plaintext 
INSERT INTO uzytkownicy (`nick`, `haslo`, `data`) VALUES ('nick' , 'haslo' , 'data')";
[SQL] pobierz, plaintext 

2) co Ty chcesz dokladnie zrobic? czy to co zaprezentowales to kompletny kod? najpierwsz pobierasz dane z jakiejs tabeli a pozniej co? nic z tym nie robisz? w nastepnej kolejnosci dodajesz jakies dane do tej samej tabeli? czy sa to moze jakies fragmenty kodu...
- zauwaz co do dodajesz do tabeli uzytkownicy: 'nick', 'haslo', 'data' - przypadkiem nie powinny tam byc jakies zmienne.. czy akurat chcesz dodawac te dane... ale watpie...

3) nie chce mi sie pisac wszystkiego od poczatku wiec przytaj sobie odpowiedz delorian'a z tego watku http://forum.php.pl/index.php?showtopic=20218

Ten post edytował mhs 23.08.2004, 13:51:47

[Yarecki]

mysql_connect (localhost, root, krasnal);
powinno być chyba tak:
mysql_connect ('localhost', 'root', 'krasnal');

BTW. wszędzie gdzie masz funkcje mysql_ dodaj na końcu or die(mysql_error()); np.:
mysql_connect ('localhost', 'root', 'krasnal') or die(mysql_error());
Teraz powinieneś zobaczyć komunikat o błędzie.

[mhs]

mysql_connect (localhost, root, krasnal);
powinno być chyba tak:
mysql_connect ('localhost', 'root', 'krasnal');

BTW. wszędzie gdzie masz funkcje mysql_ dodaj na końcu or die(mysql_error()); np.:
mysql_connect ('localhost', 'root', 'krasnal') or die(mysql_error());
Teraz powinieneś zobaczyć komunikat o błędzie.

ech... cale 5 sekund bylem szybszy..

[Yarecki]

Dzieki za pomoc. To nie jest kompletny kod a jak mozecie to jak zrobic aby zapisywal skrypt zmienne. A moze takie cos wystarczy:

[PHP] pobierz, plaintext 
<?php
INSERT INTO uzytkownicy (`nick`, `haslo`, `data`) VALUES ('$nick' , '$haslo' , '$data')&#092;";
?>
[PHP] pobierz, plaintext 

Ja używam takiej składni:

[PHP] pobierz, plaintext 
<?php
	$mixQuery = 'INSERT into tabela( pole1, pole2 ) VALUES( 5, '' . $strZmienna . '' )';
	$mysql_query( $mixQuery );
?>
[PHP] pobierz, plaintext 

można też tak:

[PHP] pobierz, plaintext 
<?php
	$mixQuery = &#092;"INSERT into tabela( pole1, pole2 ) VALUES( 5, '$strZmienna' )\";
	$mysql_query( $mixQuery );
?>
[PHP] pobierz, plaintext 

Ten post edytował Yarecki 23.08.2004, 14:25:06

[MarcinJarek]

Kurde - teraz mam kod taki:

[PHP] pobierz, plaintext 
<?php
mysql_connect ('localhost', 'root', 'krasnal') or die(mysql_error());
 
mysql_select_db(&#092;"php\");
$data = date('Y-m-d G:i:s');
$q =&#092;"INSERT INTO uzytkownicy (`login`, `haslo`, `imie`, 'nazwisko', 'mail', 'typ', 'data') VALUES ('$login' , '$haslo' , '$imie, '$nazwisko', '$mail', 'testowani', '$data')\";
$w=mysql_query($q);
?>
[PHP] pobierz, plaintext 

I nie chce dzialac - nie dodaje. ID - to jest primary key - wiec omijam go.

Dzieki z goryy:)

[Nitryt14]

Omijasz pole ID?

nie chce dzialac - nie dodaje. ID - to jest primary key - wiec omijam go.

A czy przy polu ID ustawiłeś na AUTO_INCREMENT ?

Jeżeli nie to ustaw bo inaczej nie będzie działać!

A jeżeli nie chcesz tego zmieniać ta wpisz taki kod:

[PHP] pobierz, plaintext 
<?php
 
mysql_connect ('localhost', 'root', 'krasnal') or die(mysql_error());
 
mysql_select_db(&#092;"php\");
$sql_num = &#092;"SELECT * FROM uzytkownicy;\";
$query_num = mysql_query($sql_num);
$result = mysql_num_rows($query_num);
$result = $result + 1;
$data = date('Y-m-d G:i:s');
$q =&#092;"INSERT INTO uzytkownicy ('id',`login`, `haslo`, `imie`, 'nazwisko', 'mail', 'typ', 'data') VALUES ('$result','$login' , '$haslo' , '$imie, '$nazwisko', '$mail', 'testowani', '$data')\";
$w=mysql_query($q);
 
?>
[PHP] pobierz, plaintext 

Pozdrawiam

[MarcinJarek]

Mam AUTO_INCREMENT i w polu artybury mam "UNSIGNED"

Co do kodu - wolalbym, aby dzialalo to autouzupelnianie...

[Nitryt14]

To zmień ustawienie żeby nie było UNSIGNED tzn. wyłącz to!

[MarcinJarek]

Nadal nie dziala. Wklejam caly kod bo to moze tego wina...

[PHP] pobierz, plaintext 
<?
if ($haslo==$haslo2) {
echo &#092;"Podane hasla sa inne\";
}else {
 
mysql_connect ('localhost', 'root', 'krasnal') or die(mysql_error());
 
mysql_select_db(&#092;"php\");
$zapytanie = &#092;"SELECT * FROM uzytkownicy\";
$wykonaj = mysql_query($zapytanie);
$zapytanie = &#092;"SELECT login FROM uzytkownicy WHERE login='$login'\";
 
if(!zapytanie) {
echo &#092;"Ten login jest juz zajety!\";
} else {
mysql_connect ('localhost', 'root', 'krasnal') or die(mysql_error());
 
mysql_select_db(&#092;"php\");
$data = date('Y-m-d G:i:s');
$q =&#092;"INSERT INTO uzytkownicy (`login`, `haslo`, `imie`, 'nazwisko', 'mail', 'typ', 'data') VALUES ('$login' , '$haslo' , '$imie, '$nazwisko', '$mail', 'testowani', '$data')\";
$w=mysql_query($q);
echo &#092;"dodano\";
 
 }
}
 
?>
[PHP] pobierz, plaintext 

P.S moze to wina, ze mam ustawione NULL - NIE

Ten post edytował MarcinJarek 23.08.2004, 15:47:12

[tumczasty]

$q ="INSERT INTO uzytkownicy (`login`, `haslo`, `imie`, 'nazwisko', 'mail', 'typ', 'data') VALUES ('$login' , '$haslo' , '$imie, '$nazwisko', '$mail', 'testowani', '$data')";

przy imieniu nie jest domknięty apostrof

[MarcinJarek]

.....Nadal nie dziala ;(

[Nitryt14]

A na końcu tego zapytania

[PHP] pobierz, plaintext 
<?php
 
$q =&#092;"INSERT INTO uzytkownicy (`login`, `haslo`, `imie`, 'nazwisko', 'mail', 'typ', 'data') VALUES ('$login' , '$haslo' , '$imie, '$nazwisko', '$mail', 'testowani', '$data')\";
 
?>
[PHP] pobierz, plaintext 

Brakuje średnika zamień na:

[PHP] pobierz, plaintext 
<?php
 
$q =&#092;"INSERT INTO uzytkownicy (`login`, `haslo`, `imie`, 'nazwisko', 'mail', 'typ', 'data') VALUES ('$login' , '$haslo' , '$imie, '$nazwisko', '$mail', 'testowani', '$data');\";
 
?>
[PHP] pobierz, plaintext 

Pozdrawiam

[mhs]

jezeli to jest koncowy kod to:

1) jezeli $haslo itd. to nazwy zmiennych prosto pobieranych z formularza to nie rob tak, tylko korzystaj z $_POST -> generalnie poczytaj o register globals

2) po co te dwa zapytania?

[SQL] pobierz, plaintext 
$zapytanie = "SELECT * FROM uzytkownicy"; $wykonaj = mysql_query($zapytanie);
$zapytanie = "SELECT login FROM uzytkownicy WHERE login='$login'";
[SQL] pobierz, plaintext 

wystarczy, ze wykonasz drugie zapytanie i sprawdzisz czy jest taki uzytkownik

3) mozesz nalozyc w projekcie bazy na pole login artybut UNIQUE przez co bedziesz mial zapewniona integralnosc danych w bd (przy zapytaniu, probie dodania uzytkownika o istniejacym loginie system uniemozliwi taka czynnosc)

4) nie musisz tworzyc zmiennej $date wytarczy ze zapiszesz:

[PHP] pobierz, plaintext 
<?php
 
, '\".date('Y-m-d G:i:s').\"')&#092;";
 
?>
[PHP] pobierz, plaintext 

5) nie sprawdzasz czy zapytanie zostalo faktycznie wykonane, tylko z gory zakladasz, ze jest ok...

[PHP] pobierz, plaintext 
<?php
$w=mysql_query($q);
echo &#092;"dodano\";
?>
[PHP] pobierz, plaintext 

ps. to kilka uwag na poczatek... (jeszcze mozna by napisac o braku walidacji danych... i innych)



---edit---

Brakuje średnika zamień na:

Kod php:
1
2
3
4
5
<?php

$q ="INSERT INTO uzytkownicy (`login`, `haslo`, `imie`, 'nazwisko', 'mail', 'typ', 'data') VALUES ('$login' , '$haslo' , '$imie, '$nazwisko', '$mail', 'testowani', '$data');";

?>



Pozdrawiam

nie brakuje... w zapytaniach wysylanych z poziomu skryptu nie stosuje sie tego, jezeli natomiast pracujesz na konsoli - to tak...

Ten post edytował mhs 23.08.2004, 16:06:43

[MarcinJarek]

Wkelilem caly skrypt. A wiec w czym jest blad?

Aha nie ma takego czegosc jak 'UNIQUE"

Ten post edytował MarcinJarek 23.08.2004, 16:18:33

[mhs]

[PHP] pobierz, plaintext 
<?php
if ($haslo != $haslo2) {
    echo &#092;"Podane hasla sa inne\";
}
else {
    mysql_connect ('localhost', 'root', 'krasnal') or die(mysql_error());
    
    mysql_select_db(&#092;"php\");
    
    $zapytanie = &#092;"SELECT login FROM uzytkownicy WHERE login='$login'\";
    
    $wynikZapytania = mysql_query($zapytanie);
    
    if(mysql_num_rows($wynikZapytania) > 0) {
        echo &#092;"Ten login jest juz zajety!\";
    }
    else
    {        
        $zapytanie =&#092;"INSERT INTO uzytkownicy (`login`, `haslo`, `imie`, 'nazwisko', 'mail', 'typ', 'data') VALUES ('$login' , '$haslo' , '$imie, '$nazwisko', '$mail', 'testowani', '\".date('Y-m-d G:i:s').\"')\";
        
        $wynikZapytania = mysql_query($zapytanie);
        
        if (mysql_affected_rows() > 0)
            echo 'uzytkownik zostal dodany';
        else
            echo 'nie zostal dodany';
    }
}
?>
[PHP] pobierz, plaintext 

napisalem (poprawilem) to wlasciwie z glowy wiec nie dam 100% gwarancji...
przede wszystkim to przy sprawdzaniu loginu zapomniales wyslac zapytanie do bazy danych... zobacz czy to bedzie Ci dzialas...

niestety dzisiaj juz dalej Ci nie pomoge gdyz za chwile wychodze z pracy...



---e...
UNIQUE - jest

Ten post edytował mhs 23.08.2004, 16:30:19

[MarcinJarek]

Kod powinien wygladac tak:

[PHP] pobierz, plaintext 
<?php
if ($haslo != $haslo2) {
 
mysql_connect ('localhost', 'root', 'krasnal') or die(mysql_error());
 
    mysql_select_db(&#092;"php\");
 
    $zapytanie = &#092;"SELECT login FROM uzytkownicy WHERE login='$login'\";
 
    $wynikZapytania = mysql_query($zapytanie);
 
    if(mysql_num_rows($wynikZapytania) > 0) {
        echo &#092;"Ten login jest juz zajety!\";
    }
    else
    {
$data = date('Y-m-d G:i:s');
        $zapytanie =&#092;"INSERT INTO uzytkownicy (`login`, `haslo`, `imie`, 'nazwisko', 'mail', 'typ', 'data') VALUES ('$login' , '$haslo' , '$imie', '$nazwisko', '$mail', 'testowani', '$data')\";
 
        $wynikZapytania = mysql_query($zapytanie);
 
        if (mysql_affected_rows() > 0)
            echo 'uzytkownik zostal dodany';
        else
            echo 'nie zostal dodany';
    }
}
else {
 echo &#092;"Nick zajety\";
}
?>
[PHP] pobierz, plaintext 

Ale i tak nie dziala...pojawia sie: "nie zostal dodany".

pomocy!

Ja mam UNIKALNY :-) bo mam PL, wiec to chyba to

Ten post edytował MarcinJarek 23.08.2004, 16:35:09

[Yarecki]

Czemu nie użyjesz or die( mysql_error() ) przy mysql_query ?

[Synaps]

Rozwiązanie jest raczej proste moim zdaniem , zapytanie typu insert powinno wyglądać tak

[SQL] pobierz, plaintext 
INSERT INTO uzytkownicy (login, haslo, imie, nazwisko, mail, typ, DATA) VALUES ('$login' , '$haslo' , '$imie', '$nazwisko', '$mail', 'testowani', '$data')
[SQL] pobierz, plaintext 

nazwy pól nie powinny być w '' .

[MarcinJarek]

Zapisuje, ale haslo podane zapisuje w polu LOGIN a hasla wogule nie wpisuje...