[PHP] Połączenie dwóch funkcji Opcje

[topcio]

Witajcie

Czy można jakoś połączyć poniższe funkcje w jedną, jakoś samemu nie potrafię, dlatego zrobiłem dwie oddzielne, ale chyba lepiej jest jak jest mniej zapytań do bazy zwłaszcza takich samych.

[PHP] pobierz, plaintext 
	public function login($data) {
		$data['user_pass'] = $this->code_passwd($data['user_pass']);
		$result = $this->sql->query("SELECT user_id FROM users WHERE user_login = '{$data['user_login']}' AND user_pass = '{$data['user_pass']}' LIMIT 1");
		if($result->num_rows > 0) {
			return $result->fetch_row()[0];
		} else {
			return 	['Podany login i/lub hasło są niepoprawne. Spróbuj Ponownie. <br />' .
							'Zapomniałem <a href="index.php?action=forgetten">Hasła.</a>'];
		}
	}
 
	public function loged_in($data) {
		$data['user_pass'] = $this->code_passwd($data['user_pass']);
		$result = $this->sql->query("SELECT user_id FROM users WHERE user_login = '{$data['user_login']}' AND user_pass = '{$data['user_pass']}' LIMIT 1");
		$value = mysqli_fetch_object($result);
		$user_id = $value->user_id;
		$query = "
			UPDATE users
			SET
			user_last_login_IP = '" . ip2long($this->getClientIP()) . "'
			WHERE user_id = '{$user_id}'
			";
		$result = $this->sql->query($query);
			if($result === false) {
				$errors[] = "Wystąpił błąd z bazą danych.";
				return $errors;
			}
	}
[PHP] pobierz, plaintext 

A może po prostu w drugiej funkcji sprawdzę czy pierwsza jest true (IMG:style_emoticons/default/questionmark.gif) ?
Zaraz to wypróbuję.

[topcio]

Zrobiłem tak

[PHP] pobierz, plaintext 
  case 'log_in':
    if($users->is_logged()) {
      $data = $users->get_data();
      if($data['user_range'] == 1) {
        header("Location: index.php?action=user_login");
      }
      if ($data['user_range'] == 2) {
        header("Location: index.php?action=admin_login");
      }
      if ($data['user_range'] == 3) {
        header("Location: index.php?action=devel_login");
      }
    } else {
        echo "<form action=\"index.php?action=log_in_failed\" method=\"post\">";
        echo "<input type=\"text\" placeholder=\"Login\" name=\"user_login\">";
        echo "<input type=\"password\" placeholder=\"Hasło\" name=\"user_pass\">";
        echo "<input type=\"submit\" value=\"Zaloguj\">";
        echo "</form>";
    }
  break;
 
  case 'log_in_failed':
    if(!empty($_POST)) {
      $result = $users->login($_POST);
        if(is_numeric($result)) {
          $_SESSION['user_id'] = $result;
          $users->loged_in($_SESSION['user_id']);
          header("Location: index.php?action=log_in");
        } else {
          echo "<form action=\"index.php?action=log_in_failed\" method=\"post\">";
          echo "<input type=\"text\" placeholder=\"Login\" name=\"user_login\">";
          echo "<input type=\"password\" placeholder=\"Hasło\" name=\"user_pass\">";
          echo "<input type=\"submit\" value=\"Zaloguj\">";
          echo "</form>";
          echo implode('<br>',$result).'</p>';
        }
    }
  break;
[PHP] pobierz, plaintext 

Może być?
Bo gdybym zrobił, że null jak nie zalogowany, to by mi nie wyświetliło komunikatu, że błędny login albo hasło.
A tak sprawdza czy jest numer.
A drugą funkcję skróciłem do takiej postaci

[PHP] pobierz, plaintext 
	public function loged_in() {
			$query = "
				UPDATE users
				SET
				user_last_login_IP = '" . ip2long($this->getClientIP()) . "'
				WHERE user_id = '{$_SESSION['user_id']}'
				";
			$result = $this->sql->query($query);
			if($result === false) {
				$errors[] = "Wystąpił błąd z bazą danych.";
				return $errors;
			}
		}
[PHP] pobierz, plaintext 

Ten post edytował topcio 29.04.2017, 17:09:20