3

バックグラウンドで実行されるwinformアプリケーションがあり、BackgroundWorker1時間ごとに何かを実行する無限ループがあります。私のUIFormクラスは次のようなものです：

public partial class frmAutoScript : Form
{
    private volatile bool _isDownloading = false;
    private bool IsDownloading { get { return this._isDownloading; } set { this._isDownloading = value; } }

    public frmAutoScript()
    {
        InitializeComponent();
        this.RunAutoSynchronization();
    }

    private void RunAutoSynchronization()
    {
        bool isDownloading = this.IsDownloading;

        BackgroundWorker bgwDownloader = new BackgroundWorker();
        bgwDownloader.WorkerReportsProgress = true;
        bgwDownloader.ProgressChanged += (sndr, evnt) =>
        {
            if (evnt.ProgressPercentage == 2)
                isDownloading = this.IsDownloading;
            else
            {
                this.IsDownloading = evnt.ProgressPercentage == 1;
                isDownloading = this.IsDownloading;
            }
        };
        bgwDownloader.DoWork += (sndr, evnt) =>
            {
                while (true)
                {
                    if (DateTime.Now.Hour == 16 &&
                        DateTime.Now.Minute == 0)
                    {
                        try
                        {
                            bgwDownloader.ReportProgress(2);
                            if (!isDownloading)
                            {
                                bgwDownloader.ReportProgress(1);
                                new Downloader().Download();
                            }
                            bgwDownloader.ReportProgress(0);
                        }
                        catch { }
                    }

                    System.Threading.Thread.Sleep(60000);
                }
            };
        bgwDownloader.RunWorkerAsync();
    }
}

そして、その中に、クリックするとvarialbeの値をダウンロードして変更するfrmAutoScriptという名前のボタンもあります。ボタンのイベントは次のようなものです。btnDownloadvolatile_isDownloading

private void btnDownload_Click(object sender, EventArgs e)
{
    if (IsDownloading)
        MessageBox.Show("A download is currently ongoing. Please wait for the download to finish.",
            "Force Download", MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
    else
    {
        this.IsDownloading = true;
        BackgroundWorker bgwDownloader = new BackgroundWorker();
        bgwDownloader.DoWork += (sndr, evnt) =>
        {
            try
            {
                new Downloader().Download();
            }
            catch(Exception ex)
            {
                MessageBox.Show("An error occur during download. Please contact your system administrator.\n Exception: " +
                    ex.GetType().ToString() + "\nError Message:\n" + ex.Message + " Stack Trace:\n" + ex.StackTrace, "Download Error!", MessageBoxButtons.OK, MessageBoxIcon.Error);
            }
        };
        bgwDownloader.RunWorkerCompleted += (sndr, evnt) =>
        {
            this.IsDownloading = false;
        };
        bgwDownloader.RunWorkerAsync();
    }
}

しかし、ボタンをクリックしbtnDownloadて_isDownloadingが に設定されtrue、システム時間がに達すると4:00 PM、が true に設定されていても がnew Downloader().Download();再度実行さ_isDownloadingれます。どうしてこうなった？

私のコードは C#、フレームワーク 4、プロジェクトは winforms、Visual Studio 2010 Pro でビルドされています。

4

1 に答える 1

4

volatileコードはフィールドに対してテストしていません-「ローカル」のようisDownloadingに見えますが、（キャプチャされているため）実際には通常の（非volatile）フィールドです。したがって、何らかのメモリバリアを使用するか、強制的に揮発性読み取りにするかのいずれかです。またはもっと簡単に:isDownloading 完全に削除し、プロパティをチェックします。

ちなみに、キャッシュを無効にするプロパティは、キーワードの意図でvolatileはなく、結果です。それはうまくいきますが、個人的には、結果ではなく意図的に動作するlockようにコードを書くことをお勧めしますInterlocked。

于 2012-05-28T08:52:46.967 に答える