C# と VB.NET の質問掲示板

いつも大変お世話になっております。
ちょっとお時間宜しいでしょうか？

VisualC#を使用し、プログラムを組んでいます。

配列の値をソートすると、同じタイミングで打ち込んだ値も同じ順番にソートするようにしたいのですが、どのような考え方がありますでしょうか？

すみません。
上手く書けなかったので例を上げさせていただきます。


Combobox1、textbox1,2,3があり、
Combobox1に"GJ1"を入力後、
textbox1に"1"、textbox2に"5"、textbox3に"9"を入力し、
botton1をクリックすると、その値を配列に代入するようにしています。

この操作を3回繰り返し、

combobox1[3]={GJ1,GJ3,GJ2}
textbox1[3]={1,5,9}
textbox2[3]={7,2,3}
textbox3[3]={4,6,8}

を出力したとします。

Array.Sort(combobox1);
のコマンドを使用すると、

combobox1[3]={GJ1,GJ2,GJ3}
textbox1[3]={1,5,9}
textbox2[3]={7,2,3}
textbox3[3]={4,6,8}

となると思うのですが、
同じタイミングで打ち込んだtextboxの値もソートしたいのです。

ですので、comboboxがソートされた場合、
combobox1[3]={GJ1,GJ2,GJ3}
textbox1[3]={1,9,5}
textbox2[3]={7,3,2}
textbox3[3]={4,8,6

というかたちにしたいのですが、
どのようにソートもしくは並び替えればよろしいでしょうか？

お手数おかけして申し訳ございませんが、お力添え願えませんでしょうか。

■No76532 (ゆーきゃん さん) に返信

始めに長文失礼いたします。

ファイル書き込み時にエラーが起こり、回避方法が分かりません。
どなたかご教授願えませんでしょうか。

こちらがプログラムになります

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.IO;
using System.Runtime.Serialization;
using System.Runtime.Serialization.Formatters.Binary;
using System.Collections;
using System.Drawing.Drawing2D;
using System.Runtime.InteropServices;


namespace ex
{

    public partial class Form1 : Form
    {

        public Form1()
        {
            InitializeComponent();
        }

        public class data
        {

            public string cb;
            public int tb_1;
            public int tb_2;
            public int tb_3;

            public data(string c, int t1, int t2, int t3)
            {
                this.cb = c;
                this.tb_1 = t1;
                this.tb_2 = t2;
                this.tb_3 = t3;
            }
        }

        private List<data> lst;

        private void Form1_Load(object sender, EventArgs e)
        {
            lst = new List<data>();
        }


        private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
        {
            // データの読み出し＆表示
        }

        private void comboBox1_Leave(object sender, EventArgs e)
        {
            comboBox1.Items.Add(comboBox1.Text);
        }


        public void button1_Click(object sender, EventArgs e)
        {
            lst.Add(new data(
                comboBox1.Text,
                int.Parse(textBox1.Text),
                int.Parse(textBox2.Text),
                int.Parse(textBox3.Text)));

            // ボタン１が押された際、comboBox/textBoxを空白にする
            comboBox1.Text = "";
            textBox1.Clear();
            textBox2.Clear();
            textBox3.Clear();

        }


        private void button2_Click(object sender, EventArgs e)
        {
            // comboBox1 の文字列で並び替え
            lst.Sort((x, y) => x.cb.CompareTo(y.cb));

            using (BinaryWriter w = new BinaryWriter(File.OpenWrite(@"c:\data\test.dat")))
            {
                w.Write(lst);   // ←　ここでエラー           
            }

        }

        private void button3_Click(object sender, EventArgs e)
        {

            // 保存したファイルの呼び出し

        }

        // アプリケーション終了
        private void button4_Click(object sender, EventArgs e)
        {
            Close();
        }

        // エンターを押すと次のフォーカスに移動。
        private void Form1_KeyDown(object sender, KeyEventArgs e)
        {
            if (e.KeyCode == Keys.Enter)
            {
                if (!e.Control)
                {
                    this.SelectNextControl(this.ActiveControl, !e.Shift, true, true, true);
                }
            }
        }
    }
}


「 w.Write(lst);　」でファイルに書き込もうとする際にエラーが起こります。

error CS1502: 'System.IO.BinaryWriter.Write(bool)' に最も適しているオーバーロード メソッドには無効な引数がいくつか含まれています。

error CS1503: 引数 1: 'System.Collections.Generic.List<ex.Form1.data>' から 'bool' に変換できません

という内容です。

また、「lst.Sort((x, y) => x.cb.CompareTo(y.cb));」のx,yには何の値が入っているのでしょうか？デバックするごとに違う値が入ってきています。
ソートはできているのですが、どのような動きでソートされているかが分かりませんでした。

質問ばかりで申し訳ないのですが、エラー回避方法のヒントを頂けないでしょうか。
   

■No76542 (ゆーきゃん さん) に返信
> 「 w.Write(lst);　」でファイルに書き込もうとする際にエラーが起こります。
> error CS1502: 'System.IO.BinaryWriter.Write(bool)' に最も適しているオーバーロード メソッドには無効な引数がいくつか含まれています。
> error CS1503: 引数 1: 'System.Collections.Generic.List<ex.Form1.data>' から 'bool' に変換できません

エラーの通りで、BinaryWriter.Write にあなたが作ったクラスを処理するメソッドはありません。
クラスを自分で作り、それをファイルに書き落としたいなら、どのように書き込むかをあなたが実装しなければなりません。

一例
 1.最初に List の要素数を書く。
 2.foreach ループで要素ごとに、cb, tb_1, tb_2, tb_3 を Write する。

w.Write(lst.Count);
foreach (data item in lst)
{
  w.Write(item.cb);
  w.Write(item.tb_1);
  w.Write(item.tb_2);
  w.Write(item.tb_3);
}


> また、「lst.Sort((x, y) => x.cb.CompareTo(y.cb));」のx,yには何の値が入っているのでしょうか？
> デバックするごとに違う値が入ってきています。
> ソートはできているのですが、どのような動きでソートされているかが分かりませんでした。

x と y にはソートアルゴリズムの途中で、どのように並べ替えるべきか決めるために、リストの中のある値と別の値を取り出したものです。
ソートのアルゴリズムについては Sort メソッド自体に説明があります。
https://msdn.microsoft.com/ja-jp/library/w56d4y5z

■No76546 (Azulean さん) に返信

> エラーの通りで、BinaryWriter.Write にあなたが作ったクラスを処理するメソッドはありません。
> クラスを自分で作り、それをファイルに書き落としたいなら、どのように書き込むかをあなたが実装しなければなりません。
> 
> 一例
>  1.最初に List の要素数を書く。
>  2.foreach ループで要素ごとに、cb, tb_1, tb_2, tb_3 を Write する。
> 
> w.Write(lst.Count);
> foreach (data item in lst)
> {
>   w.Write(item.cb);
>   w.Write(item.tb_1);
>   w.Write(item.tb_2);
>   w.Write(item.tb_3);
> }

お答えありがとうございます。
要素ごとにwriteする必要があったのですね。
勉強になりました。 
「w.Write(lst);」だけで、Azuleanさんが書いて下さったプログラムの働きがあると思っておりました。

 
> x と y にはソートアルゴリズムの途中で、どのように並べ替えるべきか決めるために、リストの中のある値と別の値を取り出したものです。
> ソートのアルゴリズムについては Sort メソッド自体に説明があります。
> https://msdn.microsoft.com/ja-jp/library/w56d4y5z

ソートに関するサイトのURLを載せて頂きありがとうございます。
参考にさせて頂きたいと思います。 

皆様のおかげで無事に思い通りの結果を出力することができました。
大変感謝しております。ありがとうございます。

最後にもう一つだけ宜しいでしょうか。

button3を押し、前回書き込んだファイルを読み込み、コンボボックスの要素選択時に、要素に合った文字と数値を表示させるプログラムを作成しているのですが、これもListを使用した書き方など考えられるのでしょうか？

ここからは配列でやるしか思いつかないのですが、どうなのでしょうか？

他の人の役に立つかわかりませんが、完成したプログラムを提示させて頂きました。
また、ここをこうするとよりよくなるなどありましたら、ご教授願えませんでしょうか？


using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.IO;
using System.Runtime.Serialization;
using System.Runtime.Serialization.Formatters.Binary;
using System.Collections;
using System.Drawing.Drawing2D;
using System.Runtime.InteropServices;


namespace ex
{

    public partial class Form1 : Form
    {

        public Form1()
        {
            InitializeComponent();
        }

        public class data
        {

            public string cb;
            public int tb_1;
            public int tb_2;
            public int tb_3;

            public data(string c, int t1, int t2, int t3)
            {
                this.cb = c;
                this.tb_1 = t1;
                this.tb_2 = t2;
                this.tb_3 = t3;
            }
        }

        private List<data> lst;
        int yomikomi;

        public string[] cb_hai = new string[3]; // 長さが3の整数型配列を用意。
        public int[] tb_1_hai = new int[3]; // 長さが3の整数型配列を用意。
        public int[] tb_2_hai = new int[3]; // 長さが3の整数型配列を用意。
        public int[] tb_3_hai = new int[3]; // 長さが3の整数型配列を用意。


        private void Form1_Load(object sender, EventArgs e)
        {
            lst = new List<data>();
        }


        private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
        {
            // データの読み出し＆表示
            if (yomikomi == 1)
            {

                FileStream fs = new FileStream(
                     @"c:\data\test.dat",
                         FileMode.Open,
                           FileAccess.Read);


                int fileSize = (int)fs.Length; // ファイルのサイズ

                byte[] buf = new byte[fileSize]; // データ格納用配列

                int readSize; // Readメソッドで読み込んだバイト数
                int remain = fileSize; // 読み込むべき残りのバイト数
                int bufPos = 0; // データ格納用配列内の追加位置


                while (remain > 0)
                {
                    // 14 Bytesずつ読み込む 
                    readSize = fs.Read(buf, bufPos, Math.Min(14, remain));

                    bufPos += readSize;
                    remain -= readSize;

                    for (int i = 0; i < 3; i++)
                    {
                        if (cb_hai[i] == comboBox1.SelectedItem.ToString())
                        {
                            comboBox1.Text = cb_hai[i].ToString();
                            textBox1.Text = tb_1_hai[i].ToString();
                            textBox2.Text = tb_2_hai[i].ToString();
                            textBox3.Text = tb_3_hai[i].ToString();
                        }
                    }
                }
            }
        }

        private void comboBox1_Leave(object sender, EventArgs e)
        {
            comboBox1.Items.Add(comboBox1.Text);
        }


        public void button1_Click(object sender, EventArgs e)
        {
            lst.Add(new data(
                comboBox1.Text,
                int.Parse(textBox1.Text),
                int.Parse(textBox2.Text),
                int.Parse(textBox3.Text)));

            // ボタン１が押された際、comboBox/textBoxを空白にする
            comboBox1.Text = "";
            textBox1.Clear();
            textBox2.Clear();
            textBox3.Clear();

        }


        private void button2_Click(object sender, EventArgs e)
        {
            // comboBox1 の文字列で並び替え
            lst.Sort((x, y) => x.cb.CompareTo(y.cb));

            using (BinaryWriter w = new BinaryWriter(File.OpenWrite(@"c:\data\test.dat")))
            {
                //w.Write(lst.Count);   // ←　ここでエラー 
                foreach(data Item in lst)
                {
                    w.Write(Item.cb);
                    w.Write(Item.tb_1);
                    w.Write(Item.tb_2);
                    w.Write(Item.tb_3);

                }
            }

        }

        // 保存したファイルの呼び出し
        private void button3_Click(object sender, EventArgs e)
        {

            yomikomi = 1;

            comboBox1.Items.Clear();

            // 簡単読込             
            FileStream fs = new FileStream(@"c:\data\test.dat",
                                             FileMode.Open,
                                             FileAccess.Read);

            BinaryReader br = new BinaryReader(fs);

            for (int i=0; i<3; i++)
            {
                cb_hai[i] = br.ReadString();
                tb_1_hai[i] = br.ReadInt32();
                tb_2_hai[i] = br.ReadInt32();
                tb_3_hai[i] = br.ReadInt32();

                comboBox1.Items.Add(cb_hai[i]);
            }
        }

        // アプリケーション終了
        private void button4_Click(object sender, EventArgs e)
        {
            Close();
        }

        // エンターを押すと次のフォーカスに移動。
        private void Form1_KeyDown(object sender, KeyEventArgs e)
        {
            if (e.KeyCode == Keys.Enter)
            {
                if (!e.Control)
                {
                    this.SelectNextControl(this.ActiveControl, !e.Shift, true, true, true);
                }
            }
        }
    }
}

■No76563 (魔界の仮面弁士 さん) に返信
>> if (cb_hai[i] == comboBox1.SelectedItem.ToString())
> ここでは、『comboBox1.SelectedItem.ToString()』で取得していて、
> 
>> comboBox1.Items.Add(comboBox1.Text);
> ここでは、『comboBox1.Text』で取得していますね。
> 
> コードに対象性が無いようにも見えますが、両者を意図的に使い分けているのでしょうか?
> （役割が異なるため、これらが同じ値を返すこともあれば、別の値を返すこともあります）
> 

迅速なご回答ありがとうございます。
同じ値を返すつもりなので、対称性は持たせたいです。
このままですと違う値を返すことがあるのですね。
今後はしっかり対称性も考えてプログラムを組むよう心掛けたいと思います。

>>lst.Add(new data(
>>　comboBox1.Text,
>>　int.Parse(textBox1.Text),
>>　int.Parse(textBox2.Text),
>>　int.Parse(textBox3.Text)));
> 
 
> TextBox の内容が「int 型に変換できない文字列」だった場合、たとえば
> "" や "-" や "３" や "1.5" や "9876543210" だった場合なにおいては
> .Parse が例外を発生させますが、その点は大丈夫でしょうか。
> 
> TextBox 側に、数値以外を入力できないような制限をかけているなら良いですが、
> そうしたチェックを行っていないのなら、int.TryParse メソッドを試してみると良いかもしれません。
> 数値変換するだけでなく、それが数値として変更できるかどうかの検証も同時に行ってくれますよ。
> (サンプルコードゆえに、細かい点は省略して投稿されているだけなのかもしれませんが…)

今回、textboxには整数のみしか入らないので、このような形に致しました。
 "" や "-" や "３" や "1.5" や "9876543210"や "文字" だった場合、
確かにこの形だとエラーが起こりました。
全対応する書き方とかもありそうですね。
教えて頂いたint.TryParse メソッドと一緒に調べてみたいと思います。

助けて頂いてばかりで恐縮です。
自分も早く回答できる立場になれるよう、頑張っていきたいと思います。