C# DataGridViewの選択された行を削除するプログラムについて

Question

C# のDataGridViewの選択された行の削除について質問があります。

namespace 行削除
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            dataGridView1.ColumnCount = 4;

            dataGridView1.Columns[0].HeaderText = "コード";
            dataGridView1.Columns[1].HeaderText = "商品名";
            dataGridView1.Columns[2].HeaderText = "数量";
            dataGridView1.Columns[3].HeaderText = "金額";

        }

        private void dataGridView1_CellContentClick(object sender, DataGridViewCellEventArgs e)
        {

        }

        private void button1_Click(object sender, EventArgs e)
        {
            DataGridViewSelectedRowCollection src = dataGridView1.SelectedRows; 
            for (int i = src.Count -1; i >= 0; i--)                              // この部分
            {
                dataGridView1.Rows.RemoveAt(src[i].Index);
            }
        }
    }
}

上記のコードの最後のfor分なのですが、for (int i = src.Count -1; i >= 0; i--)　が
どういう働きをしているのかが、理解できず困っています。

ネットで調べたところ、「選択した行のインデックス番号がずれないように、末尾の方から削除します。」
と書いてありました。

お手数をおかけしますが、よろしくお願いいたします。

※Visual Studio 2019を使用しています。
※for文ついては理解しているつもりです。

Comments

[YAmaGNZ]

for (int i = src.Count -1; i >= 0; i--)
このfor文でiがどのような値の遷移をするかは理解できていますか？

[Zuishin]

> 上記のコードの最後のfor分なのですが、for (int i = src.Count -1; i >= 0; i--)　が
> どういう働きをしているのかが、理解できず困っています。

なぜ逆順にしなければならないかがわからないのか、for 文の機能自体を知らないのか、それを書くことで回答の精度が上がると思います。

[fight51722]

YAmaGNZ さん
おそらくですが…。
表の3行目を選択したとき、インデックスは2なので-1をしていて、
i が0より大きい場合、i-- でデクリメントして行数を減らしている、でしょうか？

[fight51722]

Zuishin さん
確かに、説明不足でした。
ご指摘いただいたように、説明を加えようと思います。
ちなみにfor文の機能については理解しているつもりです。

Answer 1

例えば５行のデータがあって、その１～３行目が選択されていたとします。

Index	何行目	選択
0	1行目	※
1	2行目	※
2	3行目	※
3	4行目
4	5行目

for (int i = 0; i < src.Count; i++)
とインデックスを0から開始しインクリメントしていった場合
i=0の時src[i].Indexは0となるので1行目を削除します。

列1	列2	選択
0	2行目	※
1	3行目	※
2	4行目
3	5行目

次にi=1の時のsrc[i].Indexは1となるので3行目を削除することになります。

列1	列2	選択
0	2行目	※
1	4行目
2	5行目

次にi=2の時のsrc[i].Indexは2となるので5行目を削除することになります。

列1	列2	選択
0	2行目	※
1	4行目

このように本来削除してはいけない行が削除されてしまいます。
なので、後ろから削除すれば行がずれないので
for (int i = src.Count -1; i >= 0; i--)
のように後ろからループする処理として作成しています。

---

しかしながら、実はfor (int i = 0; i < src.Count; i++)としても正常に動作します。
RemoveAtを実行した時、dataGridView1.SelectedRowsが保持している行のIndexも振りなおされます。
なので、長々と説明した現象が起こりません。
また、dataGridView1.SelectedRowsは選択した順番によって格納される順番がことなりますので
昇順ループでも降順ループでも意味が無かったりします。

Comments

[fight51722]

表を記述していただき、ありがとうございます！
とてもわかりやすいです。

>昇順ループでも降順ループでも意味が無かったりします。
そうなのですね…。また一つ勉強になりました。

ご回答していただいた内容を参考にさせていただきます。

丁寧に教えてくださり、本当にありがとうございました！

[YAmaGNZ]

