Форум PHP

Вопрос.
Есть форма:

<center> Форма добавления новостей<br> <form action="news.php" method="POST"> <table style="background-color: rgb(209,209,209)"> <tr> <td>Заголовок новости: </td><td><input size="52" name="title"></td></tr> <tr> <td valign="top">Краткое описание:</td><td><textarea cols="40" rows="6"name="desсriрtion"></textarea></td></tr> <tr><td valign="top">Текст статьи: </td><td> <textarea cols="40" rows="15"name="text"></textarea></td></tr> </table> <input type="submit" name="button" value="Разместить статью"> </form>

News.php

<?php $title=$_POST['title']; $desсriрtion=$_POST['desсriрtion']; $text=$_POST['text']; $db=mysql_connect(localhost, 'root', ''); mysql_select_db('rss', $db); $ids = mysql_query("SELECT id FROM 'news'); $id = mysql_result($ids, 0, 'id'); $id++; mysql_query("INSERT INTO news (id,title,desсriрtion,text)  VALUES ('id','$title','$desсriрtion','$text')",$db); mysql_close($db); ?>

Браузер выдает:

[quote]
Parse error: syntax error, unexpected T_STRING in Z:\home\RevivalCMS.ru\www\admin\news.php on line 10
[/quote]

:D
вы тут в подсветке кода не видите не красивостей?)

А по расцветке кода не видно?

<?php $title=$_POST['title']; $description=$_POST['description']; $text=$_POST['text']; $db=mysql_connect(localhost, 'root', ''); mysql_select_db('rss', $db); $ids = mysql_query("SELECT id FROM 'news'"); $id = mysql_result($ids, 1); $id++; mysql_query("INSERT INTO news (id,title,description,text)  VALUES ('id','$title','$description','$text')",$db); mysql_close($db); ?>

Исправил, теперь выдает:
[quote]
Warning: mysql_result(): supplied argument is not a valid MySQL result resource in Z:\home\RevivalCMS.ru\www\admin\news.php on line 8
[/quote]

Вот и поглядите, каким символом начинается красный цвет, и каким заканчивается. Может тогда дойдет.

Warning: mysql_result(): supplied argument is not a valid MySQL result resource in Z:\home\RevivalCMS.ru\www\admin\news.php on line 8
Что - и теперь не можете найти?

<?php  $title=$_POST['title'];  $desсriрtion=$_POST['desсriрtion'];  $text=$_POST['text'];  $db=mysql_connect(localhost, 'root', '');  mysql_select_db('rss', $db);  $ids = mysql_query("SELECT id FROM 'news';");  $id = mysql_result($ids, 0, 'id');  $id++;  mysql_query("INSERT INTO news (id,title,desсriрtion,text)   VALUES ('id','$title','$desсriрtion','$text');"),$db);  mysql_close($db);  ?>

вот, попробуй так.

>для: Trianon (04.07.2009 в 18:12)
>вот, попробуй так.

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

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

выдает ошибку в 11 строке. скрипт мне никто не давал, писал сам.

а посчитать скобки в этой строке?

Скрипт , который сейчас приведен в (04.07.2009 в 18:07) выглядит более корректно.
Основная Ваша проблема заключается в том, что Вы не проверяете, успешно выполнен запрос сервером БД или нет.
Если запрос завершился неудачей ($ids равно нулю) то надо смотреть текст диагностики (возвращается функцией mysql_error())

Блин я не на этот коммент отвечал а на тот который выше, где говорилось о первичном ключе и auto_incriment только еще желательно тип поля интаджер(INT) выставить
Полностью согласен!
Revival1002, Накой хрен вы так обрабатываете поле id? Это чистой воды ламерство! И о каком программировании может идти речь если вы не можете отличить присвоение от сравнения.
Если вы желаете мучаться и страдать ерундой, то делайте так как я описал ниже(где-то в саомом конце) но все-таки бросте это глупое дело и обратитесь к профессионалам ссылку дал в превом комменте(но вдруг вы не заметите http://www.phpsite.z4.ru/zakaz.php

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

Эт еще зачем?

Я то то тоже не понял зачем.

<?php $title=$_POST['title']; $desсriрtion=$_POST['desсriрtion']; $text=$_POST['text']; $db=mysql_connect('localhost', 'root', ''); mysql_select_db('rss', $db); $ids = mysql_query("SELECT id FROM news ", $db); if ($ids=0) {echo mysql_error();} $id = mysql_result($ids, 0); $id++; mysql_query("INSERT INTO news (id,title,desсriрtion,text)VALUES ('$id','$title','$desсriрtion','$text')",$db); mysql_close($db); ?>

Warning: mysql_result(): supplied argument is not a valid MySQL result resource in Z:\home\RevivalCMS.ru\www\admin\news.php on line 9
По моему что то не то получилось.

А $id у вас что строковое значение?

Разве это имеет значение в PHP. Изначально не строковое.

Если оно не строковое, а числовое, то зачем Вы его берете в апострофы?

if($ids=0) это вовсе не сравнение.
Это присваивание нуля этой переменной.

Кроме того Вы не сделали попытки проверить результат второго запроса.

тогда как сравнить? if !($ids) ?

if($ids == 0)

if(!$ids)

переделал строчку так

$qu=mysql_query("INSERT INTO news (id,title,desсriрtion,text)VALUES ('$id','$title','$desсriрtion','$text')",$db);

$qu получается равным нулю, но место текста ошибки оно пишет "mysql_error". Значит ошибка в этой строчке.
---------
Написал запрос снова, уже все работает. Все большое спасибо!

Почему id текстовый?

> но место текста ошибки оно пишет "mysql_error". Значит ошибка в этой строчке.

кто пишет mysql_error ? отсель непонятно.

Блин, вчера код, работал, а сегодне, я ничего не изменял и уже не работает. ТУпой комп!

спасибо, хоть, не форум...
С таким подходом даже за топор лучше не браться.

Да, дружище ты совершенно прав! Просто программирование - это такая штука, как погода, то работает то нет.
Если потанцевать с бубном вокруг компьютера, то возможно и заработает ;)

