[PHP][MySQL]Zmienna PHP w zapytaniu SQL Opcje

[EthanLyonall]

Witam, mam następujący kod:

DatabaseManager.class.php

[PHP] pobierz, plaintext 
<?php
	class DatabaseManager
	{
		static public function getConnection()
		{
			$conn = @new mysqli(DB_SERVER, DB_USERNAME, DB_PW, DB_DB);
 
			if(mysqli_connect_errno())
			{
				echo "Błąd połączenia z bazą danych! Skontaktuj się z administratorem!";
				exit();
			}
			else
				return $conn;
		}
 
		static public function selectBySQL($SQL)
		{
			$conn = self::getConnection();
			$SQL = $conn->real_escape_string($SQL);
			$result = $conn->query($SQL);
 
			if(!$result)
			{
				echo "Błąd zapytania! Skontaktuj się z administratorem!";
				exit();
			}
			else
			{
				$resultArray = Array();
 
				while(($row = $result->fetch_array(MYSQLI_ASSOC)) !== NULL)
				{
					$resultArray[] = $row;
				}
 
				if(count($resultArray) > 0)
					return $resultArray;
				else
					return false;
			}
 
			mysqli_close($conn);
		}
	}
?>
[PHP] pobierz, plaintext 

ModuleLoader.class.php:

[PHP] pobierz, plaintext 
<?php
	class ModuleLoader
	{
		static public function load($MODULE)
		{
			switch($MODULE)
			{
/*///////////*/	case 'login':
					if(!isset($_POST['login']))
					{
						header("Location: mainmenu");
						exit();
					}
 
					$login = $_POST['login'];
					$login = htmlentities($login, ENT_QUOTES, "UTF-8");
 
					DatabaseManager::selectBySQL("SELECT * FROM users WHERE username='$login'");
 
					if($resultArray)
					{
						$password = $_POST['password'];
 
						if(password_verify($password, $resultArray['password']))
						{
 
							$_SESSION['online'] = true;
							$_SESSION['login'] = $login;
 
							$result->free_result();
 
							unset($_POST['password']);
							unset($_POST['login']);
							header("Location: home");
						}
						else
						{
							$result->free_result();
							$_SESSION['repeat_login'] = '<span style="color: red">Nieprawidłowy login lub hasło!</span>';
							header("Location: mainmenu");
						}
					}
					else
					{
						$_SESSION['repeat_login'] = '<span style="color: red">Nieprawidłowy login lub hasło!</span>';
						header("Location: mainmenu");
					}
					break;
			}
		}
	}
?>
[PHP] pobierz, plaintext 

Problem pojawia się w 18 linii pliku ModuleLoader.class.php z samym zapytaniem. Nie ważne w jaki sposób próbuje tą zmienną włożyć do zapytania uaktywnia się if z 23 linii pliku DatabaseManager.class.php świadczący o złym sformułowaniu zapytania.

Jeżeli to samo zapytanie występuje bez zmiennej, załóżmy:
"SELECT * FROM users WHERE id=1"
To wszystko działa jak powinno (tj. uaktywnia się if z 37 linii (DatabaseManager.class.php) jeżeli wyszukanie nic nie znalazło lub else z 39 linii (DatabaseManager.class.php) jeżeli zapytanie coś wyszukało).

Problem pojawia się właśnie w chwili gdy chce do zapytania włożyć zmienną php. Próbowałem na różne sposoby:
1. "SELECT * FROM users WHERE username='$login'"
2. 'SELECT * FROM users WHERE username="$login"'
3. "SELECT * FROM users WHERE username='".$login."'"
4. 'SELECT * FROM users WHERE username="'$login'"'
5. "SELECT * FROM users WHERE username='{$login}'"
6. 'SELECT * FROM users WHERE username="{$login}"'
7. "SELECT * FROM users WHERE username='.{$login}.'"
8. 'SELECT * FROM users WHERE username=".{$login}."'
9. "SELECT * FROM users WHERE username='%$login%'"

Niestety żaden ze sposób nie przyniósł żadnego lepszego efektu.

[viking]

Ta twoja klasa jest do kitu. real_escape_string powinien escapować część zapytania czyli wartość wkładaną. Do tego $SQL może zwrócić null i nawet o tym nie będziesz wiedział.

[lustfingers]

Nie znam się na PHP ale na moje oko to linia 18 powinna wyglądać tak:

[PHP] pobierz, plaintext 
DatabaseManager::selectBySQL('SELECT * FROM users WHERE username='.$login);
[PHP] pobierz, plaintext 

[kapslokk]

Na sql'u też się nie znasz, skoro chcesz wciskać string do zapytania bez apostrofów / cudzysłowów.

Ten post edytował kapslokk 9.09.2016, 20:54:38

[lustfingers]

Ciekawe forum, widzę że łatwiej jest komuś wytknąć błąd niż dać rzeczową poradę które może na kierunkować na rozwiązanie. Tak nie znam sie i nie jestem programistą, interesuje sie tym od niedawna w wolnym czasie w ramach hobby.

[kapslokk]

Wytknalem Ci błąd, bo wprowadzasz innych użytkowników w błąd, a rzeczowej porady udzielił już viking i więcej do dodania już tutaj nie ma

[viking]

Żeby nie było że mało rzeczowo. Weź sobie jakiś orm typu Propel i zobacz jak to tam wygląda. Najlepiej użyj zewnętrzne rozwiązanie albo nawet goły kod PDO / Mysqli. Poczytaj o bindowaniu, zobacz w dokumentacji jak to wygląda. To co tu przedstawiłes nadaje się tylko do kosza.

[szajens]

[PHP] pobierz, plaintext 
DatabaseManager::selectBySQL("SELECT * FROM users WHERE username='$login'");
[PHP] pobierz, plaintext 

Ten post edytował szajens 17.09.2016, 06:34:29