- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
- 33
- 34
- 35
- 36
- 37
- 38
- 39
- 40
- 41
- 42
- 43
- 44
- 45
- 46
- 47
- 48
//Такую штуку можно встретить на большинстве страниц раз по 5
$lrow = mysql_query("select title,sendn,sende,notifs,
nmails,nmails1,nmails2,nmails3,nmails4,nmails5,nmails6,nmails7,nmails8,nmails9,nmails10,
nmails11,nmails12,nmails13,nmails14,nmails15,nmails16,nmails17,nmails18,nmails19,nmails20,
nmails21,nmails22,nmails23,nmails24,nmails25,nmails26,nmails27,nmails28,nmails29,nmails30,
nmails31,nmails32,nmails33,nmails34,nmails35,nmails36,nmails37,nmails38,nmails39,nmails40,
cont,cont1,cont2,cont3,cont4,cont5,cont6,cont7,cont8,cont9,cont10,
cont11,cont12,cont13,cont14,cont15,cont16,cont17,cont18,cont19,cont20,
cont21,cont22,cont23,cont24,cont25,cont26,cont27,cont28,cont29,cont30,
cont31,cont32,cont33,cont34,cont35,cont36,cont37,cont38,cont39,cont40,
texts,texts1,texts2,texts3,texts4,texts5,texts6,texts7,texts8,texts9,texts10,
texts11,texts12,texts13,texts14,texts15,texts16,texts17,texts18,texts19,texts20,
texts21,texts22,texts23,texts24,texts25,texts26,texts27,texts28,texts29,texts30,
texts31,texts32,texts33,texts34,texts35,texts36,texts37,texts38,texts39,texts40 from $ltable where listnum = '$ulist'",$link) or die('admin-37-'.mysql_error());
list($ltitle,$sendn,$sende,$notifs,
$nmails,$nmails1,$nmails2,$nmails3,$nmails4,$nmails5,$nmails6,$nmails7,$nmails8,$nmails9,$nmails10,
$nmails11,$nmails12,$nmails13,$nmails14,$nmails15,$nmails16,$nmails17,$nmails18,$nmails19,$nmails20,
$nmails21,$nmails22,$nmails23,$nmails24,$nmails25,$nmails26,$nmails27,$nmails28,$nmails29,$nmails30,
$nmails31,$nmails32,$nmails33,$nmails34,$nmails35,$nmails36,$nmails37,$nmails38,$nmails39,$nmails40,
$cont,$cont1,$cont2,$cont3,$cont4,$cont5,$cont6,$cont7,$cont8,$cont9,$cont10,
$cont11,$cont12,$cont13,$cont14,$cont15,$cont16,$cont17,$cont18,$cont19,$cont20,
$cont21,$cont22,$cont23,$cont24,$cont25,$cont26,$cont27,$cont28,$cont29,$cont30,
$cont31,$cont32,$cont33,$cont34,$cont35,$cont36,$cont37,$cont38,$cont39,$cont40,
$texts,$texts1,$texts2,$texts3,$texts4,$texts5,$texts6,$texts7,$texts8,$texts9,$texts10,
$texts11,$texts12,$texts13,$texts14,$texts15,$texts16,$texts17,$texts18,$texts19,$texts20,
$texts21,$texts22,$texts23,$texts24,$texts25,$texts26,$texts27,$texts28,$texts29,$texts30,
$texts31,$texts32,$texts33,$texts34,$texts35,$texts36,$texts37,$texts38,$texts39,$texts40)=mysql_fetch_row($lrow);
// Это вывод строк, полученных выше
echo "
<br>
<input type=text class=xbox name=tnmails1 value=\"".htmlspecialchars($nmails1)."\" size=61 style=\"width: 150px\">";
$res1 = mysql_query ("SELECT subject,content FROM lm_saved");
$myrow1 = mysql_fetch_array ($res1);
echo "<select name=tncont1 style='width:150px'>";
if($tncont1){
echo "<option selected>$tncont1</option>";
$rrr1 = mysql_query("SELECT content from lm_saved where subject='$tncont1'");
$rrrrow1 = mysql_fetch_array ($rrr1);
mysql_query ("UPDATE lm_lists SET texts1='$rrrrow1[content]' WHERE listnum='$list'"); }
else {echo "<option selected>".htmlspecialchars($cont1)."</option>";}
do { echo "<option value='$myrow1[subject]'>$myrow1[subject]</option>";}
while ($myrow1 = mysql_fetch_array ($res1));
echo "</select>";
getUserFieldsList($ltable,$list, 2);
// И так еще раз 39 ...
Дали задание "добавить еще 20 полей в программе". Похоже автор сеиго не подозревал об fetch_assoc и циклах. Каждая страница этой "программы" включает в себя css + html + js. Самое обидное, что сделать как надо - потратить пару месяцев, посему приходиться делать тоже самое тупым копипастом. Впервые в жизни меня реально тошнило от правки кода.
Follow us!
Тебя тошнит, а меня выворачивает наизнанку от такого кода.
> меня реально ташнило от правки кода
правки??????
!!
и вообще, код написан в таком дурном тоне, что если по-хорошему, то надо рефакторить с чистого листа
0. Изучаем старый код: мы понимаем, что написали плохо, и настолько плохо, что просто рефакторинг не поможет - например, выбрана неудачная архитектура или неправильные инструменты
1. продумываем новый алгоритм, имея ввиду возможность повтора этого шага, стараясь с минимальными затратами.
2. сравниваем, чем не угождает старый код новым принципам, отмечаем места, которые потенциально будут пригодны
3. создаем новую ветку и пишем новый код по принципам шага 1, не забываем копипастить удачные куски (обычно то там то сям не более одного экрана) старый код, но не поддаваясь искушению полениться и кое-где ляпнуть кусок старого говна.
получается не всегда идеально, но с опытом получается лучше и лучше. Может получиться и хуже, если сфейлили на последней фразе последнего шага.
die