Недопустимая проблема с многопоточным доступом - PullRequest
Новая
       27

Недопустимая проблема с многопоточным доступом

26 голосов
/ 17 декабря 2009

У меня есть два класса ViewModel: PersonViewModel и PersonSearchListViewModel. Одним из полей, которые реализует PersonViewModel, является изображение профиля, которое загружается через WCF (локально кэшируется в изолированном хранилище). PersonSearchListViewModel - это контейнерный класс, который содержит список Persons. Поскольку загрузка изображений относительно тяжелая, PersonSearchListViewModel загружает только изображения для текущей, следующей и предыдущей страниц (результаты отображаются на пользовательском интерфейсе) ... чтобы еще больше улучшить загрузку изображений, я поместил загрузку изображений в другой поток. Однако многопоточный подход вызывает проблемы с многопоточным доступом.

PersonViewModel: 

public void RetrieveProfileImage()
{
    Image profileImage = MemorialDataModel.GetImagePerPerson(Person);
    if (profileImage != null)
    {
        MemorialDataModel.ImageManager imgManager = new MemorialDataModel.ImageManager();
        imgManager.GetBitmap(profileImage, LoadProfileBitmap);
    }
}

private void LoadProfileBitmap(BitmapImage bi)
{
    ProfileImage = bi;
    // update 
    IsProfileImageLoaded = true;
}

private BitmapImage profileImage;
public BitmapImage ProfileImage
{
    get
    {
        return profileImage;
    }
    set
    {
        profileImage = value;
        RaisePropertyChanged(new System.ComponentModel.PropertyChangedEventArgs("ProfileImage"));
    }
}

PersonSearchListViewModel: 

private void LoadImages()
{
    // load new images 
    Thread loadImagesThread = new Thread(new ThreadStart(LoadImagesProcess));
    loadImagesThread.Start();

    //LoadImagesProcess(); If executed on the same thread everything works fine 
}

private void LoadImagesProcess()
{
    int skipRecords = (PageIndex * PageSize);
    int returnRecords;

    if (skipRecords != 0)
    {
        returnRecords = 3 * PageSize; // page before, cur page and next page 
    }
    else
    {
        returnRecords = 2 * PageSize;   // cur page and next page 
    }

    var persons = this.persons.Skip(skipRecords).Take(returnRecords);

    // load images 
    foreach (PersonViewModel pvm in persons)
    {
        if (!pvm.IsProfileImageLoaded)
        {
            pvm.RetrieveProfileImage();
        }
    }
}

Как вы обрабатываете данные в классе ViewModel многопоточным способом? Я знаю, что вы должны использовать диспетчер на пользовательском интерфейсе для обновления. Как вы обновляете ViewModel, которая связана с пользовательским интерфейсом?

** РЕДАКТИРОВАТЬ **

Есть еще одна странная ошибка. В коде ниже: 

        public void GetBitmap(int imageID, Action<BitmapImage> callback)
        {
            // Get from server 
            bitmapCallback = callback;

            memorialFileServiceClient.GetImageCompleted += new EventHandler<GetImageCompletedEventArgs>(OnGetBitmapHandler);
            memorialFileServiceClient.GetImageAsync(imageID);
        }

        public void OnGetBitmapHandler(object sender, GetImageCompletedEventArgs imageArgs)
        {
            if (!imageArgs.Cancelled)
            {
                // I get cross-thread error right here 
                System.Windows.Media.Imaging.BitmapImage bi = new BitmapImage();
                ConvertToBitmapFromBuffer(bi, imageArgs.Result.Image);

                // call call back
                bitmapCallback.Invoke(bi);
            }
        }

При попытке создать новый объект BitmapImage в фоновом потоке я получаю перекрестную ошибку. Почему я не могу создать новый объект BitmapImage в фоновом потоке?

Ответы [ 3 ]

62 голосов
/ 18 декабря 2009

Чтобы обновить DependencyProperty в ViewModel, используйте тот же диспетчер, который вы использовали бы для доступа к любому другому UIElement: 

System.Windows.Deployment.Current.Dispatcher.BeginInvoke(() => {...});

Кроме того, BitmapImages должен создаваться в потоке пользовательского интерфейса. Это потому, что он использует DependencyProperties, которые могут быть использованы только в потоке пользовательского интерфейса. Я попытался создать экземпляр BitmapImages в отдельных потоках, и он просто не работает. Вы можете попробовать использовать другие средства для хранения изображений в памяти. Например, когда вы загружаете изображение, сохраняйте его в MemoryStream. Затем BitmapImage в потоке пользовательского интерфейса может установить свой источник в MemoryStream.

Вы можете попробовать создать экземпляр BitmapImages в потоке пользовательского интерфейса, а затем сделать все остальное с помощью BitmapImage в другом потоке ... но это может стать проблематичным, и я даже не уверен, что он будет работать. Ниже приведен пример: 

System.Windows.Media.Imaging.BitmapImage bi = null;
using(AutoResetEvent are = new AutoResetEvent(false))
{
    System.Windows.Deployment.Current.Dispatcher.BeginInvoke(() =>
    {
        bi = new BitmapImage();
        are.Set();
    });
    are.WaitOne();
}

ConvertToBitmapFromBuffer(bi, imageArgs.Result.Image);
bitmapCallback.Invoke(bi);
2 голосов
/ 18 декабря 2009

Я полагаю, что у вас возникла проблема с многопоточностью в потоке пользовательского интерфейса.

Редактирование связанного объекта может привести к обновлению пользовательского интерфейса в рабочем потоке, что не может быть успешно выполнено. Скорее всего, вам нужно будет использовать клавишу-подсказку InvokeRequired / Invoke при каждом обновлении связанного класса.

Вы сказали, что уже знали это, но для справки:

MSDN для поточно-ориентированных вызовов к пользовательскому интерфейсу

0 голосов
/ 12 сентября 2011

Этого можно достичь с помощью WriteableBitmap. 

    public void LoadThumbAsync(Stream src, 
                    WriteableBitmap bmp, object argument)  
    {  
        ThreadPool.QueueUserWorkItem(callback =>  
        {  
            bmp.LoadJpeg(src);  
            src.Dispose();  
            if (ImageLoaded != null)  
            {  
                Deployment.Current.Dispatcher.BeginInvoke(() =>  
                {  
                    ImageLoaded(bmp, argument);  
                });  
            }  
        });  
    }

Но вы должны создать WriteableBitmap в потоке пользовательского интерфейса, тогда загрузка может быть выполнена в другом потоке. 

    void DeferImageLoading( Stream imgStream )  
    {  
        // we have to give size  
        var bmp = new WriteableBitmap(80, 80);  
        imageThread.LoadThumbAsync(imgStream, bmp, this);  
    }

См. Больше объяснений в этом сообщении в блоге

...