Проблема логин скрипт в PHP где мне найти ошибку?

Question

[Mini MVC]

Когда я уже залогинился у меня должно в место Login стоять Logout но чтото у меня это не получаеться, может кто небудь мне помочь найти ошибку что я делаю не так? вот сам код:

header.php

<header>
            <div class="wrapper">
                
                <nav>
                    <p>Restaurant Tuscany</p>
                    
                    <ul>
                            <li><a href="?page=2">Menu</a></li>
                            <li><a href="?page=3">Gallery</a></li>
                            <li><a href="?page=4">Contact</a></li>
                            
                            <?php 

                                if(isset($_SESSION['username'])){ 

                                    foreach($userRows as $userRow){ 

                            ?>
                            
                            <li><a href='?action=logout'>Logud</a></li>
                            
                            <?php 
                                    } 

                                    if(isset($_GET['action'])){ 

                                        if($_GET['action'] == 'logout'){ 

                                            restLogout(); 

                                        } 

                                    } 

                                }else{ 
                                    
                                    print '<li><a href="?page=5">Login</a></li>'; 

                                } 

                                ?>
                            
                            
                    </ul><!-- ul ends here -->
                    
                    <div class="clear"></div>
                    
                </nav><!-- nav ends here -->
                 
                    <h1><br>Why travel to Italy<br>to taste the local<br>traditions?</h1>
                    
                    
                    <div class="circle">  
                        
                        <p><br>Get your<br> 
                            Tuscany<br>
                            daily special<br>
                            here</p>
                        
                    </div><!-- circle  ends here -->
                    
            </div><!-- wrapper  ends here -->
            
        </header><!-- header ends here -->

login.php

<?php 

    if(isset($_SESSION['username'])){
        
        print "<script>window.location.replace('?page=home');</script>";
        
    }else{

?>

<div class="login">
    
    <div class="loginWrapper">
        
        <div id="login">
            
            <div id="triangle"></div>
            
            <h1>Log in</h1>
            
            <form action="" method="POST">
                
                <input type="text" name="username" placeholder="Username" />
                
                    <?php 
                    
                        if($_GET['error'] == 2){
                            
                            print "Wrong Username";
                            
                        }
                    
                    ?>
                <input type="password" name="password" placeholder="Password" />
              
                    <?php 
                    
                        if($_GET['error'] == 3){
                            
                            print "Wrong Password";
                            
                        }
                    
                    ?>
                
                <input type="submit" name="submit" value="Login" />
                
            </form>
            
                    <?php 

                    if($_GET['error'] == 1){ 

                        print "All should be field out"; 

                    } 
                    
                    //Virker med Admin 
                        if(isset($_POST['submit'])){ 

                            $username = $_POST['username']; 
                            
                            $password = hash('sha256', $_POST['password']); 

                            restLogin($username, $password); 
                            
                        } 
} 
?> 
            
            
          </div>
        
    </div>
        
        
</div>

functions.php

<?php 

function restLogin($username, $password){ 

    global $objCon; 

    if($username != '' && $password != ''){ 
        
        $checkUser = $objCon->query("SELECT username FROM rest_user WHERE username = '$username' LIMIT 1"); 
        
        $checkAdmin = $objCon->query("SELECT username, password, userType FROM rest_user WHERE username = '$username' AND password = '$password' AND userType = 1"); 
        
        $checkPass = $objCon->query("SELECT username, password FROM rest_user WHERE username = '$username' AND password = '$password' LIMIT 1"); 
        
        if(mysqli_num_rows($checkUser) != 1){ 

            header('Location: ?page=5&error=2'); 

        }elseif(mysqli_num_rows($checkPass) != 1){ 

            header('Location: ?page=5&error=3'); 

        }elseif(mysqli_num_rows($checkAdmin) >= 1){ 

            $_SESSION['username'] = $username; 

            header('Location: ?page=100'); 

        }else{ 

            $_SESSION['username'] = $username; 

            header('Location: ?page=200'); 

        } 

    }else{ 

        header('Location: ?page=5&error=1'); 

    } 
} 

?>

<?php 

function restLogout(){ 

    unset($_SESSION['username']); 

    header('Location: ?page=home'); 

} 

?>

Answer 1

[Sergey Lerg]

Зачем у вас ссылка на Logout в цикле?
У вас либо пустой массив $userRows, либо он вовсе не нужен.

Comments

[Mini MVC]

ссылка для того чтобы я мог сделать выход.

вот насчёт $userRows я не пойму наверное я гдетот чтото упустил или не дописал вот не как не могу понайть что.

[Sergey Lerg]

ALLEROY 88: я имею ввиду зачем в цикле?

[Mini MVC]

Sergey Lerg: ну потомучто я так всегда делал и так меня учили этому, вот тут у меня оно не получаеться! А как мне сделать тогда чтобы оно заработало когда я сделаю вход на сайт и чтобы у меня в место вход показывало выход?

[Sergey Lerg]

ALLEROY 88: уберите цикл, оставьте только ссылку.

[Mini MVC]

Sergey Lerg: что ты иммешь веду когда говоришь цикл? можешь мне пакозать ато я чтото не пойму

[Sergey Lerg]

ALLEROY 88: foreach

[Mini MVC]

Sergey Lerg: я убрал тогда у меня показывает кнопка вход и выход

[Sergey Lerg]

ALLEROY 88: значит напортачили с if.

[Mini MVC]

Sergey Lerg: я чтото не могу найти в if проблему, где это именно я напортачил?

[Sergey Lerg]

