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

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
if ($content->num_of_subscribers == 1)
	{
		$icon_num = getExternalThemeFile("media/babble1-blue.png");
	}
	if ($content->num_of_subscribers == 2)
	{
		$icon_num = getExternalThemeFile("media/babble2-blue.png");
	}
	if ($content->num_of_subscribers == 3)
	{
		$icon_num = getExternalThemeFile("media/babble3-blue.png");
	}
	if ($content->num_of_subscribers == 4)
	{
		$icon_num = getExternalThemeFile("media/babble4-blue.png");
	}
	if ($content->num_of_subscribers == 5)
	{
		$icon_num = getExternalThemeFile("media/babble5-blue.png");
	}
	if ($content->num_of_subscribers == 6)
	{
		$icon_num = getExternalThemeFile("media/babble6-blue.png");
	}
	if ($content->num_of_subscribers == 7)
	{
		$icon_num = getExternalThemeFile("media/babble7-blue.png");
	}
	if ($content->num_of_subscribers == 8)
	{
		$icon_num = getExternalThemeFile("media/babble8-blue.png");
	}
	if ($content->num_of_subscribers == 9)
	{
		$icon_num = getExternalThemeFile("media/babble9-blue.png");
	}
	if ($content->num_of_subscribers == 10)
	{
		$icon_num = getExternalThemeFile("media/babble10-blue.png");
	}
	if ($content->num_of_subscribers > 10)
	{
		$icon_num = getExternalThemeFile("media/babble11-blue.png");
	}

И ведь не лень было копи-пэстить, а потом и цифорки менять :)

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

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

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

    mra4niy_coder:
    то есть так:
    $icon_num = getExternalThemeFile("media/babble".$content->num_of_subscribers."-blue.png");
    да? =)
    Ответить

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

    зато секурно 😀 , не надо регэкспом прочищать от левых символов строку...
    Ответить

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

    Keeper:
    Можно было хотя бы так, хотя это тоже говнокод:

    switch ($content->num_of_subscribers) {
  case 1:
  case 2:
  ...
  case 10:
    $icon_num = getExternalThemeFile("media/babble".$content->num_of_subscribers."-blue.png");
    break;
  default:
    $icon_num = getExternalThemeFile("media/babble11-blue.png");
}
    Ответить

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

    Stam:
    к секурности отношения не имеет, ибо число береться из бд (каунтом).

    $icon_num = getExternalThemeFile("media/babble".min($content->num_of_subscribers, 11)."-blue.png");
    Ответить

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

Семь раз отмерь — один отрежь, guest!

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

    ↑
    8
    ↓