[C#] Case insensitive Contains op ObjectCollection

[Oogst]

Ik wil graag case insensitive kijken of een element voorkomt in een ListBox en een ComboBox. Nou dacht ik in eerste instantie daarvoor gewoon StringComparison.OrdinalIgnoreCase toe te kunnen voegen aan de call naar Contains, dus zo:

code:

mijnListBox.Items.Contains(textBoxName.Text, StringComparison.OrdinalIgnoreCase)
mijnComboBox.Items.Contains(textBoxName.Text, StringComparison.OrdinalIgnoreCase)

Ondanks dat Linq aan Contains() toevoegt dat je er OrginalIgnoreCase op kunt gebruiken, accepteert ie dit niet, dus blijkbaar voegt Linq dat niet toe aan de Contains functie van deze specifieke datastructuur of zo? Ik ben een C++ programmeur, dus de details van dit soort dingen in C# zijn nog nieuw voor me.

Mijn volgende idee was dus om zelf een functie te schrijven die case insensitive checkt of een element in de lijsten zit. Dat krijg ik echter alleen werkend met twee losse functies voor ListBox en ComboBox:

code:

public static bool containsCaseInsensitive(ListBox.ObjectCollection collection, string text)
{
    //check all the elements in the collection
    foreach (string element in collection)
    {
        if (text.Equals(element, StringComparison.OrdinalIgnoreCase))
        {
            return true;
        }
    }

    //if we get here, then the element was not found
    return false;
}

public static bool containsCaseInsensitive(ComboBox.ObjectCollection collection, string text)
{
    //check all the elements in the collection
    foreach (string element in collection)
    {
        if (text.Equals(element, StringComparison.OrdinalIgnoreCase))
        {
            return true;
        }
    }

    //if we get here, then the element was not found
    return false;
}

Het lijkt me duidelijk dat zo dezelfde functie twee keer maken niet cool is. Ik dacht dus iets te kunnen doen met generics of met dat ze allebei overerven van IList of zo, maar dat krijg ik niet compilend. Deze twee werken bijvoorbeeld ook niet:

code:

public static bool containsCaseInsensitive<T>(IList<T> collection, string text)
public static bool containsCaseInsensitive<T>(T collection, string text)

Dus eigenlijk twee vragen:
1. Waarom werkt Contains op ListBox en ComboBox niet met StringComparison.OrdinalIgnoreCase?
2. Hoe kan ik voorkomen dat ik die functie twee keer moet copy-pasten met verschillende parameters?

Alvast bedankt!

[RobIII]

Listboxes/Comboboxes bevatten niet per-se strings maar objects (items is van type ObjectCollection) dus een StringComparison enum value meegeven zou raar zijn bij een Contains method. Wat je wel kunt doen (makkelijkst denk ik) is zoiets:

C#:

mijnListBox.Items.OfType<string>().Contains("appel", StringComparer.OrdinalIgnoreCase)

De OfType returned een collectie welke enkel strings bevat (in dit geval alle items dus) en daarna kun je wel gewoon Contains gebruiken zoals je dat wilde. Als je 100% zeker weet dat er alleen maar strings in de items collectie zitten kun je i.p.v. OfType<string> ook Cast<string> gebruiken; daarmee worden alle items naar string gecast.

[ Voor 58% gewijzigd door RobIII op 14-03-2012 14:53 ]

[Oogst]

Hmm, die pikt ie ook niet, met hele mooie errors:

NewSettingForm.cs(91,13): error CS1928: 'System.Collections.Generic.IEnumerable<string>' does not contain a definition for 'Contains' and the best extension method overload 'System.Linq.Queryable.Contains<TSource>(System.Linq.IQueryable<TSource>, TSource, System.Collections.Generic.IEqualityComparer<TSource>)' has some invalid arguments

NewSettingForm.cs(91,13): error CS1929: Instance argument: cannot convert from 'System.Collections.Generic.IEnumerable<string>' to 'System.Linq.IQueryable<string>'

NewSettingForm.cs(91,78): error CS1503: Argument 3: cannot convert from 'System.StringComparison' to 'System.Collections.Generic.IEqualityComparer<string>'

[ Voor 188% gewijzigd door Oogst op 14-03-2012 14:55 ]

[RobIII]

De code die ik postte werkt bij mij. Dus ergens doe je iets raars

Maarre, laat eens zien hoe je de listbox/combobox vult en de code die je nu gebruikt met die Contains()?

[Oogst]

Hmm, interessent, benieuwd wat ik fout doe...

Dit is hoe ik hem aanroep:

code:

if (listBoxSetting.Items.OfType<string>().Contains("QQQ", StringComparison.OrdinalIgnoreCase))

Dit is hoe die listBoxSetting wordt gedeclareerd:

code:

private System.Windows.Forms.ListBox listBoxSetting;

En hier wordt er wat ingestopt:

code:

this.listBoxSetting.Items.AddRange(new object[] {"<Default>"});

[RobIII]

Je gebruikt een StringComparison.OrdinalIgnoreCase enum; als je mijn code goed bekijkt zie je dat ik een (static) StringComparer class (instance) meegeef Het duurde effe voor ik 't zag maar de foutmelding is duidelijk: hij verwacht een stringcomparer, geen stringcomparison enum value

[ Voor 49% gewijzigd door RobIII op 14-03-2012 15:11 ]

[DarthDavy]

Je steekt geen strings in je listbox, maar een lijst van objecten met één string.

code:

this.listBoxSetting.Items.Add("<Default>");

Zo kan je bereiken, wat je wil bereiken zonder zelf functies te gaan schrijven (niet nodig, .NET heeft voldoende string functies)

code:

comboBox1.Items.OfType<string>().Contains("", StringComparer.Create(CultureInfo.CurrentCulture, true));

[RobIII]

DarthDavy schreef op woensdag 14 maart 2012 @ 15:12:
Je steekt geen strings in je listbox, maar een lijst van objecten met één string.

En dus? Dat is dus een lijst van strings.
Dit werkt net zo goed:

C#:

myListbox.Items.AddRange(new object[] { "AAA", "BBB", "CCC", "DDD" });
var f = myListbox.Items.Cast<string>().Contains("aaa", StringComparer.OrdinalIgnoreCase);
Trace.WriteLine(string.Format("Item found: {0}", f));

Item found: True

DarthDavy schreef op woensdag 14 maart 2012 @ 15:12:

code:

comboBox1.Items.OfType<string>().Contains("", StringComparer.Create(CultureInfo.CurrentCulture, true));

Ik prefereer de static instances van de StringComparers; niet zo lelijk als dat hele .Create(blatiebloe) gedoe

[ Voor 30% gewijzigd door RobIII op 14-03-2012 15:16 ]

[Oogst]

Jups, dat fixt het, nu doet ie het! Het was inderdaad StringComparer ipv StringComparison.

Dank u!

[DarthDavy]

RobIII schreef op woensdag 14 maart 2012 @ 15:13:
[...]

En dus? Dat is dus een lijst van strings.

Jah, was in de war met de AddRange...