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

    +164

    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
    function addnews($str1="", $str2="", $str3="", $str4="", $str5=""){
  $num = 0;
  $done = 0;
  while ($done == 0){
    $num++;
    $s = "";
    if ($num<10000000) {$s="0".$s;}
    if ($num<1000000)  {$s="0".$s;}
    if ($num<100000)   {$s="0".$s;}
    if ($num<10000)    {$s="0".$s;}
    if ($num<1000)     {$s="0".$s;}
    if ($num<100)      {$s="0".$s;}
    if ($num<10)       {$s="0".$s;}
	$done = 1;
	if (file_exists("news/".$s.$num.".txt")){$done=0;}
  }
  $fh1 =fopen("news/".$s.$num.".txt","w");
  fwrite($fh1, $str1."\r\n");
  fwrite($fh1, $str2."\r\n");
  fwrite($fh1, $str3."\r\n");
  fwrite($fh1, $str4."\r\n");
  fwrite($fh1, $str5);
  fclose($fh1);
}

    Предложили доработать корпоративную тикет-систему. Движок абсолютно всё хранит в txt-файлах. Да, и пароли пользователей тоже - в открытом виде. БД? Нет, не слышали :(
    Но это еще не так страшно...

    Запостил: Arris, 26 Марта 2015

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

    • ava Lokich 26.03.2015 20:57 # +1

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

      • ava Arris 26.03.2015 21:20 # 0

        На самом деле эта функция - не говнокод :) Я вот сейчас нашел другую... побольше. Не могу сказать, что она совсем уж говнокодистая, но... НО НАХРЕНА так извращаться? О.о
        Ответить

        • ava Lokich 26.03.2015 21:29 # 0

          да проблема в том, что это файлы.
          пока кто-то пишет что-то в файл, другие его ждут. конкурентный доступ? не, не слышал
          Ответить

          • ava bormand 26.03.2015 21:36 # 0

            > пока кто-то пишет что-то в файл, другие его ждут.
            Судя по коду, тут по файлу на запись. Так что конкурентность будет не хуже, чем в СУБД.
            Ответить

            • ava Arris 26.03.2015 21:56 # 0

              по одному файлу на каждый... кусочек информации - пользователь, таск-тикет, новость, сообщение в ленте итд итп.

              Да, а вот тикет и комментарии к нему лежат в одном файле.
              Ответить

            • ava Lokich 26.03.2015 22:33 # +1

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

              • ava Stallman 26.03.2015 22:52 # −3

                >распараллелить
                >PHP
                >распараллелить в один поток
                /0

                # Тред не читал, пыхобляди соснули.
                Ответить

                • ava bormand 26.03.2015 23:19 # +1

                  Дык там же новый инстанс интерпретатора на один запрос и никакой связи между инстансами. Идеальная обстановочка для распараллеливания.
                  Ответить

              • ava bormand 26.03.2015 23:21 # 0

                > это не значит что нет узких мест
                Ты так говоришь, как будто СУБД - это серебряная пуля, волшебным образом спасающая от всех узких мест.
                Ответить

                • ava Lokich 27.03.2015 15:37 # +1

                  конечно же это не панацея, но это лучше такой реализации
                  Ответить

            • ava kegdan 27.03.2015 22:19 # 0

              но не скорость
              Ответить

        • ava Vasiliy 27.03.2015 09:13 # +2

          >if ($num<10000000) {$s="0".$s;}
          if ($num<1000000) {$s="0".$s;}
          if ($num<100000) {$s="0".$s;}
          if ($num<10000) {$s="0".$s;}
          if ($num<1000) {$s="0".$s;}
          if ($num<100) {$s="0".$s;}
          if ($num<10) {$s="0".$s;}
          >На самом деле эта функция - не говнокод :)
          у меня разрыв шаблона
          Ответить

          • ava bormand 27.03.2015 09:28 # 0

            Это хуйня. Вот где истинное говно:
            while ($done == 0){
    $num++;
            Ответить

            • ava Lokich 27.03.2015 15:14 # 0 

              if (file_exists("news/".$s.$num.".txt")){$done=0;}

              а это типа совсем не хуйня?
              Ответить

              • ava Arris 27.03.2015 22:13 # +1

                Может быть мне выложить полный код файла const.php ? :)
                Ответить

          • ava Arris 27.03.2015 11:21 # 0

            по сравнению стем, что я увидел дальше - поверьте - не говнокод :)
            Ответить

        • ava Dev_18 12.05.2015 19:57 # 0

          Недавно познакомился с такой функцией как crypt() и с PDO.
          Ответить

          • ava bormand 12.05.2015 19:59 # 0

            > crypt()
            Поздновато, уже пора знакомиться с password_hash() и password_verify().
            Ответить

            • ava Arris 12.05.2015 20:07 # 0

              Нигатишно! Велосипеды надо самому изобретать, иначе жить не интересно!
              Ответить

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