Странное поведение скрипта

@Firework · Регистрация: 01.04.2013

Студворк — интернет-сервис помощи студентам

Есть система оценок отделов компании. Оценки ставятся ежемесячно. Это обычная html-форма + php-обработчик.
В этом обработчике есть проверка на существование оценок это этого пользователя за текущий месяц.

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
$query_check = " SELECT * FROM `grades` WHERE user_id = '$user_id' AND year = '$year' AND month = '$month' ";
$res_check = mysql_query($query_check);
$check_arr_count = 0;
while($row_check = mysql_fetch_array($res_check))
{
    ++$check_arr_count;
}
if ( $check_arr_count > 0 )
{
    echo '<p>Ошибка 1004</p>';
    echo '<p>Вы уже оценивали!</p>';
    exit();
}

Т.е. просто идёт выборка записей пользователя за текущий месяц. Если их количество больше 0, то вывод сообщения об ошибке и прекращение работы скрипта. Эта система работает несколько месяцев. И ещё не было такого, чтобы кто-то оценил 2 раза. Но сегодня случилось что-то странное. Сотрудница как-то поставила аж 3 оценки.

Она сказала, что при отправке данных с формы, страница зависла (иногда сервер тормозит, возможно дело в этом). И она нажала кнопку "отправить" несколько раз.

Вот я думаю, мог ли скрипт получить 3 запроса подряд из-за какого-то глюка сервера. С другой стороны даже если и так, то после первого выполнения скрипта в таблице БД уже будут записи и остальные 2 запроса закончатся неудачей. Но факт остаётся фактом - проверка дала сбой. Может есть у кого идеи?

@Vas-e-na · 08.09.2014, 13:04

Сделайте из трех полей (user_id year month) уникальный ключ и тогда на уровне базы внести дублирующую запись не получится

ЗЫ А почему бы не использовать COUNT(*) вместо подсчета на пхп???

@Firework · 08.09.2014, 13:14 **[ТС]**

Сообщение от Vas-e-na

Сделайте из трех полей (user_id year month) уникальный ключ и тогда на уровне базы внести дублирующую запись не получится

Имеете ввиду объединить 3 поля в одно? Типа ui_month_year
- 77_август_2014.

Сообщение от Vas-e-na

ЗЫ А почему бы не использовать COUNT(*) вместо подсчета на пхп???

Просто когда писал скрипт, выводил отладочную информацию с каждой итерацией цикла. Поэтому пересчёт записей остался на пхп.

Но всё-таки не ясно в чём дело. Т.к. тогда придётся менять структуру базы данных и переписывать скрипты.

@Vas-e-na · 08.09.2014, 13:59

Сообщение от Firework

Имеете ввиду объединить 3 поля в одно? Типа ui_month_year

Нет, создать ключ из трех полей аля

SQL
1
CREATE UNIQUE INDEX uid ON `grades`(`user_id`, `month`, `year`);

Но проще это через phpMyAdmin: выделяете три нужных поля и создаете уникальный индекс.

Добавлено через 1 минуту
Соответсвенно с таким ключом при INSERT новой записи с дублиующимся ключом вернётся false

@Firework · 08.09.2014, 14:05 **[ТС]**

Спасибо. Не знал о такой возможности. Но всё же меня интересует причина, по которой в скрипте не сработала проверка наличия оценки в таблице БД.

@Vas-e-na · 08.09.2014, 14:10

Скорее всего из-за того что есть промежуток времени между SELECT и INSERT при слишком быстром запросе страницы/скрипта во все SELECT'ы попали данные без INSERT, вследствии чего и произошел сбой...
Ну как-то так...

@Firework · 08.09.2014, 14:17 **[ТС]**

Т.е. не дожидаясь первого выполнения скрипта, началось второе, а потом третье. И всё это по запросу с одного и того же клиента? разве такое может быть?

@Vas-e-na · 08.09.2014, 16:11

Может быть все что угодно, зависит от кривизны рук (не Ваших конечно, а того кто жал на кнопку...)

@Firework · 08.09.2014, 17:48 **[ТС]**

Вот я и хочу узнать возможные причины на будущее.

@ads · 08.09.2014, 18:06

Сообщение от Firework

Вот я и хочу узнать возможные причины на будущее.

контроль на уровне базы, как сказал Vas-e-na, и не придется ездить на велосипеде по граблям. Следующий раз когда ваши система будет перегружена, особо нетерпеливый и десять оценок поставит..

Сообщение от Firework

тогда придётся менять структуру базы данных и переписывать скрипты.

добавление уникально ключа не потребует никакого переписывания скриптов

