Что не так с синтаксисом MySQL?

Question

Что не так с синтаксисом MySQL?

0

Я использую следующий код для подсчета и суммирования значений из базы данных.

$query = "SELECT
          COUNT(n.*) AS cnt_news,
          COUNT(a.*) AS cnt_adv,
          COUNT(c.*) AS cnt_comm,
          SUM(CASE WHEN c.approve = '1' AND c.spam = '0' THEN 1 ELSE 0 END) AS cnt_approved,
          SUM(CASE WHEN c.approve = '0' AND c.spam = '0' THEN 1 ELSE 0 END) AS cnt_unapproved,
          SUM(CASE WHEN c.spam = '0' THEN 1 ELSE 0 END) AS cnt_spam,
          SUM(a.amount) AS t_amnt,
          SUM(a.cashpaid) AS t_cpaid,
          SUM(a.balance) AS t_bal
          FROM
          news n, advertisements a, comments c";
          $result = mysql_query($query) or die(mysql_error());
          $row = mysql_fetch_array($result);

следующий код дает мне ошибку, ошибка

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '*) AS cnt_news, COUNT(a.*) AS cnt_adv, COUNT(c.*) AS cnt_c' at line 2

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

что не так с моим кодом.??

следующий код работает отлично для меня.

$query = "SELECT COUNT(*) as cnt_news FROM news";
$result = mysql_query($query);
$row = mysql_fetch_array($result);


$query = "SELECT COUNT(*) as cnt_adv FROM advertisements";
$result = mysql_query($query);
$row = mysql_fetch_array($result);

$query = "SELECT COUNT(*) as cnt_comm FROM comments";
$result = mysql_query($query);
$row = mysql_fetch_array($result);


$query = "SELECT SUM(CASE WHEN c.approve = '1' AND c.spam = '0' THEN 1 ELSE 0 END) AS cnt_approved,
          SUM(CASE WHEN c.approve = '0' AND c.spam = '0' THEN 1 ELSE 0 END) AS cnt_unapproved,
          SUM(CASE WHEN c.spam = '1' THEN 1 ELSE 0 END) AS cnt_spam
          FROM COMMENTS c";
$result = mysql_query($query);
$row = mysql_fetch_array($result);


$query = "SELECT SUM(a.amount) as t_amnt,
          SUM(a.cashpaid) as t_cpaid,
          SUM(a.balance) as t_bal
          FROM advertisements a";
$result = mysql_query($query);
$row = mysql_fetch_array($result);

Где я ошибаюсь?

Ibrahim Azhar Armar 13 сен. 2010, в 11:37

Источник

Теги:

php

mysql

4 ответа

0

Вы можете попробовать

SELECT (SELECT COUNT() FROM news) AS cnt_news, (SELECT COUNT() FROM рекламы) как cnt_adv, ...

viriathus 13 сен. 2010, в 07:31

0

Похоже, Mysql не любит эту строку. Измените COUNT(n.*) на COUNT(n.id) или на любое другое имя поля первичного ключа этой таблицы. Сделайте то же самое для a и c.

Fanis 13 сен. 2010, в 07:02

0

