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 ?
Zaraz to wypróbuję.

Nie jest czymś złym mieć dużo funkcji.
Szczególnie jeżeli każda robi co innego.

dlatego kod poniżej powinien być w osobnej funkcji

[PHP] pobierz, plaintext 
$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");
[PHP] pobierz, plaintext 

W ogóle, nie ma potrzeby tworzyć zmiennych ponad stan, jeżeli ich nie używasz w innych miejscach skryptu.

Dobrze zrozumiałem, że w tej drugiej funkcji powinienem odnieść się do funkcji login, która de facto sprawdza i loguje klienta
mniej więcej coś takiego

[PHP] pobierz, plaintext 
if($this->login() === true) {
echo "OK" ;
} else {
echo "NIE OK";
}
[PHP] pobierz, plaintext 

Czy dobrze rozumiem

Twoja funkcja login() nie zwraca ani true ani false. Wiec albo popraw to co ona zwraca albo popraw warunek. Lepiej jakbys poprawil to co zwraca

Muszę to jutro na spokojnie przemyśleć, bo w chwili obecnej funkcja login() zwraca ID użytkownika, szkoda że nie da się zrobić tak by zwracała dwie wartości, tzn ID i true

A po co ma zwracac ID i true? Bez sensu.
Niech zwraca ID lub null gdy usera nie ma.

[PHP] pobierz, plaintext 
		if($result->num_rows > 0) {
			return (int) $result->fetch_row()[0];
		}
		return null;
[PHP] pobierz, plaintext 

A potem sprawdzanie

[PHP] pobierz, plaintext 
    if($this->login() !== null) {
    echo "OK" ;
    } else {
    echo "NIE OK";
    }
[PHP] pobierz, plaintext 

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 

Jak chcesz skracać, to dlaczego formularz powielasz zamiast wrzucić do osobnej metody?

A wiesz, nie zastanowiłem się nad tym, ale jest to dobry pomysł.

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 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 {
          $users->login_form($result);
        }
    }
    else if(empty($_POST)) {
      $result = null;
      $users->login_form($result);
    }
  break;
 
 
	public function login_form($result) {
		echo "<form action=\"index.php?action=log_in\" 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>";
		if(isset($result)) {
		echo implode('<br>',$result).'</p>';
		}
	}
 
[PHP] pobierz, plaintext