진행 표시 줄이 매우 정확하지 않습니다.

Question

다른 인기있는 앱의 캐시 정리 프로그램 인 첫 번째 Android 앱에서 작업하고 있습니다. 마침내 앱 캐시를 확인한 다음 삭제하는 기능이 있습니다. 그러나 파일을 삭제하기 위해 설정 한 비동기 작업은 진행률 막대를 정확하게 업데이트하지 않는 것 같습니다. 내가 프로그래밍 n00b이기 때문에, 내 코드는 대부분 다른 소스의 copypasta입니다.

여기에 내가 의심 블록이 진행 바 계산 문제를 가지고있어하십시오 불필요 경우에만 디렉토리를 통해 이동을 필요로 루프에 대한 중첩 된 것처럼 것 같다

@Override 
    protected Void doInBackground(Void... params) { 

     // Delete Cache !! 
     File dir = new File(Environment.getExternalStorageDirectory() 
       + "/Android/Data/com.popularapp/Cache"); 

     // Progress Bar 
     for (int i = 1; i < 100; i++) { 
      if (isCancelled()) { 
       break; 
      } else { 
       System.out.println(i); 
       publishProgress(i); 
       if (dir.isDirectory()) { 
        String[] children = dir.list(); 
        for (int i2 = 0; i2 < children.length; i2++) { 
         new File(dir, children[i2]).delete(); 
        } 
       } 
      } 
     } 

     return null; 
    } 

    @Override 
    protected void onProgressUpdate(Integer... values) { 
     super.onProgressUpdate(values); 
     progressBar.setProgress(values[0]); 
     tvLoading.setText("Loading... " + values[0] + " %"); 
     tvPer.setText(values[0] + " %"); 
    } 

Answer 1

. 실제로 파일을 삭제하는 for 루프 반복에 진행 상황을 연결해야합니다.

바깥 쪽 루프는이고 'i2'카운터 변수를 사용하여 진행률을 게시 할 수 있습니다.

for 루프를 두 번째 (처음으로) for 루프로 호출하면 publishProgress (i2/children.length * 100)가 호출됩니다. 모든 파일의 삭제에 거의 같은 시간이 소요된다고 가정 할 때 더 정확한 숫자 여야합니다.

@Override 
protected Void doInBackground(Void... params) { 

    // Delete Cache !! 
    File dir = new File(Environment.getExternalStorageDirectory() 
      + "/Android/Data/com.popularapp/Cache"); 
    if (dir.isDirectory()) { 
     String[] children = dir.list(); 
     for (int i2 = 0; i2 < children.length; i2++) { 
       if (isCancelled()) break; 
       int progress = 100 * i2/children.length; 
       Log.w("Deleting files...", "Current iteration: " + i2 + " Progress: " + progress); 
       publishProgress(progress); 
       new File(dir, children[i2]).delete(); 
     } 
    } 

    return null; 
}