Обработка ошибок. Меньше sql-запросов в логике.

2020-09-25 19:27:11 +03:00
parent 5b78d00f19
commit 3ca33f38aa
3 changed files with 94 additions and 79 deletions
@@ -4,22 +4,25 @@ session_start();

 if (is_null($_SESSION['uid'])) {
    header("Location: index.php");
+    exit;
 }
 require_once "functions.php";
-if ($user['room'] != 29) {
+$user = $user ?? [];
+if ($user->room != 29) {
    header("Location: main.php");
-    die();
+    exit;
 }
-if ($user['battle'] != 0) {
+if ($user->battle != 0) {
    header('location: fbattle.php');
-    die();
+    exit;
 }
 const BANK_SESSION_NAME = "bankid";
+const SUCCESS = "Успешная операция!";
 $get = urldecode(filter_input(INPUT_SERVER, 'QUERY_STRING'));
 if ($get == 'exit') {
    $_SESSION[BANK_SESSION_NAME] = null;
 }
-$bank = new Bank($_SESSION['uid']);
+$bank = new Bank($user->id);

 if (isset($_POST['userlogin'])) {
    $_SESSION[BANK_SESSION_NAME] = $bank->user_id;
@@ -28,24 +31,23 @@ if ($_SESSION[BANK_SESSION_NAME]) {
    $toid = (int)$_POST['to_id'] ?? 0;
    $summa = (int)$_POST['summa'] ?? 0;
    $submit = $_POST['action'] ?? '';
-
-    // Зачисдение кредитов на счёт.
+    $status = '';
+    // Зачисление кредитов на счёт.
    if ($submit === 'depositMoney' && $summa) {
        $bank->depositMoney($summa);
-        unset($submit, $summa);
+        $status = SUCCESS;
    }
-
    // Снятие кредитов со счёта.
    if ($submit === 'withdrawMoney' && $summa) {
        $bank->withdrawMoney($summa);
-        unset($submit, $summa);
+        $status = SUCCESS;
    }
-
    // Перевод кредитов на другой счёт.
    if ($submit === 'sendMoney' && $summa && $toid) {
        $bank->sendMoney($toid, $summa);
-        unset($submit, $summa, $toid);
+        $status = SUCCESS;
    }
+    unset($submit, $summa, $toid);
 }
 ?>
 <!DOCTYPE html>
@@ -64,7 +66,7 @@ if ($_SESSION[BANK_SESSION_NAME]) {
    <form method="post"><input type="submit" name="userlogin" value="Управление счётом"></form>
 <?php else: ?>
    <p><a href="?exit"> ← выйти из счёта</a></p>
-    <div><?= $bank->status ?></div>
+    <div><?= $status ?></div>
    <div class="appblock appblock-main">
        <span class="legend">Cчет №<?= $bank->user_id ?></span>
        <span class="wrap">На счету: <span class="num"><?= $bank->money ?></span></span>