イメージオブジェクトをC＃で処理する場合のベストプラクティスは何ですか？

Question

私は次の2つのコードを持っています。第1のものは、第2のものが正しく処分している間に、明示的にImageオブジェクトを処分していません。プロダクションコードでどちらを使用するかをお勧めします。

private bool SavePatientChartImage(byte[] ImageBytes, string ImageFilePath, string IMAGE_NAME, int rotationAngle) 
    { 
     bool success = false; 
     System.Drawing.Image newImage; 
     try 
     { 
      using (MemoryStream stream = new MemoryStream(ImageBytes)) 
      { 
       newImage = System.Drawing.Image.FromStream(stream); 
       switch (rotationAngle) 
       { 
        case 90: 
         newImage.RotateFlip(RotateFlipType.Rotate90FlipNone); 
         break; 
        case 180: 
         newImage.RotateFlip(RotateFlipType.Rotate180FlipNone); 
         break; 
        case 270: 
         newImage.RotateFlip(RotateFlipType.Rotate270FlipNone); 
         break; 
        default: 
         newImage = newImage; 
         break; 
       } 
       newImage.Save(Path.Combine(ImageFilePath, IMAGE_NAME)); 
       success = true; 
      } 
     } 
     catch (Exception ex) 
     { 
      success = false; 
     } 
     return success; 
    } 

と

private bool SavePatientChartImage(byte[] ImageBytes, string ImageFilePath, string IMAGE_NAME, int rotationAngle) 
    { 
     bool success = false; 
     System.Drawing.Image newImage; 
     try 
     { 
      using (MemoryStream stream = new MemoryStream(ImageBytes)) 
      { 
       using(newImage = System.Drawing.Image.FromStream(stream)) 
       { 
       switch (rotationAngle) 
       { 
        case 90: 
         newImage.RotateFlip(RotateFlipType.Rotate90FlipNone); 
         break; 
        case 180: 
         newImage.RotateFlip(RotateFlipType.Rotate180FlipNone); 
         break; 
        case 270: 
         newImage.RotateFlip(RotateFlipType.Rotate270FlipNone); 
         break; 
        default: 
         newImage = newImage; 
         break; 
       } 
       newImage.Save(Path.Combine(ImageFilePath, IMAGE_NAME)); 
       success = true; 
       } 
      } 
     } 
     catch (Exception ex) 
     { 
      success = false; 
     } 
     return success; 
    } 

どちら宗教的に従うこと。お勧めします

Answer 1

常に使い捨て可能なインスタンスをどこかに配置してください。だから後者を取る。

あなたはそれが少し読みやすくできます。

 using (MemoryStream stream = new MemoryStream(ImageBytes)) 
     using(var newImage = System.Drawing.Image.FromStream(stream)) 
     { 
     // ... 

注：これは、usingステートメント外で宣言された変数を持ってしても意味がありません。あなたは外で使うべきではありません。