実際にSelectedRowsの中身がどう変化するか確認してみることをお勧めします。
また、RemoveAtの実行前と実行後のIndexの変化も確認してみてください。

Answer 2

var arr = new string[] {"1, バナナ", "2, りんご", "3, ぶどう"};

ループの中で配列の数を増減させると途中でインデックスが変わってしまいます。
つまり、インデックス0の1, バナナを削除すると次のループはインデックスは1になりますが、順序が変わっています（["2, りんご", "3, ぶどう"]）ので,インデックス1はぶどうとなり、りんごのデータを拾えません。

これが逆にりんご（インデックス2）から数え上げて、りんごを削除しても、ぶどうのインデックスは変わりません。これが逆順に数え上げる理由です。

で、逆順に数え上げる方法について説明します。

配列のカウントを取ると、アイテムの数の３が返ってきます。
配列のインデックスは０から数えるのでバナナのインデックスは０です。

ここまでは理解できていると思います。
ぶどうから拾う場合は、[2, 1, 0]というように変数を変えていかなければなりません。

このとき、初期値の2は、配列のカウントの３から1引いた数になります。というのもインデックスは0から始まるからです。

次に、1を求めるためには通常ループごとにカウントアップしているi++のところをカウントダウンする必要があります。よって、i--とします。

最後継続する条件は、-1が含まれないようにしなければなりませんし、最後のインデックスは0となるので、インデックスは0以上となります。

これをコードにすると次のようになります。

var arr = new string[] {"1, バナナ", "2, りんご", "3, ぶどう"};

for(var i = arr.Count - 1; 0 <= i; i--){
    System.Console.WriteLine(arr[i]);
} 

出力はこんなふうになります。

3, ぶどう
2, りんご
1, バナナ

Comments

[fight51722]

逆順に数え上げる方法やコード、出力結果まで…！
詳しく書いていただき本当にありがとうございます。
ループやインデックス、配列などまだまだ理解が足りていないので学習を続けたいと思います。
ご回答していただいた内容を参考委にさせていただきます。

本当にありがとうございました！

[iwamoto_takaaki]

書きすぎたね・・・
ループの動作は、各変数の状態を頭で追えるようになるまで大変だけど、順を追って行けば必ず理解できるので頑張ってください。

[fight51722]

全く書きすぎではないです！
丁寧に解説してくださる方がいて、とても心強く感じます。
この先も諦めずに学習を継続していきたいと思います！

Answer 3

リストの要素を削除する場合、最初の方から削除していくと、削除した要素以降はインデックスがずれてしまい、以降の要素を削除する場合余計な手間がかかります
なので、このような場合は、リストの後尾から削除するようにします

ということで、

for (int i = src.Count -1; i >= 0; i--)

srcの最後尾から始めて、０未満になるまで、-1しながらループを回していきます

Comments

[fight51722]

インデックスがずれないように後尾から削除する、というのはそういう意味だったのですね。
ご回答していただいたおかげで理解を深める事ができました。
ご回答ありがとうございました！

Answer 4

例えば，2番目と5番目が選択されていたとする．

• 「2番目」を先に削除すると，その削除によって「5番目」だった行は「4番目」になる．

この状態においての「5番目を削除」することは，狙いの行ではない別の行を削除してしまうことになる．

• 「5番目」の側を先に削除したならば，「2番目」は依然として「2番目」であるから，問題なく「2番目を削除」すれば所望の動作が達成できる．

つまり，indexがでかい側から削除していくと面倒が無い，という話．

（ところで，dataGridView1.SelectedRowsが返す内容はソートされている保証は無いのでは？と思う．
もしもそうであれば，上述のようなことをするためには，これをソートする必要がある．
であれば，その際のソート順を降順にすれば，forの「向き」は素直な方向で書けるだろう．）

Comments

[fight51722]

表の具体例まで書いていたき、ありがとうございます。
上の行を先に削除してしまうと、次に下の行を削除する場合
関係のない行を削除してしまうことになるのですね。

ソートについても学習していこうと思います。

丁寧にご回答していただき、ありがとうございました！