[MySQL][PHP]Czy to dobre zapytanie Opcje

[--armind--]

Witam, chciał bym się zapytać czy to dobre zapytanie ejst, i czy cały ten skrypt jest dobrze skonstruowany. Uzywam codeigniter. tak więc to moje początki.
Potrzebuje z tego zapytania wyciągnąć dane z tabel: users i role, tak wiec dlatego używam JOIN

[PHP] pobierz, plaintext 
	public function login($login, $pass)
	{
		$query = $this->db->query("SELECT * FROM users LEFT JOIN role ON role.user_id = users.id WHERE login='$login' AND pass='$pass'");
	    foreach ($query->result() as $row)
		{
			return $data = array('id' => $row->id, 'name' => $row->name, 'admin' => $row->admin);
		}
	}
[PHP] pobierz, plaintext 

Wczesniej doradzono mi zebym uzył tego kawałka kodu:

[PHP] pobierz, plaintext 
return $this->db->select('*')->from('users')->join('role', 'user_id=users.id')->where(array('login' => $login, 'pass' => $pass))->get()->row_array();
[PHP] pobierz, plaintext 

No ale niestety, tylko jeden user może się zalogować.

[Turson]

[PHP] pobierz, plaintext 
public function login($login, $pass)
	{
		$query = $this->db->query("SELECT * FROM users LEFT JOIN role ON role.user_id = users.id WHERE login='$login' AND pass='$pass'");
	    foreach ($query->result() as $row)
		{
			return $data = array('id' => $row->id, 'name' => $row->name, 'admin' => $row->admin);
		}
	}
[PHP] pobierz, plaintext 

Przecież tylko jeden użytkownik może mieć login i hasło pasujące do wzorca w warunku, więc foreach jest niepotrzebny.

Wczesniej doradzono mi zebym uzył tego kawałka kodu:

Myślę, że dobrze doradzono, bo to frameworkowy sposób zapytania.

No ale niestety, tylko jeden user może się zalogować.

Jak jeden

[--armind--]

Jeden uzytkownik moze się zalogować, inni userzy się nie zalogują - "Niewłaściwe dane".

Jak by tu wytłumaczyć...chodzi mi że pierwszy user z id np 40 się zaloguje a reszta id ma problem z logowaniem bo nie idzie się zalogować

[--armind--]

Odświeżam. prosze o pomoc.