常见的编码陷阱3

常见的编码陷阱

6.避免三元冗余

在JavaScript和PHP中，过度使用三元语句是很常见的事情：

1         //javascript

2         returnfoo.toString()!==""?true:false;

3         //php

4         return(something())?true:false;

条件判断的返回值永远只有false和true，言外之意就是你无需把true和false显示添加到三元运算中。相反，你只需简单的返回条件：

5         //javascript

6         returnfoo.toString()!=="";

7         //php

8         returnsomething();

PHP篇

7.适当的时候使用三元操作

If...else语句是大多数语言的重要组成部分。但有些简单的事情，比如根据条件进行赋值，你很有可能会这样写：

9         if($greeting)

10     {

11     $post->message='Hello';

12     }

13     else

14     {

15     $post->message='Goodbye';

16     }

其实使用三元操作只需一行代码就可以搞定，并保持了良好的可读性：

17     $post->message=$greeting?'Hello':'Goodbye';

8.抛出异常，而不是采用盗梦空间式的嵌套（Inception-Style Nesting）

多层次的嵌套是丑陋的、难以维护和不可读的。下面的代码是个简单的例子，但是随着时间的推移会变得更糟：

18     //anti-pattern

19     $error_message=null;

20     if($this->form_validation->run())

21     {

22     if($this->upload->do_upload())

23     {

24     $image=$this->upload->get_info();

25     if(!$this->image->create_thumbnail($image['file_name'],300,150))

26     {

27     $error_message='Therewasanerrorcreatingthethumbnail.';

28     }

29     }

30     else

31     {

32     $error_message='Therewasanerroruploadingtheimage.';

33     }

34     }

35     else

36     {

37     $error_message=$this->form_validation->error_string();

38     }

39     //Showerrormessages

40     if($error_message!==null)

41     {

42     $this->load->view('form',array(

43     'error'=>$error_message,

44     ));

45     }

46     //Savethepage

47     else

48     {

49     $some_data['image']=$image['file_name'];

50     $this->some_model->save($some_data);

51     }

如此凌乱的代码，是否该整理下呢。建议大家使用异常这个清洁剂：

52     try

53     {

54     if(!$this->form_validation->run())

55     {

56     thrownewException($this->form_validation->error_string());

57     }

58     if(!$this->upload->do_upload())

59     {

60     thrownewException('Therewasanerroruploadingtheimage.');

61     }

62     $image=$this->upload->get_info();

63     if(!$this->image->create_thumbnail($image['file_name'],300,150))

64     {

65     thrownewException('Therewasanerrorcreatingthethumbnail.');

66     }

67     }

68     //Showerrormessages

69     catch(Exception$e)

70     {

71     $this->load->view('form',array(

72     'error'=>$e->getMessage(),

73     ));

74     //Stopmethodexecutionwithreturn,oruseexit

75     return;

76     }

77     //Gotthisfar,mustnothaveanytrouble

78     $some_data['image']=$image['file_name'];

79     $this->some_model->save($some_data);

虽然代码行数并未改变，但它拥有更好的可维护性和可读性。尽量保持代码简单。