Вебдевелоперам. Очень, ОЧЕНЬ красивая ошибка

[dil]

Прям хоть на собеседованиях разработчикам предлагай.
Вот этот кусок кода на PHP (на самом деле, язык не принципиален) хронически вызывает двойной вызов функции и, соответственно, двойную вставку данных в базу.

<table><tr><td background=#fff7eb id=message>
<?php mysql_query("INSERT INTO `res_action` (`type`, `date`, `id`, `from`, `to`, `text`, `viewed`, `hidden`)
VALUES ('hate', '08/08/07 12:15', 1070808121557, 'foo', 'bar', '', '0', '0')"); ?>
</td></tr></table>

Но стоит убрать из td цвет, как все волшебным образом начинает работать правильно.

Утащено отсюда. Но прежде чем смотреть там правильные ответы, попробуйте понять сами.

Comments

[motto.livejournal.com]

поясни тупому в трех словах, почему второй раз парсится

[dil.livejournal.com]

потому что там вместо background должен быть bgcolor.
Дальше понял? :)

[ex-acerbial236.livejournal.com]

Перепутал человек атрибуты background и bgсolor.
В результате td background=#fff7eb вызывает страницу еще раз с якорем #fff7eb

[dil.livejournal.com]

правильно :) сколько времени ушло на ответ?

[ex-acerbial236.livejournal.com]

Я подсмотрел. =)

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

[ivlad.livejournal.com]

классно! понравилось. но сам не догадался.

[pzrk.livejournal.com]

Нихера не понял. но при наличии в "боекомплекте" двух десятков remote root против freebsd - это совершенно пофигу.

[pzrk.livejournal.com]

И, кстати, закатай код - а то pre ленту рвет.

[dil.livejournal.com]

эм. а у тебя экран какой ширины?

[pzrk.livejournal.com]

Экран - 1600, окно броузера - 1024.

[ex-1smash1100.livejournal.com]

ооооо, надо атрибуты брать в кавычки, в двойные.

[bormotov.livejournal.com]

это, как-бы понятно, непонятно почему два раза вставляет :)

[dmarck.livejournal.com]

bgcolor != background ;-)

[bormotov.livejournal.com]

да я уж почитал ответ :)

[dil.livejournal.com]

это да, но на эффект не влияет :)

[bormotov.livejournal.com]

кстати, интересно, есть валидатор линков который поймает это?

[ex-acerbial236.livejournal.com]

В ёлочки!

[ex-1smash1100.livejournal.com]

Отличная ошибка.
Браузер-то ждет что ему отдадут урл в атрибуте бекграунд.

[webushka.livejournal.com]

Пыхпых тут, действительно, не при чём. Это от незнания хатэмээля. Путают bgcolor с background и потом удивляются - чего оно два раза страницу дёргает :)

[dil.livejournal.com]

Правильно :)

[dkfl.livejournal.com]

art.lebedev завел моду кавычки опускать.

[dil.livejournal.com]

кавычки тут ни при чем, с ними ровно то же самое

[dkfl.livejournal.com]

ааа.. bgcolor жесть. никогда бы не додумался.

[maxdm.livejournal.com]

+5 to dil

[b-a-t.livejournal.com]

Про background заметил, что # напоминает anchor - вспомнил, сложить вместе - не догадался :(

Старею :(

[danfe.livejournal.com]

Та же фигня.

[dz.livejournal.com]

Блядь. Саш. Ошибка заключается в том, что стейт кода не должен передаваться через веб-страницу. А не в том, что кто-то ошибся в атрибуте. Но поскольку "не принципиальный язык" PHP не умеет хранить стейт между запросами (кроме как в БД), все пэхапэ-колбасеры пишут вот так, как тут у тебя. И получается гавнокод, поведение которого зависит от числа вызовов страниц.

То есть: если этот код протоколирует показы страниц, то двойная запись - верное поведение.
Если нет - то двойная запись - неверное поведение ВНЕ зависимости от того, вызвал её неправильный атрибут или релоад в броузере.

[dil.livejournal.com]

Дим, язык _не принципиален_. Не то, чтобы я очень любил PHP, но в данном случае ты на него наехал не по делу. Это могло быть написано на перле, ASP, JSP, чистой джаве, ColdFusion и на любом другом серверном языке. С тем же результатом.

PHP уже давно умеет хранить state в абстракции под названием "сессия". http://www.php.net/manual/en/ref.session.php

Я не знаю точно, чтО имел в виду автор кода, но по-моему, именно протоколирование показов страницы, а не запоминание состояния. Во втором случае был бы какой-нибудь UPDATE или комбинация его с INSERT'ом.
Поведение получилось формально правильное, но не то, которое имелось в виду. Потому что запросов было два, они и посчитались, а показов только один, причем про второй достоверно известно, что его никто никогда увидеть не может :)

[dz.livejournal.com]

Ну - может. Без знания более общей картины, конечно, трудно разобраться.

[kukutz.livejournal.com]

Кажется, сессии были даже в PHP3.

Так давно не перечитывал мануал?

[dz.livejournal.com]

Давно. Да и зачем. По коду видно, что ход мысли колбасеров от этого не изменился.

[kukutz.livejournal.com]

Штырит, штырит колбасёра
Наша пати до упора

[dil.livejournal.com]

Дим, а если б этот mysql_query был закопан глубоко внутри класса на сервере, а background вместо bgcolor - в совершенно отдельном xslt, тебе бы было легче? :)

[dz.livejournal.com]

Мне - нет, а некоторым страницам Яндекса, состояние которых последние шесть лет теряется при проходе через авторизацию - да.

[computer-bug.livejournal.com]

Блин. Надо же. Гениальная ошибка!