перенаправление пользователя опроса на последнюю страницу, на которой он был

Question

перенаправление пользователя опроса на последнюю страницу, на которой он был

1

Я работаю над опросом, и я почти готов, но есть одно требование, которое действительно съело меня.

Пользователь может провести опрос, и есть 6 разных наборов вопросов, и каждый набор вопросов находится в группе вопросов от 1 до 6. Когда пользователь задает вопрос 1-4 и выходит из системы, он/она должен вернуться в систему и перенаправить в группу вопросов 5. До сих пор мой код работает и перенаправляется на первые три страницы (вопросы1.php, people.php и environment.php), но отказ для остальных.

Я думаю, что проблема - это мой if elseif statement. Любая помощь для достижения желаемого результата будет действительно оценена. Ура!

Вот код

if ($row['userid']==$user){
while($row = mysql_fetch_assoc($result_set)){
if($row['question_group_id']==1 && $row['choosen']==1){
 header("location:people.php"); 
       }
elseif($row['question_group_id']==2 && $row['choosen']==1){
 header("Location:environment.php"); 
    }
elseif($row['question_group_id']==3 && $row['choosen']==1){
 header("Location:widerorganization.php"); 
   }
elseif($row['question_group_id']==4 && $row['choosen']==1){
 header("Location:procurement.php"); 
  }
elseif($row['question_group_id']==5 && $row['choosen']==1){
 header("Location: communication.php"); 
   }
elseif($row['question_group_id']==6 && $row['choosen']==1){
 header("Location: notificationpage.php"); 
    }}}
 else {
 header("Location:questions1.php"); 
    }

Dave Sari 08 июль 2015, в 14:25

Источник

2

также обратите внимание, что после header() некоторый код все еще может быть выполнен. Чтобы предотвратить это, добавьте exit(); после каждого header()
SuperDJ 08 июль 2015, в 12:27
1

Вы используете $row вне вашего оператора while , затем перезаписываете его и используете снова в последнем elseif .
vonUbisch 08 июль 2015, в 12:34
0

Привет, Дейв, форматирование твоего вопроса может действительно помочь с небольшим количеством TLC, и нам будет легче помочь тебе!
Troyseph 08 июль 2015, в 12:58
1

@SebastianTroy, лол .. круто ...
Dave Sari 08 июль 2015, в 13:16
0

@vonUbisch Я только что видел, что .. в последний раз, если я избавился от строки для 'userid' .. Я поместил userid, который находится вне цикла внутри цикла, но это делает код хуже .. он даже ничего не проверяет но перейдем прямо к questions1.php, который еще
Dave Sari 08 июль 2015, в 13:23
0

@SuperDJ, когда я использую exit (); он сразу останавливается в первый раз и перенаправляет меня в это первое место
Dave Sari 08 июль 2015, в 13:28
0

Можете ли вы обновить свой вопрос новым кодом, потому что без него я не смогу сфабриковать точный ответ из-за слишком большого количества неопределенностей из-за перезаписи.
vonUbisch 08 июль 2015, в 13:28
0

как мне обновить? не вижу опции для этого :( @vonUbisch
Dave Sari 08 июль 2015, в 13:32
0

По какой-то причине редактирование отключено в этом вопросе .. Я не уверен, почему.
vonUbisch 08 июль 2015, в 13:38
0

@vonUbisch Я только что обновил вопрос, часть кода ... я удалил $ row ['userid'] == $ user в последнем случае, если
Dave Sari 08 июль 2015, в 13:41
0

@DaveSari проверьте мой ответ ниже;)
Allen Linatoc 08 июль 2015, в 13:48
0

@DaveSari, mysql_fetch_assoc всегда возвращает одну строку, верно?
vonUbisch 08 июль 2015, в 13:48
0

@vonUbisch да, именно поэтому я включил цикл while, чтобы он зациклился ..
Dave Sari 08 июль 2015, в 14:02
0

Так что же тут делать? Если это только одна строка вы можете оставить в while цикл, но это не имеет значения , что много ..
vonUbisch 08 июль 2015, в 14:06
0

@vonUbisch изначально у меня не было цикла, но без цикла он просто проверял первое условие, если оно выполнено, и загружал эту страницу, с циклом, который мог проверять больше, чем первое условие. Или, может быть, это то, что я использовал, есть ли возможность использования заявления случая? так что он проверяет все условия перед загрузкой страницы
Dave Sari 08 июль 2015, в 14:16

Показать ещё 13 комментариев

Теги:

php

sql

mysqli

2 ответа

Ещё вопросы

