Sử dụng for-each vòng lặp trong java

Question

Trong mã của tôi,

for(City city : country.getCities()){ 
    // do some operations 
} 

Sử dụng country.getCities() là tốn kém? JVM sẽ duy trì stacktrace cho mọi cuộc gọi ..?

List<City> cityList = country.getCities(); 
for(City city : cityList){ 
    // do some operations 
} 

Cách tốt nhất để sử dụng là gì?

Answer 1

Không, vòng lặp này:

for(City city : country.getCities()) 

sẽ chỉ gọi country.getCities()lần, và sau đó lặp trên nó. Nó không gọi nó cho mỗi lần lặp của vòng lặp. Trong trường hợp của bạn, nó tương đương với:

for (Iterator<City> iterator = country.getCities().iterator(); 
    iterator.hasNext();) { 
    City city = iterator.next(); 
    // do some operations 
} 

Không có lợi ích khi viết lại theo đoạn thứ hai của bạn.

Xem section 14.14.2 of the JLS để biết thêm chi tiết.

Answer 2

Trong cả hai trường hợp, getCities() chỉ được gọi một lần.

Tuyên bố từ chối trách nhiệm mặc định: Như thường lệ, bạn hầu như không bao giờ cần phải lo lắng về hiệu suất, vì trình biên dịch tốt hơn nhiều so với một người.

Answer 3

Chúng tương đương nhau. getCities() sẽ chỉ được thực hiện sau khi. (nếu bạn dán System.out.println() vào phương pháp đó, bạn sẽ thấy điều này).

Lưu ý rằng nó sẽ không gọi getCities() nhiều lần không chỉ vì những lý do về hiệu suất, mà còn bởi vì có không đảm bảo rằng nó sẽ trở về bộ sưu tập giống nhau mỗi lần (nó có thể là trực quan mà nó trả về bộ sưu tập này, nhưng không bắt buộc)

Tôi muốn ví dụ đầu tiên của bạn về sự phù hợp.

Answer 4

Cả hai đều giống nhau, không có sự khác biệt. Và country.getCities() sẽ được gọi một lần trong vòng lặp.

Tôi nghĩ đầu tiên tốt hơn vì không có tham chiếu bổ sung nào được sử dụng.

Answer 5

JVM sẽ tối ưu hóa mã khi được biên dịch để đảm bảo phương thức country.getCities() được gọi một lần.

Bạn nên viết mã theo cách bạn và/hoặc nhóm của bạn nghĩ là dễ đọc nhất.