ALLEROY 88: если у вас в одной ветке if - ссылка на выход, а в другой ветке if - ссылка на вход, то они не могут выводится одновременно, если же они всё таки выводятся одновременно, то значит if построен не правильно.

[Mini MVC]

Sergey Lerg: Ну а тогда как мне правельно написать?

[Sergey Lerg]

ALLEROY 88: На этом мои способности к телекинезу заканчиваются. Ищите.

Answer 2

[Mini MVC]

Когда я вписываю в этот код в header.php у мнея всё праподает.

$user = $_SESSION['username']; 
    $userResult =  $objCon->query("SELECT * FROM rest_user WHERE username = '$user'"); 

    $userRows = array(); 
    while($userRow = $userResult->fetch_object()){ 
        $userRows[] = $userRow; 
    }

Answer 3

[Сергей Мельников]

Глядя на ваш код напрашивается один вывод - пожалейте тех, кто его будет читать, это просто ппц!

header.php

<?php
if (isset($_GET['action']) && ($_GET['action'] == 'logout') {
  restLogout(); 
}
?>
<header>
  <div class="wrapper">      
    <nav>
      <p>Restaurant Tuscany</p>

      <ul>
        <li><a href="?page=2">Menu</a></li>
        <li><a href="?page=3">Gallery</a></li>
        <li><a href="?page=4">Contact</a></li>

        <?php if (isset($_SESSION['username']) && count($userRows)): ?>
          <li><a href='?action=logout'>Logud</a></li>
        <?php else: ?>
          <li><a href="?page=5">Login</a></li>
        <?php endif; ?>
      </ul><!-- ul ends here -->

      <div class="clear"></div>        
    </nav><!-- nav ends here -->

    <h1><br>Why travel to Italy<br>to taste the local<br>traditions?</h1>

    <div class="circle">
      <p><br>Get your<br> Tuscany<br> daily special<br> here</p>
    </div><!-- circle  ends here -->          
  </div><!-- wrapper  ends here -->  
</header><!-- header ends here -->

login.php

<?php
//Virker med Admin 
if (isset($_POST['submit'])) {
  $username = $_POST['username'];
  $password = hash('sha256', $_POST['password']);
  restLogin($username, $password);    
}
?>

<?php if (isset($_SESSION['username'])): ?>
  <script>window.location.replace('?page=home');</script>
<?php else: ?>

<div class="login">
  <div class="loginWrapper">
    <div id="login">
      <div id="triangle"></div>

      <h1>Log in</h1>

      <form action="" method="POST">
        <input type="text" name="username" placeholder="Username" />
        <?php if($_GET['error'] == 2): ?>
          Wrong Username
        <?php endif; ?>
        
        <input type="password" name="password" placeholder="Password" />
        <?php if($_GET['error'] == 3): ?>
          Wrong Password
        <?php endif; ?>
                
        <input type="submit" name="submit" value="Login" />                
      </form>

      <?php if($_GET['error'] == 1): ?>
        All should be field out
      <?php endif; ?>                 
            
    </div>        
  </div>
</div>

<?php endif; ?>

functions.php

<?php 

function restLogin($username, $password) {

  global $objCon;

  if ($username != '' && $password != '') {

    $checkUser = $objCon->query("SELECT username FROM rest_user WHERE username = '$username' LIMIT 1");
    $checkAdmin = $objCon->query("SELECT username, password, userType FROM rest_user WHERE ".
      "username = '$username' AND password = '$password' AND userType = 1");

    $checkPass = $objCon->query("SELECT username, password FROM rest_user WHERE username = ".
      "'$username' AND password = '$password' LIMIT 1");

    if (mysqli_num_rows($checkUser) != 1) {
      header('Location: ?page=5&error=2');
    } elseif (mysqli_num_rows($checkPass) != 1) {
      header('Location: ?page=5&error=3');
    } elseif (mysqli_num_rows($checkAdmin) >= 1) {
      $_SESSION['username'] = $username;
      header('Location: ?page=100');
    } else {
      $_SESSION['username'] = $username;
      header('Location: ?page=200');
    } 

  } else {
    header('Location: ?page=5&error=1');
  } 
} 

function restLogout() {

  unset($_SESSION['username']);
  header('Location: ?page=home'); 
}

?>

Comments

[Mini MVC]

тебя я уже пожалел

[Сергей Мельников]

ALLEROY 88: это мне просто было нечего делать)

[Mini MVC]

Сергей Мельников: Да я так и понял)

[Сергей Мельников]

ALLEROY 88: но ведь согласитесь, так и красивее и другим удобнее

[Сергей Мельников]

ALLEROY 88: https://mega.co.nz/#!ZlxmHTrS!gz6qXd0eHsRPhMeFy4-D...

[Mini MVC]

Сергей Мельников: что это?

[Сергей Мельников]

ALLEROY 88: посмотри, если будет интересно (реализация сайта с авторизацией)

[Mini MVC]

Сергей Мельников: сейчас пасмотрю, оно работает? проверял?

[Сергей Мельников]

ALLEROY 88: работает, это все я писал, если будут вопросы - пишите. Могу еще версию с комментариями в коде скинуть.

[Mini MVC]

Сергей Мельников: можешь мне версию с коментариями скинуть ато много чего нового тама для меня

[Сергей Мельников]

ALLEROY 88: https://mega.co.nz/#!48YHFCZZ!jiJO3qws4gPKjAQJDZ7F...

[Mini MVC]

Сергей Мельников: спасибо есле будут вопросы напишу

[Сергей Мельников]

ALLEROY 88: ок, можешь писать в скайп - mlnkv5