Getters must not throw exceptions CS-R1038
Anti-pattern
Major
11 days ago
—
20 days old
Exception being thrown from getter
67
68 public PHPExcel writes
69 {
70 get71 {72 throw new Exception("The method or operation is not implemented.");73 }74 set
75 {
76 throw new Exception("The method or operation is not implemented.");Exception being thrown from getter
48
49 public PHPExcel writes
50 {
51 get52 {53 throw new Exception("The method or operation is not implemented.");54 }55 set
56 {
57 throw new Exception("The method or operation is not implemented.");Exception being thrown from getter
29
30 public PHPExcel writes
31 {
32 get33 {34 throw new Exception("The method or operation is not implemented.");35 }36 set
37 {
38 throw new Exception("The method or operation is not implemented.");Exception being thrown from getter
10
11 public PHPExcel writes
12 {
13 get14 {15 throw new Exception("The method or operation is not implemented.");16 }17 set
18 {
19 throw new Exception("The method or operation is not implemented.");Exception being thrown from getter
10
11 public PHPExcel writes
12 {
13 get14 {15 throw new Exception("The method or operation is not implemented.");16 }17 set
18 {
19 throw new Exception("The method or operation is not implemented.");
Description
While it is acceptable for setters to throw exceptions, getters are simply expected to return a value. Throwing any exception is considered an antipattern and is not a recommended practice. It is recommended that you refactor your code accordingly.