C# Sortierung von ArrayList

[Quaussi]

hallo user,

ich habe hier ein kleines programm geschrieben, dass leider nicht funktioniert...quelle: galileo computing
ich sehe den fehler leider nicht
vielleicht kann mir einer von euch weiter helfen
vielen lieben dank


auftretender fehler: am arrList.Sort(); -> Fehler beim Vergleichen von zwei Elementen im Array


using System;
using System.Collections;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace sortieren
{
public interface IComparable { int CompareTo(object obj); }

class Program
{
static void Main(string[] args)
{
ArrayList arrList = new ArrayList();

HoldValue obj1 = new HoldValue(17); arrList.Add(obj1);
HoldValue obj2 = new HoldValue(110); arrList.Add(obj2);
HoldValue obj3 = new HoldValue(5); arrList.Add(obj3);
arrList.Sort();

int i = 0;
foreach (HoldValue temp in arrList) { Console.WriteLine("Element{0} mit den Wert: {1}", i, temp.intVar); i++; }
Console.ReadLine();
}
}


public class HoldValue : IComparable
{
public int intVar;
public HoldValue(int x) { intVar = x; }

public int CompareTo(object obj)
{
HoldValue val = (HoldValue)obj;
if (val.intVar < this.intVar) return 1;
if (val.intVar == this.intVar) return 0;
return -1;
}
}
}

 

[Madman1209]

Hi,

dann mach einen Breakpoint bevor der Fehler auftritt und debugge das Ganze, schau dir die arrList an und ob die Elemente drin korrekt sind.

VG,
Mad

 

[Multithread]

Wenn du Sort ein Delegate übergibst, kannst du sogar selber sagen wie Sortiert werden soll

Vielleicht klappts dann damit. Auf ne ArrayList habe ich selber noch nie Sort verwendet. Probier mal ein Array aus object's, das sollte auch gehen.

 

[Quaussi]

war das erste, was ich getan hatte
"arrList" ist richtig beschrieben
so was ist ziemlich deprimierend

 

[Madman1209]

Hi,

quelle: galileo computing

Link bitte.

VG,
Mad

 

[Quaussi]

http://openbook.galileocomputing.de...05_003.htm#mjb3469db33024a1c67fec977e0ab1b8b9

ab dem Punkt: 5.3.4 Die Elemente einer »ArrayList« sortieren

 

[Marguth]

Würde empfehlen wenn dann die neuere Version zu nehmen.

Visual C# 2010


Scheint mir auf erstem blick tatsächlich nicht ganz richtig zu sein das Beispiel...

 

[1668mib]

Vielleicht solltest du das Interface IComparable nicht selbst definieren sondern das aus Collections nehmen...

Diese Zeile löschen:
public interface IComparable { int CompareTo(object obj); }

 

[Multithread]

Was genau musst du überhaupt machen? Vielleicht sind Listen eine alternative für dich

List<int> liste= new List<int>();
liste.add(3);
liste.add(2);
liste.Sort();

liste[0]=2;
liste[1]=3;

 

[TheCadillacMan]

IComparable ist ein System-Interface. Die ArrayList kann mit deiner "Nachbildung" nichts anfangen und wirft deshalb einen Fehler.
Wirf folgende Deklaration raus und es sollte klappen:

public interface IComparable { int CompareTo(object obj); }

Noch zwei Amerkungen:
- Verwende zum Code posten bitte die Code-Tags; ist einfach besser lesbar.
- ArrayList sollte man i. d. R. nicht mehr verwenden, weil List<T> die bessere Alternative ist. Wenn dein Buchbeispiel mit ArrayList arbeitet ist das natürlich ok, nur in der Praxis sollte man List<T> benutzen.

 

[Madman1209]

Hi,

genau, die eigene Implementierung von IComparable zerhaut es dir.

VG,
Mad

 

[Quaussi]

super sache.....ich habe den interface auskommentiert

ich danke euch für die schnellen rückantworten

das mit List<T> werde ich morgen versuchen


so feierabend

 

[_daniel_999]

Einfacher wäre das Ganze nochmals mit LINQ

var list = new List<HoldValue>();

list.Add(new HoldValue(17));
list.Add(new HoldValue(110));
list.Add(new HoldValue(5));

var orderedlist = list.OrderBy(x => x.intVar);

Weiß jetzt nicht ob alles syntax-mäßig korrekt ist

mfg

 

[1668mib]

Einfacher wäre es, das Array in der richtigen Reihenfolge anzulegen... er macht es um was zu lernen, und nicht um die beste Lösung zu finden.

Natürlich kann man mit solchen Closures Ausdrücke kompakter gestalten... aber ob der Lern-Effekt dadurch größer ist, sei jetzt mal dahingestellt.

In der Mathematik fängt man auch mit kleinen Zahlenbereichen an, und nicht direkt mit komplexen Zahlen... das hat schon seine Gründe...

 

[_daniel_999]

Das war bloß ein weiterer Ansatz um das Problem anzugehen und nicht die beste Lösung zu finden..