Под фразой "потанцевать с бубном вокруг компьютера" имелось ввиду почитать литературу по корректному построению БД (Trianon (05.07.2009 в 12:59)). Ну и немного непонятно как можно кодить без знания оператора присваивания в PHP?

<?php  $title=$_POST['title'];  $desсriрtion=$_POST['desсriрtion'];  $text=$_POST['text'];  $db=mysql_connect('localhost', 'root', '');  mysql_select_db('rss', $db);  $ids = mysql_query("SELECT id FROM news ", $db); if ($ids == 0) { echo 'Записей нет'; } else { $id = mysql_result($ids); $id++;  mysql_query("INSERT INTO news (id,title,desсriрtion,text)VALUES ('$id','$title','$desсriрtion','$text')",$db);  } mysql_close($db);  ?>

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

Не "записей нет" а "запрос не был выполнен"

Ну что за ерунду вы тут творите, если уж хочеться изобрести велосипед, то делайте это хоть по уму

<?  $result = mysql_query("select id from news order by id desc",$db); // вытаскиваем все поля id из базы в обратном порядке(чтобы потом последнее поле захватить if (!$result) //тут проверка прошел ли запрос   { echo "<p>Возникла ошибка с базой данных. <strong>Сообщите администратору:****</strong></p>"; exit (mysql_error());   } if(mysql_num_rows($result) > 0) // проверка вытащилось ли хоть 1 строчка из базы { $myrow = mysql_fetch_array($result);  //вытаскивается первая строка из $result Так как мы сортировали в по полю id по убыванию то и ниже $id получиться самая большая что есть в базе $id = $myrow['id'];  $id++; // вот и все  $result = mysql_query("INSERT INTO news (id,title,desсriрtion,text) VALUES  ('$id','$title','$desсriрtion','$text')",$db);  // тут у вас скорей всего ошибка в том что нет подходящих полей в таблице news if($result == true)//проверка успешности добавления информации {exit("Все ок информация добавилась");} else {exit("Не обновилось, произошла ошибка!");}  } ?>

К тому же mysql поддерживает собственный индекс, а так как вы никто не делает
И вообще,Revival1002, судя по всему вам еще рановато заниматься практикой, нужно сначала изучить теорию.
Если хотите обновить скрипт сайта или доработать его, то вот вам адрес, там вам помогут практически за копейки : http://www.phpsite.z4.ru/zakaz.php - тут есть и форум где вы можете оставить вопрос администратору и вам дадут профессиональную консультацию.

_______________________________
НУ КАК КРАСИВО ПОЛУЧИЛОСЬ?

Не совсем. Если требуется ОДНА запись, зачем тянуть ВСЕ? Использовать LIMIT 1 вера не позволяет?
К тому же вытаскивать ID из базы , чтобы вставить на 1 больший - это не верный подход. Одновременно могут быть два и более запроса, прочиавших один и тот же ID, и затем попытающихся записать снова один и тот же ID. Автоинкеремнтные поля не от лени придумывались.

LIMIT 1 использовать не позволяет вера. А в запросе напрашивается SELECT MAX(id) . В том гипотетическом варианте, когда нужен MAX(id) . :)

Дак я об этом и говорю!!! Зачем вообще таким способом обрабатывать id! Ну раз человеку захотелось именно так вот я и привел ему более менее приличный пример, можно и limit можно и MAX(id), но все равно как ни крути место такому способу обработки индекса в мусорном баке! Думаю в этом вы со мной согласитесь?
У меня на хосте например, если не создашь нормальный индекс в БД, то сразу высвечивается замечание.

Улыбнуло, посмотрев Ваш суперпрофессиональный сайт, особенно дергающееся меню.
А насчет индексов Вы заблуждаетесь. Индексы ускоряют выборку и ничего более. Можно успешно работать и без них, особенно если в таблицу много пишется, и только изредка читается, ибо индекс перестраивается при каждой вставке.

Такое ощущение, что вы думаете - это я так id обрабатываю. Читайте внимательнее!

А так, Во всем с вами полностью согласен :)
Может был немножко резковат в комментах просто настроение никакое.
Классный форум нашел, пока искал ответы на свой вопрос(правда так и не нашел).
Как-нибудь еще загляну.