Новый день, старая тема.
Пишу человечку админку, все работает кроме радактирования записей в базе!
Вот код:

Код

switch ($mode) { //оставила чтобы видно было откуда берутся переменные. case'meny':{     echo "<h3>Товары:</h3> <a href=collection_admin.php?mode=add>Добавить в коллекцию</a><br> <a href=collection_admin?mode=del>Удалить из коллекции </a><br> <a href=collection_admin?mode=edit>Редактировать коллецию</a><br>"; } break; //до сюда работает case 'edit':{ echo "<table cellspacing=0 cellpadding=0 border> <form method=POST> <tr><th colspan=6 height=50>Форма для редактирования данных базы Coolection</th></tr> <tr><td align=center height=20>Артикул (только номер)</td> <td align=center height=20>Текст(описание материалов)</td> <td align=center height=20>Размер</td> <td align=center height=20>Цена</td> <td align=center>Категория</td> </tr>"; $result = mysql_query("SELECT id_col,art,text,size,price,date,id_cat FROM collection") or die(mysql_error());                 for($i=0;$i<mysql_num_rows($result);$i++) {     $mis=mysql_fetch_row($result); echo "<tr> <td align=center valign=top><input type=text name=ar value='$mis[1]'></td> <td align=center valign=top><input type=text name=tex value='$mis[2]'></td> <td align=center valign=top><input type=text name=raz value='$mis[3]'></td> <td align=center valign=top><input type=text name=zena value='$mis[4]'></td> <td align=center valign=top><input type=text name=cate value='$mis[6]'></td> </tr>"; } ?> <tr><td colspan=5 align=center><Input Type=submit name=red Value=Редактировать style=cursor:hand></td></tr><table></form> <? if($red) { $time=date("Y-m-d"); //вот тут начинаются проблемы, база может отредактировать только последнюю запись, выводимую  // на экран mysql_query ("UPDATE collection SET `art`='$ar', `text`='$tex', `size`='$raz', `price`='$zena', `id_cat`='$cate' WHERE id_col=$mis[0]") or die(mysql_error());                       header('location: collection_admin.php?mode=edit'); exit; }  } break; }

Пожалейте! Сижу с кодом 3 день. Незнаю уже с какой стороны исправить!

А в чем конкретно? mysql_error выведи после запроса.

Код

WHERE id_col=$mis[0]") or die(mysql_error());

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

Это сообщение отредактировал(а) PARROT - 11.5.2005, 17:10

Цитата(PARROT @ 11.5.2005, 18:02)

switch ($mode) {

Почему так-то, нужно ведь $_POST['mode']

Цитата(Aliance @ 11.5.2005, 17:18)

Цитата(PARROT @ 11.5.2005, 18:02) switch ($mode) {

Дело как раз не в switch, он работает как часы.

Цитата(PARROT @ 11.5.2005, 18:10)

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

А какие тебе надо?
Он у тебя обновляет ту запись, ID которой ты берешь из $mas[0]

В базе пока 15 записей, почему он обновляет именно 15-ю, а остальные нет?
А переменная $mas[0]; у всех разная, так как это номер записи.
Может можно обновлять только 1 или только 2 и т.д. записи, сразу 3 или 5 обновременно нельзя?

Дык поидеи массивом нуно пробежаЦа

Так мы же поля в базе обновляем...?
Загружаем их в текстареа и там редактируем, после просто обновляем по идее, а?
Может я не права? Запуталась?
Я уже ничего не понимаю.

В упор не вижу ниодной textarea

Ну в input, разница не большая
Смысл один и тот же, редактирование и обновление данных в базе.
Мне уже смешно. Я сейчас по полу кататься начну!
Добавлено @ 21:30
грузим из базы в input, value-м для которых будут являться поля из базы.

Это сообщение отредактировал(а) PARROT - 11.5.2005, 21:30

Может попробовать сheckbox ввести, чтобы отмечать те сообщения, которые ты редактируешь? А потом указать в запросе к базе, что обнавлять только те записи, которые отмечены.

Цитата

WHERE id_col=$mis[0]")

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

Код

If или while(isset($checkbox)) {mysql_query("UPDATE collection 'art'=$art ......... where id_cat=$mis[0]");

Это сообщение отредактировал(а) cosmos33 - 11.5.2005, 22:21

Конечно будет редактировать только последнюю

Объясняю:
Ты выбираешь из базы, и делаешь построчный fetch (обработка строк)
Выводишь построчно таблицу для редактирования. Когда цикл вывода заканчивается, то $mis содержит в себе последнюю строчку, выбранную из БД. И соответственно $mis[0] - идентификатор последней записи.
Потом запускается редактирование (кстати его лучше поставить сначала, иначе результаты сразу видны не будут)
Потому и редактируется только она.
Это не совсем верный способ.


зачем чекбоксы?

насколько я понял смысл, нужно сделать редактирование сразу всех записей.

тогда код по идее должен выглядеть вот так:

Код

switch ($mode) { //оставила чтобы видно было откуда берутся переменные. case'meny':{     echo "<h3>Товары:</h3> <a href=collection_admin.php?mode=add>Добавить в коллекцию</a><br> <a href=collection_admin?mode=del>Удалить из коллекции </a><br> <a href=collection_admin?mode=edit>Редактировать коллецию</a><br>"; } break; //до сюда работает case 'edit':{ //Блок редактирования лучше поместить сначала, чтобы результаты при следующей выборке были видны сразу if($_SERVER['REQUEST_METHOD']=="POST") { $time=date("Y-m-d"); //вот тут начинаются проблемы, база может отредактировать только последнюю запись, выводимую  // на экран exctract($_POST); foreach($id_record as $id => $value) {     mysql_query ("UPDATE collection SET `art`='$ar[$id]', `text`='$tex[$id]', `size`='$raz[$id]', `price`='$zena[$id]', `id_cat`='$cate[$id]' WHERE id_col=$id") or die(mysql_error());     } // Теперь это хеадер не нужен //header('location: collection_admin.php?mode=edit'); //exit; }  echo "<table cellspacing=0 cellpadding=0 border> <form method=POST> <tr><th colspan=6 height=50>Форма для редактирования данных базы Coolection</th></tr> <tr><td align=center height=20>Артикул (только номер)</td> <td align=center height=20>Текст(описание материалов)</td> <td align=center height=20>Размер</td> <td align=center height=20>Цена</td> <td align=center>Категория</td> </tr>"; $result = mysql_query("SELECT id_col,art,text,size,price,date,id_cat FROM collection") or die(mysql_error()); while ($mis=mysql_fetch_object($result)) { echo "<tr> <td align=center valign=top> <input type=hidden name=id_record[$mis->id_col] value='$mis->id_col'> <input type=text name=ar[$mis->id_col] value='$mis->art'></td> <td align=center valign=top><input type=text name=tex[$mis->id_col] value='$mis->text'></td> <td align=center valign=top><input type=text name=raz[$mis->id_col] value='$mis->size'></td> <td align=center valign=top><input type=text name=zena[$mis->id_col] value='$mis->price'></td> <td align=center valign=top><input type=text name=cate[$mis->id_col] value='$mis->id_cat'></td> </tr>"; } ?> <tr><td colspan=5 align=center><Input Type=submit name=red Value=Редактировать style=cursor:hand></td></tr><table></form> <? } break; }

p.s. не тестировал, возможно где-то опечатался

Это сообщение отредактировал(а) oxygen - 12.5.2005, 03:23

Цитата(oxygen @ 12.5.2005, 03:16)

Это не совсем верный способ.

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

Спасибо, oxygen. Я посмотрю, подумаю.

Добавлено @ 07:49

Цитата(cosmos33 @ 11.5.2005, 22:00)

Может попробовать сheckbox

На дух checkbox-сы не переношу, как и select, внешний вид их вызывает отвращение.

Даже не могу объяснить почему.

Тогда уж лучше делать ссылками или уж на худой конец вместо ссылок вставить img

Цитата(PARROT @ 12.5.2005, 07:46)

Тогда уж лучше делать ссылками или уж на худой конец вместо ссылок вставить img

да нет же, не надо так мучаться
просто пробежаться по массиву и каждую строчку проапдейтить.

Цитата(PARROT @ 12.5.2005, 07:46)

На дух checkbox-сы не переношу, как и select,

Зря - иногда очень экономит место в интерфейсе

Да конечно, проплетемся.

Ща только подумаем... Просто вставить код и пользоваться уже неинтересно.

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

Для себя удобство тоже создавать нужно, а не только о пользователях заботиться.

Цитата(oxygen @ 12.5.2005, 09:27)

да нет же, не надо так мучаться

Если бы так......... то давно бы все работало.
Добавлено @ 09:56

Цитата(DENNN @ 12.5.2005, 09:29)

Цитата(PARROT @ 12.5.2005, 07:46) select Зря - иногда очень экономит место в интерфейсе

А видуха у них, прям скажем никакая... даже если селект привести в порядок css, то его стрелка повергнет в ужас.

Это сообщение отредактировал(а) PARROT - 12.5.2005, 09:54

Цитата(PARROT @ 12.5.2005, 10:54)

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

Почему? Ты думаешь на отд. странце это делать...мдя...
Берешь, выводишь всю БД. Рядом с каждым столбцом - чекбаттон. Если он помечен - ее изменяем, если нет - не трогаем.
Далее отмечаешь что нужно - нажимаешь кнопку вперед - там уже будут только те столбцы, что пометила. Их редактируешь и воуля

Мы общим собранием и большинство голосов решили с чеками не связываться, так как они портят общий дизайн страниц.
У меня еще руки не дошли доделать и сил нет. И работы много, да еще директор пасет, вот!
Добавлено @ 15:54
Чем лучше пользоваться все таки: for или while ?
Знаю, что while используется для масивов.
Я никак не могу его понять, этот while. Даже боюсь.

PARROT в данном случае однозначно while() потому что for грузит на каждой итерации еще mysql_num_rows - лишняя операция*количество итераций.

Понять его можно так
ПОКА($var равна mysql_fetch) ДЕЛАТЬ что-либо
ИНАЧЕ конец цикла

как только fetch обработает последний ряд, на следующей итерации $var получит FALSE и цикл закончится

Это сообщение отредактировал(а) oxygen - 12.5.2005, 16:30

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

Это сообщение отредактировал(а) cosmos33 - 12.5.2005, 16:33

Цитата(cosmos33 @ 12.5.2005, 16:30)

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

собственно что я и сделал в коде

Мое мнение, лучше все-таки юзать поле id, которое будет задаваться автоматически, при добавлении в базу новой записи.

Удобство, простота - главное!

Короче правил нет.

Получается, кто на что горазд?

Это сообщение отредактировал(а) PARROT - 12.5.2005, 17:22

Уникальность поля не определяется тем, что оно создано автоматически.

Цитата(PARROT @ 12.5.2005, 16:38)

Мое мнение, лучше все-таки юзать поле id, которое будет задаваться автоматически, при добавлении в базу новой записи. Удобство, простота - главное! Короче правил нет. Получается, кто на что горазд?

так и надо. Автоинкрементное поле.