-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathDataTypes.java
149 lines (113 loc) · 4.08 KB
/
DataTypes.java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
1.
// before
float amount
// after
double amount
amount - кол-во едениц, может переводиться из веса и быть не целым.
// Изменил тип вещественной переменной на тип с большей точностью.
2.
// before
usedPhysicalMemorySizeAccurate = getUsedPhysicalMemorySize() / 1024L / 1024L
// after
const long BYTE_ORDER_INCREASER = 1024L
usedPhysicalMemorySizeAccurate = getUsedPhysicalMemorySize() / BYTE_ORDER_INCREASER / BYTE_ORDER_INCREASER
//вынесение "магического числа" в константу
3.
// before
if(i == (uriSplitted.length - 1)){}
// after
boolean isLastChar = i == (uriSplitted.length - 1)
if(isLastChar){}
// Добавил логическую переменную для упрощения сложного условия
4.
// before
if(uriSplitted[i].equals(" ")){}
// after
boolean isSpace = uriSplitted[i].equals(" ")
if(isSpace){}
// Добавил логическую переменную для упрощения сложного условия
5.
// before
if(uriSplitted[i + indexOffset].equals(")")){}
// after
boolean isCloseBracket = uriSplitted[i + indexOffset].equals(")")
if(isCloseBracket){}
// Добавил логическую переменную для упрощения сложного условия
6.
// before
if((uriSplitted[i].equals(" ")) || (i == (uriSplitted.length - 1))
|| (uriSplitted[i + indexOffset].equals(")"))){}
// after
boolean isEndQuery = isLastChar || isSpace || isCloseBracket;
if(isEndQuery){}
// Добавил логические переменные для упрощения сложных условий
7.
// before
var max = <some code>
var used = <some code>
var usage = used / max;
// after
var max = <some code>
var used = <some code>
var usage;
if(!(max==null) && max != 0L){
usage = used / max;
}else{
throw new Exception("The "max" is 0 (zero)");
}
// Добавил код, предупреждающий появление ошибки деления на 0.
8.
// before
while(bytes <= -999_950 || bytes >= 999_950){}
// after
const int DANGEROUSLY_ROUNDED_NUMBER = 999_950;
const int NEGATIVE_DANGEROUSLY_ROUNDED_NUMBER = -999_950;
while(bytes <= NEGATIVE_DANGEROUS_ROUNDED_NUMBER || bytes >= DANGEROUS_ROUNDED_NUMBER){
// вынесение "магического числа" в константу
9.
// before
if((naturalPersonInfo.mainPhone
&& naturalPersonInfo.mainPhone.communicationValue)
|| (naturalPersonInfo.additionalPhone
&& naturalPersonInfo.additionalPhone.communicationValue)){}
// after
boolean isSetMainPhone = naturalPersonInfo.mainPhone && naturalPersonInfo.mainPhone.communicationValue
boolean isSetAdditionalPhone = naturalPersonInfo.additionalPhone && naturalPersonInfo.additionalPhone.communicationValue
boolean isSetPhone = isSetMainPhone || isSetAdditionalPhone
if(isSetPhone){}
// Добавил логические переменные для упрощения сложных условий
10.
// before
if(fields.address || registration.steps.save.address.mainAddress){}
// after
boolean isSetAddress = fields.address || registration.steps.save.address.mainAddress;
if(isSetAddress){}
// Добавил логические переменные для упрощения сложных условий
11.
// before
if (getWidth == "480"){}
// after
const targetWitdth = "480"
if (getWidth == targetWitdth){}
// вынесение "магического числа" в константу
12.
// before
String parameter = code.Substring(20, 22);
switch (parameter) {
case ("104"):
Код1;
break;
case ("531"):
Код2;
break;
// after
const weightCode = "104"
const amountCode = "531"
switch (parameter) {
case (weightCode):
Код1;
break;
case (amountCode):
Код2;
break;
// вынесение "магического числа" в константу