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

Поиск:

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


Опытный
**


Профиль
Группа: Участник
Сообщений: 429
Регистрация: 28.9.2005
Где: HellCity (Sofia, Bulgaria)

Репутация: 1
Всего: 29



Мой любимий кусок кода, не помню где его видел, но сразу в WTF отправил
Код
if (unlink("../news_photos/smalls/".$filename_old) == TRUE) {
     unlink("../news_photos/smalls/".$filename_old);
 }


P.S. это из винграда:
http://forum.vingrad.ru/index.php?showtopic=47964&hl=

Это сообщение отредактировал(а) Darhazer - 24.8.2006, 18:24


--------------------
I'm a wheel, I'm a wheel, I can roll, I can feel
But you can't stop me turning
'Cause I'm the sun, I'm the sun, I can move, I can run
But you'll never stom me burning
PM MAIL WWW ICQ YIM   Вверх
IZ@TOP
Дата 24.8.2006, 13:31 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


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


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

Репутация: 1
Всего: 73



Darhazer, ЖЖЕШЬ НЕ ПАДЕЦКИ! У нас весь офис лежал!!!

Добавлено @ 13:33 
Мне тут рассказывали про комменты в коде, значит идет какой-то if, и там коммент:
Код


if(что-то != чего-то) { // По идее логика программы сюда доходить не должна
   $_variable = что-то;
}



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

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


Мудрый...
****


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

Репутация: 1
Всего: 261



Цитата(IZ@TOP @  24.8.2006,  14:31 Найти цитируемый пост)
Мне тут рассказывали про комменты в коде, значит идет какой-то if, и там коммент:

Вспомнилось. На вовебе одно ламо код JS не рабочий дало. Код содержал комментарий:
Цитата
<!--Dummy comment to hide code from non-JavaScript browsers-->

PM ICQ   Вверх
smartov
Дата 24.8.2006, 18:07 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


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


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

Репутация: 2
Всего: 259



Darhazer, ааааа smile ууужос smile smile smile
PM MAIL   Вверх
Ignat
Дата 24.8.2006, 18:34 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


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


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

Репутация: 2
Всего: 73



Цитата(smartov @  24.8.2006,  13:59 Найти цитируемый пост)
млиин.. 29 раз... упорный тип, без вопросов... он наверное себе макросы в редакторе писал

Хуже... Я такого кода, наверное, километров 10 перефакторил. Так пишут поклонники копипаста.


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


Эксперт
***


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

Репутация: нет
Всего: 36



Не далее, как вчера, наткнулся на код:
Код

$a = addslashes( stripslashes( addslashes( $str ) ) );


То ли чел покурил не того.
То ли я не понимаю всей глубинной сути.
PM WWW   Вверх
Vreden
Дата 24.8.2006, 21:20 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Idiotic Admin
**


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

Репутация: -1
Всего: 1



Так ребятки, знаю код безобразный, признаюсь мой, упорный тип, да, знаю, не кричите.
Код

<?
$login = $_POST['login'];
$parol = $_POST['parol'];
$parol_povtor = $_POST['parol_povtor'];
if (preg_match('~[^qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNMéöóêåíãøùçõúôûâàïðîëäæýÿ÷ñìèòüáþ¸ÉÖÓÊÅÍÃØÙÇÕÚÔÛÂÀÏÐÎËÄÆÝß×ÑÌÈÒÜÁÞ¨012345678 9]+~', $login))
{echo "<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Âû èñïîëüçîâàëè çàïðåùåííûå ñèìâîëû â èìåíè ïåðñîíàæà. Ïîïðîáóéòå åùå ðàç. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>";}
else
  {if (is_numeric($login[0]))
  {echo "<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Èìÿ âàøåãî ïåðñîíàæà íà÷èíàåòñÿ ñ öèôðû. Ïîïðîáóéòå åùå ðàç. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>";}
  else
    {if ($parol != $parol_povtor)
    {echo "<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ââåäåííûå âàìè ïàðîëè íå ñîâïàäàþò. Ïîïðîáóéòå åùå ðàç. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>";}
    else
      {if ($login[0] == "")
      {echo "<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ââåäèòå ïîæàëóéñòà Âàø ëîãèí. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>";}
      else
        {if ($parol[0] == "")
        {echo "<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ââåäèòå ïîæàëóéñòà Âàø ïàðîëü. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>";}
        else
          {if ($login[0] == " ")
          {echo "<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ëîãèí íå ìîæåò íà÷èíàòüñÿ ñ ïðîáåëà. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>";}
          else
            {$leng_login = strlen($login);
            {if ($leng_login < 3)
            {echo "<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ëîãèí ñîñòîèò ìåíåå, ÷åì èç òðåõ ñèìâîëîâ. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>";}
            else
              {$leng_parol = strlen($parol);
              {if ($leng_parol < 8)
              {echo "<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Ïàðîëü ñîñòîèò ìåíåå, ÷åì èç âîñüìè ñèìâîëîâ. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>";}
              else
                {mysql_connect("localhost","root")or die (mysql_error());
                mysql_select_db("test") or die (mysql_error());
                $del = mysql_query (sprintf("select * from `user` where  login='%s'",$login)) or die (mysql_error());
                $del= mysql_fetch_array($del);
                if ($del[1]== $login)
                  {echo "<html><body bgcolor='black' text='green'><p align='center'> Îøèáêà. <br> Òàêîé ëîãèí óæå ñóùåñòâóåò. Ïðèäóìàéòå ïîæàëóéñòà äðóãîé. <br><a href='registr.html'> Åù¸ ðàç </></p></body></html>";}
                  else
                  {session_start();
                  session_register("loginuser");
                  session_register("paroluser");
                  $loginuser = $login;
                  $paroluser = $parol;
                  #setcookie ('loginuser', $login);
                  #setcookie ('paroluser', $parol);
                  header("location: registr2.html");}}}}}}}}}}}
