Selon la réponse acceptée sur " Justification de préférer les variables locales aux variables d'instance? ", Les variables devraient vivre dans la plus petite portée possible.
Simplifier le problème dans mon interprétation, cela signifie que nous devrions reformer ce type de code:
public class Main {
private A a;
private B b;
public ABResult getResult() {
getA();
getB();
return ABFactory.mix(a, b);
}
private getA() {
a = SomeFactory.getA();
}
private getB() {
b = SomeFactory.getB();
}
}
dans quelque chose comme ça:
public class Main {
public ABResult getResult() {
A a = getA();
B b = getB();
return ABFactory.mix(a, b);
}
private getA() {
return SomeFactory.getA();
}
private getB() {
return SomeFactory.getB();
}
}
mais selon "l'esprit" de "les variables doivent vivre dans la plus petite portée possible", "les variables n'ayant jamais" n'ont-elles pas une portée plus petite que "les variables"? Je pense donc que la version ci-dessus devrait être refactorisée:
public class Main {
public ABResult getResult() {
return ABFactory.mix(getA(), getB());
}
private getA() {
return SomeFactory.getA();
}
private getB() {
return SomeFactory.getB();
}
}
donc cela getResult()n'a pas de variables locales du tout. Est-ce vrai?
72
La création de variables explicites présente l’avantage de devoir les nommer. L'introduction de quelques variables peut rapidement transformer une méthode opaque en une méthode lisible.
—
Jared Goguen
Honnêtement, un exemple en termes de noms de variables a et b est trop artificiel pour démontrer la valeur des variables locales. Heureusement, vous avez quand même de bonnes réponses.
—
Doc Brown
Dans votre deuxième extrait, vos variables ne sont pas vraiment des variables . Ce sont effectivement des constantes locales, car vous ne les modifiez pas. Le compilateur Java les traitera de la même manière si vous les définissez final, car ils sont dans la portée locale et le compilateur sait ce qui va se passer avec eux. C'est une question de style et d'opinion, si vous devez réellement utiliser un
—
Hyde
finalmot-clé ou non.
@JaredGoguen La création de variables explicites s'accompagne de la charge de devoir les nommer et de l'avantage de pouvoir les nommer.
—
Bergi
Absolument zéro des règles de style de code comme « variables doivent vivre dans la plus petite portée possible » sont applicables universellement sans pensée. En tant que tel, vous ne voudrez jamais baser une décision de style de code sur le raisonnement du formulaire de cette question, dans lequel vous prenez une simple directive et l'étendez dans une zone moins couverte ("les variables devraient si possible être de petite taille" -> "les variables ne devraient pas exister si possible"). Soit de bonnes raisons justifient votre conclusion, soit votre conclusion est une extension inappropriée; De toute façon, vous n'avez pas besoin de la directive d'origine.
—
Ben