List <>の最後の要素を見つけるにはどうすればよいですか？

以下は私のコードからの抜粋です：

public class AllIntegerIDs 
{
    public AllIntegerIDs() 
    {            
        m_MessageID = 0;
        m_MessageType = 0;
        m_ClassID = 0;
        m_CategoryID = 0;
        m_MessageText = null;
    }

    ~AllIntegerIDs()
    {
    }

    public void SetIntegerValues (int messageID, int messagetype,
        int classID, int categoryID)
    {
        this.m_MessageID = messageID;
        this.m_MessageType = messagetype;
        this.m_ClassID = classID;
        this.m_CategoryID = categoryID;
    }

    public string m_MessageText;
    public int m_MessageID;
    public int m_MessageType;
    public int m_ClassID;
    public int m_CategoryID;
}

私は私のmain（）関数コードで以下を使用しようとしています：

List<AllIntegerIDs> integerList = new List<AllIntegerIDs>();

/* some code here that is ised for following assignments*/
{
   integerList.Add(new AllIntegerIDs());
   index++;
   integerList[index].m_MessageID = (int)IntegerIDsSubstring[IntOffset];
   integerList[index].m_MessageType = (int)IntegerIDsSubstring[IntOffset + 1];
   integerList[index].m_ClassID = (int)IntegerIDsSubstring[IntOffset + 2];
   integerList[index].m_CategoryID = (int)IntegerIDsSubstring[IntOffset + 3];
   integerList[index].m_MessageText = MessageTextSubstring;
}

問題はここにあります：forループを使用してリストのすべての要素を印刷しようとしています：

for (int cnt3 = 0 ; cnt3 <= integerList.FindLastIndex ; cnt3++) //<----PROBLEM HERE
{
   Console.WriteLine("{0}\t{1}\t{2}\t{3}\t{4}\n", integerList[cnt3].m_MessageID,integerList[cnt3].m_MessageType,integerList[cnt3].m_ClassID,integerList[cnt3].m_CategoryID, integerList[cnt3].m_MessageText);
}

forループでcnt3と同等にして、リストのすべてのエントリを出力できるように、最後の要素を見つけたいと思います。リストの各要素は、コードサンプルで前述したように、AllIntegerIDsクラスのオブジェクトです。リストの最後の有効なエントリを見つけるにはどうすればよいですか？

integerList.Find（integerList []。m_MessageText == null;

それを使用する場合、0から最大値までの範囲のインデックスが必要になります。使用するつもりのない別のforループを使用する必要があることを意味します。より短い/より良い方法はありますか？

ありがとう、Viren

リストの最後のアイテムにアクセスしたいだけなら

if(integerList.Count>0)
{
   var item = integerList[integerList.Count - 1];
}

リスト内のアイテムの総数を取得するには、Countプロパティを使用できます

var itemCount = integerList.Count;

コレクションの最後の項目を取得するには、LastOrDefault（）およびLast（）拡張メソッドを使用します

var lastItem = integerList.LastOrDefault();

または

var lastItem = integerList.Last();

追加することを忘れusing System.Linq;ないでください。そうしないと、このメソッドは使用できなくなります。

問題の根源、リストの最後の要素に安全に対処する方法を見てみましょう...

想定

List<string> myList = new List<string>();

その後

//NOT safe on an empty list!
string myString = myList[myList.Count -1];

//equivalent to the above line when Count is 0, bad index
string otherString = myList[-1];

最初にリストが空でないことを保証しない限り、「count-1」は悪い習慣です。

空のリストをチェックする便利な方法は、それを行うこと以外にありません。

私が考えることができる最も短い方法は

string myString = (myList.Count != 0) ? myList [ myList.Count-1 ] : "";

すべてを出て、常にtrueを返すデリゲートを作成し、それをFindLastに渡すと、最後の値（またはリストが空の場合はデフォルトで作成されたvalye）が返されます。この関数はリストの最後から始まるため、メソッドは通常O（n）ですが、Big O（1）または一定時間になります。

//somewhere in your codebase, a strange delegate is defined
private static bool alwaysTrue(string in)
{
    return true;
}

//Wherever you are working with the list
string myString = myList.FindLast(alwaysTrue);

FindDastメソッドは、デリゲート部分を数えると見苦しいですが、宣言する必要があるのは1か所だけです。リストが空の場合、文字列のリストタイプ ""のデフォルトの構成値が返されます。alwaysTrueデリゲートをさらに一歩進めて、文字列型ではなくテンプレートにすると、さらに便利です。

int lastInt = integerList[integerList.Count-1];

変化する

for (int cnt3 = 0 ; cnt3 <= integerList.FindLastIndex ; cnt3++)

に

for (int cnt3 = 0 ; cnt3 < integerList.Count; cnt3++)

Countプロパティを使用します。最後のインデックスはになりますCount - 1。

for (int cnt3 = 0 ; cnt3 < integerList.Count; cnt3++)

あなたは最初にリストの要素の数を数えることでそれを見つけることができます例えば。

int count = list.Count();

次に、カウントにインデックスを付けることができます-1でリストの最後の要素を取得します。

int lastNumber = list[count - 1];

C＃8.0では、^演算子の完全な説明で最後のアイテムを取得できます

List<char> list = ...;
var value = list[^1]; 

// Gets translated to 
var value = list[list.Count - 1];

リストのCountプロパティを使用しないのはなぜですか？

for(int cnt3 = 0; cnt3 < integerList.Count; cnt3++)

元の質問とは関係なく、リストに複数回インデックスを付けるよりもローカル変数への参照をキャプチャすると、パフォーマンスが向上します。

AllIntegerIDs ids = new AllIntegerIDs();
ids.m_MessageID = (int)IntegerIDsSubstring[IntOffset];
ids.m_MessageType = (int)IntegerIDsSubstring[IntOffset + 1];
ids.m_ClassID = (int)IntegerIDsSubstring[IntOffset + 2];
ids.m_CategoryID = (int)IntegerIDsSubstring[IntOffset + 3];
ids.m_MessageText = MessageTextSubstring;
integerList.Add(ids);

そしてあなたのforループでは：

for (int cnt3 = 0 ; cnt3 < integerList.Count ; cnt3++) //<----PROBLEM HERE
{
   AllIntegerIDs ids = integerList[cnt3];
   Console.WriteLine("{0}\t{1}\t{2}\t{3}\t{4}\n",
      ids.m_MessageID,ids.m_MessageType,ids.m_ClassID,ids.m_CategoryID, ids.m_MessageText);
}

私はforeachの方がずっと簡単だと同意しなければなりません

foreach(AllIntegerIDs allIntegerIDs in integerList)
{
Console.WriteLine("{0}\t{1}\t{2}\t{3}\t{4}\n", allIntegerIDs.m_MessageID,
allIntegerIDs.m_MessageType,
allIntegerIDs.m_ClassID,
allIntegerIDs.m_CategoryID,
allIntegerIDs.m_MessageText);
}

また、私はあなたが同じようにそれを追加することができ、あなたの.NETのバージョンに応じて、公共分野の代わりにあなたの情報にアクセスするためのプロパティを追加示唆しているpublic int MessageType {get; set;}と取り除くm_ことが可能はずのようなどのパブリックフィールド、プロパティから。

これはあなたに役立つと思います。チェックしてください

    TaxRangers[TaxRangers.Count]. max