[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ę.

[Tomplus]

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.

[topcio]

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

[nospor]

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

[topcio]

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

[nospor]

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 

[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

[Tomplus]

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

[topcio]

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