Zgodnie z przyjętą odpowiedzią „ Racjonalne uzasadnienie preferowania zmiennych lokalnych zamiast zmiennych instancji? ” Zmienne powinny mieć możliwie najmniejszy zakres.
Uprość problem do mojej interpretacji, oznacza to, że powinniśmy zmienić kod tego rodzaju:
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();
}
}
w coś takiego:
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();
}
}
ale zgodnie z „duchem” zmiennych zmienne powinny żyć w najmniejszym możliwym zakresie, czy „nigdy nie mieć zmiennych” ma mniejszy zakres niż „mieć zmiennych”? Myślę więc, że powyższa wersja powinna zostać refaktoryzowana:
public class Main {
public ABResult getResult() {
return ABFactory.mix(getA(), getB());
}
private getA() {
return SomeFactory.getA();
}
private getB() {
return SomeFactory.getB();
}
}
więc getResult()nie ma żadnych lokalnych zmiennych. Czy to prawda?
72
Tworzenie wyraźnych zmiennych ma tę zaletę, że trzeba je nazwać. Wprowadzenie kilku zmiennych może szybko przekształcić nieprzejrzystą metodę w czytelną.
—
Jared Goguen
Szczerze mówiąc, przykład w zakresie nazw zmiennych aib jest zbyt wymyślony, aby wykazać wartość zmiennych lokalnych. Na szczęście i tak masz dobre odpowiedzi.
—
Doc Brown
W drugim fragmencie zmienne nie są tak naprawdę zmiennymi . Są to stałe lokalne, ponieważ ich nie modyfikujesz. Kompilator Java potraktuje je tak samo, jeśli zdefiniujesz je jako ostateczne, ponieważ są one w zasięgu lokalnym, a kompilator wie, co się z nimi stanie. To kwestia stylu i opinii, czy rzeczywiście powinieneś użyć
—
hyde
finalsłowa kluczowego, czy nie.
@JaredGoguen Tworzenie wyraźnych zmiennych wiąże się z koniecznością ich nazwania i korzyścią z możliwości ich nazwania.
—
Bergi,
Absolutnie zero zasad stylu kodu, takich jak „zmienne powinny żyć w najmniejszym możliwym zakresie”, są uniwersalnie stosowane bez zastanowienia. Jako taki, nigdy nie chcesz opierać decyzji w stylu kodu na uzasadnieniu formularza w tym pytaniu, w którym bierzesz prostą wytyczną i rozszerzasz ją na mniej oczywisty obszar („zmienne powinny mieć małe zakresy, jeśli to możliwe” -> „zmienne nie powinny istnieć, jeśli to możliwe”). Albo istnieją uzasadnione powody, które potwierdzają twoją konkluzję, albo twoja konkluzja jest niewłaściwym przedłużeniem; tak czy inaczej, nie potrzebujesz oryginalnej wytycznej.
—
Ben