Dlaczego Bitmap powoduje regułę CA2000, ale obraz nie?

Question

Jest wiele pytań na temat SO lamentujących na tym, że Code Analysis reguła CA2000 jest stosowana zbyt rygorystycznie przez VS2010, ale wydaje mi się, że natknąłem się na przypadek, w którym należy go zastosować, ale nie jest.

Rozważmy następujący kod:

Image srcImage = Image.FromFile(source); 
Bitmap newImage = new Bitmap(newWidth, newHeight); 

using (Graphics gr = Graphics.FromImage(newImage)) 
{ 
    gr.DrawImage(srcImage, new Rectangle(0, 0, newWidth, newHeight)); 
} 
newImage.Save(destination, ImageFormat.Jpeg); 

Teraz jeśli biegnę analizy kodu w Visual Studio 2010 na ten temat, to narzekają newImage nie są umieszczone (Easy Fix, umieścić go w innym wykorzystaniem bloku) , ale nie narzeka na srcImage (która również ma metodę Dispose(), której nigdy nie wywołuję). Czy ktoś wie, dlaczego analiza kodu nie narzeka tutaj?

Answer 1

Dobrze powinno „narzekają srcImage” zbyt jednak Chyba że nie narzekać, bo jesteś przekazaniem go do metody DrawImage „gr.DrawImage(srcImage, new Rectangle(0, 0, newWidth, newHeight));”, więc albo nie jest wystarczająco inteligentny, aby wiedzieć, że nie będzie używał więcej akcji po zwróceniu metody, a może założył, że użyłeś jej w instancji gr, która zostanie usunięta. W każdym razie powinieneś użyć using dla srcImage, tak jak to, co robisz z newImage i nie postępuj zgodnie z analizą kodu w tym zakresie.

Answer 2

CA2000 i podobne/pokrewne zasady CA2213 (DisposableFieldsShouldBeDposed) i CA1001 (TypesThatOwnDisposableFieldsShouldBeDisposable) są raczej surowe, jeśli chodzi o sposób, w jaki rozpoznają "własność" produktu jednorazowego użytku. Biorą pod uwagę, że Twój kod jest właścicielem jednorazowej instancji, jeśli do utworzenia instancji bezpośrednio w kodzie wykorzystywany jest konstruktor instancji. Ponieważ używasz Image.FromFile do utworzenia instancji dla srcImage, reguła nie rozpoznaje Twojego kodu jako właściciela.

Jeśli nie zgadzasz się z tą zasadą, możesz utworzyć raport o błędzie pod numerem https://connect.microsoft.com/visualstudio/feedback. (Jeśli dbasz o zasady pola jednorazowego, możesz zagłosować na istniejącą propozycję https://connect.microsoft.com/VisualStudio/feedback/details/485291/typesthatowndisposablefieldsshouldbedisposable-rule-ca1001-is-too-permissive, gdy jesteś na tym.)