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

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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
if (!empty($forum_actions))
{

?>
<div class="linksb">
	<div class="inbox crumbsplus">
		<p class="subscribelink clearb"><?php echo implode(' - ', $forum_actions); ?></p>
	</div>
</div>
<?php

}

?>
<div id="brdstats" class="block">
	<h2><span><?php echo $lang_index['Board info'] ?></span></h2>
	<div class="box">
		<div class="inbox">
			<dl class="conr">
				<dt><strong><?php echo $lang_index['Board stats'] ?></strong></dt>
				<dd><span><?php printf($lang_index['No of users'], '<strong>'.forum_number_format($stats['total_users']).'</strong>') ?></span></dd>
				<dd><span><?php printf($lang_index['No of topics'], '<strong>'.forum_number_format($stats['total_topics']).'</strong>') ?></span></dd>
				<dd><span><?php printf($lang_index['No of posts'], '<strong>'.forum_number_format($stats['total_posts']).'</strong>') ?></span></dd>
			</dl>
			<dl class="conl">
				<dt><strong><?php echo $lang_index['User info'] ?></strong></dt>
				<dd><span><?php printf($lang_index['Newest user'], $stats['newest_user']) ?></span></dd>
<?php

if ($pun_config['o_users_online'] == '1')
{
	// Fetch users online info and generate strings for output
	$num_guests = 0;
	$users = array();
	$result = $db->query('SELECT user_id, ident FROM '.$db->prefix.'online WHERE idle=0 ORDER BY ident', true) or error('Unable to fetch online list', __FILE__, __LINE__, $db->error());

	while ($pun_user_online = $db->fetch_assoc($result))
	{
		if ($pun_user_online['user_id'] > 1)
		{
			if ($pun_user['g_view_users'] == '1')
				$users[] = "\n\t\t\t\t".'<dd><a href="profile.php?id='.$pun_user_online['user_id'].'">'.pun_htmlspecialchars($pun_user_online['ident']).'</a>';
			else
				$users[] = "\n\t\t\t\t".'<dd>'.pun_htmlspecialchars($pun_user_online['ident']);
		}
		else
			++$num_guests;
	}

главная страница форумного движка fluxbb
moar тут по сцылке https://github.com/fluxbb/fluxbb/blob/fluxbb-1.4/index.php

Блядь... зато скока у них там этого самого прекрасно пафоса про секурность и прочие ништяки.

Имхо пора на написание опенсорсного говнокода давать лицензию сродни водительским правам

Запостил: brainstorm brainstorm, (Updated )

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

    • мешанина пхп и хтмл.
      нет префикса БД - как правило в движках БД нормальных систем таблицы ставят в спец. скобки и нет вот этой хуйни:
      'SELECT user_id, ident FROM '.$db->prefix.'online
      если $db хранит префикс - то почему бы не встроит его обработку внутрь слоя БД?
      говнокод короче.
      Ответить
      • Есть строка запроса 'SELECT * FROM `online` LEFT JOIN `offline` USING `my_field` WHERE 10050'.
        Засовывание префикса перед online и offline будет ещё худшим говнокодом.
        Ответить
        • в друпал эта строка выглядит так
          db_query('SELECT * FROM {online} LEFT JOIN {offline} USING `my_field` WHERE ......

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

        А с базой реальное говнище, эталонное.
        Но говнокод с базой укладывается в одну строку, нахуя 50-то постить ?
        Ответить
        • я урл дал. там моар этого. говнокод равномерно размазан по файлу.
          насчет мешанины - СМОТРЯ ГДЕ.
          есть функции темизации в друпал например - для вывода массивов и тд - да ниче против не имаю что в строку html складывают и возвращают. но не такое же как тут.....
          Ответить
      • В огороде бузина, а в Киеве дядька...

        Про секурность скажите чего или нет?
        Ответить
  • если бы "давать лицензию" - то php бы не существовало...
    Ответить
    • c dl dd все ясно в общем то - иногда надо делать таблицы с "плавающей" длинной - чтобы картинками обтекались и тд. вот для этого фокуса игра с ними пользительна.
      Ответить
        • опечатка.
          ну у меня былы задача сделать аналог table который бы обтекался внешними картинками. dd dl это делают на ура.
          Ответить
  • "$pun_user_online" - хехе, что-то мне кажется, что код частично спижжен с punbb.
    когда-то сам ковырялся в этом движке, там были как раз такие названия...
    Ответить
    • да походу все чиста форумные движки в коде являют адЪ
      мне из всех в свое время jforum оч понравился. но он сука на жабе.
      Ответить
    • ...Если бы твоя ветреная мамаша вовремя придушила тебя в детстве подгузником, ты...
      ☑ Не стал бы частью социума;
      ☑ Не стал бы частью антисоциума;
      ☑ Не пользовался бы канализацией;
      ☑ Не потреблял бы кислород и продукты питания;
      Не спамил бы ГК
      Ответить

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

Где здесь C++, guest?!

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


    8