安卓中的volatile关键字无效？

Question

我有一个活动类，在其中有一个静态标志，假设为

public static volatile flag = false;

在类中，我启动了一个线程，该线程检查标志并执行不同的操作。

我还有一个广播接收器，用于将标志设置为true或false。

我认为volatile会强制将标志设置为最近的值。但是我可以看到我的广播接收器将静态标志设置为true，但我的线程仍然将其视为false。

我错过了一些基本的东西吗？任何帮助都将不胜感激！

简化代码（已更新）-因此，该标志应在一分钟后更改为true。但它从未这样做。但来自广播接收器的消息显示已将其更改为true

TestappActivity.java：

package com.test;

import java.util.Calendar;

import android.app.Activity;
import android.app.AlarmManager;
import android.app.PendingIntent;
import android.content.Intent;
import android.os.Bundle;

public class TestappActivity extends Activity {
    /** Called when the activity is first created. */
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);

        Intent intent0 = new Intent(this, TestService.class);
        this.startService(intent0);

        Intent intent = new Intent(this, TestReceiver.class);
        AlarmManager am = (AlarmManager) getSystemService(ALARM_SERVICE);
        PendingIntent sender = PendingIntent.getBroadcast(this,
                1, intent,
                PendingIntent.FLAG_UPDATE_CURRENT);
        Calendar slot = Calendar.getInstance();
        int min = slot.get(Calendar.MINUTE);
        slot.set(Calendar.MINUTE, min+1);
        am.set(AlarmManager.RTC_WAKEUP, slot.getTimeInMillis(), sender);
    }
}

TestService.java:

package com.test;

import android.app.Service;
import android.content.Intent;
import android.os.IBinder;
import android.util.Log;

public class TestService extends Service {

    private static final String TAG = "TestService";

    public static volatile boolean flag = false;

    private MyTopThread mTopThread;

    public TestService() {
    }

    @Override
    public IBinder onBind(Intent intent) {
        return null;
    }


    @Override
    public void onCreate() {

    }

    @Override
    public void onDestroy() {

    }

    @Override
    public int onStartCommand(Intent intent, int flags, int startId) {
        protect();

        // We want this service to continue running until it is explicitly
        // stopped, so return sticky.
        return START_STICKY;
    }


    /**
     * Run protection
     * 
     */
    private void protect() {

        mTopThread = new MyTopThread();
        mTopThread.start();
    }


    private class MyTopThread extends Thread {

        @Override
        public void run() {
            while (true) {
                try {
                    Thread.sleep(150);
                    Log.d(TAG, "Flag is " + TestService.flag);
                } catch (InterruptedException e) {
                    e.printStackTrace();
                }

            }
        }

    }
}

TestReceiver.java:

package com.test;

import android.content.BroadcastReceiver;
import android.content.Context;
import android.content.Intent;
import android.util.Log;

public class TestReceiver extends BroadcastReceiver {
    final static private String TAG = "TestReceiver";

    @Override
    public void onReceive(Context context, Intent intent) {
        Log.d(TAG, "onReceive is triggered ...");
        TestService.flag = true;
        Log.d(TAG, "flag is changed to " + TestService.flag);

    }
}

AndroidManifest.xml:

<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="com.test"
    android:versionCode="1"
    android:versionName="1.0" >

    <uses-sdk android:minSdkVersion="8" />

    <application
        android:icon="@drawable/ic_launcher"
        android:label="@string/app_name" >
        <activity
            android:name=".TestappActivity"
            android:label="@string/app_name" >
            <intent-filter>
                <action android:name="android.intent.action.MAIN" />

                <category android:name="android.intent.category.LAUNCHER" />
            </intent-filter>
        </activity>

        <service android:name=".TestService" />

        <receiver
            android:name=".TestReceiver"
            android:process=":remote" >
        </receiver>
    </application>

</manifest>

- Safecoder

---

应该可以工作。你能发一下你的代码吗？ - Ted Hopp

我没有看到 flag 的值被改变了。 - Ted Hopp

你的清单文件中是否声明了该服务在自己的进程中运行？ - Ted Hopp

出于好奇，如果Android的组件出现问题，如果您从活动的内部线程更改变量会发生什么？（例如运行另一个线程（Timer类），并在1分钟后更改值）。 基本上你是对的，这段代码应该可以工作，我不太确定你是否遗漏了什么... - codeScriber

@Ted Hopp，不，我没有在自己的进程中运行服务。 我也更新了我的清单文件，请查看一下。 - Safecoder

Answer 1

我认为问题在于你在独立的进程中运行接收器。从 <receiver> 的 android:process 属性文档中可以看到，如果分配给该属性的名称以冒号（':'）开头，则需要创建一个新的应用程序专用进程，并且广播接收器会在该进程中运行。

我认为接收器正在修改本地进程版本的 TestService.flag，而不是由 TestService 使用的版本。请尝试从你的清单文件中的 <receiver> 标签中删除 android:process 属性。

- Ted Hopp

---

你说得对！在看到你的服务流程问题后，我也在考虑同样的事情。我刚试了一下，它确实有效！非常感谢！ - Safecoder

Answer 2

从这个链接

http://www.javamex.com/tutorials/synchronization_volatile.shtml

基本上，volatile 用于指示变量的值将被不同的线程修改。

- Bill Gary

---

是的，它在BroadcastReceiver中被修改，这与我的服务中运行的线程不同。 - Safecoder

Answer 3

我真的希望你的服务线程不是这个（我没有看到其他的）：

private class MyTopThread extends Thread {

    @Override
    public void run() {
        while (true) {
            try {
                Thread.sleep(150);
                Log.d(TAG, "Flag is " + TestService.flag);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }

        }
    }

}

因为你这里使用了while(true)，而不是应该用的while(!flag)。

- Tudor

---

这只是我快速组合的一个测试程序，以展示问题。它使用while(true)，因为我想显示标志值，以查看它是否被广播接收器更改。但是logcat显示即使在我从广播接收器看到“flag is changed to true”之后，它仍然保持为“Flag is false”。 - Safecoder

运行它，你就会明白我的意思。你必须设置一个过滤器，以便在 while 循环中继续打印出消息“Flag is false”时，“flag 被更改为 true”。 - Safecoder

@Howard Li：你确定你设置为false的onReceive被执行了吗？ - Tudor

在 onReceive 中，我将其设置为 true，并且可以从 logcat 中看到打印出了“flag is changed to true”。 - Safecoder

你是否尝试过只测试 while(!flag) 并查看线程是否停止？ - Tudor

尝试过了，发现了同样的问题。 - Safecoder