이 파일 검색 스레드를 중단하는 것이 안전합니까?

Question

첫째, 코드 :

lblFileNbr.Text = "?/?"; 
lblFileNbr.ToolTipText = "Searching for files..."; 
lock(_fileLock) 
{ 
    _dirFiles = new string[0]; 
    _fileIndex = 0; 
} 
if(_fileThread != null && _fileThread.IsAlive) 
{ 
    _fileThread.Abort(); 
} 
_fileThread = new Thread(() => 
    { 
     string dir = Path.GetDirectoryName(fileName) ?? "."; 
     lock (_fileLock) 
     { 
      _dirFiles = GetImageFileExtensions().SelectMany(f => Directory.GetFiles(dir, f, _searchOption)).OrderBy(f => f).ToArray(); 
      _fileIndex = Array.IndexOf(_dirFiles, fileName); 
     } 
     int totalFileCount = Directory.GetFiles(dir, "*.*", _searchOption).Length; 

     Invoke((MethodInvoker)delegate 
     { 
      lblFileNbr.Text = string.Format("{0}/{1}", NumberFormat(_fileIndex + 1), NumberFormat(_dirFiles.Length)); 
      lblFileNbr.ToolTipText = string.Format("{0} ({1} files ignored)", dir, NumberFormat(totalFileCount - _dirFiles.Length)); 
     }); 
    }); 
_fileThread.Start(); 

나는 작은 이미지보기 프로그램을 짓고 있어요. 이미지를 열면 동일한 디렉토리에있는 파일 수가 나열됩니다. 다른 파일 (예 : 150K)이 많은 디렉토리에서 이미지를 열면 파일 목록을 작성하는 데 몇 초가 걸립니다. 따라서이 작업을 다른 스레드에 위임합니다.

그러나 파일 검색을 완료하기 전에 다른 이미지를 열면 이전 카운트는 더 이상 적합하지 않으므로 스레드를 중단합니다.

내가 사진 사이를 전환 할 몇 가지 왼쪽 및 오른쪽 키 기능을 추가하려는 때문에 _dirFiles 및 _fileIndex 잠금, 그래서 나는 다른 곳에서 그 액세스해야합니다 (그러나 UI 스레드에서).

안전한가요? C#에서 스레드를 다루는 수십 가지 방법이있는 것처럼 보입니다. 저는 단순한 것을 원했습니다.

---

fileName는 (? 오른쪽은 익명 함수에 "복사"됩니다 의미) 지역 변수이며, _searchOption는 읽기 전용, 그래서 나는 그 둘은 접근 안전 상상한다.

Answer 1

>이 파일 검색 스레드를 중단해도 안전합니까?

짧은 답변은 입니다.

스레드를 중단하는 것이 거의 절대 안전하지 않으며,이 조언은 네이티브 코드를 실행할 때 더 많이 적용됩니다.

시간이 걸리는 Directory.GetFiles에 대한 전화이므로 협조적으로 빨리 종료 할 수없는 경우 가장 좋은 방법은 스레드를 포기하는 것입니다. 정상적으로 끝내지 만 결과는 무시하십시오.

은 언제나처럼, 나는 Joe Albahari's free ebook

Answer 2

을 읽는 것은 Thread.Abort를 사용하여 스레드를() 중단 안전하지 않습니다 좋습니다. 하지만 대신 제어 된 방식으로 스레드를 안전하게 가져올 수있는 자체 중단을 구현할 수 있습니다.

GetFiles 대신 EnumerateFiles를 사용하는 경우 스레드를 중단해야하는지 여부를 확인하기 위해 플래그를 확인하면서 총 파일 수를 얻기 위해 카운터를 증가시킬 때 각 파일을 반복 할 수 있습니다. .

() 현재 GetFiles에 대신이 같은 무언가를 호출 길이 :

private bool AbortSearch = false; 
private int NumberOfFiles(string dir, string searchPattern, SearchOption searchOption) 
{ 
    var files = Directory.EnumerateFiles(dir, searchPattern, searchOption); 
    int numberOfFiles = 0; 
    foreach (var file in files) 
    { 
     numberOfFiles++; 

     if (AbortSearch) 
     { 
      break; 
     } 
    } 
    return numberOfFiles; 
} 

그런 다음

_fileThread.Abort(); 

와

AbortSearch=true; 
_fileThread.Join(); 

당신은 얻을 것이다 대체 할 수 현재 Thread.Abort()에서 무엇을하고 있는가?하지만 모든 스레드가 원할 때 끝내도록 허용합니다. 에.