не удалось сгенерировать и вставить quiz_id и question_id

Question

не удалось сгенерировать и вставить quiz_id и question_id

0

У меня есть следующий код для простой программы викторины, и по какой-то причине он не может сгенерировать (и сохранить в базе данных) идентификатор quiz и идентификатор вопроса, что означает, что ответы не имеют никакой ссылки.

Код в панели администратора: (я предполагаю, что ошибка здесь, но я не уверен)

//inserting the questions into the database
 //checking if the required data has been filled
    if(isset($_POST['desc'])){
        if(!isset($_POST['iscorrect']) || $_POST['iscorrect'] == ""){
            echo "Sorry, important data to submit your question is missing. Please press back in your browser and try again and make sure you select a correct answer for the question.";
            exit();
        }

        if(!isset($_POST['type']) || $_POST['type'] == ""){
            echo "Sorry, there was an error parsing the form. Please press back in your browser and try again";
            exit();
        }

     //connecting to the database
        require_once("scripts/connect_db.php");

     //initializing the variables
        $question = $_POST['desc'];
        $program = $_POST['code_desc'];
        $programType = $_POST['prog-lang'];
        $answer1 = $_POST['answer1'];
        $answer2 = $_POST['answer2'];
        $answer3 = $_POST['answer3'];
        $answer4 = $_POST['answer4'];
        $type = $_POST['type'];
        $quizID = $_POST['quizID'];

     //replacing everything except 0-9 with nothing as its values are - 1/2/3...
        $quizID = preg_replace('/[^0-9]/', "", $quizID);

     //replacing everything except a-z with nothing as its values are - mc/tf
        $type = preg_replace('/[^a-z]/', "", $type);

     //replacing everything except 0-9 & a-z with nothhing as value is - answer1/2/3/4
        $isCorrect = preg_replace('/[^0-9a-z]/', "", $_POST['iscorrect']);

     //getting and converting strings as they are
        $question = htmlspecialchars($question);
        $question = mysqli_real_escape_string($con,$question);

        $program = htmlspecialchars($program);
        $program = mysqli_real_escape_string($con,$program);

        $answer1 = htmlspecialchars($answer1);
        $answer1 = mysqli_real_escape_string($con,$answer1);

        $answer2 = htmlspecialchars($answer2);
        $answer2 = mysqli_real_escape_string($con,$answer2);

        $answer3 = htmlspecialchars($answer3);
        $answer3 = mysqli_real_escape_string($con,$answer3);

        $answer4 = htmlspecialchars($answer4);
        $answer4 = mysqli_real_escape_string($con,$answer4);



     //if its a true/false question, do this-
        if($type == 'tf'){
         //if any field is null or empty, say sorry
            if((!$question) || (!$answer1) || (!$answer2) || (!$isCorrect)){
                if($answer1=='0' || $answer2=='0')
                {
                    //do nothing
                }else{
                    echo "Sorry, All fields must be filled in to add a new question to the quiz. Please press back in your browser and try again.";
                    exit();
                }
            }
        }

     //if its a multiple choice question, do this-
        if($type == 'mc'){
         //if any field is null or empty, say sorry
            if((!$question) || (!$answer1) || (!$answer2) || (!$answer3) || (!$answer4) || (!$isCorrect)){
                if($question=='0' || $answer1=='0' || $answer2=='0' || $answer3=='0' || $answer4=='0')
                {
                    //do nothing
                }else{
                    echo "Sorry, All fields must be filled in to add a new question to the quiz. Please press back in your browser and try again.";
                    exit();
                }
            }
        }

     //inserting the question and type into table question
        $sql = mysqli_query($con,"INSERT INTO questions (quiz_id, question, code, code_type, type) VALUES ('$quizID', '$question', '$program', '$programType', '$type')")or die(mysqli_error());
        //lastId is there, so we can insert the id, question_id in our table
            $lastId = mysqli_insert_id();
            mysqli_query($con,"UPDATE questions SET question_id='$lastId' WHERE id='$lastId' LIMIT 1")or die(mysqli_error());

Что он должен делать: (пример) генерируемого идентификатора quiz

INSERT INTO 'quizes' ('id', 'quiz_id', 'quiz_name', 'total_questions', 'display_questions', 'time_allotted', 'set_default') VALUES
(1, 1, 'LEVEL1(EASY)', 22, 20, 30, 0),
(2, 2, 'LEVEL2(HARD)', 9, 10, 20, 1);

И вопросы: (что он должен делать)

INSERT INTO 'questions' ('id', 'quiz_id', 'question_id', 'question', 'code', 'code_type', 'type') VALUES
(1, 1, 1, 'If the output of the question is  hai , find the error in the program?', 'main()\r\n { \r\nprintf(&quot;\\nab&quot;);\r\nprintf(&quot;\\bsi&quot;);\r\nprintf(&quot;\\aha&quot;);\r\n\r\n}\r\n', 'cpp', 'mc'),
(2, 1, 2, 'find the output?', 'void main()\n{\nint i=1,y;\ny=i---i---i;\ncout&lt;&lt;y&lt;&lt;â€,â€&lt;&lt;i;\ngetch();\n}\n', 'cpp', 'mc'),
(3, 1, 3, 'find the output?', '#include&lt;stdio.h&gt;\r\n\r\nint main()\r\n{\r\ncharstr[20], *s;\r\nprintf(&quot;Enter a string\\n&quot;);\r\nscanf(&quot;%s&quot;, str);\r\n    s=str;\r\nwhile(*s != ''\\0'')\r\n    {\r\nif(*s &gt;= 97&amp;&amp; *s &lt;= 122)\r\n            *s = *s-32;\r\n        s++;\r\n    }\r\nprintf(&quot;%s&quot;,str);\r\nreturn0;\r\n}\r\n', 'cpp', 'mc'),
(4, 1, 4, 'find the error', '#include&lt;stdio.h&gt;\r\nint main()\r\n{\r\nint P = 10;\r\nswitch(P)\r\n    {\r\ncase10:\r\nprintf(&quot;Case 1&quot;);\r\n\r\ncase20:\r\nprintf(&quot;Case 2&quot;);\r\nbreak;\r\n\r\ncase P:\r\nprintf(&quot;Case 2&quot;);\r\nbreak;\r\n    }\r\nreturn0;\r\n}\r\n\r\n', 'cpp', 'mc'),
(5, 1, 5, 'find the correct valid function call...assuming the function exists', '', '', 'mc'),

Вместо этого для Quizid и questionID он просто регистрируется в базе данных как 0.

Может ли кто-нибудь указать мне в правильном направлении, чтобы исправить это, или предложить, где еще я могу посмотреть.

Ниже приведен код для вставки ответов (снова в admin.php)

//if inserting a true/false question, insert answers by this-
        if($type == 'tf'){
         //if answer1 is marked correct, do this--
            if($isCorrect == "answer1"){
                $sql2 = mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer1', '1')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer2', '0')")or die(mysqli_error());
                $msg = 'Thanks, question no.'.$lastId.' has been added';
                header('location: admin.php?msg='.$msg.'');
                exit();
            }
         //if answer2 is marked correct, do this--
            if($isCorrect == "answer2"){
                $sql2 = mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer2', '1')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer1', '0')")or die(mysqli_error());
                $msg = 'Thanks, question no.'.$lastId.' has been added';
                header('location: admin.php?msg='.$msg.'');
                exit();
            }   
        }

     //if inserting a multiple choice question, insert answers by this-
        if($type == 'mc'){
         //if answer1 is marked correct, do this--
            if($isCorrect == "answer1"){
                $sql2 = mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer1', '1')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer2', '0')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer3', '0')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer4', '0')")or die(mysqli_error());
                $msg = 'Thanks, question no.'.$lastId.' has been added';
                header('location: admin.php?msg='.$msg.'');
                exit();
            }
         //if answer2 is marked correct, do this--
            if($isCorrect == "answer2"){
                $sql2 = mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer2', '1')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer1', '0')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer3', '0')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer4', '0')")or die(mysqli_error());
                $msg = 'Thanks, question no.'.$lastId.' has been added';
                header('location: admin.php?msg='.$msg.'');
                exit();
            }
         //if answer3 is marked correct, do this--
            if($isCorrect == "answer3"){
                $sql2 = mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer3', '1')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer1', '0')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer2', '0')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer4', '0')")or die(mysqli_error());
                $msg = 'Thanks, question no.'.$lastId.' has been added';
                header('location: admin.php?msg='.$msg.'');
                exit();
            }
         //if answer4 is marked correct, do this--
            if($isCorrect == "answer4"){
                $sql2 = mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer4', '1')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer1', '0')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer2', '0')")or die(mysqli_error());
                mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES ('$quizID', '$lastId', '$answer3', '0')")or die(mysqli_error());
                $msg = 'Thanks, question no.'.$lastId.' has been added';
                header('location: admin.php?msg='.$msg.'');
                exit();
            }

