[W7/XP/C#]Items groeperen in ListView

[Glashelder]

Ik ben aan het knutselen in C# en loop tegen een probleempje aan.. Ik ben een programmatje aan het maken waarin ik bij kan houden welke films er hier allemaal in huis zijn.

Hiervoor heb ik een listview waarin ik films wil groepen op genre. Zolang ik een groep heb gaat het goed:



Zodra ik met meerdere groepen ga werken gaat het fout:


(die rechter krijg ik als ik met de muis boven 'Back to the Future' ga hangen)

Relevant stukje code:

C#:

foreach (Categorie Genre in Result)
{
    ListViewGroup GenreGroep = new ListViewGroup(Genre.Naam, HorizontalAlignment.Left);
    lstFilms.Groups.Add(GenreGroep);

    ArrayList FilmResultaat = new ArrayList();
    FilmResultaat = DB.getFilms(Genre.ID);

    if (FilmResultaat != null) //Voor als een categorie (genre) geen films bevat
    {
        foreach (Film Filmpje in FilmResultaat)
        {
            lstFilms.Items.Add(Filmpje.Titel).Group = GenreGroep;
        }
    }
}

C#:

public class Categorie
{
    public string Naam;
    public int ID;
}

public class Film
{
    public int FilmID;
    public string Titel;
    public string Genre;
    public bool BlueRay;
    public bool DVD;
    public bool VCD;
    public int Jaar;
    public int AfbeeldingID;
}

Wat doe ik hier fout? Ben al een tijdje aan het zoeken en prutsen maar hij blijft dit resultaat geven.. Eerst voegde ik eerst alle genres toe en daarna pas alle films. Zoals je in de code kan zien voegt hij nu een genre toe en daarna direct alle bijbehorende films. Maar ook dat geeft hetzelfde resultaat

Had ook al een en ander gevonden over bugjes in het .NET framework maar dat was uit 2006..neem aan dat dat inmiddels wel opgelost is..

[ Voor 12% gewijzigd door Glashelder op 27-04-2009 12:49 ]

[Haan]

Kan je niet beter een ListViewGroup maken, daar films aan toe voegen en dan de group aan de ListView toevoegen?
En volgens mij kan dit makkelijker als je .Net 3.5 en Linq gebruikt, maar daar moet ik me zelf ook nog altijd een keer in verdiepen.

[Glashelder]

Als ik dat doe dan krijg ik helemaal niets te zien in de ListView. De films voeg ik dan toe aan de groep met items.Add en die groep voeg ik dan toe aan de listview..

[urk_forever]

Heb je nog andere code die iets doet met de listview of instellingen die je op de listview ingesteld hebt? Ik heb dit bij mij net even uitgeprobeerd en hier werkt het gewoon.

[Janoz]

Ik vermoed dat er iets mis gaat met het afhandelen van de redraw events oid. Het ziet er heel erg naar uit dat het bepalen van de afmeting van de componenten gedaan wordt op het moment dat ze nog niet (helemaal) gevuld is.

Ben je soms dingen aan het uitvoeren in je UI thread die eigenlijk in een andere thread zouden moeten?

[Haan]

Heb het hier ook net zitten testen, het werkt met jouw code wel.

[Glashelder]

@Janoz:
Nee ik doe verder helemaal niets boeiends volgens mij.. Welke versie van Visual Studio gebruik je? Ik gebruik 2008 op Windows7.

Ik heb toevallig net toch een nieuwe HDD gehaald, dus ik zal Vista even proberen

[Janoz]

Ik gebruik geen Visual studio. Ik doe niet eens aan C# .

[RobIII]

Glashelder schreef op maandag 27 april 2009 @ 20:07:
Nee ik doe ... niets boeiends volgens mij.. ... Ik gebruik ... Windows7.

Het is niet gezegd dat het hier aan ligt maar wat als je eens test op een OS dat UIT de beta fase is? En dat soort zaken lijken me ook wel relevant om voortaan in je topicstart op te nemen en sowieso iets om eerst even te testen op een gereleased OS voordat je een topic überhaupt opent

[ Voor 24% gewijzigd door RobIII op 27-04-2009 20:58 ]

[Glashelder]

Windows XP staat met een reden in de titel

@RobIII hieronder:
je hebt gelijk, maar ik had het OS erin gezet omdat dit bovenin het scherm stond toen ik een nieuw topic ging openen

Topictitel: Zet taal en/of platform tussen [blokhaken] en omschrijf je probleem kort en krachtig.

Nou staat er inderdaad en/of...maargoed ik heb het er nu wel bijgezet.
Zal het de volgende keer wel laten

[ Voor 80% gewijzigd door Glashelder op 27-04-2009 21:08 ]

[RobIII]

Glashelder schreef op maandag 27 april 2009 @ 20:57:
Windows XP staat met een reden in de titel

Dat was mij nog niet opgevallen Sowieso zetten we doorgaans alleen de taal in de tag Anders krijgen we straks: "[XP/W7/C#/.Net20/Winforms/VS2008SP1/EN] Items groeperen in ListView"
Je tag zou nu zelfs kunnen suggereren dat 't onder Vista/Win2K wél goed werkt.

[ Voor 37% gewijzigd door RobIII op 27-04-2009 21:02 ]

[ThaStealth]

Je code doet het hier (Windows Vista en Windows 7 RC1)

Wat wel opvalt is dat Windows 7 het iets anders uitlijnt dan Windows Vista doet (zie screen voor meer details, de linker is de applicatie in een Windows 7 VM, de rechter is de applicatie op Vista (host)).

Screen

Ik denk dat het probleem in een ander gedeelte van de code zit, dus wat hierboven genoemd word dat je in de GUI thread dingen zit te doen die daar niet in horen.

Kun je misschien een sample bouwen van je applicatie (waar je hard coded) de boxen vult zonder dat er een database achter hangt, dan kun je dat posten en kunnen wij op onze systemen testen.

p.s.

Je kan beter een enum maken met mediadragers ipv bools voor DVD/VCD. Als binnekort Blu Ray wat ingeburgerd is (of de opvolger daar weer van) kom je je hele applicatie aan het omschrijven omdat je overal true's en falses zit rond te slingeren .

Je kan trouwens bij Genre beter het GenreID opgeven ipv de string zelf, (zoals je het gedaan heb bij AfbeeldingsID) op die manier voorkom je dat (als je ergens ooit een spellingsfout maakt in een genrenaam) je deze niet bij elke film afzonderlijk moet controleren en eventueel aanpassen.

Tenslotte is het misschien handig om een lege ArrayList terug te geven bij je DB.getFilms() methode, op die manier voorkom je de ==null check die erna komt (en dus een exception als je het ergens vergeet). Mocht de ArrayList leeg zijn dan zal de foreach gewoon niets doen (ipv zoals nu een exception throwen).