L'objet est actuellement en cours d'utilisation ailleurs

J'obtiens cette erreur, et il semble que c'est parce que l'image même de l'objet est accessible par différents threads. Cependant, je suis en utilisant des verrous partout avec elle.

public class MySingleInstanceClass
{
    private Object locker = new Object();

    private Bitmap myImage = new Bitmap(100, 100);

    public Bitmap MyImage
    {
        get
        {
            lock (locker)
                return myImage;
        }
        private set
        {
            lock (locker)
                myImage = value;
        }
    }

    private void Refresh()
    {
        lock (locker)
        {
            var g = Graphics.FromImage(myImage);
            //do more processing
        }
    }
}

Classe MySingleInstanceClass aura qu'une seule instance. Les appels à MyImage et Refresh() peut venir de différents threads. Comme je le comprends, le code à l'intérieur lock(locker) ne sera pas exécutée jusqu'à ce qu'il a fini dans un autre thread, mais j'ai toujours l'erreur. Quelqu'un peut-il point une faille dans le code?

Exception ressemble à ça:

Une exception de première chance de type 'System.InvalidOperationException' s'est produite dans System.Drawing.dll

Erreur: l'Objet est actuellement en usage ailleurs.

au Système.De dessin.Les graphiques.FromImage(Image)

à (les points de la ligne contenant var g = Graphiques.FromImage(myImage);)

  • la serrure à l'intérieur de la getter/setter semble inutile....
  • que pensez-vous de ce que vous atteindre avec verrouillage de la lecture (et le privé setter?) de MyImage? Je suis juste en pensant à haute voix pour ne présumez pas que j'ai la solution, mais ne serait-il pas plus de sens si le verrouillage de comportement dans la classe de consommateurs de cette classe?
  • et bien, je vais faire en sorte qu'une thread qui appelle MyImage attend jusqu'à ce que le thread qui appelle Refresh() termine le bloc de code là, sinon je vais retourner myImage avant qu'il soit terminé le traitement. Pensez-vous que je n'en avez pas besoin? Pourquoi?
  • Pouvez-vous afficher de message d'exception à l'intérieur de l'exception le cas échéant?
  • C'est fondamentalement ce qui est en question le titre; j'ai ajouté au bas de la page.
InformationsquelleAutor Georgii Oleinikov | 2013-02-02
  1. 9

    la locker objet n'est pas statique; ainsi, chaque nouvelle instance crée son propre casier; vous devez créer locker comme statique afin d'empêcher l'accès à partir d'autres threads si l'utilisation de plusieurs objets.

    private static Object locker = new Object();

    Pour unique objet de scénario, l'utilisation d'un non statique de la classe de niveau variable comme un casier est bon. Si vous êtes employant ce scénario j'ai l'impression que la mise en œuvre de Singleton a quelques problèmes.

    Mise à JOUR:

    public sealed class MySingleInstanceClass
{
    private static volatile MySingleInstanceClass instance;
    private static object syncRoot = new Object();
    private Bitmap myImage;

    private MySingleInstanceClass() 
    {
        myImage = new Bitmap(100, 100);
    }

    public static MySingleInstanceClass Instance
    {
        get
        {
            if (instance == null)
            {
                lock (syncRoot)
                {
                    if (instance == null)
                        instance = new MySingleInstanceClass();
                }
            }

            return instance;
        }
    }  

    public Bitmap MyImage
    {
        get
        {
            lock (syncRoot)
                return myImage;
        }
        private set
        {
            lock (syncRoot)
                myImage = value;
        }
    }

