バグ修正

今日の目標→

今日はバグの修正だけ。

●修正箇所

前回のForm1.csで、ReadNetImage()メソッドの次の部分に問題があった。

return Image.FromStream(stream);

これは次の3行のコードに差し替える。

MemoryStream mem = new MemoryStream();
stream.CopyTo(mem);
return Image.FromStream(mem);

そもそもWebClient.OpenRead()の説明で、「ストリームはすぐ閉じろ」的なことが書いてあるから、即クローズしていたのである。しかも、確実にストリームをクローズできるように、わざわざtry～catch～finally構文とか使ってた。

ところが、Image.FromStream()の説明を見たら、「読み込んだ画像の使用中はストリームを閉じちゃダメ」だって。なんだよそれ。

結局、ネットデータのストリームはメモリにコピーして接続を切る。そしてメモリストリームはクローズせず後始末をガベコレにまかせる、という折衷案にたどり着いたのであった。

バグ修正後のForm1.csが下記である。

// プロジェクト：NetJob // ファイル名：Form1.cs using System; using System.Net; using System.Drawing; using System.IO; using System.Windows.Forms; using MarugeProject; namespace NetJob { public partial class Form1 : Form { // コントロールなど private SplitContainer MySplitBox; private RichTextBox MyRichTextBox; private PictureBox MyPictBox; private TextBox MyInputBox; private Button MyGetButton; private RichReporter MyReporter; // フォームのコンストラクタ public Form1() { InitializeComponent(); // タイトルバーにプログラム名を表示する this.Text = Application.ProductName; // ウィンドウサイズを400x400にする this.Width = 400; this.Height = 400; // 上下分割コンテナを作る MySplitBox = new SplitContainer(); MySplitBox.Name = "MySplitBox"; MySplitBox.Anchor = AnchorStyles.Left | AnchorStyles.Right | AnchorStyles.Top | AnchorStyles.Bottom; MySplitBox.Left = 8; MySplitBox.Top = 8; MySplitBox.Width = this.ClientSize.Width - 16; MySplitBox.Height = this.ClientSize.Height - 40; MySplitBox.BorderStyle = BorderStyle.FixedSingle; MySplitBox.Orientation = Orientation.Horizontal; MySplitBox.SplitterDistance = 200; // 分割コンテナの上側にピクチャボックスを配置する MyPictBox = new PictureBox(); MyPictBox.Name = "MyPictBox"; MyPictBox.BorderStyle = BorderStyle.None; MyPictBox.Dock = DockStyle.Fill; MySplitBox.Panel1.Controls.Add(MyPictBox); // 分割コンテナの下側にリッチテキストボックスを配置する MyRichTextBox = new RichTextBox(); MyRichTextBox.Name = "MyRichTextBox"; MyRichTextBox.BorderStyle = BorderStyle.None; MyRichTextBox.Dock = DockStyle.Fill; MySplitBox.Panel2.Controls.Add(MyRichTextBox); // 「画像を読み込む」ボタンを作る MyGetButton = new Button(); MyGetButton.Name = "MyGetButton"; MyGetButton.Anchor = AnchorStyles.Right | AnchorStyles.Bottom; MyGetButton.Left = this.ClientSize.Width - 128; MyGetButton.Top = this.ClientSize.Height - 30; MyGetButton.Width = 120; MyGetButton.Text = "画像を読み込む(&R)"; MyGetButton.Click += MyGetButton_Click; // テキスト入力ボックスを作る MyInputBox = new TextBox(); MyInputBox.Name = "MyInputBox"; MyInputBox.Anchor = AnchorStyles.Left | AnchorStyles.Right | AnchorStyles.Bottom; MyInputBox.Left = 8; MyInputBox.Top = MyGetButton.Top + 2; MyInputBox.Width = this.ClientSize.Width - MyGetButton.Width - 18; MyInputBox.Text = "http://"; // 各コントロールをフォームに配置する this.SuspendLayout(); this.Controls.Add(MySplitBox); this.Controls.Add(MyGetButton); this.Controls.Add(MyInputBox); this.ResumeLayout(); // テキスト出力の準備 MyRichTextBox.DetectUrls = false; MyRichTextBox.Font = new Font("MS UI Gothic", 12); MyReporter = new RichReporter(MyRichTextBox); // 初期メッセージを出力 MyReporter.Print("使用方法：\n"); MyReporter.Print("URLを入力 → 「画像を読み込む」をクリック\n\n"); } // 「画像を読み込む」ボタンをクリック private void MyGetButton_Click(object sender, EventArgs e) { MyRichTextBox.Clear(); MyReporter.Print("接続します\n"); Image image = ReadNetImage(MyInputBox.Text); if (image != null) { MyReporter.Print("成功しました\n"); MyPictBox.Image = image; } else { MyReporter.ColorPrint(Color.Red, "失敗しました\n"); } } // ネットにアクセスして画像を読み込む private Image ReadNetImage(string URL) { Stream stream = null; try { // ネットから画像を読み込み、Image型で返す WebClient client = new WebClient(); stream = client.OpenRead(URL.Trim()); // return Image.FromStream(stream); // ←バグるので下記に変更 MemoryStream mem = new MemoryStream(); stream.CopyTo(mem); return Image.FromStream(mem); } catch { // 読み込み失敗ならnullを返す return null; } finally { // 成功でも失敗でも、戻る前に必ずストリームを閉じる if (stream != null) { stream.Close(); stream.Dispose(); } } } } }

旧版のプログラムはアニメーションGIFを読み込ませると不正終了した。今度は大丈夫だろう…たぶん…。