[C#]IComparer implementatie probleem

[4of9]

Hallo,

Ik probeer een arraylist met Uri objecten te sorteren.
Daarvoor moet ik een custom comparer maken.

Nu het ik de class UriComparer gemaakt die de interface IComparer implementeerd.

Het gekke is dat als ik de boel probeer te compilen dat de compiler zegt dat mijn class de Compare method van de interface niet implementeerd.

Wat doe ik fout?

C#:

    public class UriComparer : IComparer
    {

        public int Compare(Uri url,Uri currenturl)
        {
            string firstUrl = url.AbsoluteUri as firstUrl;
            string secondUrl = currenturl.AbsoluteUri as secondUrl;
        
            return firstUrl.CompareTo(secondUrl);
        }

    }

mijn sort aanroep:

C#:

public void RemoveDoubleUrlsFromResults(ref ArrayList results)
{
            UriComparer uriComparer = new UriComparer();
            results.Sort(uriComparer);
}

Alvast bedankt voor de hulp!

[BRAINLESS01]

ik kan mij vergissen, maar ik zou iets verwachten als

'as String' in plaats van 'as firsturl'

[Woy]

Als je IComparer implementeerd dan moet je de volgende functie hebben

C#:

public int Compare(object a, object b )
{}

Anders moetj e IComparer<Uri> implementeren. Dan klopt jouw implementatie wel

[whoami]

Ik denk dat _BRAINLESS de nagel op de kop slaat.
Wellicht krijg je naast die ene compiler error ook nog andere errors, en is de foutmelding die je nu vernoemt, het gevolg van de syntax-errors in je Compare method.

[Gerco]

Volgens MSDN moet je het zo doen:

C#:

   public class myReverserClass : IComparer  {

      // Calls CaseInsensitiveComparer.Compare with the parameters reversed.
      int IComparer.Compare( Object x, Object y )  {
          return( (new CaseInsensitiveComparer()).Compare( y, x ) );
      }

Dus de parameters zijn van type Object ipv het type Uri (of wat je dan ook aan het comparen bent).

[edit]
Lekker voorbeeld trouwens, je zal op die manier maar 10k objecten gaan vergelijken, zit je 10k CaseInsensitiveComparer's te creeren. Ik hoop dat .NET ook "escape analysis" ondersteund... (al vermoed ik dat dat hier niet toe te passen is omdat het om een externe class gaat)

[ Voor 22% gewijzigd door Gerco op 05-09-2006 10:58 ]

[whoami]

oeps, idd...
Alhoewel _BRAINLESS ook een punt heeft, moet de method natuurlijk dezelfde signature hebben als de signature in de interface.

[4of9]

de signature was idd verkeerd en de "as" operator niet nodig.

Ik had een verkeerd voorbeeld gevolgd blijkbaar.

compilende code:

C#:

        int IComparer.Compare(object url,object currenturl)
        {
            string firstUrl = ((Uri)url).AbsoluteUri;
            string secondUrl = ((Uri)currenturl).AbsoluteUri;
        
            return firstUrl.CompareTo(secondUrl);
        }

Hij compiled wel maar de sort werkt niet naar behoren, maar dat zal nog wel opgelost worden!

thx allemaal

[Not Pingu]

4of9 schreef op dinsdag 05 september 2006 @ 10:59:

Hij compiled wel maar de sort werkt niet naar behoren, maar dat zal nog wel opgelost worden!

Dat is nou juist de reden dat je zelf een comparer schrijft. Je laat nu een string zichzelf met een andere string vergelijken en dat zal inderdaad geen goede sortering opleveren. Je zou bijv. kunenn sorteren op het domein (bijv. tweakers.net) en daarna op alle subdomeinen binnen dat domein (bijv gathering.tweakers.net, pda.tweakers.net, www.tweakers.net).

Daarvoor kun je zelf de verschillende onderdelen van een URI met elkaar vergelijken en de juiste returnwaarde berekenen.