Накосячил в коде

[penguen]

Накосячил в коде

Делаю мини-сайт о регистрации Заявок на поломки
Есть спец стр. для редактирования принятых заявок.
Так вот, удаление работает, обновление данных НЕТ.
Помогите отладить, буду очень признателен
Даю свой код :

http://phpclub.ru/paste/1850 (код > 10-15 строк рекомендуется размещать там)

И база
[sql]
SET FOREIGN_KEY_CHECKS=0;
-- ----------------------------
-- Table structure for zayavka
-- ----------------------------
CREATE TABLE `zayavka` (
`id_zav` int(10) unsigned NOT NULL auto_increment,
`date` varchar(30) NOT NULL default '',
`address` varchar(200) NOT NULL,
`telephone` varchar(30) NOT NULL default '',
`comment` varchar(255) NOT NULL default '',
`plan` varchar(30) NOT NULL default '',
`vidrabot` varchar(255) NOT NULL default '',
`ispolnitel` varchar(255) NOT NULL default '',
`vipolnenie` varchar(255) NOT NULL default '',
PRIMARY KEY (`id_zav`)
) ENGINE=MyISAM AUTO_INCREMENT=33 DEFAULT CHARSET=cp1251;

-- ----------------------------
-- Records
-- ----------------------------
INSERT INTO `zayavka` VALUES ('1', '07.08.07', 'Shevchenko 5 kv 6', '56789', '', '07.08.07', 'podkluchenie', 'Simba', 'Da');
[/sql]

 

[Mozart]

www.phpfaq.ru/debug

 

[Leshas0]

//удалить цитату
$sql = "UPDATE zayavka SET comment='{$_GET['comment']}' WHERE id_zav = '{$_GET['update']}'";

после этого пробовал поставить print mysql_error ();?

 

[penguen]

Leshas0
За чем удалять данный запрос?
http://path/abon/admin/readzav.php?update=1&page=1&comment=11111111111
Добавляет в базу, но если не прописывать ручками 1111111
переменную коммент=коммент, правую часть не понимает

 

[Leshas0]

penguen
Погоди... ты хотел узнать, почему у тебя не обновляется запись в таблице, так?

После запроса на `update` выведи mysql_error () и посмотри, в чём косяк.

p.s. А в коде своём можешь `удалить цитату` заменить на `обновить цитату`

и вообще... не лучше ли цитату (я так понимаю, это `comment`) передавать POST'ом?

 

[penguen]

Да, не обновляет, но код рабочий. Т.к. переменную $_GET[comment] не понимает,как будь-то ее не объявили.
Дебуг вкл. вывалил ошибки интерпретатор.
Удаление, работает, менять стр.? Понт.

 

[Leshas0]

если в запросе

$sql = "UPDATE zayavka SET comment='{$_GET['comment']}' WHERE id_zav = '{$_GET['update']}'";

$_GET['update'] - это ID, то убери одинарные кавычки

$sql = "UPDATE `zayavka` SET `comment`='" . $_GET['comment'] . "' WHERE `id_zav` = " . $_GET['update'];

 

[penguen]

Добавил мускул-еррор.
Вывалил ошибку
Notice: Undefined index: comment in C:\wamp\www\abon\admin\readzav.php on line 103
style="background-color: #ffffff;" size="60" maxlenght="100" name="comment">

 

[Leshas0]

ну вот тебе и ошибка
разбирайся, где твоя переменная

 

[ustas]

не пиши так, читай про уколы в sqlы.
comment='{$_GET['comment']}' - про это на каждом углу написано

 

[HraKK]

Leshas0
Не позорился бы.

penguen
Попробуйте сами по той статье что вам дали сверху отладить, ведь не надо пугаться что неработает. Это совершенно нормально. Как и совершенно, нормально самому отлаживать.

 

[penguen]

Короче объясняю.
Данная стр. показыв. данные из базы, а именно:

Адресс, телефон, на когда планир. ремонт, вид работ, кто делал, и выволнена ли заявка.

Вот код:

<TD width=5%><?php echo $n++ ?></TD>

<TD width=15%><?php echo $row['2']?></TD>

<TD valign="middle" align="left" width=25%>
<input type="text" value=<?php echo $row['4']?> style="background-color: #ffffff;" size="60" maxlenght="100" name="comment"></TD> <<< и переменная коммент

<TD><input type=text value=<?php echo $row['5']?> style="background-color: #ffffff;" size="10" maxlenght="30" name="plan"></TD></TD>

