c# Ermittel das zweithöchste Datum einer List<DateTime>

[csharper (Gast)]

Hallo!

Ich versuche verzweifelt das zweitweit entfernteste Datum einer Liste 
mit Daten zu ermitteln.

Folgenden Code habe ich dafür geschrieben:

 public static DateTime SecondWarrantyDate(List<DateTime> Warrantys)
        {
            Warrantys.Sort();

            DateTime _SecondWarrantyDate = new DateTime();
            _SecondWarrantyDate = DateTime.MinValue;
            DateTime dummy = new DateTime();
            dummy = DateTime.MinValue;
            bool foundsecond = false;
            bool firstround = true;
            foreach (DateTime dt in Warrantys)
            {
                Console.WriteLine("Funktionsinhalt -> " + dt.ToString());
                //Kleiner als 0 (null) 
                //Diese Instanz liegt vor value.
                //Zero 
                //Diese Instanz entspricht value. 
                //Größer als 0 (null) 
                //Diese Instanz liegt nach value, oder value ist null.
                if ((dummy.CompareTo(dt) < 0) && firstround)
                {
                    dummy = dt;
                    firstround = false;
                }

                if ((dummy.CompareTo(dt) < 0) && !firstround && !foundsecond)
                {
                    foundsecond = true;
                    _SecondWarrantyDate = dt;
                }
            }



            return _SecondWarrantyDate;
        }

Wenn ich die Funktion mit diesen Daten befeuere, funktioniert sie:

 List<DateTime> dtlist = new List<DateTime>();
               dtlist.Add(DateTime.Now);
               dtlist.Add(DateTime.Now.AddYears(5));
               dtlist.Add(DateTime.Now.AddYears(10));
               dtlist.Add(DateTime.Now.AddYears(10));
                dtlist.Add(DateTime.Now.AddYears(6));

              Console.WriteLine("SecondWarrantyDate:");
               Console.WriteLine(SecondWarrantyDate(dtlist).ToString());

Aber leider nicht, wenn ich folgende Daten benutze, welche mir die 
Dell-Schnittstelle mit Garantiedaten zurückliefert...

->

14.10.2015 00:00:00
14.10.2015 00:00:00
14.10.2015 00:00:00
01.05.2022 00:00:00

Mein Verdacht: Die Funktion datetime.compareto() vergisst das Jahr beim 
Vergleich?!

Kann mir jemand einen Tipp geben?

Gruß!

[bluppdidupp (Gast)]

Wäre es nicht einfacher, die Liste einfach mit .Sort() aufsteigend zu 
sortieren und dann einfach das zweite Element aus der sortierten Liste 
zu ziehen?
(und bei Bedarf ggf. noch Duplikate vor dem Sortieren entfernen)

[csharper (Gast)]

bluppdidupp schrieb:
> Wäre es nicht einfacher, die Liste einfach mit .Sort() aufsteigend
> zu
> sortieren und dann einfach das zweite Element aus der sortierten Liste
> zu ziehen?
> (und bei Bedarf ggf. noch Duplikate vor dem Sortieren entfernen)

Sicher eine Möglichkeit, aber wie lösche ich denn ohne weiteres alle 
Duplikate aus einer Liste?

List.RemoveDuplicateEntrys() gibt es jedenfalls nicht, also müsste ich 
mir die Logik wohl selber schreiben?

Und dann ist es auch wieder nur ein anderer Lösungsansatz, der ähnlich 
viel Aufwand benötigt.

[bluppdidupp (Gast)]

Also z.B. so irgendwie:

// Input:
List<DateTime> inputDates=new List<DateTime>();
inputDates.Add(new DateTime(2015,10,14));
inputDates.Add(new DateTime(2015,10,14));
inputDates.Add(new DateTime(2015,10,14));
inputDates.Add(new DateTime(2022,5,1));

// Doppelte Werte entfernen:
List<DateTime> inputDatesDistinct=inputDates.Distinct().ToList();

// Aufsteigend sortieren:
inputDatesDistinct.Sort();

// Ausgeben:
if (inputDatesDistinct.Count>0)
{
    Console.WriteLine("Liegt am längsten zurück: "+inputDatesDistinct[0]);
}

if (inputDatesDistinct.Count>1)
{
    Console.WriteLine("Liegt am zweit-längsten zurück: "+inputDatesDistinct[1]);
}

[Thomas Z. (thomas_z41)]

Bist du dir sicher dass dein Datumsformat richtig geparsed wird?

[csharper (Gast)]

