CODEWITHSUNDEEP
vb.netsortingarraylisticomparer
Zifendale
Zifendale

Reputation: 140

Casting Error when sorting with IComparer

I have an ArrayList of strings of the form "Q19_1_1", "Q19_10_1", "Q19_5_1". With the normal sort method the list will be sorted as

"Q19_1_1"  
"Q19_10_1"
"Q19_5_1"

But I would like to sort it numerically based off the second integer in name and then the third. So I would like:

"Q19_1_1"
"Q19_5_1"  
"Q19_10_1"

My Sub:

    Dim varSet As New ArrayList

    varSet.Add("Q19_1_1")
    varSet.Add("Q19_10_1")
    varSet.Add("Q19_5_1")

    varSet.Sort(New VariableComparer())

I have a IComparer:

Public Class VariableComparer
Implements IComparer(Of String)

Public Function Compare(ByVal x As String, ByVal y As String) As Integer Implements System.Collections.Generic.IComparer(Of String).Compare

    Dim varPartsX As Array
    Dim varPartsY As Array
    varPartsX = x.Split("_")
    varPartsY = y.Split("_")

    Return String.Compare(varPartsX(1), varPartsY(1))
End Function
End Class

But when I attempt to sort I get the error:

Unable to cast object of type 'VBX.VariableComparer' to type 'System.Collections.IComparer'.

VariableComparer implements IComparer but I'm guessing it can't be of type IComparer(Of String)?

How can I resolve this issue? What am I missing?

Upvotes: 1

Views: 1959

Answers (2)

tinstaafl
tinstaafl

Reputation: 6948

Using a List(Of String) also gives you access to the LINQ extensions. Specifically the OrderBy and the ThenBy extensions. You could do it something like this:

    Dim test3 As New List(Of String)({"Q19_1_1", "Q19_10_1", "Q19_5_1", "Q19_5_2"})
    test3 = test3.OrderBy(Of Integer)(Function(s) Integer.Parse(s.ToString.Split("_"c)(1))) _
                 .ThenBy(Of Integer)(Function(s2) Integer.Parse(s2.ToString.Split("_"c)(2))).ToList

Casting to Integer gives you the proper sorting without using a new IComparer interface

Upvotes: 2

Reed Copsey
Reed Copsey

Reputation: 564403

You are correct - the issue is that you implemented IComparer(Of String), but not IComparer, which is a completely different interface.

If you switch to use a List(Of String) instead of ArrayList, it will work correctly.

This will also give you type safety within your collection.

In general, ArrayList (and the other System.Collections types) should be avoided in new development.

Upvotes: 1

Related Questions