Android volatile не работает?

Question

У меня есть класс Activity, в котором у меня есть статический флаг, скажем,

public static volatile flag = false;

Затем в классе я запускаю поток, который проверяет флаг и делает разные вещи.

У меня также есть широковещательный приемник, который устанавливает флаг в значение true или false.

Я, хотя volatile заставит флаг установить самое последнее значение. Но я вижу, что мой вещательный приемник устанавливает статический флаг в true, но мой поток все еще получает его как false.

Я что-то упустил здесь? Любая помощь будет оценена!

Упрощенный код (обновлено) - поэтому флаг должен измениться на 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>

Answer 1

Я думаю, проблема в том, что вы запускаете приемник в своем собственном процессе. Из документов для атрибута android:process <receiver>:

Если имя, присвоенное этому атрибуту, начинается с двоеточия (':'), новый частный для приложения процесс создается тогда, когда это необходимо, и приемник вещания запускается в этом процессе.

Я думаю, что получатель модифицирует локальную для процесса версию TestService.flag, а не ту, которая используется TestService. Попробуйте удалить атрибут android:process из тега <receiver> в манифесте.

Answer 2

Я действительно надеюсь, что ваш служебный поток не тот (я не вижу другого):

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), как должно быть.

Answer 3

По этой ссылке

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

По сути, volatile используется для указания того, что значение переменной будет изменено различными потоками.