Thomas Z. schrieb:
> Bist du dir sicher dass dein Datumsformat richtig geparsed wird?

Jop, folgender Code ergibt:

foreach (DateTime dt in Warrantys)
            {
              Console.WriteLine("Jahr->" + dt.Year);
              Console.WriteLine("Monat->" + dt.Month);
              Console.WriteLine("Tag->" + dt.Day);
...
...

->

Jahr->2015
Monat->10
Tag->14
...
Jahr->2022
Monat->5
Tag->1

[csharper (Gast)]

bluppdidupp schrieb:
> Also z.B. so irgendwie:// Input:
> List<DateTime> inputDates=new List<DateTime>();
> inputDates.Add(new DateTime(2015,10,14));
> inputDates.Add(new DateTime(2015,10,14));
> inputDates.Add(new DateTime(2015,10,14));
> inputDates.Add(new DateTime(2022,5,1));
>
> // Doppelte Werte entfernen:
> List<DateTime> inputDatesDistinct=inputDates.Distinct().ToList();
>
> // Aufsteigend sortieren:
> inputDatesDistinct.Sort();
>
> // Ausgeben:
> if (inputDatesDistinct.Count>0)
> {
>     Console.WriteLine("Liegt am längsten zurück:
> "+inputDatesDistinct[0]);
> }
>
> if (inputDatesDistinct.Count>1)
> {
>     Console.WriteLine("Liegt am zweit-längsten zurück:
> "+inputDatesDistinct[1]);
> }

Deine Lösung hat ebenfalls das selbe Programm... ich verstehe nicht 
warum...

Dann muss es ja an DateTime.Sort() liegen?!

[csharper (Gast)]

Programm = Problem.

[csharper (Gast)]

Ja, es lag an List.Sort()...

inputDatesDistinct.Sort((a, b) => b.CompareTo(a));

Damit funktioniert es.

Danke für eure Bemühungen.

[csharper (Gast)]

csharper schrieb:
> Ja, es lag an List.Sort()...
>
> inputDatesDistinct.Sort((a, b) => b.CompareTo(a));
>
> Damit funktioniert es.
>
> Danke für eure Bemühungen.

Sorry, hab mich zu früh gefreut... das macht gar keinen Sinn, daran 
liegt es nicht. Ich hab die Liste nur in der anderen Reihenfolge 
abgegriffen...

Ich weiß immer noch nicht, warum es nicht funktioniert. Hat noch jemand 
einen Tipp?

[Fuzzi59 (Gast)]

csharper schrieb:
> Sorry, hab mich zu früh gefreut... das macht gar keinen Sinn, daran
> liegt es nicht. Ich hab die Liste nur in der anderen Reihenfolge
> abgegriffen...

Und das ist auch richtig so! Wenn du nämlich aufsteigend sortierst und 
den zweiten Wert ausliest, erhälst du den zweitkleinsten Wert. 
Absteigend sortiert hingegen ist der zweite Wert auch der zweithöchste.

Ich schätze mal dein Problem entstand dadurch, dass du nur zwei 
verschiedene Werte genutzt hast und so bei falscher Sortierung den 
entsprechend falschen Wert erhalten hast.

[Arc N. (arc)]

csharper schrieb:
> bluppdidupp schrieb:
>> Wäre es nicht einfacher, die Liste einfach mit .Sort() aufsteigend
>> zu
>> sortieren und dann einfach das zweite Element aus der sortierten Liste
>> zu ziehen?
>> (und bei Bedarf ggf. noch Duplikate vor dem Sortieren entfernen)
>
> Sicher eine Möglichkeit, aber wie lösche ich denn ohne weiteres alle
> Duplikate aus einer Liste?
>
> List.RemoveDuplicateEntrys() gibt es jedenfalls nicht, also müsste ich
> mir die Logik wohl selber schreiben?
>
> Und dann ist es auch wieder nur ein anderer Lösungsansatz, der ähnlich
> viel Aufwand benötigt.

            List<DateTime> dtList = new List<DateTime>();
            dtList.Add(DateTime.Now);
            dtList.Add(new DateTime(2015, 10, 14, 0, 0, 0));
            dtList.Add(new DateTime(2015, 10, 14, 0, 0, 0));
            dtList.Add(new DateTime(2015, 10, 14, 0, 0, 0));
            dtList.Add(new DateTime(2015, 10, 14, 0, 0, 0));
            dtList.Add(new DateTime(2015, 10, 14, 0, 0, 0));
            dtList.Add(DateTime.Now.AddYears(5));
            dtList.Add(DateTime.Now.AddYears(5));
            dtList.Add(DateTime.Now.AddYears(5));
            dtList.Add(DateTime.Now.AddYears(10));
            dtList.Add(DateTime.Now.AddYears(10));
            dtList.Add(DateTime.Now.AddYears(6));

            var sortedListOfDistinctDates = dtList.Distinct().OrderBy(date => date).ToList();

[csharper (Gast)]

Fuzzi59 schrieb:
> csharper schrieb:
>> Sorry, hab mich zu früh gefreut... das macht gar keinen Sinn, daran
>> liegt es nicht. Ich hab die Liste nur in der anderen Reihenfolge
>> abgegriffen...
>
> Und das ist auch richtig so! Wenn du nämlich aufsteigend sortierst und
> den zweiten Wert ausliest, erhälst du den zweitkleinsten Wert.
> Absteigend sortiert hingegen ist der zweite Wert auch der zweithöchste.
>
> Ich schätze mal dein Problem entstand dadurch, dass du nur zwei
> verschiedene Werte genutzt hast und so bei falscher Sortierung den
> entsprechend falschen Wert erhalten hast.

Du hast Recht. :) Es funktioniert so, vielen Dank... ich war wohl so 
verzweifelt, dass ich gar nichts mehr geblickt habe.

Ich verstehe trotzdem nicht, warum mein Anfangscode nicht funktioniert 
hat...

Naja, manchmal soll mans einfach so hinnehmen.

Danke euch.

Arc N. schrieb:
> csharper schrieb:
>> bluppdidupp schrieb:
>>> Wäre es nicht einfacher, die Liste einfach mit .Sort() aufsteigend
>>> zu
>>> sortieren und dann einfach das zweite Element aus der sortierten Liste
>>> zu ziehen?
>>> (und bei Bedarf ggf. noch Duplikate vor dem Sortieren entfernen)
>>
>> Sicher eine Möglichkeit, aber wie lösche ich denn ohne weiteres alle
>> Duplikate aus einer Liste?
>>
>> List.RemoveDuplicateEntrys() gibt es jedenfalls nicht, also müsste ich
>> mir die Logik wohl selber schreiben?
>>
>> Und dann ist es auch wieder nur ein anderer Lösungsansatz, der ähnlich
>> viel Aufwand benötigt.
>             List<DateTime> dtList = new List<DateTime>();
>             dtList.Add(DateTime.Now);
>             dtList.Add(new DateTime(2015, 10, 14, 0, 0, 0));
>             dtList.Add(new DateTime(2015, 10, 14, 0, 0, 0));
>             dtList.Add(new DateTime(2015, 10, 14, 0, 0, 0));
>             dtList.Add(new DateTime(2015, 10, 14, 0, 0, 0));
>             dtList.Add(new DateTime(2015, 10, 14, 0, 0, 0));
>             dtList.Add(DateTime.Now.AddYears(5));
>             dtList.Add(DateTime.Now.AddYears(5));
>             dtList.Add(DateTime.Now.AddYears(5));
>             dtList.Add(DateTime.Now.AddYears(10));
>             dtList.Add(DateTime.Now.AddYears(10));
>             dtList.Add(DateTime.Now.AddYears(6));
>
>             var sortedListOfDistinctDates =
> dtList.Distinct().OrderBy(date => date).ToList();

Auch dir danke!

[Andreas E. (andreas_e_2)]

Spontan und ungetestet könnte es auch so gehen

            var a = Warrantys.Distinct().OrderByDescending(b => b.Date).Skip(1).Take(1);
            return a.First();

[bluppdidupp (Gast)]

Unter "zweitweit entfernteste" hätte ich eigentlich das Datum das am 
zweit-weitesten in der Vergangenheit liegt verstanden, daher hatte ich 
aufsteigend sortiert, dann liegt das kleinere Datum (=weiter in der 
Vergangenheit) vorne ;D

[Andreas E. (andreas_e_2)]

Stimmt. Also nur OrderBy(b=> ...

sry, war noch nicht ganz wach.

[Georg (Gast)]

csharper schrieb:
> Sicher eine Möglichkeit, aber wie lösche ich denn ohne weiteres alle
> Duplikate aus einer Liste?

Warum eigentlich? das 2. ist das 2. Und wenn das denselben Wert hat wie 
das 1. oder das 3., woher weisst du dann überhaupt, welches du jetzt 
löschen musst? Das ist nicht wirklich ein Problem der Software, sondern 
des Konzepts - was sind denn überhaupt Duplikate? Doppelgänger, geklonte 
Kunden oder was?

Georg