1. PHP / Говнокод #1823

    +150.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    // Выведем все ошибки на чистую воду :)
    error_reporting(E_ALL);
    
    /**
     * Рекурсивное изменение прав
     * @param string $path - Путь (файл или каталог)
     * @param int $filemode - Права, например 0755
     * @return - bool FALSE если не удалось, TRUE если удалось
     */
    function chmodTree($path, $filemode) {
      if (!is_dir($path)) return chmod($path, $filemode);
      $dir = opendir($path);
      while ($file = readdir($dir))
        if ($file != '.' && $file != '..') {
          $fullpath = $path.'/'.$file;
          if (is_link($fullpath)) return FALSE;
          if (is_dir($fullpath))
            if (!chmodTree($fullpath, $filemode)) return FALSE;
          if (is_file($fullpath))
            if (!chmod($fullpath, $filemode)) return FALSE;
        }
      closedir($dir);
      return chmod($path, $filemode);
    }
    
    // Функция file_put_contents появляется только в PHP 5.0 ;(
    if (!function_exists('file_put_contents')) {
      function file_put_contents($filename, $data) {
        if (!($f = @fopen($filename, 'w'))) {
          return FALSE;
        } else {
          $bytes = fwrite($f, $data);
          fclose($f);
          return $bytes;
        }
      }
    }
    
    /**
     * Получает файл с удалённого хоста, показывает и сохраняет в кеше
     * @param string $RemoteFile - файл на удалённой машине
     * @param string $LocalFile - локальный файл для сохранения
     */
    function CashedFile($RemoteFile, $LocalFile) {
      // Пытаемся получить файл с удалённого хоста
      $text = @file_get_contents($RemoteFile); // Игнорируем ошибки
      if ($text === FALSE) {
        // Если не удалось получить файл, тогда читаем копию из кеша
        $text = file_get_contents('cache/'.$LocalFile);
      } else {
        // Если каталога с кешем нет => создаём его
        if (!file_exists("cache"))mkdir("cache");
        // Ставим права на каталог
        chmodTree("cache", 0777);
        // Записываем полученный документ в файл
        file_put_contents('cache/'.$LocalFile, $text);
      }
      // Отдаём браузеру то что удалось скачать :)
      echo $text;
    }

    Кусок быстосляпанного кода для подгрузки частей сайта из Google Docs.
    Практически нечитаемый код, но работает, подгружает файлы с удаленного хоста, складывает их в каталог, потом оттуда читает.
    Непонятно, зачем нужна функция chmodTree, каталог cache же одноуровневый.

    Запостил: denis, 10 Сентября 2009

    Комментарии (13) RSS

    • аффтар кода убей сибя сразбегу
      Ответить
      • Почему?
        Ответить
        • патамушта тут чужой говнокод пинают а не советы спрашивают
          Ответить
          • ну а это мой говнокод и я хотел чтобы его попинали,
            а мож и объяснили хоть немного, как ПРАВИЛЬНО кодить на PHP :))
            Ни одной книжки толковой по PHP найти не могу.. Попробовал читать быдлокнижку "PHP глазами хакера", в ней полно ошибок (или просто опечаток) и советы очевидные только, не ясно, как всё сделать красиво
            Ответить
        • Не помешало бы провести реструктуризацию кода. Сделать его читаемей
          Ответить
          • Это как? :)
            Я только начинаю программировать на PHP и не понимаю пока его дзен.
            Пытался найти какой-нибудь сайт с примерами хорошей практики на PHP - не нашёл, нашёл только govnokod.
            Есть ли сайты с действительно хорошими советами, как писать большие проекты на PHP.
            Ответить
    • все в полне читаемое ... у меня есть нечто подобное ... может кому понравиться ;)
      ---------------
      static function menu_catalog_parent($parent_id,$array) {

      $return = false;

      $template = shell_catalog::_shell_menu_cat();

      self::$tab = self::$tab+20;

      foreach($array as $id => $parent) {

      if(isset($parent['parent_id-'.$parent_id])) {

      $parent['parent_id-'.$parent_id] = charset_x_win($parent['parent_id-'.$parent_id]);
      $id = str_replace('id-','',$id);

      $temp = $template[2];

      if(isset($_GET['cat']) && (int)$_GET['cat'] == $id) {
      $parent['parent_id-'.$parent_id] = '<b>'.$parent['parent_id-'.$parent_id].'</b>';
      $temp = str_replace('[BG]',$template[3],$temp);
      if(!preg_match('/(,'.$id.')/',$_SESSION['level_menu_cat']))
      $_SESSION['level_menu_cat'] .= ','.$id;
      } else {
      $temp = str_replace('[BG]','',$temp);
      }

      $temp = str_replace('[ID]',$id,$temp);
      $temp = str_replace('[NAME]',$parent['parent_id-'.$parent_id],$temp);
      $return .= $temp;

      $parent_cat = catalog::menu_catalog_parent($id,$array) ;

      if($parent_cat) {
      $parent_cat = str_replace('[CAT]',$parent_cat,$template[4]);
      $parent_cat = str_replace('[ID]',$id,$parent_cat);
      $return .= $parent_cat;
      }

      }

      }

      return $return;

      }
      Ответить
    • Мне кажется, вполне читаемый код. А зачем для chmod такой слон? chmod -R через system позвать было бы проще.
      Ответить
      • нет, не проще.

        C:\Users\TWINc>chmod
        'chmod' is not recognized as an internal or external command,
        operable program or batch file.
        Ответить
    • По коду комментарии дебильные... какой смысл тупо писать, что делает каждая строчка?

      <?php
      $i = 0; // устанавиваем i в ноль
      $++; // увеличиваем i на 1
      //...
      Ответить

    Добавить комментарий