я пробовал ваш путь ошибка исчезает, но почему, черт возьми, он печатает неправильные значения? у меня есть два числа строк в таблице базы данных, и вместо этого он показывает 30. :(
Ibrahim Azhar Armar 13 сен. 2010, в 08:53
1

Вы выполняете полное объединение трех таблиц: новости n, реклама a, комментарии c ... попробуйте заменить COUNT (n.id) (и эквивалент для каждой из ваших таблиц) на COUNT (DISTINCT n.id)
Mark Baker 13 сен. 2010, в 08:56
0

@ Марк Бейкер хороший улов, я даже не заметил остальную часть запроса. 3 таблицы объединены в каком-либо столбце? Я предполагаю, что у новостей есть комментарии и реклама.
Fanis 13 сен. 2010, в 09:04
0

@ Марк Бейкер, ваш код работает для первых трех запросов, я использовал код. COUNT (DISTINCT n.id) AS cnt_news, и это прекрасно работает, однако, если я добавляю отличные от четвертого запроса, он все равно дает неправильное значение. ?
Ibrahim Azhar Armar 13 сен. 2010, в 09:09
0

В исходном наборе запросов, которые объединяются, также нет никаких объединений, но лучшим ответом было бы сделать соответствующее объединение (требуется больше ввода от OP, чтобы помочь идентифицировать объединения), а не использовать DISTINCT.
Mark Baker 13 сен. 2010, в 09:09
0

Если я использую Distinct в седьмом запросе SUM (DISTINCT a.amount) AS t_amnt, он просто берет значения из первых двух строк и игнорирует остальные.
Ibrahim Azhar Armar 13 сен. 2010, в 09:12
0

@ Марк, я уже написал рабочий код с количеством запросов, как мне сделать правильное соединение для таблиц?
Ibrahim Azhar Armar 13 сен. 2010, в 09:15
0

@Ibrahim Чтобы проконсультировать вас по поводу объединений, нам нужно знать отношения между таблицами (например, первичные и внешние ключи и т. Д.)
Mark Baker 13 сен. 2010, в 09:33
0

@ Марк Я обновил пост, пожалуйста, посмотрите. ти
Ibrahim Azhar Armar 13 сен. 2010, в 09:38

Показать ещё 7 комментариев

0

Вы не можете использовать count(tablename.*), попробуйте использовать count(tablename.columnname)

JohnoBoy 13 сен. 2010, в 06:14

0

я попытался изменить на COUNT (n.id) как cnt_news, но работает, но выводит неправильные значения, ... есть идеи, почему?
Ibrahim Azhar Armar 13 сен. 2010, в 08:54
0

Если указанный вами столбец имеет значения NULL, команда count будет игнорировать их, попробуйте использовать счетчик в столбце id таблицы или в ненулевом столбце
JohnoBoy 13 сен. 2010, в 10:05

Ещё вопросы

я пробовал ваш путь ошибка исчезает, но почему, черт возьми, он печатает неправильные значения? у меня есть два числа строк в таблице базы данных, и вместо этого он показывает 30. :(
Вы выполняете полное объединение трех таблиц: новости n, реклама a, комментарии c ... попробуйте заменить COUNT (n.id) (и эквивалент для каждой из ваших таблиц) на COUNT (DISTINCT n.id)
@ Марк Бейкер хороший улов, я даже не заметил остальную часть запроса. 3 таблицы объединены в каком-либо столбце? Я предполагаю, что у новостей есть комментарии и реклама.
@ Марк Бейкер, ваш код работает для первых трех запросов, я использовал код. COUNT (DISTINCT n.id) AS cnt_news, и это прекрасно работает, однако, если я добавляю отличные от четвертого запроса, он все равно дает неправильное значение. ?
В исходном наборе запросов, которые объединяются, также нет никаких объединений, но лучшим ответом было бы сделать соответствующее объединение (требуется больше ввода от OP, чтобы помочь идентифицировать объединения), а не использовать DISTINCT.
Если я использую Distinct в седьмом запросе SUM (DISTINCT a.amount) AS t_amnt, он просто берет значения из первых двух строк и игнорирует остальные.
@ Марк, я уже написал рабочий код с количеством запросов, как мне сделать правильное соединение для таблиц?
@Ibrahim Чтобы проконсультировать вас по поводу объединений, нам нужно знать отношения между таблицами (например, первичные и внешние ключи и т. Д.)
@ Марк Я обновил пост, пожалуйста, посмотрите. ти
я попытался изменить на COUNT (n.id) как cnt_news, но работает, но выводит неправильные значения, ... есть идеи, почему?
Если указанный вами столбец имеет значения NULL, команда count будет игнорировать их, попробуйте использовать счетчик в столбце id таблицы или в ненулевом столбце

Ibrahim Azhar Armar · Accepted Answer · 2010-09-13T10-49-00.000Z

Ну, я отказался от идеи сделать мои запросы в одну, и, как предложил Col.Shrapnel, я сделал для нее настраиваемую функцию, и мне было очень легко поддерживать код таким образом. спасибо, полковник Шарпнель, я отправляю предложенный им ответ.

это созданная пользователем функция i.

function dbgetvar($query) {
             $res = mysql_query($query);
         if( !$res) {
             trigger_error("dbget: ". mysql_error(). " in " .$query);
             return false;
             }
             $row = mysql_fetch_array($res);
             if(!$row) return "";
             return $row;
             }

а затем я вызвал свою функцию с помощью этого кода.

     $news = dbgetvar("SELECT COUNT(*) as count FROM news");
 $comments = dbgetvar("SELECT SUM(CASE WHEN c.approve = '1' AND c.spam = '0' THEN 1 ELSE 0 END) AS approved,
                       SUM(CASE WHEN c.approve = '0' AND c.spam = '0' THEN 1 ELSE 0 END) AS pending,
                       SUM(CASE WHEN c.spam = '1' THEN 1 ELSE 0 END) AS spam,
                       COUNT(*) AS count
                       FROM COMMENTS c");
$advertise = dbgetvar("SELECT SUM(a.amount) AS amount,
                       SUM(a.cashpaid) AS cashpaid,
                       SUM(a.balance) AS balance,
                       COUNT(*) AS count
                       FROM advertisements a");

приведенный выше код работает отлично для меня.