取消“一会儿”的问题Android 中的循环

Question

首先，对于这个不太具有描述性的标题感到抱歉。

我有以下代码，它无法正常工作（或如我想要的那样）。

即使我调用“Cancel()”，但时间仍在继续......

private boolean cancelled;

private synchronized Bitmap renderBitmap(PatchInputStream pis){
              File file = new File(Environment.getExternalStorageDirectory()+url);

              FileOutputStream fos=new FileOutputStream(file);
              byte buf[]=new byte[2000];
              int len;
              while((len=pis.read(buf))>0 && !isCancelled() ){
                  fos.write(buf,0,len);
              }
              fos.close();
              if (isCancelled()){
                cancelled=false;
                return null;
              }

              Bitmap bm = saveBitmapToFile(file);
              return bm;
}

public boolean isCancelled(){
    return cancelled;
}

public void Cancel(){
    cancelled=true;
}

这段代码有什么问题？我错过了什么吗？

Answer 1

至少代码有一个并发错误，可以通过设置 cancelled volatile 来解决。否则，无法保证其他线程看到新值。
（显然情况并非如此。）

另一种可能性是未调用 cancel() 方法。
（显然情况并非如此。）

还有一种可能是在不同的对象实例上调用 cancel() 方法。尝试将 System.out.println(System.identityHashCode(this)) 添加到 cancel() 方法和 while 循环，看看它们是否打印相同的值。

Answer 2

按照 @mibollma 的建议，在已取消的实例变量上使用 volatile 。这将确保没有线程正在使用变量的缓存版本。如果这不起作用，您可以尝试以下建议：

您的 cancel() 方法可能没有被调用，带有 while 循环的线程不允许这样做。使用 Thread.yield() 方法。

使当前正在执行的线程对象暂时暂停并允许其他线程执行。

在你的 while 循环中：

          while((len=pis.read(buf))>0 && !isCancelled() ){
              fos.write(buf,0,len);
              Thread.yield();
          }