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

C#にて、現状は以下のソースコードのようにしていますが、foreach等を使ったうまいやり方はないのでしょうか？

private static List<Hoge> _objs = new List<Hoge>(); 

for( int i = 0; i < _objs.Count; i++ ) {
    _objs[i].update();
    if( _objs[i].isEnd ) { // 役目は終わったか？...
        _objs.RemoveAt(i--);
    }
}

気持ち的には

foreach( Hoge obj in _objs ) {
    obj.update();
    if( obj.isEnd ) {
        _objs.Remove(obj);
    }
}

ができたらいいのに（ちがうな…のような書き方ないかな？）と思いまして。

■No10106 (てぃあの さん) に返信
> C#にて、現状は以下のソースコードのようにしていますが、foreach等を使ったうまいやり方はないのでしょうか？

_objs.RemoveAll(delegate(Hoge obj)
{
  obj.update();
  return obj.isEnd;
});

ではダメなのでしょうか？

> foreach(Hoge obj n _objs.ToArray()) {
>   obj.update();
>   if (obj.isEnd) _objs.Remove(obj);
> }

これだと_objs.Remove(obj)んとこで毎回_objsを頭っから舐めてobjを探しちゃうよな。

List<Hoge> tmp = new List<Hoge>();
foreach ( Hoge obj in _objs ) {
  obj.update();
  if ( !obs.isEnd ) tmp.Add(obj)　// 必要なもんだけコピって
}
_objs = tmp; // 差し替え

とかいかがでしょ。

昔似たような質問に回答した気がします。

そのときは「別リストを作って削除 *しない* ものを追加していく」とか

private static List<Hoge> _objs = new List<Hoge>();
private static List<Hoge> _objs2 = new List<Hoge>();

for( int i = 0; i < _objs.Count; i++ ) {
    _objs[i].update();
    if( !_objs[i].isEnd ) { // 役目は終わったか？...
        _objs2.Add(_objs[i]);
    }
}

# しまった、これεπιστημηさんが先に書いてるや…

for で ++ じゃなく -- で検索、とか

private static List<Hoge> _objs = new List<Hoge>(); 

for( int i = _objs.Count-1; i >= 0; i-- ) {
    _objs[i].update();
    if( _objs[i].isEnd ) { // 役目は終わったか？...
        _objs.RemoveAt(i);
    }
}

そんなことを書いたような気がします。

なんにせよ、ひどりさんも書かれているとおり foreach の途中でリストの操作をしてはダメです。

■No10128 (επιστημη さん) に返信

objs.erase(remove_if(objs.begin(), objs.end(), update_and_validate), objs.end());
これ！ぺっかーん（！）って感じで思い出しました！！

仕事のときは先輩にこれって言われてやってましたわー。
記憶が正しければ、必要なものを前詰めにして、remove_ifの戻り値が不要な箇所の先頭indexなんでしたっけ？
そんでeraseでちょっきん！って感じだったと思うんですけど。
処理的にもバブルソートみたいな感じで詰め替えて行くからそんなんでもない…だったような（The うるおぼえ）。

※結局…C#ではどういう作法がいいんでしょうかね…？

■No10134 (渋木宏明(ひどり) さん) に返信
>>そもそも現状のコードは作法的にどうなんでしょう？まずかったりしますでしょうか？
> 
> バグってマス (^^;

for( int i = 0; i < _objs.Count; i++ ) {
    _objs[i].update();
    if( _objs[i].isEnd ) { // 役目は終わったか？...
        _objs.RemoveAt(i--);
    }
}

ありぃ？これ↑ダメ？
はずかぴぃ〜。

RemoveAll(Predict<T> match) で書いてみますｗ

> for( int i = 0; i < _objs.Count; i++ ) {
>     _objs[i].update();
>     if( _objs[i].isEnd ) { // 役目は終わったか？...
>         _objs.RemoveAt(i--);
>     }
> }
> 
> ありぃ？これ↑ダメ？

3番目が引っ掛かったとすると
_objs.RemoveAt(i--) だから 
3番目を引っこ抜いて、iが2になって、i++で3に戻るから
元々(削除前の)4番目が次の対象になりますよね。
...よさげな気がすんだけど。