problème constructeur en java

Question

public class StateQueryFilter extends FieldQueryFilter { 
    // private static final Log LOG = LogFactory.getLog(RecommendedParser.class.getName()); 

    public StateQueryFilter() { 
     super("state", 5f); 
     super("city", 5f); 
     super("notdirectory", 5f); 
     //LOG.info("Added a state query"); 
    } 

} 

Et il rapporte:

Constructor call must be the first statement in a constructor 

est-il un moyen de le faire fonctionner?

Answer 1

Vous pouvez uniquement appeler super une fois dans un constructeur, et ce doit être la première ligne.

Cela n'a même pas de sens de l'appeler plus d'une fois comme vous le faites. L'appel super ne crée pas un nouvel objet; il laisse simplement la superclasse initialiser ses champs. Essayez-vous vraiment d'initialiser les mêmes champs plus d'une fois? Vous essayez probablement de créer trois objets distincts, mais ce n'est pas la façon de le faire. Voir également Using the keyword super dans les didacticiels Java.

Answer 2

Non. Vous essayez de construire l'objet 3 fois, la logique est imparfaite.

Answer 3

De section 8.8.7 of the Java Language Specification:

La première déclaration d'un constructeur corps peut être une invocation explicite de un autre constructeur de la même classe ou de la superclasse directe (§8.8.7.1).

Remarque "première" instruction - vous ne pouvez pas en avoir plusieurs. Je trouve des constructeurs qui appellent des setters moche.

Answer 4

Une autre option consiste à avoir un constructeur qui prend les arguments.

public StateQueryFilter() { 
    super(/*state*/ 5, /* city */ 5, /*notdirectory*/ 5); 
    //LOG.info("Added a state query"); 
} 

Cependant il semble que vous devriez faire quelque chose comme

public class StateQueryFilter extends FieldQueryFilter { 
    private final String city; 
    private final String state; 
    private final boolean noDirectory; 

    public StateQueryFilter(String city, String state, boolean noDirectory) { 
     this.state = state; 
     this.city = city; 
     this.notDirectory = notDirectory; 
    } 
} 
// later 
FieldQueryFilter filter = new StateQueryFiler("Los Angeles", "California", true); 

BTW: Ne pas utiliser les valeurs flottantes, ils ne sont presque jamais la meilleure option.