Wenn Sie nicht aufpassen, handelt es sich in der Regel um redundanten Code Wir haben die Punkte zusammengefasst, die es lesbarer machen.
java
if (a == 1 && b == 2 && c == 3 && d == 4){
System.out.println("test");
}
//Schließen Sie den bedingten Ausdruck der if-Anweisung ein
if (a == 1 &&
b == 2 &&
c == 3 &&
d == 4){
System.out.println("test");
}
//Erstellen Sie eine Methode zur Verbesserung der Lesbarkeit
if (output()){
System.out.println("test");
}
private boolean isValidType(){
//wird bearbeitet
return true;
}
java
//NG
if(a == 1){
if(b == 2){
if(c == 3){
if(d == 4){
if(e == 5){
System.out.println("test");
}
}
}
}
}
//OK
////Erstellen Sie dieselbe Methode, um die Lesbarkeit zu verbessern
if (isValidType()){
System.out.println("test");
}
private boolean isValidType(){
//wird bearbeitet
return true;
}
Gemäß Namenskonvention deklariert
java
String user_name = "Taro"
java
String userName = "Taro"
java
String UserName = "Taro"
java
String user-name = "Taro"
java
//NG
class Calculator {
public static int num = 0;
public static void alterNumber(int _num) {
num = _num;
}
public static void main(String args[]) {
alterNum(23);
}
}
//OK
//Kann aufgrund eines Zugriffscontrollers nur innerhalb einer Klasse ausgeführt werden
class Calculator {
private static int num = 0;
private static void alterNumber(int _num) {
num = _num;
}
public static void main(String args[]) {
alterNum(23);
}
}
java
//NG
public class StandAction extends BaseAction {
public void Stand(Account account) {
if(account.getFirstName() == null || account.getFirstName().length()== 0) {
return;
}
if(account.getLastName() == null || account.getLastName().length() == 0) {
return;
}
}
}
//OK
//Methode zur Verbesserung der Lesbarkeit hinzugefügt
public class Stand extends BaseAction {
public void Stand(Account account){
if(isEmpty(account.getFirstName())){
return;
}
if(isEmpty(account.getLastName())) {
return;
}
}
public boolean isEmpty(String str) {
return str == null || str.length() == 0;
}
}
Recommended Posts