0

Не могу понять в чем заключается проблема, есть вот такой sql запрос, который должен выдавать статьи только той категории, которая передана в запросе GET: `

$cat = "SELECT d.id,d.cat,d.text,d.title,d.description,d.date,c.text,c.cat,c.titlecat FROM data d LEFT JOIN categories c ON d.cat = c.id WHERE d.cat = '$one'";

Вот собственно проверка входящего запроса:

if(isset($_GET['cat'])) {   $_GET['cat'] = $cats[$_GET['cat']];     $one = $_GET['cat']; }

Если вместо переменной $one в запросе написать цифру, которая соответствует категории новости, то все прекрасно работает, если оставить $one, то на страницу ничего не выводит. Проверил что содержится в $one через echo, вывело цифру, то бишь то что нужно, но почему в запросе воспринимается только цифра а не переменная, ведь значения одни и те же?

7 ответов 7

0

Сделай вывод твоего SQL и ONE - по-любому у тебя в SQL вместо ONE - пустое место, т.е. если поставишь '', то ищет d.cat = '', если кавычки убираешь, то SQl считает, что строка не завершена, т.е. d.cat = - а чему оно равно? Проблема, скорее всего в проверке, к тому же - если у тебя cat не передается через GET, то чему он равен будет? добавь что-то типа: if ... else $one = 1;
И в SQL неважно сколько пробелов, trim и прочая ерунда - это для нормального добавления, чтоб потом с поиском не париться, а для WHERE - сколько хотите их ставьте, главное чтобы пробелы не в самой строке были, т.е. можно: WHERE id = 'много-пробелов 1 опять-много-пробелов' - если id - число, но нельзя WHERE name = 'много-пробелов Иван Иванович опять-много-пробелов' - если name - строка

0

уберите одинарные кавычки вокруг $one, если это число, а так же лучше сделать так:

$one = intval($_GET['cat']);

UPD:

А зачем такая белиберда:

$_GET['cat'] = $cats[$_GET['cat']];
$one = $_GET['cat'];

Не проще ли сразу:

$one = intval($cats[$_GET['cat']]);
4
  • Если убираю одинарные кавычки пишет что ошибка синтаксиса mysql. Переменной $one присвоил то, что вы написали, но пока без изменений.
    – dikardRus
    5 дек 2011 в 13:49
  • В строке запроса ошибки не вижу... @DemoS - кавычки это не ошибка
    – user3545
    5 дек 2011 в 13:51
  • @Илья Карлеоне, обновил ответ
    – DemoS
    5 дек 2011 в 13:53
  • Проще, спасибо. Проблема осталась.
    – dikardRus
    5 дек 2011 в 14:17
0

Я конечно ничего против не имею, но Вам стоит читать какие именно ошибки выводит, и если уж не выполняется запрос - выводить то, как он будет выглядеть после инъекции переменных, а не сами переменные, затем тестировать его вручную.

3
  • И старайтесь избегать подобного синтаксиса - $cats[$_GET['cat']]. Ни к чему хорошему он не приведет.
    – stck
    5 дек 2011 в 23:11
  • Если проверить входящие данные и наличие такого элемента в массиве то лично мне непонятно к чему плохому это может привести? Разве что не совсем красиво :)
    – user3545
    6 дек 2011 в 10:23
  • Я имел ввиду без проверки.
    – stck
    6 дек 2011 в 10:52
0

а если trim(), вдруг пробелы закралис

0

Джоин не верный...

LEFT JOIN categories c ON c.id=d.cat
0

ДУмаю, чтоит убрать пробелы между key=value.

$cat = "SELECT d.id,d.cat,d.text,d.title,d.description,d.date,c.text,c.cat,c.titlecat FROM `data` d LEFT JOIN categories c ON d.cat=c.id WHERE d.cat=".$one;

Попробуй такой запрос :)))

0

Если вы попытаетесь выполнить следующее: echo '$one', то интерпретатор будет считать '$one' строкой и выведет содержимое кавычек, а без них это переменная, которая содержит нужное вам значение, во избежание подобных ошибок изучите поподробнее переменные и типы данных в PHP.

4
  • проверил так <?=is_numeric($one)?> , вывело 1, то бишь число всё правильно ведь.
    – dikardRus
    5 дек 2011 в 13:56
  • какбы у ТСа в строка запроса в двойных кавычках так что то что вы написали не имеет абсолютно никакого отношения к данной проблеме
    – user3545
    5 дек 2011 в 13:58
  • проверте так <?=is_numeric('$one')?> !!!
    – triplustri
    5 дек 2011 в 14:14
  • пробовал, ничего не выводит
    – dikardRus
    5 дек 2011 в 14:28

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.