[JAVA] Être conscient du code facile à lire

Soyez conscient du code facile à lire

Si vous ne faites pas attention, cela a tendance à être du code redondant, donc pour la sensibilisation Nous avons résumé les points qui le rendent plus lisible.

Simplifier les expressions conditionnelles

java

if (a == 1 && b == 2 && c == 3 && d == 4){
  System.out.println("test");
}

//Enveloppez l'expression conditionnelle de l'instruction if

if (a == 1 && 
   b == 2 &&
   c == 3 &&
   d == 4){
  System.out.println("test");
}

//Créer une méthode pour améliorer la lisibilité
if (output()){
  System.out.println("test");
}

private boolean isValidType(){
//En traitement
return true;
}

Nidification trop profonde

java

//NG
if(a == 1){
    if(b == 2){
        if(c == 3){
            if(d == 4){
                if(e == 5){
                   System.out.println("test");
                }
            }
        }
    }
}

//OK
////Créez la même méthode pour améliorer la lisibilité
if (isValidType()){
  System.out.println("test");
}

private boolean isValidType(){
//En traitement
return true;
}

Nom de variable ou nom de méthode inapproprié

Déclaré selon la convention de dénomination

Cas de serpent

java

String user_name = "Taro"

Caisse inférieure de chameau

java

String userName = "Taro"

Cas supérieur de chameau

java

String UserName = "Taro"

Cas de la chaîne

java

String user-name = "Taro"

Créer automatiquement des noms de variables à l'aide de codic

Connaissance de la portée du champ d'application

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
//Ne peut être exécuté que dans une classe grâce à un contrôleur d'accès
class Calculator {
  private static int num = 0;
  private static void alterNumber(int _num) {
    num = _num;
  }
  public static void main(String args[]) {
    alterNum(23);
  }
}

Intégration de code

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
//Ajout d'une méthode pour améliorer la lisibilité
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;
  }
}