Forum PHP.pl

Witam,
Ostatnio napisałem skrypt do logowania i rejestracji i on nie działa pomoże mi ktoś?
Błędy to:

Notice: Undefined index: login in /var/www/vhosts/muxiu.5v.pl/httpdocs/Skrypty/register.php on line 3

Notice: Undefined index: haslo in /var/www/vhosts/muxiu.5v.pl/httpdocs/Skrypty/register.php on line 4

Fatal error: Function name must be a string in /var/www/vhosts/muxiu.5v.pl/httpdocs/Skrypty/register.php on line 5

A kod:
Tylko rejestracja*(Edit)

[PHP] pobierz, plaintext 
<!--c1--><div class='codetop'>Kod</div><div class='codemain'><!--ec1--><?PHP
 
$user = $_POST['login'];
$pass = $_POST['haslo'];
$email = $_POST['email'];
 
$con = mysql_connect("****","****","****") or ("Blad polaczenia: "  . mysql_error());
if (!$con)
    die('Nie mozna polaczayc: ' . mysql_error());
    
mysql_select_db("****" , $con) or die ("Nie mozna wczytac bazy danych" . mysql_error());
 
$check = mysql_query("SELECT * FROM Konta WHERE `login`='".$user."'");
$numrows = mysql_num_rows($check);
if ($numrows == 0)
{
    $pass = md5($pass);
    $ins = mysql_query("INSERT INTO  `Konta` (`login` , `haslo` , 'email') VALUES ('".$user."' ,  '".$pass."' ,  '".$email."'); ");
    if ($ins)
        die ("Register-SUCCESS");
    else
        die ("Blad: " . mysql_error());
}
else
{
    die("Podany login juz istnieje!");
}
 
 
?><!--c2--></div><!--ec2-->
[PHP] pobierz, plaintext 

Proszę o pomoc
Z góry dziękuje.
:-)

Ten post edytował AgeraR 18.09.2017, 19:38:00

Notice to nie błąd a informacja, że zmienna nie istnieje. Będzie istniała, jak wyślesz formularz... Można zablokować wyświetlanie tych informacji, ale lepszym wyjściem jest napisanie kodu tak, żeby sam skrypt zapobiegał ich wyświetlaniu.

Dobra ale po
1: Jak napisać to
2: Tam jest jeszcze Fatal Error

Ja nie widzę w linii 5 funkcji o nazwie "name"... wogóle takiej funkcji nie widzę!

Cały ten kod jest do przepisania. Stare rozszerzenie mysql, pełno dziur, md5 do hasła. Może kilka lat temu było poprawnie. Poczytaj o PDO, isset / ?? (PHP7).

To co mam zrobić żeby ten Fatal error nie wyskakiwał i hasła sie zapisywały do bazy?

Nazwa funkcji musi być stringiem i wydaje się, że jest...

No to jak jest wszystko poprawnie napisane no to dlaczego mi to nie działa?

zgubiłeś die w 5 linijce.

@Kshyhoo Notice to też błąd. Ma niski priorytet ale ma.

Dodałem die

Teraz Błąd

Parse error: syntax error, unexpected '$email' (T_VARIABLE) in /var/www/vhosts/muxiu.5v.pl/httpdocs/Skrypty/register.php on line 5