    public void Refresh()
    {
        lock (syncRoot)
        {
            var g = Graphics.FromImage(myImage);
            //do more processing
        }
    }

}
    • Il y est seulement d'une seule instance, et donc il y a seulement un casier (la classe est singletone)
    • ensuite le code que vous avez fournie ne correspond pas à la question?
    • Pouvez-vous expliquer pourquoi les non-statique de l'objet ne peut pas être utilisé pour bloquer l'accès à des membres de l'instance?
    • Est-il vraiment comment je mettre en œuvre la singletone pour corriger cette erreur?
    • Pouvez-vous expliquer ce qui s'est passé si je vais l'enlever static mot-clé de la déclaration de syncRoot objet dans les deux singletion et multi-instance scénario?
    • Merci pour l'excellent exemple de thread-safe singletone, mais comme je l'ai dit je n'ai qu'une instance de la classe (je ne suis pas en utilisant singletonne dans sa mise en œuvre parce que cadre, je suis à l'aide d'malheureusement interdit aux champs statiques). Il semble que le problème est résolu, en retournant une copie de l'image Bitmap.
    • Toute autre option autre que le clonage de l'image bitmap..je ne suis pas autorisé à cloner

    InformationsquelleAutor daryal
  2. 5

    Il n'a pas d'importance si l'objet est verrouillé est statique ou non. Le problème est que le lock(locker) à l'intérieur de la méthode de lecture déverrouille dès que l'image est renvoyée. Le retour de la référence à l'image n'est pas protégé par la serrure et peut être modifié dans le même temps comme un appel à Refresh se produit.

    Une solution possible serait de se verrouiller sur le bitmap elle-même, mais qui peut introduire des blocages si pas fait avec soin.

    • Que faire si j'ai retourner la copie de l'image bitmap dans la lecture?
    • Si vous remplacez le return myImage avec return (Bitmap)myImage.Clone() vous permettra également d'éviter l'erreur. Mais sans voir le code qui utilise MySingleInstanceClass il est difficile de dire si ce serait une bonne solution.
    • Il semble que je ne peux pas directement ajouter un commentaire à l'autre solution proposée MySingleInstanceClass donc je l'écris ici. Imaginez 2 threads. Thread 1 seulement appelle à plusieurs reprises MySingleInstanceClass.Instance.Refresh(). Thread 2 appels MySingleInstanceClass.Instance.MyImage, puis modifie le retour de l'image Bitmap. Ce sera la cause de l'exception que vous nous avez montré dans votre question.
    • c'est exactement la même erreur, merci pour ça!
    InformationsquelleAutor Dirk
  3. 1

    Dans mon application, la meilleure solution était:

    • copie dir avec des fichiers à un autre, tmp. catalogue (avec nom guid)
    • utilisation tmp fichiers par utilisateur
    • supprimer tmp catalogue avec des fichiers

    Dans mon application il y a:

    • chaque demande est à 1 min de long
    • les utilisateurs maximal est de 120 (application intranet)
    • personne n'a envie d'attendre 5 à 10 min par raport à générer

    Copier quelques fichiers ajouter environ 0,01-0,2 sek. pour chaque demande, c'est mieux que statique de verrouillage sur l'app et l'utilisateur n'est pas neet attendre 10 min pour raport à générer (10 utilisateurs, cliquez sur le bouton générer au même moment).

            private void DirectoryCopy(string sourceDirName, string destDirName, bool copySubDirs)
{
//Get the subdirectories for the specified directory.
DirectoryInfo dir = new DirectoryInfo(sourceDirName);
DirectoryInfo[] dirs = dir.GetDirectories();
if (!dir.Exists)
{
throw new DirectoryNotFoundException(
"Source directory does not exist or could not be found: "
+ sourceDirName);
}
//If the destination directory doesn't exist, create it. 
if (!Directory.Exists(destDirName))
{
Directory.CreateDirectory(destDirName);
}
//Get the files in the directory and copy them to the new location.
FileInfo[] files = dir.GetFiles();
foreach (FileInfo file in files)
{
string temppath = Path.Combine(destDirName, file.Name);
file.CopyTo(temppath, false);
}
//If copying subdirectories, copy them and their contents to new location. 
if (copySubDirs)
{
foreach (DirectoryInfo subdir in dirs)
{
string temppath = Path.Combine(destDirName, subdir.Name);
DirectoryCopy(subdir.FullName, temppath, copySubDirs);
}
}
}
private void DeleteReportExecutionDirectory(string dirPath)
{
System.IO.DirectoryInfo downloadedMessageInfo = new DirectoryInfo(dirPath);
foreach (FileInfo file in downloadedMessageInfo.GetFiles())
{
file.Delete();
}
foreach (DirectoryInfo dir in downloadedMessageInfo.GetDirectories())
{
dir.Delete(true);
}
downloadedMessageInfo.Delete();
}
    InformationsquelleAutor zchpit
  4. 0

    vous pouvez cloner l'image avant de l'envoyer à la méthode 

                    Image newimg = (Image)img.Clone();
    InformationsquelleAutor user3290286