Zapytanie do bazy Opcje

[Dukov]

Wszystko już jest na dobrej drodze, problem mam jeszcze z zapytaniem do bazy

[PHP] pobierz, plaintext 
<?php
	$avatar;
	if ($_POST['name'] != null){
		$userName = $_POST['name'];	
	}
	if ($_POST['city'] != null ) {
		$userCity = $_POST['city'];
	}
	if ($_POST['desc'] != null){
		$userDesc = $_POST['desc'];
	}
 
	if ($_POST['login'] != '' && $_POST['pass'] != '' && $_POST['mail'] != '') {
 
 
		if (strlen($_POST['login']) > 3 && strlen($_POST['login']) < 10  && strlen($_POST['pass']) > 5 && strlen($_POST['pass']) < 16  ) {
			if (preg_match('/^[a-zA-Z0-9\.\-_]+\@[a-zA-Z0-9\.\-_]+\.[a-z]{2,4}$/D', $_POST['mail']) && preg_match('/^[a-zA-Z0-9\.\-_]/', $_POST['login']) ) {
 
 
				$userLogin = $_POST['login'];
				$userPass = $_POST['pass'];	
				$userMail = $_POST['mail'];
				$len = 20;
				$userID = substr(sha1(rand(1,10000)),0,$len);
				echo $userID;
				if($_FILES['filename']['name'] != ''){
					$avatar = fileReceive();
				}
				reg();
 			}
 			else
 			{
 				header("Location: ../reg.php?reg=badchars");
 			}	
 		}
 		else
 		{
 			header("Location: ../reg.php?reg=shortpole");
 		}
	}
	else
	{
		header("Location: ../reg.php?reg=emptyf");
	}
	function reg(){
		global $userID, $userMail, $userPass, $userLogin, $userDesc, $userCity, $userName, $avatar;
		try {
 			$host = 'localhost';
			$baza = 'user';
			$uzytkownik = 'adamek';
			$haslo = 'aaaaa5';
 
 
    		$conn = new PDO("mysql:host=$host;dbname=$baza", $uzytkownik, $haslo);
			$conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			$stmt = $conn->prepare("SELECT Nazwa AND mail FROM Users WHERE Nazwa = '$userLogin' OR mail = '$userMail'"); 
			$stmt->execute();
			$row = $stmt->fetch();
			if($row['Nazwa'] == 0 && $row['mail'] == 0){
				$stmt = $conn->exec("INSERT INTO Users ( ID ,  Nazwa , Haslo ,  mail , name ,  city ,  image ,  desci ,  activate ,  admin ) VALUES ('$userID','$userLogin', '$userPass', '$userMail', '$userName', '$userCity, $avatar', '$userDesc', 0 , 0 "); 
 
 				echo "ok";
				#$sendMail();
			}
 
			else{
				header("Location: ../reg.php?reg=userexists");
			}
   		}
   		catch(PDOException $e)
    	{
   	 		echo "duppppp";
    	}
	}
 
 
 
 
	function fileReceive(){
		$max_size = 1024*1024;
	if (is_uploaded_file($_FILES['filename']['tmp_name'])) {
    	if ($_FILES['filename']['size'] > $max_size) {
        	echo 'Błąd! Plik jest za duży!';
    	} 
    	else {
        	echo 'Odebrano plik. Początkowa nazwa: '.$_FILES['filename']['name'];
        	echo '<br/>';
        		if (isset($_FILES['filename']['type'])) {
            		echo 'Typ: '.$_FILES['filename']['type'].'<br/>';
        		}
       	 	$strExt = substr( $_FILES['filename']['name'], strrpos( $_FILES['filename']['name'], '.' ) + 1 );
       	 	echo "Rozszerzenie: ". $strExt;
       	 	$len = 20;
       	 	$randstring =  substr(sha1(rand(1,10000)),0,$len).".jpg";
       	 	echo $randstring;
       	 	echo $_FILES['filename']['name'];
 
        if ($strExt == 'jpg' || 'JPG' || 'jpeg' || 'JPEG'){
        	move_uploaded_file($_FILES['filename']['tmp_name'],
                	$_SERVER['DOCUMENT_ROOT'].'/avatars/'.$_FILES['filename']['name']);
 
        }	
    	else{ 
    		echo "niy!!";}
    }
} else {
   echo 'Błąd przy przesyłaniu danych!';
}
 
 	rename('../avatars/'.$_FILES['filename']['name'], "../avatars/". $randstring);
 	return $randstring;
	} 
 
 
 
function sendMail(){
	ob_start();
	$addressee = 'ajjambor912@gmail.com'; 	
	@$email = $_POST['email'];
 
	@$content = 'Name '.$_POST['username'].'  Phone: '.$_POST['userphone'].' Content: '.$_POST['message'];
 
	$header = 	"From: ".$_POST['email']." \nContent-Type:".
			' text/plain;charset="iso-8859-2"'.
			"\nContent-Transfer-Encoding: 8bit";
	if (mail($addressee, 'Message from NAhandyman: ', $content , $header))
 
			header('Location:index.php?send=true');
	else 
			header('Location:index.php?send=false');
}
 ?>
[PHP] pobierz, plaintext 

Tutaj jest błąd

[PHP] pobierz, plaintext 
$stmt = $conn->exec("INSERT INTO Users ( ID ,  Nazwa , Haslo ,  mail , name ,  city ,  image ,  desci ,  activate ,  admin ) VALUES ('$userID','$userLogin', '$userPass', '$userMail', '$userName', '$userCity, $avatar', '$userDesc', 0 , 0 "); 
[PHP] pobierz, plaintext 

Skąd wiem? Bo jak wykomentuje to skrypt leci do kolejnej lini i mam wyświetla się ok, z echa. Jeśli skasuje komentarz to przechodzi do instrukcji catch i z echa wyświetla się duppp.

[Dukov]

AuthorIP miałem jako INT, a w adresie są kropki. Dzięki, na przyszłość będę wiedział, że jak coś wygląda ok, to żeby sprawdzić sam SQL, aczkolwiek możesz mi powiedzieć, czy dobrze robię, ze każda funkcja, która potrzebuje coś z bazy łączy się z bazą? Czy lepiej napisać osobną funkcje do łączenia z bazą? Albo łączyć się tylko w głównej funkcji i wywoływać w niej pozostałe?

EDIT
Działa.

Ten post edytował Dukov 7.08.2017, 20:38:18