?>

Переделайте плиз так, чтобы была та же самая логика проги, но выглядела правильно с точки зрения этой темы. И вообще скажите плиз, что не так?


--------------------
Чё? Как? ©
PM MAIL WWW   Вверх
jzZ
Дата 24.8.2006, 22:36 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


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

Репутация: нет
Всего: 2



Код

}}}}}}}}}}}

Это вообще что такое  smile ))

Добавлено @ 22:38 
Весь код сущий кошмар, это я еще не говорю о стиле кодирования...
PM MAIL   Вверх
Ignat
Дата 25.8.2006, 08:40 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


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


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

Репутация: 2
Всего: 73



Код

if (preg_match('~[^qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNMéöóêåíãøùçõúôûâàïðîëäæýÿ÷ñìèòüáþ¸ÉÖÓÊÅÍÃØÙÇÕÚÔÛÂÀÏÐÎËÄÆÝß×ÑÌÈÒÜÁÞ¨012345678 9]+~', $login))

Ой, мама... Где я?
Цитата(Vreden @  24.8.2006,  22:20 Найти цитируемый пост)
И вообще скажите плиз, что не так? 

Вообще, что здесь так?


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


Idiotic Admin
**


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

Репутация: -1
Всего: 1



нууууууууууууу и кто-нибудь исправит?  smile 


--------------------
Чё? Как? ©
PM MAIL WWW   Вверх
Ignat
Дата 25.8.2006, 10:05 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


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


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

Репутация: 2
Всего: 73



Цитата(Vreden @  25.8.2006,  11:04 Найти цитируемый пост)
нууууууууууууу и кто-нибудь исправит? 

Вероятно, нет. Здесь переписывать надо  smile 


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


Idiotic Admin
**


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

Репутация: -1
Всего: 1



ну тогда больше код я здесь не буду показывать


--------------------
Чё? Как? ©
PM MAIL WWW   Вверх
Ignat
Дата 25.8.2006, 10:12 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


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


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

Репутация: 2
Всего: 73



Цитата(Vreden @  25.8.2006,  11:09 Найти цитируемый пост)
ну тогда больше код я здесь не буду показывать 

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


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


Idiotic Admin
**


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

Репутация: -1
Всего: 1



знал бы я как, все пишут мол это не так, это не так, а примеров кода, как сделать это не так - так!, не дают


--------------------
Чё? Как? ©
PM MAIL WWW   Вверх
Ignat
Дата 25.8.2006, 11:00 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


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


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

Репутация: 2
Всего: 73



Тогда начнем:
1. С какой целью перечислены все алфавитно-цифровые символы в регулярке?
2. Для чего служит оператор || , это я не увидел у вас в коде, но он там напрашивается.
3. Откуда навыки комментировать знаком # ?
4. Если есть повторы кода, то почему не используются функции?




--------------------
Теперь при чем :P
PM   Вверх
Ответ в темуСоздание новой темы Создание опроса
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | PHP: Избранное | Следующая тема »


 




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


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

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