Вывод тестов по конкретному ID MySQL?

Question

[YTGoodFox]

Добрый день, столкнулся с проблемой: не могу вывести тесты на экран. Ну то есть сами тесты выводятся и работают, но когда я указываю что скажем у условного пользователя ВАСЯ есть поле группа и там стоит значение 1, то должны выводиться тесты у которых поле GroupId тоже равно единице. Пользователя авторизируется у меня при помощи ссесии.
В файле Functions.php у меня расписаны собственно все функции. Вывести тесты я пытаюсь на 32 строчке. Собственно я пытаюсь сравнить значение таблицы user поля login с 'logged_user', а 'logged_user' это у меня ссесия пользователя.

Заливаю архив со всем проектом на гугл диск, т.к. кода много что бы вставлять весь, поэтому опишу все файлы.
Файл Registration.php страница, где можно зарегистрировать пользователя.
Файл Login.php страница, где можно так сказать войти в аккаунт.
Файл Index.php страница основная, с которой осуществляется переход на другие.
Файл db.php содержит код, подключающий к бд MYSQL и там стартует ссесия пользователя.
Файл Logout.php это собственно выход из ссесии.
Файл Functions.php это файл с функциями для страницы Tests.php
Файл Tests.php это страница с тестами.
Как я и говорил ранее проблема в том что я не могу получить тесты для человека с конкретной группой, это всё в файле Functions.php и сама проблема на 32 строке, я вставлю код ниже:

// Получение данных теста
function get_test_data($test_id){
  if( !$test_id ) return;
  global $db;
  $query = "SELECT q.question, q.parent_test, a.id, a.answer, a.parent_question, u.group
    FROM questions q
      LEFT JOIN answers a
        ON q.id = a.parent_question
      LEFT JOIN test
        ON test.id = q.parent_test
      INNER JOIN users u
        ON u.group = test.GroupId
            WHERE q.parent_test = $test_id AND test.enable = '1' AND u.login = 'logged_user' "; // НЕ РАБОТАЕТ НАДО ПОФИКСЕТЬ
  $res = mysqli_query($db, $query);
  $data = null;
  while ($row = mysqli_fetch_assoc($res)) {
    if( !$row['parent_question'] ) return false;
    $data[$row['parent_question']][0]= $row['question'];
    $data[$row['parent_question']][$row['id']] = $row['answer'];
  }
  return $data;
}

Я не ожидаю что кто то конечно его будет скачивать но всё же если кто то захочет посмотреть что у меня там как работает, то очень многое закоментировано. Так же в архиве лежит экспортированная таблица musql, что бы было удобнее всё просмотреть. ССЫЛКА

Заливаю на GitHub , однако файл БД MYSQL всё ещё в архиве на гугл диске.

Comments

[Daria Motorina]

Перезалейте проект на github с и обновите вопрос с ссылкой на него и конкретным фрагментом кода, где проблема. Гугл драйв все равно не показывает содержимое php файлов

Answer 1

[Валера Карманов]

Первая ошибка!

Почему используешь $_POST, ты же $_GET параметр передаешь....
Вот и условие в 9-й строке не работает

И еще, что конкретно должен возвращать функция get_test_data? (список вопросов теста)

Comments

[YTGoodFox]

Если использовать GET, то у меня не выводятся тесты а срабатывает сразу завершение теста

[Валера Карманов]

Да и к тому же у тебя нет пользователя с логином logged_user

[Валера Карманов]

YTGoodFox, значит не правильно написан код, $_POST там даже и не пахнет...

[YTGoodFox]

Валера Карманов, Это ссесия у меня в файле Login.php, я просто думал что я буду сравнивать столбец login с вошедшим пользователем

<?php
     $data = $_POST;
     if(isset($data['do_login']))
     {
       $errors = array();
       $user = R::findOne('users','login = ?', array($data['login']));
       if($user)
          {

              // логин существует
              if(password_verify($data['password'], $user->password))
                  {
                    // Всё хорошо, логиним пользователя
                    $_SESSION['logged_user'] = $user;
                    $_SESSION['logged_user'] = $_POST['login'];
                    echo '<meta http-equiv="refresh" content="1; URL=index.php" />';
                  } else {
                            {
                              $errors[] = 'Не правильно введён пароль!';
                            }
                         }
          } else {
                    $errors[] = 'Пользователь с таким логином не найден!';
                 }
                if( ! empty($errors))
                  {
                    echo '<div class="alert alert-danger" role="alert">'.array_shift($errors).'</div><hr>';
                  }
    }
?>