<TD><input type=text value=<?php echo $row['7']?> style="background-color: #ffffff;" size="10" maxlenght="30" name="ispolnitel"></TD></TD>

-~{}~ 14.08.07 11:28:

Notice: Undefined index: delete in C:\wamp\www\abon\admin\readzav.php on line 10

Notice: Undefined index: comment in C:\wamp\www\abon\admin\readzav.php on line 15

Notice: Undefined index: update in C:\wamp\www\abon\admin\readzav.php on line 15

and

Notice: Undefined index: comment in C:\wamp\www\abon\admin\readzav.php on line 103
style="background-color: #ffffff;" size="60" maxlenght="100" name="comment">
-------------------------------------
Да я не боюсь отладки. Я не могу понять почему обновление не идет, переменная коммент, якобы пуста.

 

[HraKK]

<input type="text" value=<?php echo $row['4']?> style="background-color: #ffffff;" size="70" maxlenght="30" name="comment"></TD>

<TD><input type=text value=<?php echo $row['5']?> style="background-color: #ffffff;" size="10" maxlenght="30" name="comment"></TD></TD>

<TD><input type=text value=<?php echo $row['7']?> style="background-color: #ffffff;" size="10" maxlenght="30" name="comment"></TD></TD>

у тебя тут 3 поля коммент.


penguen
Попробуй отладить сам. Это тебе необходимо, пока ты не научишся отлаживать, а будешь бегать по форумам программистом не станешь.

-~{}~ 14.08.07 10:32:

А где у тебя форма?

 

[penguen]

Notice: Undefined index: comment in C:\wamp\www\abon\admin\readzav.php on line 117

Notice: Undefined index: update in C:\wamp\www\abon\admin\readzav.php on line 117

ругается на этот кусок кода

$sql = "UPDATE zayavka SET comment='{$_GET['comment']}' WHERE id_zav = '{$_GET['update']}'";

-~{}~ 14.08.07 11:35:

HraKK
Спасибо за советы.
Вопрос. Аська у вас есть? Так легче было б, а то форум есть форум. Пхп знаю как с неделю, до этого на С++ писал, заставили учить пых-пых

 

[Гравицапа]

penguen

Пхп знаю как с неделю, до этого на С++ писал, заставили учить пых-пых

PHP FAQ: Хочу изучать PHP и Mysql. С чего
начать?

Аська у вас есть? Так легче было б, а то форум есть форум.

История от bokal:

Я столкнулся на работе с чуваком, он устроился через 6–7 месяцев после меня, он с php только начал работать. Так вот он дёргал меня каждые 5–10 минут. Я терпел это неделю, подсказывал, объяснял в чём ошибки. Просто начальство, так сказать, сделало меня наставником. Я ему и zend поставил, разрулил как юзать... Вторую неделю я просто тыкал в мануал... Но он всё равно меня дёргал постоянно. Как-то вышли покурить, я ему говорю, мол, человек, есть поиск, мануалы, ide зашибатая! Что ещё надо-то? Быстрее самому разобраться! Ответ был гениальный – «А зачем мене читать мануалы и искать, если Я У ТЕБЯ МОГУ СПРОСИТЬ!» После этого я чувака в полный игнор послал.

Мораль: не хотят помогать не новичкам, а тем, кто сам себе не хочет помочь. Тот, кто сам себе хочет помочь – быстро становится профессионалом, а тот, кто не хочет себе помочь – остается вечным новичком.

 

[HraKK]

1 час = 15$ номер WMZ - Z245775092811
переводите и общаемся по аське.

С++ писал

А что в С++ отладки нету? Как по мне там еще хуже.

 

[penguen]

Гравицапа
Намек понял. Мэн, мне проэкт сдать нужно к вечеру. Наваял все сам с 6_00 и по сей час. Маны, хотво читаю и буду читать.
Просто не врубаюсь где косяк.

-~{}~ 14.08.07 11:45:

HraKK
Верно С++ это жесть. Ладно, буду дебуг вкл. и глядеть что да по чем. Спасибо всем за монолог.

 

[jonjonson]

penguen, ты с этим скриптом как минимум с 11 июля упражняешься (судя по форуму nulled.ws). Может всё же послушаешь умных людей и начнёшь построчно код разбирать?

 

[Фанат]

Да я не боюсь отладки. Я не могу понять почему переменная коммент якобы пуста.

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

 

[penguen]

jonjonson
Тот код я отладил и он уже рабочий
*****
Я смотрю здесь больше понтов, чем дела.
Я не прошу дать мне код=рабочий и я забуду все.
Ткните носом где я не прав, и я приму к сведению, что не так.