- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
<?php
if (isset($_POST['update'])) {
$kolcen=mysql_real_escape_string($_POST['kolcen']);
$tovar=mysql_real_escape_string($_POST['tovar']);
$descr=mysql_real_escape_string($_POST['descr']);
$price=intval($_POST['price']);
$price2=intval($_POST['price2']);
$price3=intval($_POST['price3']);
$id=intval($_POST['update']);
$zag1=mysql_real_escape_string($_POST['zag1']);
$zag2=mysql_real_escape_string($_POST['zag2']);
$zag3=mysql_real_escape_string($_POST['zag3']);
switch($kolcen){
case '3':$updatetovar = mysql_query ("UPDATE `tovar3` SET `tovar`='$tovar',`descr`='$descr',`price`='$price',`price2`='$price2',`price3`='$price3' WHERE `id`='$id' and `id`='$serv'");
break;
case '2': $updatetovar = mysql_query ("UPDATE `tovar2` SET `tovar`='$tovar',`descr`='$descr',`price`='$price',`price2`='$price2' WHERE `id`='$id' and `id`='$serv'");
break;
case '1':$updatetovar = mysql_query ("UPDATE `tovar` SET `tovar`='$tovar',`descr`='$descr',`price`='$price' WHERE `id`='$id' and `pizzaid`='$service'");
break;
case '2images': $updatetovar = mysql_query ("UPDATE `2images` SET `tovar`='$tovar',`descr`='$descr',`price`='$price',`price2`='$price2',`price3`='$price3',`img`='$img',`zag1`='$zag1',`zag2`='$zag2',`zag3`='$zag3' WHERE `id`='$id' and `id`='$serv'");
break;
}
?>
Народ зацените мой код плиз на предмет говнокода, и если чето не так то дайте совет . Этот код для админки добавления товаров. Есть 4 типа категорий, товар с одной ценой, с двумя, с тремя, и товар с двумя картинками. В зависимости от типа категории делаем запрос к базе
guest 21.06.2011 15:49 # +11
7ion 21.06.2011 20:19 # 0
rO_ot 21.06.2011 16:05 # −1
По хорошему запросы можно было бы обернуть в какой-то минимальный ActiveRecord,
например добавишь ты поле в таблицу, придется менять большинство запросов, а с ActiveRecord только в модельке поле добавить;
Свитч-кейс перепроектировать, совсем невнятное говно получилось;
$kolcen=mysql_real_escape_string($_POST['kolcen']);
$tovar=mysql_real_escape_string($_POST['tovar']);
$descr=mysql_real_escape_string($_POST['descr']);
$price=intval($_POST['price']);
$price2=intval($_POST['price2']);
$price3=intval($_POST['price3']);
$id=intval($_POST['update']);
$zag1=mysql_real_escape_string($_POST['zag1']);
$zag2=mysql_real_escape_string($_POST['zag2']);
$zag3=mysql_real_escape_string($_POST['zag3']);
это всё привести к чему-то общему, что принимает response, чистит его, и из него брать чистые значения..
В целом - гавно всё
bugmenot 21.06.2011 21:59 # 0
Lure Of Chaos 21.06.2011 22:55 # +5
striker 22.06.2011 01:37 # +5
bugmenot 22.06.2011 02:19 # 0
nobody 21.06.2011 16:07 # −1
И еще, транслит в коде программисты очень не любят, учи английский и обзывай нормально переменные ...
Ну и также изучай фреймворк аля симфони или хотя бы ООП и пиши свои велосипеды ... Такое построение кода не орех, просто я уже представил что у тебя идет дальше ...
frie 21.06.2011 16:23 # 0
frie 21.06.2011 16:26 # −1
yasenhui 21.06.2011 16:35 # 0
Повтори вопрос "где почитать как нормально кодить или курсы антиговнокода" например в "вопросах и ответах" Гугла.
nobody 21.06.2011 16:40 # 0
Я лично самоучка, наставников у меня тоже не было, если работать будешь в опытном коллективе, всему научишься, иди куда нибудь на не большую ставку ради опыта ...
message 21.06.2011 16:56 # +1
rO_ot 21.06.2011 22:41 # 0
Lure Of Chaos 21.06.2011 18:05 # +8
задача "добавить еще одну цену" или "добавить еще одну картинку" превращается в ад.
читать книжки по best practices
SadKo 22.06.2011 10:16 # +1
Lure Of Chaos 24.06.2011 17:27 # 0
guest8 09.04.2019 11:01 # −999
guest8 09.04.2019 18:07 # −999
guest8 12.04.2019 14:23 # −999
guest8 19.04.2019 18:29 # −999