[PHP][PostgreSQL] Problem z dodaniem danych do bazy Opcje

[wariusz]

Witam mam prosty formularz, który należy wypełnić aby się zarejestrować można go zobaczyć na stronie:

www.trucki.pl/log/rejestracja.php

3 pola są obowiązkowe mianowicie login, hasło i mail. Po wypełnieniu pól (nieważne czy tylko obowiązkowe czy też dodatkowe) i naciśnięciu wyślij pojawiają się błędy:

[HTML] pobierz, plaintext 
Warning: pg_exec() [function.pg-exec]: Query failed: ERROR: column "admin" does not exist in /home/cls/ftp/log/rejestracja.php on line 42
false czyli nie ma takiego loginu, wynik jest:
Warning: pg_numrows(): supplied argument is not a valid PostgreSQL result resource in /home/cls/ftp/log/rejestracja.php on line 49
pg_numrows:
Warning: pg_numrows(): supplied argument is not a valid PostgreSQL result resource in /home/cls/ftp/log/rejestracja.php on line 51
 
Warning: pg_exec() [function.pg-exec]: Query failed: ERROR: invalid input syntax for integer: "" in /home/cls/ftp/log/rejestracja.php on line 76
[HTML] pobierz, plaintext 

poniżej miejsca w kodzie gdzie są błędy:

[PHP] pobierz, plaintext 
<?php
38  $db_handle = pg_connect("dbname =$pg_baza user=$pg_login password=$pg_haslo host=$pg_host");
39  if($db_handle) 
40  {
41   $query="SELECT * FROM $pg_tabela WHERE login=$login";
42   $wynik = pg_exec ($db_handle, $query);
43   if($wynik){echo" true jest taki login wynik ".$wynik;}else{echo" false czyli nie ma takiego loginu, wynik jest: ".$wynik;}
44   //echo "wynik".$wynik;
45   pg_close();
46  }
47
48  else echo "Nie można połączyć się z serwerem PSQL";
49    $wynik_pg_numrows=pg_numrows($wynik);
50    echo "pg_numrows: ".$wynik_pg_numrows;
51    if (pg_numrows($wynik)<>0) { $blad++;
52      echo "<span style=\"color:red;\">Login już został przez kogoś użyty!.
53      Zaproponuj inny!</span><br />";
54    }
55  }
 
69 $kod = uniqid(rand());
70    $haslo = md5($haslo); // zaszyfrowanie hasla
71    
72      $db_handle = pg_connect("dbname =$pg_baza user=$pg_login password=$pg_haslo host=$pg_host");
73      if($db_handle) 
74      {
75    $query="INSERT INTO $pg_tabela VALUES(DEFAULT, '$login', '$haslo', '$kod', 1, '$imie', '$miejscowosc', NOW(), '$email', '$nazwa', '$telefon', '$wojewodztwo', '$produkty' )";
76        $wynik = pg_exec ($db_handle, $query);
77        pg_close();
78      }
?>
[PHP] pobierz, plaintext 

i jeszcze kod tworzący tabele w której są umieszczane wszystkie te informacje:

[SQL] pobierz, plaintext 
CREATE TABLE dostawcy (
  lp_dostawcy INT NOT NULL DEFAULT NEXTVAL('public.dostawcy_lp_dostawcy_seq'::text) PRIMARY KEY,
  login VARCHAR(50) NOT NULL,
  haslo VARCHAR(32) NOT NULL,
  kod VARCHAR(32) NOT NULL,
  STATUS INT NOT NULL,
  imie VARCHAR(32),
  miejscowosc VARCHAR(32),
  data_dodania TIMESTAMP NOT NULL,
  email VARCHAR(120) NOT NULL,
  nazwa VARCHAR(32),
  telefon INT,
  wojewodztwo VARCHAR(32),
  produkty VARCHAR(100)
 
);
[SQL] pobierz, plaintext 

może ma ktoś jakiś pomysł co z tym może być nie tak ?

[phpion]

ERROR: column "admin" does not exist

W zmiennej $login masz zapewne wpisane "admin". Zmień zawartość tej zmiennej na 'admin' (czyli musisz ją otoczyć apostrofami) i powinno śmigać.

[wariusz]

ta zmiana o której mówisz niestety nic nie daje:
jest tak:

[PHP] pobierz, plaintext 
<?php
41   $query="SELECT * FROM $pg_tabela WHERE login=$login";
?>
[PHP] pobierz, plaintext 

zmieniłem na :

[PHP] pobierz, plaintext 
<?php
41   $query="SELECT * FROM $pg_tabela WHERE login='$login' ";
?>
[PHP] pobierz, plaintext 

i zrobiłem jeszcze kilka innych kombinacji ale nie działa.

próbowałem również zmienić w miejscu gdzie jest przypisywana wartość wprowadzona do formularza :

jest tak:

[HTML] pobierz, plaintext 
<input type="text" name="login" value="$login" /></td>
[HTML] pobierz, plaintext 

próbowałem zamiast cudzysłowów dać apostrofy lub w ogóle bez niczego ale nadal nie działa?

Ten post edytował wariusz 23.10.2008, 08:24:11

[phpion]

Napisałem Ci, że w zmiennej masz już zapisane ". Twoja modyfikacja sprawiła, że masz teraz '"admin"'. Jeśli nie wiesz gdzie i co zmienić zrób tak:

[PHP] pobierz, plaintext 
<?php
$query="SELECT * FROM $pg_tabela WHERE login=".str_replace('"', "'", $login);
?>
[PHP] pobierz, plaintext 

Przekopiuj tą linijkę bo możesz pomieszać " z '. Jest tak: ' " ' zmieniane na " ' " (oczywiście bez spacji).

[wariusz]

po wstawieniu tej linijki, którą mi podałeś nadal jest ten sam błąd.

[phpion]

Możesz wyechować to zapytanie?

[PHP] pobierz, plaintext 
<?php
// ...
$query="SELECT * FROM $pg_tabela WHERE login=".str_replace('"', "'", $login);
echo $query;
$wynik = pg_exec ($db_handle, $query);
// ...
?>
[PHP] pobierz, plaintext 

[wariusz]

po echo wypisuje:

[SQL] pobierz, plaintext 
SELECT * FROM dostawcy WHERE login=admin
[SQL] pobierz, plaintext 

[phpion]

Aj, poszedłem w złą stronę:

[PHP] pobierz, plaintext 
<?php
$query="SELECT * FROM $pg_tabela WHERE login='$login'";
?>
[PHP] pobierz, plaintext 

Tak też wywala błąd z unknown column? Nie powinno...

[batman]

Ludzie, zacznijcie w końcu poprawnie i bezpiecznie używać stringów.

[PHP] pobierz, plaintext 
<?php
$query = "SELECT * FROM $pg_tabela WHERE login='".$login."'";
?>
[PHP] pobierz, plaintext 

Ponadto użyj funkcji mysql_real_escape_string.


edit
Nie zauważyłem, że używasz postgresa - użyje funkcji pg_escape_string.

[wariusz]

to już próbowałem wcześniej ale po wklejeniu Twojej linijki wyskakuje:

[SQL] pobierz, plaintext 
SELECT * FROM dostawcy WHERE login='admin' 
 
true jest taki login wynik Resource id #2pg_numrows: 0
[SQL] pobierz, plaintext 

ta linijka true jest... wygląda na to jakby był już taki login w bazie a baza jest nadal pusta?

po wklejeniu linijki od Batmana wyskakuje:

[SQL] pobierz, plaintext 
SELECT * FROM dostawcy WHERE login='admin'
 true jest taki login wynik Resource id #2pg_numrows: 0
[SQL] pobierz, plaintext 

[batman]

Jeszcze jedna rzecz. Nie pg_exec, a pg_query.


edit
No i oczywista rzecz. Źle sprawdzasz, czy login istnieje. Dopiero przejrzałem Twój kod nieco dokładniej i oto co znalazłem.

[PHP] pobierz, plaintext 
<?php
$query="SELECT * FROM $pg_tabela WHERE login=$login";
$wynik = pg_exec ($db_handle, $query);
?>
[PHP] pobierz, plaintext 

Wynik zawsze będzie różny od false, ponieważ zapytanie wykonuje się poprawnie. Jeśli chcesz sprawdzić, czy login jest już bazie, to albo robisz to przy użyciu pg_num_rows lub poprzez pobranie wiersza (pg_fetch_*) i sprawdzenie, czy tablica nie jest pusta.

[wariusz]

sprawdzania poprawności odbywa się tak:

