За последние 24 часа нас посетил 59171 программист и 1810 роботов. Сейчас ищет 961 программист ...

Безопасность сайта

Тема в разделе "PHP для новичков", создана пользователем Yahoo Yeyoo!, 12 сен 2007.

  1. 440Hz

    440Hz Старожил
    Команда форума Модератор

    С нами с:
    21 дек 2012
    Сообщения:
    8.003
    Симпатии:
    1
    Адрес:
    Оттуда
    редкое свойство - думать. видно не все поряно еще.
    =)

    +1
     
  2. Yahoo Yeyoo!

    Yahoo Yeyoo! Активный пользователь

    С нами с:
    2 май 2007
    Сообщения:
    111
    Симпатии:
    0
    Адрес:
    г.Жуковский
    ну собственно да, пока это так называется
     
  3. 440Hz

    440Hz Старожил
    Команда форума Модератор

    С нами с:
    21 дек 2012
    Сообщения:
    8.003
    Симпатии:
    1
    Адрес:
    Оттуда
    найди отличия?

    было:
    PHP:
    1.  
    2. <?
    3.  
    4. if(!isset($_SESSION['mode'])){$_SESSION['mode'] = "client";}
    5. require_once("function_list.php");
    6. if(!isset($_GET['act']) and !isset($_GET['mannage'])){
    7. if(file_exists("design.htm")){
    8. require_once("design.htm");
    9. }else require_once("test_design.htm");
    10. }
    11. require_once("actions.php");
    12. ?>
    стало:
    PHP:
    1.  
    2. <?
    3.  
    4. /**
    5. * головной файл
    6. *
    7. * бла-бла-бла поподробнее
    8. *
    9. * @copyright я и еще раз я
    10. *
    11. */
    12.  
    13. // стартуем сессию
    14.  
    15. // если нет моды - тогда клиент
    16. if( !isset($_SESSION['mode']) ) {
    17.     $_SESSION['mode'] = "client";
    18. }
    19.  
    20. // подключаем фнукции
    21. require_once("function_list.php");
    22.  
    23. // если не нет экшена или менеджера
    24. if( !isset($_GET['act']) AND !isset($_GET['mannage']) ) {
    25.     if( file_exists("design.htm") ) {
    26.         // если есть файл дизайна
    27.         require_once("design.htm");
    28.     } else {
    29.         // тогда тестовый дизайнг
    30.         require_once("test_design.htm");
    31.     }
    32. }
    33.  
    34. // поключаем выдачу страниц
    35. require_once("actions.php");
    36.  
    37. ?>
    ты пойми, твой код будут читать другие люди и очевидные для тебя вещи - для них будут открытием.
    уважай себя и их.
    уважай свой труд и труд чужих.
     
  4. Yahoo Yeyoo!

    Yahoo Yeyoo! Активный пользователь

    С нами с:
    2 май 2007
    Сообщения:
    111
    Симпатии:
    0
    Адрес:
    г.Жуковский
    аа, так дело только в каментах?
     
  5. 440Hz

    440Hz Старожил
    Команда форума Модератор

    С нами с:
    21 дек 2012
    Сообщения:
    8.003
    Симпатии:
    1
    Адрес:
    Оттуда
    блин... ты тупить-то заканчивай...

    я говорю о принципах, которые должны воспитывать в себе те, кто хочет профессионально заниматься программированием, а не слепить пару сайтов и пойти торговать картошкой.

    я бы, получив такой код, даже читать бы его не стал. читаю исключительно что б ты, дружок, научится для начала уважению, а написаить 100К говняного, никому не нужного кода, может почти каждый.
     
  6. Yahoo Yeyoo!

    Yahoo Yeyoo! Активный пользователь

    С нами с:
    2 май 2007
    Сообщения:
    111
    Симпатии:
    0
    Адрес:
    г.Жуковский
    так давай без выражений,
    уж кто бы говорил о неуважении, насколько я понимаю, я к тебе без хамства обращасюсь, тему поднял специально в форуме для новичков, чтобы узнать минусы кода, а вообще в чем выражается мое неуважение?
     
  7. 440Hz

    440Hz Старожил
    Команда форума Модератор

    С нами с:
    21 дек 2012
    Сообщения:
    8.003
    Симпатии:
    1
    Адрес:
    Оттуда
    в том коде, который я от тебя получил.
    что не понятного?

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

    я всегда готов помочь, тем, кто нуждается в помощи.
     
  8. Hawk

    Hawk Активный пользователь

    С нами с:
    30 авг 2007
    Сообщения:
    201
    Симпатии:
    0
    Адрес:
    Беларусь
    Yahoo Yeyoo! и 440Hz верно говорит помог не раз, форум дружелюбный главное к людям нормально относится;)
     
  9. Anonymous

    Anonymous Guest

    ога. Хуже чем 440hz токо я =)
     
  10. Yahoo Yeyoo!

    Yahoo Yeyoo! Активный пользователь

    С нами с:
    2 май 2007
    Сообщения:
    111
    Симпатии:
    0
    Адрес:
    г.Жуковский
    однако я даже не знал что кодом можно проявлять столь глубокое и наглое неуважение, кое ты увидел. если честно, не вижу никакой связи.

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

    Вообще исходя из того что код сырой, я обратился за помощью, чтобы показали недостатки. а тут все дело свелось к тому, якобы я тут всех обсираю, и неуважительно отношусь.
     
  11. 440Hz

    440Hz Старожил
    Команда форума Модератор

    С нами с:
    21 дек 2012
    Сообщения:
    8.003
    Симпатии:
    1
    Адрес:
    Оттуда
    предлагаю проехать и забыть.
    лучше будем друг другу помогать...
     
  12. Yahoo Yeyoo!

    Yahoo Yeyoo! Активный пользователь

    С нами с:
    2 май 2007
    Сообщения:
    111
    Симпатии:
    0
    Адрес:
    г.Жуковский
    .
    Согласен, просто недоразумение.
    Так всетки, скажите кто что думает, как лучше сделать исходник. именно потому что он сырой я и обращаюсь сюда
     
  13. 440Hz

    440Hz Старожил
    Команда форума Модератор

    С нами с:
    21 дек 2012
    Сообщения:
    8.003
    Симпатии:
    1
    Адрес:
    Оттуда
    ну для начала начать форматировать и коментить код.
    =)

    переделать и прислать мне...
     
  14. Yahoo Yeyoo!

    Yahoo Yeyoo! Активный пользователь

    С нами с:
    2 май 2007
    Сообщения:
    111
    Симпатии:
    0
    Адрес:
    г.Жуковский
    окей :) а на мыло прислать?
     
  15. 440Hz

    440Hz Старожил
    Команда форума Модератор

    С нами с:
    21 дек 2012
    Сообщения:
    8.003
    Симпатии:
    1
    Адрес:
    Оттуда
    да. я посмотрю и займемся организацией файловой структуры и архитектурой приложения.
    весчь очень нужная, хотя мало кто про это пишет и говорит.
     
  16. Luge

    Luge Старожил

    С нами с:
    2 фев 2007
    Сообщения:
    4.680
    Симпатии:
    1
    Адрес:
    Минск
    Yahoo Yeyoo!
    PHP:
    1. <?php
    2.     $safe_name= mysql_escape_string($_POST['login']);
    3.     $safe_pass = mysql_escape_string($_POST['pass']);
    4.     $safe_pass_n = md5($safe_pass);
    5.     require_once('config.php');
    6.     if($safe_name !== $config['adm_user'] and $safe_pass_n !== md5($config['adm_pass']))
    7. ?>
    если у меня в пароле будет кавычка, то пароли не совпадут. Экранировать символы в пароле не надо.
     
  17. Yahoo Yeyoo!

    Yahoo Yeyoo! Активный пользователь

    С нами с:
    2 май 2007
    Сообщения:
    111
    Симпатии:
    0
    Адрес:
    г.Жуковский
    Спасибо, я не обратил внимание на это
     
  18. Luge

    Luge Старожил

    С нами с:
    2 фев 2007
    Сообщения:
    4.680
    Симпатии:
    1
    Адрес:
    Минск
    PHP:
    1. <?php if(mysql_query($sql) or die(myssql_error())){  ?>
    :)
    любое действие из actions.php при обновлении страницы будет выполнено повторно

    а вообще действительно, очень тяжело неотформаченный код читать
     
  19. Yahoo Yeyoo!

    Yahoo Yeyoo! Активный пользователь

    С нами с:
    2 май 2007
    Сообщения:
    111
    Симпатии:
    0
    Адрес:
    г.Жуковский
    я поправлю и выложу повторно, сейчас на работе, немогу этим заняться
     
  20. 440Hz

    440Hz Старожил
    Команда форума Модератор

    С нами с:
    21 дек 2012
    Сообщения:
    8.003
    Симпатии:
    1
    Адрес:
    Оттуда
    я предлагаю дать человеку отформатировать код, закоментить его, а потом будем править?
     
  21. Luge

    Luge Старожил

    С нами с:
    2 фев 2007
    Сообщения:
    4.680
    Симпатии:
    1
    Адрес:
    Минск
    так я же не против :) Просто что сразу бросилось в глаза, то и написал.
     
  22. 440Hz

    440Hz Старожил
    Команда форума Модератор

    С нами с:
    21 дек 2012
    Сообщения:
    8.003
    Симпатии:
    1
    Адрес:
    Оттуда
    а я ниче не разобрал... толи плохо вижу, старый стал, толи код в мешанине...

    =)
     
  23. Luge

    Luge Старожил

    С нами с:
    2 фев 2007
    Сообщения:
    4.680
    Симпатии:
    1
    Адрес:
    Минск
    а мне уже после прочтения скрипта написанного в одну (1!) строчку, который, после форматирования и избавления от echo через каждые 15-20 символов, имел 500 строк уже ничего не страшно :D
     
  24. 440Hz

    440Hz Старожил
    Команда форума Модератор

    С нами с:
    21 дек 2012
    Сообщения:
    8.003
    Симпатии:
    1
    Адрес:
    Оттуда
    ты еще регулярки не отлаживал, малыш...
    =)
     
  25. Luge

    Luge Старожил

    С нами с:
    2 фев 2007
    Сообщения:
    4.680
    Симпатии:
    1
    Адрес:
    Минск
    от почему всегда всё к мерянью пиписьками сводится? :(