Forum PHP.pl

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 ?

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

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

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).

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

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 

po echo wypisuje:

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

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.

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 

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.

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 

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.

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,?

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 

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?

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