Модераторы: Aliance, skyboy, MoLeX, ksnk

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Библиотека самых бесполезных кусков кода, или как не стоит писать на РНР 
:(
    Опции темы
Ignat
Дата 2.9.2006, 09:48 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Флудератор
****


Профиль
Группа: Экс. модератор
Сообщений: 4030
Регистрация: 19.4.2004
Где: غيليندزيك مدينة

Репутация: 2
Всего: 73
Vaulter, только не говори, что это писал ты smile


--------------------
Теперь при чем :P
PM   Вверх
RomanK
Дата 2.9.2006, 18:00 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 81
Регистрация: 25.6.2005
Где: Россия, Выборг

Репутация: нет
Всего: 2
Вот... нашел. Цитирую только кусок кода:
Код

<?
print"
  <tr>
    <td width=754 height=19 colspan=3 background=img/1.gif class=text><font color=#FFFFFF><b>Информация на форуме</b></font></td>
  </tr>
  <tr>
    <td width=754 height=35 colspan=3 bgcolor=#EFEFEF class=niz>
Сегодня : $chislo $mesyac, $den_nedeli<br>
Всего тем оставленных за всю работу скрипта : $count <br>
В настоящее время присутствуют пользователей: $online<br>
<b>Статистика</b> :
<br>Уникальных: $unique
<br>Сегодня: $all_today
<br>Всего: $all

";


case("lol") :

print"";

break;
endswitch;
?>


Больше всего понравились строки - 16, 19, 21. smile

Это сообщение отредактировал(а) RomanK - 2.9.2006, 18:01
PM MAIL WWW ICQ   Вверх
smartov
Дата 2.9.2006, 22:48 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


свой собственный
****


Профиль
Группа: Экс. модератор
Сообщений: 4225
Регистрация: 2.2.2006
Где: NJ

Репутация: 2
Всего: 259
Не оценил. Просто какой-то недоработанный код и все :\

Написан конечно дубово но индусский стиль присутствует очень мало...
Да и не рабочий код по-идее...
PM MAIL   Вверх
Vaulter
Дата 3.9.2006, 10:11 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Участник Клуба
Сообщений: 1724
Регистрация: 30.12.2002
Где: бункер

Репутация: нет
Всего: 22
Ignat, чур меня! тебе вот смешно! smile а мне пришлось ловить бугага на протяж. двух месяцев переделывая систему...
в итоге, с нуля свою накропал smile



--------------------
PM MAIL WWW ICQ   Вверх
RomanK
Дата 3.9.2006, 19:16 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 81
Регистрация: 25.6.2005
Где: Россия, Выборг

Репутация: нет
Всего: 2
smartov, если это ты про код в моем посте, то это просто вырезка из большого кода, в котором мне понравились именно строки 16, 19, 21. Особенно понравилась строка 21. smile 
Интересно что этим хотел показать автор! smile
PM MAIL WWW ICQ   Вверх
smartov
Дата 3.9.2006, 21:52 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


свой собственный
****


Профиль
Группа: Экс. модератор
Сообщений: 4225
Регистрация: 2.2.2006
Где: NJ

Репутация: 2
Всего: 259
Ну строка 16 - строка как строка, ну может хотел он переносов в конце текста.
19, 21 - такое впечатление что кто-то что-то дебажил или временно проверял, а потом забыл убрать...
PM MAIL   Вверх
IZ@TOP
Дата 4.9.2006, 17:40 (ссылка) | (голосов:1) Загрузка ... Загрузка ... Быстрая цитата Цитата


Панда-бир!
****


Профиль
Группа: Участник
Сообщений: 4795
Регистрация: 3.2.2003
Где: Бамбуковый лес

Репутация: 1
Всего: 73
Всем офисом дружно катались под столами после просмотра такого кода:

Код

switch(TRUE) {
   default:
     // Что-то выполняется...
}



--------------------
Один из розовых плюшевых-всадников апокалипсиса... очень злой...

Семь кругов ада для новых элементов языка
Мои разрозненные мысли
PM MAIL WWW ICQ Skype GTalk   Вверх
smartov
Дата 4.9.2006, 20:21 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


свой собственный
****


Профиль
Группа: Экс. модератор
Сообщений: 4225
Регистрация: 2.2.2006
Где: NJ

Репутация: 2
Всего: 259
Мощно! smile)
Мы такой код называем Uber_really_что-то
В твоем случае Uber_realy_true smile
PM MAIL   Вверх
BuShaRt
Дата 11.9.2006, 18:00 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1391
Регистрация: 29.6.2006

