Forum PHP.pl > [PHP] error przy if

Pomoc - Szukaj - Użytkownicy - Kalendarz

GoOx

5.01.2012, 17:40:23

Witam ostatnio troszeczke poduczyłem się i wiem conieco zorbić

ale mam problem a mianowicie robię skrypt rejestracji lecz gdy wgrałem go na mój server mam errory

mój kod:

[PHP] pobierz, plaintext 
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
"http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<title>Untitled Document</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
Rejstracja:
Login:<input type="text" name="login"></br>
Haslo:<input type="text" name="haslo"></br>
<input type="submit" name="rejestruj" value="Rejestruj">
<?
$loginr = $_POST['login'];
$haslor = $_POST['haslo'];
include('polaczenie.php');
$dodaj = "INSERT INTO `uzytkownicy` set `nick` = '$loginr' , `haslo` = '$haslor'";
if(!mysql_query($dodaj, $db_lnk))
{
echo('Wystapil blad. Konto nie zostalo utworzone<br>');
}
Else
{
echo('Konto zostalo pomyslnie utworzone<br>');
}
?>
</body>
</html>
 
[PHP] pobierz, plaintext

a to mój error

Parse error: syntax error, unexpected T_STRING in /home/ecompute/public_html/przyklady/skrypt/rejestracja.php on line 17

toffiak

5.01.2012, 17:43:24

[PHP] pobierz, plaintext 
$dodaj = 'INSERT INTO `uzytkownicy` set `nick` = '.$loginr.' , `haslo` = '.$haslor;
[PHP] pobierz, plaintext

GoOx

5.01.2012, 18:00:19

Ok dzięki erroru nie ma zapomniałem tez action...

ale gdy próbuje się rejestrować to wyskakuje mi że wystąpił błąd itd...

poprawiony kod:

[PHP] pobierz, plaintext 
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
"http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<title>Untitled Document</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
Rejstracja:
<form action="rejestracja.php?rejestruje" method="post">
Login:<input type="text" name="login"></br>
Haslo:<input type="text" name="haslo"></br>
<input type="submit" name="rejestruj" value="Rejestruj">
</form>
<?
$loginr = $_POST['login'];
$haslor = $_POST['haslo'];
include('polaczenie.php');
$dodaj = 'INSERT INTO `uzytkownicy` set `nick` = '.$loginr.' , `haslo` = '.$haslor;
if(isset($_POST['rejestruj'])) 
if(!mysql_query($dodaj, $db_lnk))
{
{
echo('Wystapil blad. Konto nie zostalo utworzone<br>');
}
}
Else
{
echo('Konto zostalo pomyslnie utworzone<br>');
}
?>
</body>
</html>
 
[PHP] pobierz, plaintext

proszę o pomoc.

jeśli ktoś chce to tu jest demo: http://e-computer.xaa.pl/przyklady/skrypt/rejestracja.php

ublizzard

5.01.2012, 18:06:28

[PHP] pobierz, plaintext 
$loginr = $_POST['login'];
$haslor = $_POST['haslo'];
include('polaczenie.php');
$dodaj = 'INSERT INTO `uzytkownicy` set `nick` = '.$loginr.' , `haslo` = '.$haslor;
if(isset($_POST['rejestruj'])){
if(!mysql_query($dodaj, $db_lnk))
{
echo('Wystapil blad. Konto nie zostalo utworzone<br>');
}
Else
{
echo('Konto zostalo pomyslnie utworzone<br>');
}
}
?>
</body>
</html>
[PHP] pobierz, plaintext

darko

5.01.2012, 18:09:11

Spróbuj tak:

[PHP] pobierz, plaintext 
$dodaj = "INSERT INTO uzytkownicy (nick, haslo) values('$loginr' , md5('$haslor'))";
if(isset($_POST['rejestruj']))
{
  $result = mysql_query($dodaj, $db_lnk) or die(mysql_error());
  if(!$result)
  {
    echo('Wystapil blad. Konto nie zostalo utworzone<br>');
  }
  else
  {
     echo('Konto zostalo pomyslnie utworzone<br>');
  }
}
[PHP] pobierz, plaintext

Używaj PDO

GoOx

5.01.2012, 18:12:27

darko juz prawie działa

jak klikam rejestruj to mam taki error:

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in /home/ecompute/public_html/przyklady/skrypt/rejestracja.php on line 19

PS co oznacza też wykrzyknik w if ? bo to się używa ale tak naprawdę nie wiem co to daje.

ublizzard

5.01.2012, 18:18:31

wykrzyknik oznacza negację.
Twój błąd kryje się w linii 19.

mortus

5.01.2012, 18:19:12

Błąd oznacza, że zmienna $db_lnk nie jest zasobem połączenia z bazą danych, czyli brakuje gdzieś tego:

[PHP] pobierz, plaintext 
$db_lnk = mysql_connect('username','password');
[PHP] pobierz, plaintext

albo przypisujesz zmiennej nie to, co trzeba.

Wykrzyknik w instrukcji warunkowej oznacza zaprzeczenie/negację, np. !true to zaprzeczenie prawdy, czyli nieprawda.

GoOx

5.01.2012, 18:21:47

ach no tak zapomniałem dać include do pliku z połączeniem

ech ;/ rejestracja działa ale do bazy danych zapisuje tylko haslo z nick nie zapisuje.

mortus

5.01.2012, 18:26:28

A może kolumna nazywa się login lub username, a nie nick. Sprawdź to.

GoOx

5.01.2012, 18:28:01

to moja struktura bazy danych.

[SQL] pobierz, plaintext 
   CREATE TABLE `obrazki` (
    `id` INT NOT NULL AUTO_INCREMENT PRIMARY KEY ,
    `nick` VARCHAR( 255 ) NOT NULL
    `haslo` VARCHAR( 255 ) NOT NULL
    ) ENGINE = MYISAM ;
[SQL] pobierz, plaintext

i jeszcze przed chwilą zaważyłem że jak wpisuję hasło to w wszystkich kontach hasło w md5 jest identyczne.

mortus

5.01.2012, 18:29:18

To przed wysłaniem zapytania sprawdź, co kryje się w $_POST['login']: var_dump($_POST['login']);.

darko

5.01.2012, 18:32:12

Zamykaj inputy w formularzu: <input type="text" name="login" />
Spróbuj tak:

[PHP] pobierz, plaintext 
$loginr = mysql_real_escape_string($_POST['login']);
$haslor = md5(mysql_real_escape_string($_POST['haslo']));
$dodaj = "INSERT INTO uzytkownicy (nick, haslo) values('$loginr' , '$haslor')";
[PHP] pobierz, plaintext

I sprawdzaj przynajmniej długość przesyłanych zmiennych jak nie możesz/nie chcesz używać PDO. Dla pola hasło użyj input type="password"

GoOx

5.01.2012, 18:34:53

ahh moja wina wcześniej przez przypadek skasowałem to:

[PHP] pobierz, plaintext 
$loginr = $_POST['login'];
$haslor = $_POST['haslo'];
[PHP] pobierz, plaintext

wraz z include z polaczeniem

można zamknąć skrypt działa

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.