경고 :

Question

편집 "는 람다 식에 반복 변수를 사용하여 예기치 않은 결과를 가질 수있다"

Dim numbers1 As New List(Of Int32)({1, 2, 3}) 
Dim numbers2 As New List(Of Int32)({3, 4, 5}) 
For Each n1 In numbers1 
    ' no warning ' 
    Dim contains = numbers2.Contains(n1) 
Next 
For Each n1 In numbers1 
    ' warning on n1' 
    Dim contains = (From num In numbers2 Where num = n1).Any 
Next 

: 다음은이 문제를 (내가 deleted 내 원래의 질문을했습니다) 훨씬 더 간단한 예입니다 그래서 저는 컴파일러가 왜 두 번째 반복에서 예상치 못한 결과를 얻을 수 있다고 생각하는지 이해하지 못합니다. 반면에 처음에는 안전합니다. 나는 @ ee-m의 interesting link이이 동작에 대한 이유를 제공하지 않는다고 생각한다. (for-each 이슈가 아니기 때문에 For n1 As Int32 = 1 To 3도 컴파일러 경고가된다.)

내가 다음은 "모범 사례"를해야한다고 정말 확신 아니에요 :

For Each n1 In numbers1 
    Dim number1 = n1 
    ' no warning' 
    Dim contains = (From num In numbers2 Where num = number1).Any 
Next 

지역 변수 number1가 중복 및 메타 나이트가 이미 강조했다 @ 덜 읽을 수있는 코드를합니다. 참고 : 세 가지 방법 모두 안전하며 올바른 결과를 제공합니다.

Answer 1

메시지에 말한대로 원치 않는 결과가있을 수 있습니다. 귀하의 경우에는 .ToList()가 안전하지만 컴파일러가 확인하기가 어렵습니다.

에릭 Lippert의가 (코드 예제는 C#에서 VB 없음) 진행되는이 주제에 대한 블로그 게시물의 몇 가지를 썼다

Answer 2

표준 '모범 사례'와 같은 로컬 VAR (Dim exc = excel)에 복사를 채택하도록 제안 흥미로운 찾을 수 있습니다 코드의 종류에서 발생할 수있는 "개는"일부 이상 :

Closing over the loop variable considered harmful