Репутация: -1
Всего: 6
 smile 
А зачем топ был созда? =)) Переименуйте его в "Забавные ошибки" 
Просто не чего интересного не узнал...
PM MAIL   Вверх
smartov
Дата 11.9.2006, 19:39 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


свой собственный
****


Профиль
Группа: Экс. модератор
Сообщений: 4225
Регистрация: 2.2.2006
Где: NJ

Репутация: 2
Всего: 259
Цитата(BuShaRt @  11.9.2006,  17:00 Найти цитируемый пост)
Просто не чего интересного не узнал...

Это хорошо! smile
PM MAIL   Вверх
BuShaRt
Дата 11.9.2006, 20:25 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1391
Регистрация: 29.6.2006

Репутация: -1
Всего: 6
smartov, 
Ну да, я конечно посмеялся, почуствовал себя не полным нубом, но все же было бы гораздо лучше, если бы люди рассказывали тут как стоит избавляться от определенных ошибок.... В самом первом посте были комменты, к которым можно прислушаться, а больше я нечего не увидел, лишь куча кода... 

Я думаю всем стоит чему либо учиться, поэтому стоит не только смеяться над чужими ошабками, но и критиковать их... Вохможно возникнут споры, но ведь в спорах рождаеться истина...

Вот я кстате не понял вот тут:

Код

if (preg_match('/[^\w\d]+/', $login)) //non alphanumeric symbols in login    
    print_and_Die("<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Âû èñïîëüçîâàëè çàïðåùåííûå ñèìâîëû â èìåíè ïåðñîíàæà. Ïîïðîáóéòå åùå ðàç. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>");    
if (is_numeric($login[0])) //only numeric symbols in login    
    print_and_Die("<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Èìÿ âàøåãî ïåðñîíàæà íà÷èíàåòñÿ ñ öèôðû. Ïîïðîáóéòå åùå ðàç. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>");    
if ($parol != $parol_povtor) //passwords mismatch    
    print_and_Die("<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ââåäåííûå âàìè ïàðîëè íå ñîâïàäàþò. Ïîïðîáóéòå åùå ðàç. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>");    
if (trim($login[0]) == "") //login is empty    
    print_and_Die("<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ââåäèòå ïîæàëóéñòà Âàø ëîãèí. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>");    
if ($parol[0] == "") // password is empty    
    print_and_Die("<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ââåäèòå ïîæàëóéñòà Âàø ïàðîëü. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>");    
if (strlen($login) < 3) //login length is less 3    
    print_and_Die("<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ëîãèí ñîñòîèò ìåíåå, ÷åì èç òðåõ ñèìâîëîâ. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>");    
if (strlen($parol) < 8) //minimal passwd length = 8    
    print_and_Die("<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ïàðîëü ñîñòîèò ìåíåå, ÷åì èç âîñüìè ñèìâîëîâ. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>"
);


А зачем каждый раз прерывать? не проще ли делать так?

Код

if (preg_match('/[^\w\d]+/', $login)) 
    $messages[] = ошибка.


Потом перенести в отдельный файл...

Код

<p align='center'> Ошибки:</p>
<p align='center'> Îøèáêà. <br> <messages> <br><a href='registr.html'> Åù¸ ðàç </></p>


и сделать так в подытоживание ошибок:
Код

$error = file ("html/error.html");
print $error[0];
foreach ($message as $val)
    {
        print (ereg_replace ("<messages>",$val,$error[1]));
    }    
