|
|
|
|
|
для: Trianon
(08.06.2007 в 01:17)
| | Я отказываюсь от авторства!
Да, действительно, не заметил каунта.. Если б сам с нуля писал - не ошибся бы.. | |
|
|
|
|
|
|
|
для: mr Bin
(08.06.2007 в 01:10)
| | >>mysql_num_rows всегда вернет 1 независимо от количества строк в таблице
>Странно, меня никогда это ещё не подводило. Если такого нет в таблице, то значит нет.
Вы не заметили, что запрос - SELECT COUNT(id) - Такой запрос всегда вернет одно значение независимо от числа строк в таблице.
Я заметил, что большинство огрехов - автора.
Но Вы же назвали код своим... Получите, что называется, и распишитесь :) | |
|
|
|
|
|
|
|
для: Trianon
(08.06.2007 в 00:57)
| | >mysql_num_rows всегда вернет 1 независимо от количества строк в таблице
Странно, меня никогда это ещё не подводило. Если такого нет в таблице, то значит нет.
>непонятен источник возникновения переменной $id
Ну я тоже не знаю, а втору виднее.. я только подстроился под него..
>отсутствует восстановление данных после искажения механизмом магических кавычек,
>отсутствует экранирование при образовании литеральных констант MySQL-кода.
Ну я не вдавался в такие подробности, это лишь краткий пример..
>целочисленное (вроде как) поле id сравнивается со строковым литералом,
>отсутствует явное перечисление имен полей в операторе INSERT.
Опять же.. какой запрос автор опубликовал, такой я и вставил для примера, чтобы ему было понятнее, что откуда берется..
p.s. Называется напросился :( | |
|
|
|
|
|
|
|
для: mr Bin
(07.06.2007 в 23:09)
| | у меня? Претензий - нет.
Замечания... как обычно:
Логические ошибки: функция mysql_num_rows всегда вернет 1 независимо от количества строк в таблице,
непонятен источник возникновения переменной $id,
отсутствует восстановление данных после искажения механизмом магических кавычек,
отсутствует экранирование при образовании литеральных констант MySQL-кода.
стилистические ошибки:
целочисленное (вроде как) поле id сравнивается со строковым литералом,
отсутствует явное перечисление имен полей в операторе INSERT. | |
|
|
|
|
|
|
|
для: Trianon
(07.06.2007 в 22:21)
| | Я видел, что Вы ответили ему. Но часто так бывает, что независимо от ника отвечают...
И я подумал, что Вы каким-то образом адресовали первую строчку мне... Понял.
И всё же, я правильный код составил? :) Или и ко мне есть претензии? | |
|
|
|
|
|
|
|
для: mr Bin
(07.06.2007 в 22:06)
| | Включите линейный режим форума. В нем написано, кто кому отвечает. | |
|
|
|
|
|
|
|
для: Trianon
(07.06.2007 в 21:04)
| | Эм, я не правильно написал? | |
|
|
|
|
|
|
|
для: Specter
(07.06.2007 в 20:34)
| | Это не одно и то же.
Неправильно.
В $num будет записано число строк в таблице, поле id в которых равно искомому. | |
|
|
|
|
|
|
|
для: Specter
(07.06.2007 в 18:41)
| | Я бы переписал так:
<?php
$SQL = mysql_query("SELECT COUNT(id) FROM votes WHERE id='".$id."'");
if(mysql_num_rows($SQL) == 0) { mysql_query("INSERT INTO `votes` VALUES(0, '".$id."', '".$_POST['q']."')"); } else { mysql_query("UPDATE `votes` SET `text` = '".$_POST['q']."' WHERE `id` = '".$id."'"); }
?>
|
| |
|
|
|
|
|
|
|
для: Trianon
(07.06.2007 в 20:07)
| | Я просто код с скрипта скопировал с скобкой. А так ведь я правильно понял, что если id существует, то $num будет равен true? | |
|
|
|
|