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

ASP.NET、C++/CLI、Java 何でもどうぞ

C# と VB.NET の入門サイト

Re[4]: リストボックスから削除する

分類:[C#]　

C#です。
リストボックスにファイルをドラッグアンドドロップした時に、ワードファイル以外を除外したいと考えています。
下記のようにしてみましたが、複数をドラッグアンドドロップした場合に、1つだけ不要なリストが残ってしまいます。
これを解決するにはどうすればよいでしょうか?

for (int i = 0; i <= listBox1.Items.Count - 1; i++)
{
if (Path.GetExtension(listBox1.Items[i].ToString()) != ".doc")
{
listBox1.Items.RemoveAt(i);
i = 0;
}

ご存知の方がいらっしゃいましたら、ご教示ください。

if内で、
> listBox1.Items.RemoveAt(i);
> i = 0;
この記述がありますが、
このあとにiがインクリメントされる(i++)されるのかと。
それで、自らRemoveAtしているので、
インデックスがずれていくのかな、と。
（つまり、最初のアイテムが削除対象だとすると、当初[1]にあったアイテムは判断対象にならない。）

パッと見ですが。

■No49219 (かつさん) に返信
> C#です。
> リストボックスにファイルをドラッグアンドドロップした時に、ワードファイル以外を除外したいと考えています。
> 下記のようにしてみましたが、複数をドラッグアンドドロップした場合に、1つだけ不要なリストが残ってしまいます。
> これを解決するにはどうすればよいでしょうか?
>
> for (int i = 0; i <= listBox1.Items.Count - 1; i++)
> {
> if (Path.GetExtension(listBox1.Items[i].ToString()) != ".doc")
> {
> listBox1.Items.RemoveAt(i);
> i = 0;
> }
>
> ご存知の方がいらっしゃいましたら、ご教示ください。

後ろから検索した方が簡単だと思いますよ。

つfor (int i = listBox1.Items.Count - 1; i >=0 ; i--)

            for (int i = 0; i < listBox1.Items.Count ; i++)
            {
                if (Path.GetExtension(listBox1.Items[i].ToString()) != ".doc")
                {
                    listBox1.Items.RemoveAt(i);
                    i = 0;
                }
            }
            if (Path.GetExtension(listBox1.Items[0].ToString()) != ".doc")
            {
                listBox1.Items.RemoveAt(0);
                
            }

上記のように、listBox1.Items[0]のみ別で判定するようにすれば、解決ですが、あまりきれいではないですね。
もっと精進します。ありがとうございました。

どうしても先頭から処理したいのなら。

for (int i = 0; i < listBox1.Items.Count; )
{
    if (Path.GetExtension(listBox1.Items[i].ToString()) != ".doc")
    {
        listBox1.Items.RemoveAt(i);
    }
    else
    {
        i++;
    }
}

＞上記のように、listBox1.Items[0]のみ別で判定するようにすれば、解決ですが、あまりきれいではないですね。
＞もっと精進します。ありがとうございました。
それだと、forでItemsの中が空になった場合listBox1.Items[0]で落ちますよ。

>>みきぬさん、noriさん、doorさん
いろいろとご教授いただきましてありがとうございました。
前からする必要もありませんので、みきぬさんの方法でいきたいと思います。
for (int i = listBox1.Items.Count - 1; i >=0 ; i--)

ありがとうございました。

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

ASP.NET、C++/CLI、Java 何でもどうぞ

C# と VB.NET の入門サイト

Re[4]: リストボックスから削除する

過去ログには書き込み不可