Новые блоги и статьи Все статьи Все блоги /
Тестирование Pull Request в Kubernetes с vCluster Mr. Docker 19.07.2025 Часто сталкиваюсь с серьезной дилемой при настройке тестовых окружений для проверки Pull Request в Kubernetes. С одной стороны, каждый PR требует изолированной среды — только так можно гарантировать,. . .	Мой 7 минутный ролик с крамольным предложением про шахматы, предлагаю заценить _Ivana 18.07.2025 p2UhJNMGY94	Десять Middleware Node.js для эффективного кодинга Reangularity 18.07.2025 Когда я только начинал работать с Node. js, количество пакетов в npm меня буквально парализовало. Сегодня их больше 1,3 миллиона — попробуй разберись, что стоит твоего внимания, а что нет. Я потратил. . .	Context и глубины Android mobDevWorks 18.07.2025 В Android разработки Context напоминает воздух - он везде, жизненно необходим, но мало кто может детально объяснить его природу. Мы привыкли получать его как параметр, передавать дальше и. . .	Результаты исследования от команды MCM (июль 2025 г.) Programma_Boinc 18.07.2025 Результаты исследования от команды MCM (июль 2025 г. ) Как сообщалось в наших предыдущих публикациях, мы изучаем гены, которые имеют наибольший рейтинг и ассоциируются с различными видами рака, в. . .
ИИ-чатбот на React с OpenAI и LangChain.js Reangularity 17.07.2025 React давно стал для меня золотым стандартом фронтенд-разработки. Его компонентная структура, виртуальный DOM и однонаправленный поток данных идеально подходят для создания динамичных интерфейсов. . .	Пишем адаптер для локального хранилища S3 на C# stackOverflow 16.07.2025 Разработка современных приложений часто требует интеграции с объектными хранилищами, и Amazon S3 стал де-факто стандартом в этой области. Однако работа с облачными сервисами в процессе разработки. . .	Старые замки kumehtar 16.07.2025 Смотрел тут фото, попались пара старых замков. И сразу бросилось в глаза из отличие. Например: Замок Бистон, в англии. Разрушенное сооружение. Но - не испорченное людьми, по крайней мере - на. . .	Java и Eclipse Store: Сверхбыстрые приложения с In-Memory DB Javaican 15.07.2025 Eclipse Store — это микро-движок персистентности для Java, который позволяет хранить и извлекать нативные Java-объекты без необходимости преобразования данных или использования объектно-реляционного. . .	EmBitz, создание проекта, отладка, прошивка locm 15.07.2025 Создание проекта для Blue Pill (STM32F103C8T6) в EmBitz 2. 30, написания кода blink, запуск отладки в ОЗУ, заливка релизной прошивки во flash используя ST-Link и др. . . .

@Vas-e-na 420 / 417 / 79 Регистрация: 21.06.2010 Сообщений: 1,362
	08.09.2014, 13:04
	Сделайте из трех полей (user_id year month) уникальный ключ и тогда на уровне базы внести дублирующую запись не получится ЗЫ А почему бы не использовать COUNT(*) вместо подсчета на пхп??? 1

@Firework 102 / 93 / 36 Регистрация: 01.04.2013 Сообщений: 654
	08.09.2014, 14:05 [ТС]
	Спасибо. Не знал о такой возможности. Но всё же меня интересует причина, по которой в скрипте не сработала проверка наличия оценки в таблице БД. 0

@Vas-e-na 420 / 417 / 79 Регистрация: 21.06.2010 Сообщений: 1,362
	08.09.2014, 14:10
	Скорее всего из-за того что есть промежуток времени между SELECT и INSERT при слишком быстром запросе страницы/скрипта во все SELECT'ы попали данные без INSERT, вследствии чего и произошел сбой... Ну как-то так... 0

@Firework 102 / 93 / 36 Регистрация: 01.04.2013 Сообщений: 654
	08.09.2014, 14:17 [ТС]
	Т.е. не дожидаясь первого выполнения скрипта, началось второе, а потом третье. И всё это по запросу с одного и того же клиента? разве такое может быть? 0

@Vas-e-na 420 / 417 / 79 Регистрация: 21.06.2010 Сообщений: 1,362
	08.09.2014, 16:11
	Может быть все что угодно, зависит от кривизны рук (не Ваших конечно, а того кто жал на кнопку...) 0

@Firework 102 / 93 / 36 Регистрация: 01.04.2013 Сообщений: 654
	08.09.2014, 17:48 [ТС]
	Вот я и хочу узнать возможные причины на будущее. 0

Опции темы