"PHP" / Говнокод #2806 Ссылка на оригинал

0

  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 Sergunik, (Updated )

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

    • ava Sergunik # Ссылка на оригинал 0 ⇈

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

  • ava guest # Ссылка на оригинал 0

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

  • ava guest # Ссылка на оригинал 0

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

    • ava Sergunik # Ссылка на оригинал 0 ⇈

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

  • ava guest # Ссылка на оригинал 0

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

  • ava nur # Ссылка на оригинал 0

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

  • ava guest # Ссылка на оригинал 0

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

  • ava leprosus # Ссылка на оригинал 0

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

  • ava guest # Ссылка на оригинал 0

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

    • ava guest # Ссылка на оригинал 0 ⇈

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

      • ava guest # Ссылка на оригинал 0 ⇈

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

    • ava guest # Ссылка на оригинал 0 ⇈

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

  • ava guest # Ссылка на оригинал 0

    ололо а вот тот самый 'илитный код' нашего сергуника (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 # Ссылка на оригинал 0 ⇈

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

      • ava guest # Ссылка на оригинал 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 # Ссылка на оригинал 0

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

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

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