Jak mogę znaleźć ostatni element na liście <>?

Poniżej znajduje się wyciąg z mojego kodu:

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;
}

Próbuję użyć następującego kodu w moim kodzie funkcji 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;
}

Problem jest tutaj: próbuję wydrukować wszystkie elementy z mojej listy przy użyciu pętli 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);
}

Chcę znaleźć ostatni element, aby zrównać cnt3 w mojej pętli for i wydrukować wszystkie wpisy na liście. Każdy element listy jest obiektem klasy AllIntegerIDs, jak wspomniano powyżej w przykładowym kodzie. Jak znaleźć ostatni ważny wpis na liście?

Czy powinienem użyć czegoś takiego jak integerList.Find (integerList []. M_MessageText == null;

Jeśli go użyję, będzie potrzebny indeks, który będzie w zakresie od 0 do dowolnego maksimum. Znaczy, będę musiał użyć innej pętli for, której nie zamierzam używać. Czy istnieje krótszy / lepszy sposób?

Dzięki, Viren

Jeśli chcesz tylko uzyskać dostęp do ostatniej pozycji na liście, możesz to zrobić

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

aby uzyskać całkowitą liczbę elementów na liście, możesz użyć właściwości Count

var itemCount = integerList.Count;

Aby uzyskać ostatni element kolekcji, użyj metod rozszerzających LastOrDefault () i Last ()

var lastItem = integerList.LastOrDefault();

LUB

var lastItem = integerList.Last();

Pamiętaj, aby dodać using System.Linq;, w przeciwnym razie ta metoda nie będzie dostępna.

Przejdźmy do sedna pytania, jak bezpiecznie zająć się ostatnim elementem listy ...

Zarozumiały

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

Następnie

//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” to zły nawyk, chyba że najpierw zagwarantujesz, że lista nie jest pusta.

Nie ma wygodnego sposobu sprawdzenia pustej listy poza zrobieniem tego.

Najkrótszy sposób, jaki przychodzi mi do głowy, to

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

możesz pójść na całość i utworzyć delegata, który zawsze zwraca wartość true, i przekazać go do FindLast, który zwróci ostatnią wartość (lub domyślną wartość skonstruowaną, jeśli lista jest pusta). Ta funkcja zaczyna się na końcu listy, więc będzie Wielkie O (1) lub stały czas, mimo że metoda zwykle to 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);

Metoda FindLast jest brzydka, jeśli policzysz część delegata, ale musi zostać zadeklarowana tylko w jednym miejscu. Jeśli lista jest pusta, zwróci domyślną skonstruowaną wartość typu listy „” dla łańcucha. Pójście dalej z delegatem alwaysTrue, czyniąc go szablonem zamiast typu string, byłoby bardziej przydatne.

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

Zmiana

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

do

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

Skorzystaj z Countnieruchomości. Ostatni indeks będzie Count - 1.

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

Możesz go znaleźć licząc najpierw liczbę elementów na liście, np

int count = list.Count();

Następnie możesz zindeksować liczbę - 1, aby uzyskać ostatni element na liście, np

int lastNumber = list[count - 1];

W C # 8.0 można uzyskać ostatni element z pełnym wyjaśnieniem operatora ^

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

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

Dlaczego po prostu nie użyć właściwości Count na liście?

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

Niezależnie od pierwotnego pytania uzyskasz lepszą wydajność, jeśli będziesz przechwytywać odwołania do zmiennych lokalnych zamiast wielokrotnie indeksować listę:

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

A w twojej forpętli:

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

Musiałbym się zgodzić, że dla każdego byłoby o wiele łatwiejsze

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

Sugerowałbym również dodanie właściwości, aby uzyskać dostęp do swoich informacji zamiast pól publicznych, w zależności od wersji .net możesz dodać to jak public int MessageType {get; set;}i pozbyć się m_z pól publicznych, właściwości itp., Ponieważ nie powinno ich tam być.

Myślę, że to ci pomaga. Proszę sprawdzić

    TaxRangers[TaxRangers.Count]. max