Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/topic.php on line 737
Форумы портала PHP.SU :: Как лучше писать код?

 PHP.SU

Программирование на PHP, MySQL и другие веб-технологии
PHP.SU Портал     На главную страницу форума Главная     Помощь Помощь     Поиск Поиск     Поиск Яндекс Поиск Яндекс     Вакансии  Пользователи Пользователи


 Страниц (1): [1]   

> Без описания
event
Отправлено: 10 Марта, 2012 - 22:18:29
Post Id


Частый посетитель


Покинул форум
Сообщений всего: 542
Дата рег-ции: Май 2011  


Помог: 0 раз(а)




Привет, какой из ниже наведенных вариантов лучше, первый или второй?
PHP:
скопировать код в буфер обмена
  1. <?PHP
  2.  
  3. # ... 1 вариант ... #
  4.  
  5. $action = $_GET["action"];
  6.  
  7. if ($action == 'one'){
  8.        
  9. # ... тут какой-то код ... #
  10.        
  11. }
  12.  
  13. if ($action == 'two'){
  14.  
  15. # ... тут какой-то код ... #
  16.  
  17. }
  18.  
  19. elseif ($action == 'three'){
  20.        
  21. # ... тут какой-то код ... #
  22.  
  23. }
  24.  
  25. ?>
  26.  
  27.  
  28. <?PHP
  29.  
  30. # ... 2 вариант ... #
  31.  
  32. $action = $_GET['action'];
  33.  
  34. switch ($action){
  35.        
  36. case "one":
  37.  
  38. # ... тут какой-то код ... #
  39.                
  40. break;
  41.        
  42.         switch ($action){
  43.        
  44. case "two":
  45.  
  46. # ... тут какой-то код ... #
  47.                
  48. break;
  49.  
  50. switch ($action){
  51.        
  52. case "three":
  53.  
  54. # ... тут какой-то код ... #
  55.                
  56. break;
  57.  
  58. }
  59.  
  60.  
  61. ?>
 
 Top
LIME
Отправлено: 10 Марта, 2012 - 22:23:16
Post Id


Активный участник


Покинул форум
Сообщений всего: 10732
Дата рег-ции: Нояб. 2010  


Помог: 322 раз(а)




не принципиально
кому как нравится
если внутри пункта много кода то от свитча толку мало
он только читаемость повышает
 
 Top
event
Отправлено: 10 Марта, 2012 - 22:28:49
Post Id


Частый посетитель


Покинул форум
Сообщений всего: 542
Дата рег-ции: Май 2011  


Помог: 0 раз(а)




А почему толку мало, если кода в свитче много?)
 
 Top
LIME
Отправлено: 10 Марта, 2012 - 22:32:38
Post Id


Активный участник


Покинул форум
Сообщений всего: 10732
Дата рег-ции: Нояб. 2010  


Помог: 322 раз(а)




потому-что всеравно листать придется
удобства не прибавится
 
 Top
event
Отправлено: 10 Марта, 2012 - 22:37:21
Post Id


Частый посетитель


Покинул форум
Сообщений всего: 542
Дата рег-ции: Май 2011  


Помог: 0 раз(а)




ясн, получается, + только в читаемости?
 
 Top
Okula
Отправлено: 10 Марта, 2012 - 22:50:41
Post Id



Участник


Покинул форум
Сообщений всего: 1389
Дата рег-ции: Окт. 2010  


Помог: 42 раз(а)




В таких ситуациях предпочитаю использовать конструкцию switch.
Я нашёл + в том что не нужно дополнительно в фигурные скобки обрамлять.

(Отредактировано автором: 11 Марта, 2012 - 00:01:10)

 
 Top
DelphinPRO
Отправлено: 10 Марта, 2012 - 22:51:03
Post Id



Активный участник


Покинул форум
Сообщений всего: 7187
Дата рег-ции: Февр. 2012  


Помог: 353 раз(а)




можно еще так Улыбка

PHP:
скопировать код в буфер обмена
  1.  
  2. $action = $_GET["action"];
  3. call_user_func($action);
  4.  
  5. function one(){
  6.          # ... тут какой-то код ... #
  7. }
  8.  
  9. function two(){
  10.          # ... тут какой-то код ... #
  11. }
  12.  
  13. function three(){
  14.          # ... тут какой-то код ... #
  15. }
  16.  
  17.  


-----
Чем больше узнаю, тем больше я не знаю.
 
 Top
LIME
Отправлено: 10 Марта, 2012 - 23:12:14
Post Id


Активный участник


Покинул форум
Сообщений всего: 10732
Дата рег-ции: Нояб. 2010  


Помог: 322 раз(а)




Okula пишет:
Я нашёл + в том что не нужно дополнительно в кавычки обрамлять.
может имелись ввиду фигурные скобки?
тогда {} VS : break ))
(Добавление)
event кстати у вас неправильный синтаксис свитча
 
 Top
Okula
Отправлено: 11 Марта, 2012 - 00:00:30
Post Id



Участник


Покинул форум
Сообщений всего: 1389
Дата рег-ции: Окт. 2010  


Помог: 42 раз(а)




LIME пишет:
может имелись ввиду фигурные скобки?

ага, думал ободном, а написал другое Улыбка))
 
 Top
event
Отправлено: 11 Марта, 2012 - 00:11:04
Post Id


Частый посетитель


Покинул форум
Сообщений всего: 542
Дата рег-ции: Май 2011  


Помог: 0 раз(а)




LIME
Где именно косяк?
 
 Top
Bio man
Отправлено: 11 Марта, 2012 - 00:18:45
Post Id


Постоянный участник


Покинул форум
Сообщений всего: 2751
Дата рег-ции: Июль 2010  
Откуда: Даугавпилс, Латвия


Помог: 52 раз(а)




PHP:
скопировать код в буфер обмена
  1. switch ($action){
  2.        
  3. case "one":
  4.  
  5. # ... тут какой-то код ... #
  6.              
  7. break;
  8.        
  9. case "two":
  10.  
  11. # ... тут какой-то код ... #
  12.              
  13. break;
  14.        
  15. case "three":
  16.  
  17. # ... тут какой-то код ... #
  18.              
  19. break;
  20.  
  21. }
 
 Top
Страниц (1): [1]
Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)
« Вопросы новичков »


Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.
 



Powered by PHP  Powered By MySQL  Powered by Nginx  Valid CSS  RSS

 
Powered by ExBB FM 1.0 RC1. InvisionExBB