[Валера Карманов]

Не мне судить как кто пишет код, но что за бред в 34 и 35 строках!!!!

выбери что то одно...

[YTGoodFox]

Валера Карманов, Писал по одному видео, сам я плохо принципе кодю, это мне для курсовой

[Валера Карманов]

YTGoodFox, используй ID пользователя, он уникальный для каждого пользователя

[YTGoodFox]

Не знаю что лучше выбрать, там были обе строчки и я обе оставил, хотя по логике я смотрю что они же по идеи одни и тоже делают.

[YTGoodFox]

Валера Карманов, Для чего использовать ID, для вывода тестов? У меня будет много пользователей и по этому мне их надо разбить по группам.

[Валера Карманов]

убери из запроса AND u.login = 'logged_user', и все должно заработать. На счет $_POST и $_GET забудь, я тупанул и не сразу понял зачем этот код нужен

[Валера Карманов]

YTGoodFox, может я тупой?! Кто знает!!!
Когда объясни зачем в запросе указано
AND u.login = 'logged_user'

[YTGoodFox]

Валера Карманов, Так весь прикол в том что AND u.login = 'logged_user'
нужен, для того что бы скажем Пользователь1 с группой ВТ получаем тесты принадлежащие к такой же группе.
Я знаю что без этой строчки работает, но мне нужна сортировка по группам Что бы у ВТ были одни тесты, что бы у БУХ другие, у меня в бд kuser есть таблица users, в которой есть поле group, есть просто таблица group с id номером каждой группы и в таблице test есть поле GroupId.

[YTGoodFox]

Валера Карманов,

[Валера Карманов]

YTGoodFox, получается что тесты надо выводить согласно группе в которой находиться пользователь?

[YTGoodFox]

Валера Карманов, Да

[Валера Карманов]

YTGoodFox, исправление для функции get_tests

global $db;

$ulogin = null;

if(isset($_SESSION['logged_user'])) {
  $ulogin = $_SESSION['logged_user'];
}

$query = "SELECT *
  FROM
    test AS t
  LEFT JOIN users AS u
    ON u.login = '$ulogin'
  WHERE
    t.enable = '1'
  AND
    t.GroupId = u.group";

$res = mysqli_query($db, $query);
if(!$res) return false;
$data = array();
while($row = mysqli_fetch_assoc($res)){
  $data[] = $row;
  }
return $data;

[Валера Карманов]

YTGoodFox, далее исправления для функции get_test_data

if( !$test_id ) return;

  $ulogin = null;

  if(isset($_SESSION['logged_user'])) {
    $ulogin = $_SESSION['logged_user'];
  }

  global $db;
  $query = "SELECT q.question, q.parent_test, a.id, a.answer, a.parent_question, u.group
    FROM questions q
      LEFT JOIN answers a
        ON q.id = a.parent_question
      LEFT JOIN test
        ON test.id = q.parent_test
      INNER JOIN users u
        ON u.group = test.GroupId
            WHERE q.parent_test = $test_id AND test.enable = '1' AND u.login = '$ulogin'"; 
  $res = mysqli_query($db, $query);
  $data = null;
  while ($row = mysqli_fetch_assoc($res)) {
    if( !$row['parent_question'] ) return false;
    $data[$row['parent_question']][0]= $row['question'];
    $data[$row['parent_question']][$row['id']] = $row['answer'];
  }
  return $data;

также добавь условие на не найденный тест

если тест не найден или не относиться к текущему пользователю, то будет выводиться ошибка что тест не найден.

[YTGoodFox]

Как то не правильно он выбирает id теста

[YTGoodFox]

Для 2го теста у него 1 id и он его не выводит а для второго 3ий

[Валера Карманов]

YTGoodFox, в 17 строке в фанкшине замени * на t.*

[YTGoodFox]

Валера Карманов, Можете пожалуйста скриншотом показать?

[Валера Карманов]

[YTGoodFox]

Валера Карманов, Спасибо большое, а не можете пожалуйста сразу подсказать а как можно скрывать все страницы кроме login.php для не авторизированых людей? и скажем только администраторам давать доступ к registratio.php?

[Валера Карманов]

YTGoodFox, добавь условие где нужно

if(!isset($_SESSION['logged_user'])) {
  header('Location: /'); exit;
}

для регистрации вот такое условие

if($_SESSION['logged_user'] != 'admin') {
  header('Location: /'); exit;
}

admin — логин админа

[YTGoodFox]

Валера Карманов, ,ОГРОМНЕЙШЕЕ ВАМ СПАСИБО