![]() |
Модераторы: skyboy, MoLeX, Aliance, ksnk |
![]() ![]() ![]() |
|
beginner1 |
|
||||||
Новичок Профиль Группа: Участник Сообщений: 2 Регистрация: 10.5.2014 Репутация: нет Всего: нет |
Здравствуйте. Помогите пожалуйста. Я вывожу вопрос и ответы из БД. В данный момент я пытаюсь сделать подсчет правильных ответов, но к сожалению не получается помогите пожалуйста что не так? Пожалуйста помогите очень нужна помощь(((.
Это сообщение отредактировал(а) beginner1 - 12.5.2014, 09:45 |
||||||
|
|||||||
igorold |
|
|||
![]() Опытный ![]() ![]() Профиль Группа: Участник Сообщений: 557 Регистрация: 22.12.2005 Где: Россия->Урал-& gt;Миасс Репутация: нет Всего: 17 |
что не получается? какой результат работы скрипта?
Этот ответ добавлен с нового Винграда - http://vingrad.com |
|||
|
||||
beginner1 |
|
|||
Новичок Профиль Группа: Участник Сообщений: 2 Регистрация: 10.5.2014 Репутация: нет Всего: нет |
не получается подсчет правильных и не правильных ответов результат скрипта, выводит вопрос и ответы из БД, а правильные ответы не считает и не заносит в БД по нажатию кнопки "следующий вопрос".
|
|||
|
||||
beginner1 |
|
|||
Новичок Профиль Группа: Участник Сообщений: 2 Регистрация: 10.5.2014 Репутация: нет Всего: нет |
всегда ответ не верный и считает не по нажатию кнопки
Этот ответ добавлен с нового Винграда - http://vingrad.com |
|||
|
||||
beginner1 |
|
|||
Новичок Профиль Группа: Участник Сообщений: 2 Регистрация: 10.5.2014 Репутация: нет Всего: нет |
||||
|
||||
Aliance |
|
|||
![]() I ♥ <script> ![]() ![]() ![]() ![]() Профиль Группа: Модератор Сообщений: 6418 Регистрация: 2.8.2004 Где: spb Репутация: 3 Всего: 137 |
Вы слышали, что такое switch/case? Про конструкцию else while я промолчу ![]() Что касается вопроса, тут логика скрипта, как мне кажется, страдает. У вас в 88 строчке используется переменная data, а откуда она возьмется, если до этого выполниться условие if? |
|||
|
||||
beginner1 |
|
|||
Новичок Профиль Группа: Участник Сообщений: 2 Регистрация: 10.5.2014 Репутация: нет Всего: нет |
да я знаю что логика скрипта хромает((( Aliance помогите пожалуйста исправить, чтобы правильно работало правильно ли ответил пользователь на вопрос или нет и подсчитывалось количество правильных ответов.
Этот ответ добавлен с нового Винграда - http://vingrad.com |
|||
|
||||
Aliance |
|
|||
![]() I ♥ <script> ![]() ![]() ![]() ![]() Профиль Группа: Модератор Сообщений: 6418 Регистрация: 2.8.2004 Где: spb Репутация: 3 Всего: 137 |
А итак помогаю, пока не вижу вариантов исправленного вами кода с учетом замечаний, озвученных в данной теме. Писать-то за вас это никто не будет ;-)
|
|||
|
||||
beginner1 |
|
|||
Новичок Профиль Группа: Участник Сообщений: 2 Регистрация: 10.5.2014 Репутация: нет Всего: нет |
Aliance, заменила на switch/case.
Этот ответ добавлен с нового Винграда - http://vingrad.com |
|||
|
||||
beginner1 |
|
||||||
Новичок Профиль Группа: Участник Сообщений: 2 Регистрация: 10.5.2014 Репутация: нет Всего: нет |
Aliance, заменила на switch/case.
Этот ответ добавлен с нового Винграда - http://vingrad.com |
||||||
|
|||||||
beginner1 |
|
|||
Новичок Профиль Группа: Участник Сообщений: 2 Регистрация: 10.5.2014 Репутация: нет Всего: нет |
"У вас в 88 строчке используется переменная data, а откуда она возьмется, если до этого выполниться условие if?"
вот так надо было исправить?
Этот ответ добавлен с нового Винграда - http://vingrad.com |
|||
|
||||
Aliance |
|
|||
![]() I ♥ <script> ![]() ![]() ![]() ![]() Профиль Группа: Модератор Сообщений: 6418 Регистрация: 2.8.2004 Где: spb Репутация: 3 Всего: 137 |
Нет, не так. У вас используется переменная $data, а задается они в блоке else условия. По логике не знаю, но по идеи этот блок может не выполнится - значит переменной не будет.
Далее, у вас логика представления и бизнес-логика смешана в одном месте, это не есть гуд. Почитайте, что такое MVC. Это, скорее на заметку, т.к. сейчас будет, полагаю, весьма сложным. Основная проблема, что вижу я, в том, что вывод вопросов, запись ответов и подсчет результатов располагается в одном месте и выполняется без особых условий. Нужно построить логику примерно так:
|
|||
|
||||
beginner1 |
|
|||
Новичок Профиль Группа: Участник Сообщений: 2 Регистрация: 10.5.2014 Репутация: нет Всего: нет |
Вот так вот? только вопрос и ответы правильно перестали выводиться
Этот ответ добавлен с нового Винграда - http://vingrad.com |
|||
|
||||
Aliance |
|
|||
![]() I ♥ <script> ![]() ![]() ![]() ![]() Профиль Группа: Модератор Сообщений: 6418 Регистрация: 2.8.2004 Где: spb Репутация: 3 Всего: 137 |
Нет, не так. Такое ощущение, что вы просто наугад заменяете строки. Тут на форуме могут помочь, если у вас есть способности самому что-то исправить. В исключительных случаях, если задача простая, могут написать за вас, потому что там проще. Но не стоит ожидать, что за вас полностью переделают более-менее большой скрипт, в котором нужно разбираться. Обычно такое делается в разделе работа, но уже не за бесплатно ;-)
Я выше дал вам примерный алгоритм, у вашем же "исправлении" до сих пор осталась строчка "if(isset($_POST['submit'])){". И зачем вы выкладываете закомментированные строчки, если они не несут смысловой нагрузки? Так же пароли подключения к БД, это не нужно и занимает только лишнее место. Нужно не тупо копипастить код всего файла, а выкладывать только значимые куски, тогда возможно и желающих вам помочь будет больше, т.к. их не будет пугать большой размер исходников. Что касается кодстайла, я рекомендую вам выносить сами запросы в отдельные переменные, чтобы визуально улучшить читаемость (но это чисто мое субъективное мнение), так же у вас там есть уязвимости (sql injection). |
|||
|
||||
beginner1 |
|
|||
Новичок Профиль Группа: Участник Сообщений: 2 Регистрация: 10.5.2014 Репутация: нет Всего: нет |
Просто я недавно начала изучать php. и у меня буквально 2 недели осталось, чтобы написать программу и отчет по ней. ну точнее диплом написать.
Этот ответ добавлен с нового Винграда - http://vingrad.com |
|||
|
||||
![]() ![]() ![]() |
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей) | |
0 Пользователей: | |
« Предыдущая тема | PHP: Базы Данных | Следующая тема » |
|
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности Powered by Invision Power Board(R) 1.3 © 2003 IPS, Inc. |