[PHP] pobierz, plaintext 
<?php
else echo "Nie można połączyć się z serwerem PSQL";
    $wynik_pg_numrows=pg_numrows($wynik);
    echo "pg_numrows: ".$wynik_pg_numrows;
    if (pg_numrows($wynik)<>0) { $blad++;
      echo "<span style=\"color:red;\">Login już został przez kogoś użyty!.
      Zaproponuj inny!</span><br />";
    }
?>
[PHP] pobierz, plaintext 

i chyba po tym co podałeś jest ok został tylko jeden błąd :

[HTML] pobierz, plaintext 
Warning: pg_query() [function.pg-query]: Query failed: ERROR: invalid input syntax for integer: "" in /home/cls/ftp/log/rejestracja.php on line 81
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
74 $kod = uniqid(rand());
75    $haslo = md5($haslo); // zaszyfrowanie hasla
76    
77      $db_handle = pg_connect("dbname =$pg_baza user=$pg_login password=$pg_haslo host=$pg_host");
78      if($db_handle) 
79      {
80        $query="INSERT INTO $pg_tabela VALUES(DEFAULT, '$login', '$haslo', '$kod', 1, '$imie', '$miejscowosc', NOW(), '$email', '$nazwa', '$telefon', '$wojewodztwo', '$produkty' )";
81        $wynik = pg_query ($db_handle, $query);
82        pg_close();
83      }
?>
[PHP] pobierz, plaintext 

[batman]

Pamiętaj, że poniższy kod zawsze Ci zwróci "true jest taki login", ponieważ funkcja zwraca false tylko w przypadku błędu w zapytaniu.

[PHP] pobierz, plaintext 
<?php
$query="SELECT * FROM $pg_tabela WHERE login=$login";
$wynik = pg_exec ($db_handle, $query);
if($wynik){echo" true jest taki login wynik ".$wynik;}else{echo" false czyli nie ma takiego loginu, wynik jest: ".$wynik;}
?>
[PHP] pobierz, plaintext 

A co o drugiego problemu:

[PHP] pobierz, plaintext 
<?php
echo $query;
?>
[PHP] pobierz, plaintext 

i będziesz wiedział gdzie jest błąd. Z komunikatu błędu wynika, że zapisujesz do pola typu int pusty ciąg.

[wariusz]

query wypisuje mi:

[SQL] pobierz, plaintext 
INSERT INTO dostawcy VALUES(DEFAULT, 'admin', 'aaaaaaaaggggggggg', '209gkjfdkljglsk45353495834' ,1, '', '', NOW(), 'mail', '', '', 'Województwo', '')
[SQL] pobierz, plaintext 

nie wiem dlaczego wypiuje "Default" nie powinno już się generować jakaś liczba w bazie ta kolumna to:

[SQL] pobierz, plaintext 
lp_dostawcy INT NOT NULL DEFAULT NEXTVAL('public.dostawcy_lp_dostawcy_seq'::text) PRIMARY KEY,
[SQL] pobierz, plaintext 

ale nawet jak za default wpiszę jakąś liczbę to błąd zostaje,?

[batman]

To co się wyświetliło to jest tylko to, co leci do bazy. Dopiero na bazie wykonują się odpowiednie operacje.
A problemem są puste stringi - ''
Któreś z pól ma tym integer i dlatego generuje błąd. Musisz zrzutować odpowiednią zmienną na int-a:

[PHP] pobierz, plaintext 
<?php
(int)$zmienna
?>
[PHP] pobierz, plaintext 

[wariusz]

problem z int nie występuje jak typ pola telefon zmieniłem na varchar (nie wiem czy takie pole powinno być takiego typu).
Ale wyskoczył kolejny błąd :

[HTML] pobierz, plaintext 
Warning: pg_query() [function.pg-query]: Query failed: ERROR: relation "public.dostawcy_lp_dostawcy_seq" does not exist in /home/cls/ftp/log/rejestracja.php on line 82
[HTML] pobierz, plaintext 

nie wiem czy nie chodzi ciągle do tego default?

[batman]

To może być problem z DEFAULT. Nie używałem go już dłuższy czas. Zawsze podaję nazwy kolumn, do których zapisuję dane.

[wariusz]

witam użyłem typu serial i dodałem nazwy kolumn i wszystko działa dzięki wielkie za pomoc dla batmana i phpion