также обратите внимание, что после header() некоторый код все еще может быть выполнен. Чтобы предотвратить это, добавьте exit(); после каждого header()
Вы используете $row вне вашего оператора while , затем перезаписываете его и используете снова в последнем elseif .
Привет, Дейв, форматирование твоего вопроса может действительно помочь с небольшим количеством TLC, и нам будет легче помочь тебе!
@vonUbisch Я только что видел, что .. в последний раз, если я избавился от строки для 'userid' .. Я поместил userid, который находится вне цикла внутри цикла, но это делает код хуже .. он даже ничего не проверяет но перейдем прямо к questions1.php, который еще
@SuperDJ, когда я использую exit (); он сразу останавливается в первый раз и перенаправляет меня в это первое место
Можете ли вы обновить свой вопрос новым кодом, потому что без него я не смогу сфабриковать точный ответ из-за слишком большого количества неопределенностей из-за перезаписи.
как мне обновить? не вижу опции для этого :( @vonUbisch
По какой-то причине редактирование отключено в этом вопросе .. Я не уверен, почему.
@vonUbisch Я только что обновил вопрос, часть кода ... я удалил $ row ['userid'] == $ user в последнем случае, если
@DaveSari, mysql_fetch_assoc всегда возвращает одну строку, верно?
@vonUbisch да, именно поэтому я включил цикл while, чтобы он зациклился ..
Так что же тут делать? Если это только одна строка вы можете оставить в while цикл, но это не имеет значения , что много ..
@vonUbisch изначально у меня не было цикла, но без цикла он просто проверял первое условие, если оно выполнено, и загружал эту страницу, с циклом, который мог проверять больше, чем первое условие. Или, может быть, это то, что я использовал, есть ли возможность использования заявления случая? так что он проверяет все условия перед загрузкой страницы

Allen Linatoc · Answer 1 · 2015-07-08T12-21-00.000Z

Это вопрос рендеринга чистых кодов.

Я просто переформатировал ваш код. Вот ваш код до:

if ($row['userid']==$user)
{
    while($row = mysql_fetch_assoc($result_set))
    {
        if ($row['question_group_id']==1 && $row['choosen']==1)
        {
            header("location:people.php");
        }
        else if($row['question_group_id']==2 && $row['choosen']==1)
        {
            header("Location:environment.php"); 
        }
        else if ($row['question_group_id']==3 && $row['choosen']==1)
        {
            header("Location:widerorganization.php");
        }
        else if ($row['question_group_id']==4 && $row['choosen']==1)
        {
            header("Location:procurement.php"); 
        }
        else if ($row['question_group_id']==5 && $row['choosen']==1)
        {
            header("Location: communication.php");
        }
        else if ($row['question_group_id']==6 && $row['choosen']==1)
        {
            header("Location: notificationpage.php"); 
        }

    }
}
else
{
    header("Location:questions1.php"); 
}

Решение: Так как каждое состояние имеет предсказуемый характер, то почему бы нам не поставить эти страницы в ассоциативный массив вместо создания утомительного, if условие? Во всяком случае, это моя предлагаемая настройка вашего кода.

<?php

$locations = array(
    // Question Group ID => Landing page
    1 => 'people.php',
    2 => 'environment.php',
    3 => 'wideorganization.php',
    4 => 'procurement.php',
    5 => 'communication.php',
    6 => 'notificationpage.php'
);

if ($row['userid']==$user)
{
    while($row = mysql_fetch_assoc($result_set))
    {
        // Here the Question Group ID
        $questionGroupID = $row['question_group_id'];

        // Now we determine here
        if (isset($locations[$questionGroupID]) && $row['choosen'] == 1)
        {
            // Defined from the associative array above
            header(sprintf("location:%s", $locations[$questionGroupID]));
            exit();
        }
    }
}
else
{
    header("location:questions1.php"); 
}

Теперь его намного легче отлаживать. Попробуйте этот код;)

код выглядит чище, чем у меня .. хотя при запуске он перенаправляется на первое место people.php
Я уверен, что этот код выполняет свою работу :) Если это всегда перенаправляет вас на people.php , это означает, что значение столбца question_group_id равно 1. Для всех будет полезнее, если вы покажете запрос Вы использовали и структуру исходной таблицы из базы данных.
Я провожу тестирование с пользователем, который провел опрос для первых 3 групп .. поэтому человек должен быть направлен на procurement.php..lemme показать запрос и структуру таблицы

vonUbisch · Answer 2 · 2015-07-08T12-16-00.000Z

Здесь ваш код отформатирован без цикла и с оператором switch, как вы просили. Это может не работать напрямую, потому что все еще слишком много неопределенностей в отношении содержимого $newrow и источника $row. Но это отличное начало и действительно полезно для отладки. Оператор switch позволяет вам сделать некоторые изменения в каждом случае (как и до того, как вы изменили свой вопрос).

$page = 'questions1.php';
if ($row['userid'] == $user):
    $newrow = mysql_fetch_assoc($result_set);
    if ($newrow['choosen'] == 1):
        switch ($newrow['question_group_id']):
            case 1:
                $page = 'people.php';
                break;
            case 2:
                $page = 'enviroment.php';
                break;
            case 3:
                $page = 'widerorganization.php';
                break;
            case 4:
                $page = 'procurement.php';
                break;
            case 5:
                $page = 'communication.php';
                break;
            case 6:
                $page = 'notificationpage.php';
                break;
        endswitch;
    endif;
endif;
header("Location: {$page}");
exit;