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

    +144

    1. 1
    2. 2
    3. 3
    4. 4
    if (mysql_num_rows($res)) {while($row = mysql_fetch_assoc($res)) {
   $aSingStat[] = $row;
   }
}

    толи строчек жалко, толи решил обфусцыровать код для меньшей узнаваемости.
    написал уже б так: if (mysql_num_rows($res)) while($row = mysql_fetch_assoc($res)) $aSingStat[] = $row;
    Тогда б я вообще не парился и дропнул нафиг строчку.

    Андрюха, не пиши больше так!
    Вот пример как надо писать http://www.sergunik.name/?p=178

    Запостил: Sergunik, 16 Марта 2010

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

    • ava murz 16.03.2010 18:41 # +1.8

      Решил пропиарить сайтик? ;)
      Ответить

      • ava Sergunik 16.03.2010 18:48 # −2.4

        никак нет)
        Если Админам будет удобней - то пускай возьмут кусок кода (который под ссылкой) и добавлят его в комент. А ссылку дропнут.
        Я ничуть не обижусь.
        Ответить

    • ava guest 16.03.2010 19:12 # +0.2

      > Вот пример как надо писать
      Категорично до невозможности. Алсо, не говнокод.
      Ответить

    • ava guest 16.03.2010 19:12 # +1

      Автор сам написал эту строчку, не приводя безчислених примере такого его стиля кодження в проекте. В результате сам забыл свой стиль кодження. А причина почему он так глуметься из других людей это заниженная самооценка. И комплекс недоразвития. P.S. Я работаю на одной фирме с автором этого сообщения.
      Ответить

      • ava Sergunik 16.03.2010 19:17 # −1.4

        Андруха, почему не авторизовался, а с под гостя пишешь?
        я же тебе говорил, что это тебе "для общего развития" написал, а ти обиделся.
        не умеешь ты критику воспринимать(
        Ответить

      • ava striker 17.03.2010 00:37 # 0

        ёма, не ругайтесь :)
        Ответить

    • ava guest 16.03.2010 19:29 # −1

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

    • ava nur 16.03.2010 20:02 # +2.4

      хохлы ссорятся) как мило=)
      зы сабж не говнокод, просто стиль программирования немного отличен от всеми приемлемого
      Ответить

    • ava mrbig66 16.03.2010 20:18 # +0.4

      Кто эти люди?
      Ответить

    • ava guest 16.03.2010 20:26 # +3

      Обалдеть, мы стали свидетелями чужой личной ДРАМЫ =) Ребят, вы бы это... решали свои проблемы в своей локальной области видимости =)
      Ответить

    • ava leprosus 16.03.2010 23:35 # +0.2

      Товарищи прогеры, не нужно личные ссоры переносить на этот сайт.
      IMHO: это не говнокод!
      FYI: и этот сайт не связан с продажей кЭпок [головной убор], сЭмок [еда гопников] =)
      Так что, свои разборки разрешайте на других площадках.
      Ответить

    • ava guest 17.03.2010 02:46 # +1.2

      Юзайте ООП, или хотябы классы для работы с БД, чтобы хотя бы не постить один и тоже в другом формате ...
      Ответить

      • ava guest 17.03.2010 05:53 # −1

        был бы еще в ПоцХаПэ нормальный ООП а не унылая пародия на него
        Ответить

        • ava guest 17.03.2010 08:03 # +0.8

          Либо ты отстал от жизни и не знаешь, что в PHP5 ООП уже вполне сносное, либо обоснуй и скажи, по сравнению с чем.
          Ответить

      • ava guest 17.03.2010 08:05 # +1

        Куда деваться, некоторые не пользуются ООП, потому что лень выучить, некоторые просто его не любят. Для простых задач оно и не нужно, ящитаю. Хотя лично я, изучив его, о процедурном стиле вспоминать больше не горю желанием.
        Ответить

    • ava guest 17.03.2010 05:58 # +1.4

      ололо а вот тот самый 'илитный код' нашего сергуника (http://www.sergunik.name/?p=178)
      function fetchRows($sQuery) {
  $res = mysql_query($sQuery);
  $iNumRows = mysql_num_rows($res);
  $aRet = array();
  if ($iNumRows) {
    while ($iNumRows-- > 0) {
      $aRet[] = mysql_fetch_assoc($res);
    }
  } else {
    $aRet = false;
  }
  mysql_free_result($res);

  return $aRet;
}

$aData = fetchRow('SELECT * FROM `users`');
      Ответить

      • ava guest 17.03.2010 08:02 # 0

        Хехе, что-то многовато тут лишних телодвижений.
        Ответить

      • ava mindfreakthemon 17.03.2010 08:19 # 0

        $res = mysql_query($sQuery);
        while($array[] = mysql_fetch_assoc($res) or array_pop($array));

        о, да.
        Ответить

        • ava guest 17.03.2010 11:32 # 0

          Оригинальненько и компактненько, но использовать бы не стал. Телодвижение с извлечением false-элемента массива тут таки лишнее. И да, присоединюсь к стройному ряду велосипедистов, а чё нет-та =3
          function govnoFunction($query)
{
  if (false === ($r = @mysql_query($query)))
  {
    return false;
  }
  
  $out = array();
  
  while ($row = mysql_fetch_assoc($r))
  {
    $out[] = $row;
  }
  
  mysql_free_result($out);
  return $out;
}
          Ответить

    • ava guest 17.03.2010 11:23 # +0.6

      ни поста, ни комментов не читал, ставлю минус, потому что я быдло и куда толпа, туда и я.
      Ответить

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