Общая структура таблицы

Изображение прилагается:

ОБНОВЛЕНИЕ: в соответствии с комментарием ниже, также включая код для добавления в таблицу викторины

<?php

    include('scripts/connect_db.php');

        if(isset($_POST['quizName']) && $_POST['quizName'] != ""
        && isset($_POST['quizTime']) && $_POST['quizTime'] != ""
        && isset($_POST['numQues']) && $_POST['numQues'] != ""){

            $qName=mysql_real_escape_string($_POST['quizName']);
            $qTime=mysql_real_escape_string($_POST['quizTime']);
            $nQues=mysql_real_escape_string($_POST['numQues']);

            $qTime = preg_replace('/[^0-9]/', "", $qTime);
            $nQues = preg_replace('/[^0-9]/', "", $nQues);

            $fetch=mysql_query("SELECT id FROM quizes 
                                WHERE quiz_name='$qName'")or die(mysql_error());
            $count=mysql_num_rows($fetch);
            if($count!="")
            {
                $user_msg = 'Sorry, but \ '.$qName.' \ already exists!';
                header('location: admin.php?msg='.$user_msg.'');
            }else{
                mysql_query("INSERT INTO quizes (quiz_name, display_questions, time_allotted) 
                    VALUES ('$qName','$nQues','$qTime')")or die(mysql_error());

                $lastId = mysql_insert_id();
                mysql_query("UPDATE quizes SET quiz_id='$lastId' 
                                WHERE id='$lastId' LIMIT 1")or die(mysql_error());

                $user_msg = 'Quiz, \ '.$qName.' \ has been created!';
                header('location: admin.php?msg='.$user_msg.'');
            }
        }else{
            $user_msg = 'Sorry, but Something went wrong';
            header('location: admin.php?msg='.$user_msg.'');
        }
?>

MissComputing 14 янв. 2018, в 17:31

Источник

0

Я не вижу в вашем коде запроса для вставки в таблицу тестов. Кроме того, множественное число тестов - это тесты с двойным z.
jhilgeman 14 янв. 2018, в 15:42
0

Вы вставляете в таблицу вопросов, а затем обновляете таблицу вопросов. Может быть, эта вставка для викторин?
Will Hines 14 янв. 2018, в 15:50
0

@jhilgeman, см. обновление с кодом для вставки в таблицу теста. И спасибо за урок правописания (я работаю над существующим кодом и, следовательно, пишу ошибки!)
MissComputing 14 янв. 2018, в 15:54
0

@Will Hines - все работает отлично (вставьте тесты и т. Д.), За исключением генерации идентификационных номеров для теста и вопроса, при вставке в базу данных предположительно. Я хотел бы, в качестве ответа, некоторые предложения кода или другие объяснения относительно того, в чем может быть проблема.
MissComputing 14 янв. 2018, в 15:57
0

Правописание было написано просто потому, что чем дольше они существуют, тем труднее исправить ошибки такого рода (поскольку со временем создается больше кода, ссылающегося на имя таблицы). Поэтому я стараюсь указывать на ошибки только для того, чтобы исправить их до того, как их использование станет слишком широким, чтобы их было легко исправить.
jhilgeman 14 янв. 2018, в 16:52
0

Высоко ценится @hjilgeman .... вы можете ответить на главный вопрос, который заключается в том, что ответы и вопросы и т. Д., По-видимому, не создают / не добавляют в базу данных уникальный опросник (для вопросов) или опросник и вопросный (для ответы). Это главная проблема, так как ответы не имеют никакого отношения.
MissComputing 14 янв. 2018, в 18:49

Показать ещё 4 комментария

Теги:

php

mysql

database

1 ответ

Ещё вопросы

Я не вижу в вашем коде запроса для вставки в таблицу тестов. Кроме того, множественное число тестов - это тесты с двойным z.
Вы вставляете в таблицу вопросов, а затем обновляете таблицу вопросов. Может быть, эта вставка для викторин?
@jhilgeman, см. обновление с кодом для вставки в таблицу теста. И спасибо за урок правописания (я работаю над существующим кодом и, следовательно, пишу ошибки!)
@Will Hines - все работает отлично (вставьте тесты и т. Д.), За исключением генерации идентификационных номеров для теста и вопроса, при вставке в базу данных предположительно. Я хотел бы, в качестве ответа, некоторые предложения кода или другие объяснения относительно того, в чем может быть проблема.
Правописание было написано просто потому, что чем дольше они существуют, тем труднее исправить ошибки такого рода (поскольку со временем создается больше кода, ссылающегося на имя таблицы). Поэтому я стараюсь указывать на ошибки только для того, чтобы исправить их до того, как их использование станет слишком широким, чтобы их было легко исправить.
Высоко ценится @hjilgeman .... вы можете ответить на главный вопрос, который заключается в том, что ответы и вопросы и т. Д., По-видимому, не создают / не добавляют в базу данных уникальный опросник (для вопросов) или опросник и вопросный (для ответы). Это главная проблема, так как ответы не имеют никакого отношения.

jhilgeman · Answer 1 · 2018-01-14T14-41-00.000Z

Во-первых, число строк будет числовым, поэтому:

if($count!="")

Должно быть:

if($count > 0)

Затем вы должны использовать расширение mysqli или PDO. Функции mysql_ ненадолго устарели и полностью удалены в новых версиях PHP, поэтому ваш код сломается, если вы не измените его.

Наконец, у вас, похоже, есть "id" как первичный ключ с автоматическим приращением, но вы сразу же копируете это значение в поле quiz_id. Это отбрасывает запрос и смешивает воды в этом поле. Вы, вероятно, должны просто избавиться от quiz_id и использовать if (или переименовать id для quiz_id)

Для повышения производительности и надежности просто добавьте уникальный индекс в поле имени викторины. Затем вы можете просто попробовать вставить в таблицу квестов без проверки имени. Если это не удается из-за дублированного имени, ошибка сообщит вам, что у вас уже есть запись с этим именем. Затем вы можете избавиться от дополнительного SELECT.

EDIT: если вы спрашивали об уникальной части индекса, в основном SQL для добавления уникального индекса в поле quiz_name:

ALTER TABLE 'quizes'
ADD UNIQUE INDEX 'quiz_name' ('quiz_name');

Тогда код будет выглядеть так:

...
$qTime = preg_replace('/[^0-9]/', "", $qTime);
$nQues = preg_replace('/[^0-9]/', "", $nQues);
// End of original code

$result = mysql_query("INSERT INTO quizes (quiz_name,...other fields...) VALUES ('Quiz A',...other values...)");
if(!$result)
{
  // Failed to insert new quiz
  $error_msg = mysql_error();

  // Check the content of the error message
  if(strpos($error_msg,"Duplicate entry") !== false)
  {
    // A quiz with that name already exists, redirect to the admin page with the desired message
  }
  else
  {
    // Some other error happened, redirect to the admin page with the desired message
  }
}
else
{
  // Successful quiz insert
  $newQuizID = mysql_insert_id();
  echo $newQuizID . "\n";
}

ВАЖНО: я использовал функции "mysql_", чтобы оставаться в курсе того, что у вас есть прямо сейчас, но я не могу подчеркнуть, насколько важно преобразовать его в mysqli или PDO. Простейшим преобразованием является, вероятно, mysqli, так как синтаксис будет таким схожим.

EDIT # 2: еще одно последнее уточнение для вашего кода, в которое вставляются ответы с несколькими вариантами ответов - вы можете увеличить производительность INSERT и упростить свой код следующим образом:

if($type == 'mc'){
  $sql2 = mysqli_query($con,"INSERT INTO answers (quiz_id, question_id, answer, correct) VALUES 
      ('$quizID', '$lastId', '$answer1', '". ($isCorrect == "answer1" ? "1" : "0") ."'),
      ('$quizID', '$lastId', '$answer2', '". ($isCorrect == "answer2" ? "1" : "0") ."'),
      ('$quizID', '$lastId', '$answer3', '". ($isCorrect == "answer3" ? "1" : "0") ."'),
      ('$quizID', '$lastId', '$answer4', '". ($isCorrect == "answer4" ? "1" : "0") ."')") or die(mysqli_error());
  $msg = 'Thanks, question no.'.$lastId.' has been added';
  header('location: admin.php?msg='.$msg.'');
  exit();
}

Этот код будет использовать функцию MySQL для вставки нескольких строк одновременно, так что вы выполняете только один запрос вместо 4, и он использует некоторый базовый PHP для выбора "1" или "0" для "правильного" столбца соответствующим образом.

Можете ли вы вставить какой-нибудь код для «Наконец-то» части вашего предложения. Что вы подразумеваете под переименованием идентификатора в quiz_id - в каком бите? И как бы все это выглядело в этом случае?
Моя главная проблема, и ответ, который я ищу, связан с вопросами и ответами
Таким образом, у вас есть два поля, которые содержат числовой идентификатор теста, «id» и «quiz_id». Поле «id» является вашим первичным ключом и является частью, которая получает автоматический новый / следующий идентификационный номер, а ваш код копирует его в столбец «quiz_id». Предполагая, что quiz_id всегда является просто копией id, я предлагал удалить поле quiz_id из вашей таблицы, а затем просто переименовать столбец id в вашей таблице в quiz_id (чтобы quiz_id автоматически получал новые идентификаторы). Таким образом, весь ваш код остается неизменным, и вы можете просто удалить строку кода, которая выполняет запрос «UPDATE quizes SET quiz_id».