Wyskakuje ;(

Nikt już nie pomoże?

Sorka. Mój błąd. To nie w 5 linijce a 7.

Wklej cały kod gdzieś np http://bin.devsphp.pl i daj go tu

Te błędy występują dlatego że jak wyżej wspomniano nie ma akcji dla $_POST. Musisz zrobić warunek przed przypisaniem metody $_POST do zmiennych $user, $pass, $email. Najpierw sprawdź czy istnieją zmienne $_POST. Jeżeli nie to $user = null, $pass = null, $email = null, inaczej przypisz metodę $_POST do zmiennych.
Odnośnie zapytań do bazy danych to wyrzuć podwójne apostrofy oraz kropki przy zmiennych $user, $pass, $email, gdyż są zbędne.

Zmienione teraz wyskakuje błąd:

Parse error: syntax error, unexpected '$con' (T_VARIABLE) in /var/www/vhosts/muxiu.5v.pl/httpdocs/Skrypty/register.php on line 7

Tu masz Kod

O kurczaki...

Namieszałeś w warunkach przy łączeniu z bazą danych w tej linii.

Tam gdzie jest napisane Host itd?
jak tak to to jest specjalnie zeby nie bylo widac (IMG:style_emoticons/default/biggrin.gif)

Usuń warunki w tej linii i zobacz czy się łączy z bazą

[PHP] pobierz, plaintext 
 <?
 $mysql_host = 'host';
$port = '3306';
$username = 'nazwa użytkownika';
$password = 'hasło bazy danych';
$database = 'nazwa bazy danych';
 
try{
	$pdo = new PDO('mysql:host='.$mysql_host.';dbname='.$database.';port='.$port, $username, $password );
 
}catch(PDOException $e){
	echo 'Połączenie nie mogło zostać utworzone.';
}
 
 @mysql_query("set names 'utf8'");   
 
 
 
 
 
 
$email = substr($_POST['email'],0,32);
$pos = strpos($email, "@");
$pos2 = strpos($email, ".");
 
//WYBIERAMY USTAWIENIA Z BAZY
 
//JEŚLI KLIKNIĘTO WYŚLIJ
if(isset($_POST['send'])){
 
 
		if($pos == false OR $pos2 == false){
					 		echo "
<script type='text/javascript'>window.alert('Niepoprawny adres e-mail');document.location.href = 'index.php';</script>
";
		EXIT();
 
}else{
	$login = trim($_POST['imie']);
	$email = trim($_POST['email']);
	$errors = NULL;
	if(strlen($login)<6) $errors .= 'Login musi zawierać co najmniej 6 znaków<br>';
	if(!preg_match('/\@/', $email) || strlen($email)<5) $errors .= 'Podany adres e-mail jest nieprawidłowy<br>';
	$stmt = $pdo->prepare("SELECT COUNT(id) FROM Konta WHERE login=:login");
	$stmt->bindValue(":login", $login, PDO::PARAM_STR);
	$stmt->execute();
	$row = $stmt->fetch();
if($row[0]>0) $errors .= 'Konto o takim loginie już istnieje<br>';
		$stmt2 = $pdo->prepare("SELECT COUNT(id) FROM uzytkownicy WHERE email=:email");
	$stmt2->bindValue(":email", $email, PDO::PARAM_STR);
	$stmt2->execute();
	$row2 = $stmt2->fetch();
 
if(empty($row2[0]>0)){
	function passwordGenerator($length) {
		$uppercase = array('A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'R', 'S', 'T', 'U', 'W', 'Y', 'Z');
		$lowercase = array('a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'r', 's', 't', 'u', 'w', 'y', 'z');
		$number = array(0, 1, 2, 3, 4, 5, 6, 7, 8, 9);
		$password = NULL;
		for ($i = 0; $i < $length; $i++) {
				$password .= $uppercase[rand(0, count($uppercase) - 1)];
				$password .= $lowercase[rand(0, count($lowercase) - 1)];
				$password .= $number[rand(0, count($number) - 1)];
		}
		return substr($password, 0, $length);
}
$kod = rand(11111111,99999999);
$wzrost = rand(165,210);
$waga = rand(60,120);
//WYGENEROWANE HASLO
	$myPassword = passwordGenerator(8);
	$myPassword1 = sha1($myPassword);
	$repka = rand(1,64);
  $data = date("Y-m-d H:i:s");
 
	$dodaj = $pdo->prepare("INSERT INTO `Konta`(login, haslo, email) VALUES(:login, :haslo, :email)");
	$dodaj->bindValue(":login", $login, PDO::PARAM_STR);
	$dodaj->bindValue(":haslo", $myPassword1, PDO::PARAM_STR);
	$dodaj->bindValue(":email", $email, PDO::PARAM_STR);
	$dodaj->execute();
 
 
				 		echo "
<script type='text/javascript'>window.alert('Utworzono konto sprawdź swoją skrzynke pocztową na nią wysłaliśmy Ci wygenerowane hasło !!!!');document.location.href = 'index.php';</script>
";
 
 
/// poniżej funkcja mail() wysyła na pocztę dane do logowania !!
 
$to      = ''.$_POST['email'].'';
$subject = 'Rejestracja Serwisie';
$message = "Witaj ".$_POST['login']."!
 
   Twoje dane w serwisie to::
    Login: ".$_POST['login']."
    E-mail: ".$_POST['email']."
    Hasło: ".$myPassword."
 
 
    Wiadomość wygenerowana automatycznie, prosimy nie odpowiadać.
    Pozdrawiamy,
    Administracja Serwisu";
$headers = 'From: SERWIS XYZ' . "\r\n" .
    'Reply-To: noreply@serwis.xyz' . "\r\n" .
    'X-Mailer: PHP/' . phpversion();
 
mail($to, $subject, $message, $headers);
 
}else{
			 		echo "
<script type='text/javascript'>window.alert('Konto o takim adresie e-mail już istnieje');document.location.href = 'index.php';</script>
";
 
}
}
 
 
 
}
 
 
 
 
 
 
 
?>
 
<html>
<head>
<title> Rejestarcja w serwisie</title>
</head>
<body>
 
<form action="index.php" method="POST" class="cd-form">
					<p class="fieldset">
						<label>Login</label>
						<input name="imie" type="text" placeholder="Login">
 
					</p>
 
					<p class="fieldset">
						<label>E-mail</label>
						<input  name="email" type="email" placeholder="E-mail">
					</p>
 
 
			  <p class="fieldset">
						<input type="submit" name="send" value="Załóż konto">
					</p>
				</form>
 
</body>
 
 
 
 
</html>
 
 
 
[PHP] pobierz, plaintext 

W bazie danych ustaw haslo,email oraz login jako varchar i np daj im 255 znaków. W tym skrypcie tylko sobie zamień dane do połączenia z bazą. Tu masz już PDO. Pamiętaj!