c#でリストボックスの内容を選択した後ボタンを押すと選択した内容が勝手に変わる

前提・実現したいこと

私は現在.NET Frameworkを使用してc#で簡単なサウンドプレイヤーを作成しています。
今回は作成途中に以下の問題が発生しているのでどうすればいいか分からず困っています。
実現したいことは、項目を選択した後、再生・停止ボタンをクリックしてもラベルの内容が変更されず、選択した音楽ファイルが正しく再生されるようにすることです。まだまだ初心者の私ですが、教えていただければ幸いです。

###発生する問題
リストボックスの中身を2つ以上にした時に,どれか1つの項目を選択するとラベルには選択した項目のファイルの名前が表示される。しかし、再生・停止ボタンをクリックすると、ラベルにリストボックスの最後の項目のファイルの名前が表示され、リストボックスの最後の項目の音楽ファイルが再生されてしまう。

＜コンパイルしたファイルを実行したときの状態＞

'a.wav'がデフォルトで表示されている

＜'b.wav'をクリックし選択したときの状態＞

'b.wav'(選択した項目の名前)が正しく表示されている

＜'再生'ボタンをクリックしたときの状態＞

'b.wav'を選択していたのに勝手に'c.wav'になりc.wavの音楽ファイルが再生されてしまう

ソースコード

Formクラスを使用し、以下のようにソースコードを書きました。

//コード
using System;
using System.Windows.Forms;
class part05
{
	[STAThread]
	static void Main()
    {
		Application.EnableVisualStyles();
		Application.Run(new Player()); 
    }

}

class Player : Form
{
	Label label = new Label();
	ListBox listbox;
	Button Play;
	Button Stop;
	System.Media.SoundPlayer soundplayer;
	string[] paths;
	string[] safepaths;
	public Player()
    {
		label.Location = new Point(115, 20);
		label.AutoSize = true;
		this.FormBorderStyle = FormBorderStyle.FixedSingle;
		this.MaximizeBox = false;
		this.Size = new Size(750, 600);
		this.Location = new Point(100, 50);
		soundplayer = new System.Media.SoundPlayer();
		paths = new string[]{ /*音楽ファイルのパス*/ };

		listbox = new ListBox(){
			Location = new Point(375, 10),
			Size = new Size(300, 500),
			DrawMode = DrawMode.OwnerDrawFixed, };

		safepaths = new string[]{ /*音楽ファイルの名前*/ };

        listbox.Items.AddRange(safepaths);
	    listbox.DrawItem += new DrawItemEventHandler(DrawPlaylist);

		this.Controls.Add(listbox);
		this.Controls.Add(label);

		Play = new Button(){
			Location = new Point(20, 50),
			Text = "再生", };

		Play.Click += new EventHandler(PlayClick);

		Stop = new Button(){
			Location = new Point(100, 50),
			Text = "停止",};

		Stop.Click += new EventHandler(StopClick);

		this.Controls.AddRange(new Control[]{ Play, Stop } );
	}

	void DrawPlaylist(object sender, DrawItemEventArgs e){

	    e.DrawBackground();
		string text = (sender as ListBox).Items[e.Index].ToString();
		Brush fb = new SolidBrush(e.ForeColor);
		e.Graphics.DrawString(text, e.Font, fb, e.Bounds);
		e.DrawFocusRectangle();
		PlaySoundChange(e.Index, text); }

	void PlaySoundChange(int index, string text){
		soundplayer.Stop();
//X　　　Console.WriteLine(index);
		label.Text = text;
		soundplayer.SoundLocation = paths[index];
		soundplayer.Load(); }

	void PlayClick(object sender, EventArgs e){ soundplayer.Play(); }

	void StopClick(object sender, EventArgs e){ soundplayer.Stop(); } 
}

試したこと

リストの項目が3つのときにPlaySoundChange()のXの部分に、'Console.WriteLine(index);' を記述し、実行しました。結果は次のようになりました。
0
0
0
0
0
0
0
1
2
0　<<実行した瞬間にここまで表示された
0
0
0
0　<<1つ目の項目を選択した瞬間にここまで表示された
0
1
2 <<再生ボタンをクリックした瞬間にここまで表示された

自分でもググって調べて見たのですが、初心者の私にはどのサイトがこの現象について書かれているのかが分かりませんでした。自分が分かったのは、再生ボタンをクリックすると最終的にe.Indexの部分が最大の数になるということだけでした。回避方法として２回連続でe.Indexが同じ値だったときだけPlaySoundChange()を呼び出すという方法も思いつきましたが、もっと簡単に実装できる方法がないか知りたいです。

補足情報

C:/Windows/Microsoft.NET/Framework\v4.0.30319\csc.exeを利用してコンパイルしています。
Visual Studio は使用していません。

退会済みユーザー

2020/12/11 17:17

ソースがインデントが崩れて読みづらいので、コードの挿入で記入してください。

XPChannel

2020/12/12 04:06

ソースコードの挿入によってソースコードを記述しました。見づらくなってしまい申し訳ありませんでした。

行動規範の内容に同意します

回答2件

ベストアンサー

DrawItemイベントはあくまでListBoxの項目を描画するイベントなので、そこで PlaySoundChange を呼んでいるのに問題があると思われます。そもそも、描画用のイベントでそれに関係ない処理をするべきではありません。ListBoxの選択に連動する処理をするとすれば、SelectedIndexChanged イベントですね。

自分が分かったのは、再生ボタンをクリックすると最終的にe.Indexの部分が最大の数になるということだけでした。回避方法として２回連続でe.Indexが同じ値だったときだけPlaySoundChange()を呼び出すという方法も思いつきましたが、もっと簡単に実装できる方法がないか知りたいです。

何故そのような挙動になるかというと、DrawItemイベントで実行された DrawItemEventArgs で再生曲のインデックスを指定しているからです。ListBox項目の再描画は上から順に行われるので、一番最後の項目が PlaySoundChange に指定されたインデックスになるでしょう。

C#
1    void DrawPlaylist(object sender, DrawItemEventArgs e){
2
3        e.DrawBackground();
4        string text = (sender as ListBox).Items[e.Index].ToString();
5        Brush fb = new SolidBrush(e.ForeColor);
6        e.Graphics.DrawString(text, e.Font, fb, e.Bounds);
7        e.DrawFocusRectangle();
8
9        //ここの e.Index は描画対象のListBoxの項目のインデックス
10        //（選択している項目ではない）
11        PlaySoundChange(e.Index, text); }

あと、VisualStudioを使用しない理由は良く判りませんが、基本的に使用すべきです。デバッグ効率やフォームデザインからコードに連携するまでの効率が全然違います。

投稿2020/12/12 04:12

編集2020/12/12 04:59

退会済みユーザー

総合スコア0

XPChannel

2020/12/12 05:02

回答ありがとうございます。なるほどよく分かりました。PlaySoundChangeはSelectedIndexChangeを使用し別に呼び出すことにしようと思います。ちなみにVisualStudioを使用した方がいいのは自分でもわかってるのですがめちゃくちゃ重くなるんです。しかも起動に長いときは５分ぐらいかかるんですよ。なので使用していません。そこの部分もご教授いただきありがとうございました。

退会済みユーザー

2020/12/12 05:04

起動5分はちときついですね。可能であれば、メモリの増設を検討したほうがよいでしょう。IDEが使えない事によるタイムロスも大きいので、すぐにペイ出来る筈です。タイムイズマネーです。

XPChannel

2020/12/12 05:09

分かりました。検討してみます。

行動規範の内容に同意します