Как я могу найти последний элемент в 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);
}

Я хочу найти последний элемент, чтобы приравнять cnt3 в цикле for и распечатать все записи в списке. Каждый элемент в списке является объектом класса AllIntegerIDs, как указано выше в примере кода. Как мне найти последнюю действительную запись в списке?

Должен ли я использовать что-то вроде integerList.Find (integerList []. M_MessageText == null;

Если я использую это, ему понадобится индекс, который будет варьироваться от 0 до любого максимального значения. Значит, мне придется использовать другой цикл for, который я не собираюсь использовать. Есть более короткий / лучший способ?

Спасибо вирен

Если вы просто хотите получить доступ к последнему элементу в списке, вы можете сделать

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, если список пуст). Эта функция запускается в конце списка, поэтому будет Big O (1) или постоянным временем, несмотря на то, что метод обычно O (n).

//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);

Метод FindLast уродлив, если вы считаете часть делегата, но его нужно объявить только в одном месте. Если список пуст, он вернет созданное по умолчанию значение типа списка "" для строки. Дальнейшее продвижение делегата 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