PM MAIL   Вверх
smartov
Дата 11.9.2006, 20:35 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


свой собственный
****


Профиль
Группа: Экс. модератор
Сообщений: 4225
Регистрация: 2.2.2006
Где: NJ

Репутация: 2
Всего: 259
BuShaRt, можно еще много чего красивого сделать. Я написал поправку а 5 минут чтобы показать как от этого можно хотя бы _минимально_ здыхаться. 
Код что я написал тоже корявый, но гораздно менее индусский чем оригинал.
Потом ребята еще навернули. Ты ж видел.

Цитата(BuShaRt @  11.9.2006,  19:25 Найти цитируемый пост)
Потом перенести в отдельный файл...

Даже в файл отдельный переносить не надо. Зачем? Накапливай в переменку.
PM MAIL   Вверх
BuShaRt
Дата 11.9.2006, 20:38 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1391
Регистрация: 29.6.2006

Репутация: -1
Всего: 6
Цитата(smartov @  11.9.2006,  20:35 Найти цитируемый пост)
Даже в файл отдельный переносить не надо. Зачем? Накапливай в переменку.


Ты наверно не правилно понял... Я про перенос HTML кода в отдельный файл...

мешать HTML и PHP не хочеться не в каком виде...


Это сообщение отредактировал(а) BuShaRt - 11.9.2006, 20:43
PM MAIL   Вверх
BuShaRt
Дата 12.9.2006, 09:07 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1391
Регистрация: 29.6.2006

Репутация: -1
Всего: 6
Вот сейчас разгребаю один из модудей RunCMS, вроде пример для подражания, а вот такой код увидеть обчное дело:

Код

<table border="0" cellpadding="0" cellspacing="0" width="100%" align="center">
        <tr>
<?php
        if ( $start >= $nombre && $total > $nombre ) {
?>
        <td align="left"><a href="index.php?start=<?php echo ($start - $nombre);?>&nombre=<?php echo $nombre;?>&let=<?php echo $let;?>&ordre=<?php echo $ordre;?>"><?php echo $PdmPrec;?></a></td>
<?php
        }
?>
        <td align="center"><b><?php echo $n_pages;?></b> <?php echo $TotalPages;?></td>
<?php
        if ( ($total - $start) > $nombre ) {
                ?>
                <td align="right"><a href="index.php?start=<?php echo ($start + $nombre);?>&nombre=<?php echo $nombre;?>&let=<?php echo $let;?>&ordre=<?php echo $ordre;?>"><?php echo $PdmSucc;?></a></td>
                <?php
        }
                ?>
        </tr>
</table> 


В чем же дело? Почему программисты позволили себе написать такой стращный код?

Добавлено @ 09:13 
Код

<? /*                                         </td>
                                </tr>
                        </table>
                </td>
        </tr>
</table> */ ?>
PM MAIL   Вверх
smartov
Дата 12.9.2006, 12:16 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


свой собственный
****


Профиль
Группа: Экс. модератор
Сообщений: 4225
Регистрация: 2.2.2006
Где: NJ

Репутация: 2
Всего: 259
Цитата(BuShaRt @  11.9.2006,  19:38 Найти цитируемый пост)
Ты наверно не правилно понял... Я про перенос HTML кода в отдельный файл...
мешать HTML и PHP не хочеться не в каком виде...

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

Цитата(BuShaRt @  12.9.2006,  08:07 Найти цитируемый пост)
Вот сейчас разгребаю один из модудей RunCMS, вроде пример для подражания

Значит не такой уж пример для подражание как кажется.
Или писало много программеров. Ты бы видел код довольон широко рапсространенного движка OSCOmmerce. Там просто жесть.
А phpBB? Сколько там дыр. Так что не все что массово- пример для подражания.
PM MAIL   Вверх
Ответ в темуСоздание новой темы Создание опроса
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | PHP: Избранное | Следующая тема »

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.1544 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.