Bug in CollectionView Filterlogik

  • WPF MVVM

Es gibt 6 Antworten in diesem Thema. Der letzte Beitrag () ist von kafffee.

    Bug in CollectionView Filterlogik

    Hallo allerseits an diesem wunderschön grauen Novembertag,

    nach längerer Zwangspause bin ich mal wieder online ;)

    ich habe ein Problem mit einer Filterlogik, bzw. zweier Filterlogiken im Zusammenspiel. Ich glaube jedenfalls es liegt an der Filterlogik, kann aber auch andere Ursachen haben...

    Der Zusammenhang ist recht komplex, aber ich versuch mal mein Bestes das zu erklären:

    Ich habe drei Listboxen: Interpeten, Alben und Musiktitel. Diese sind jeweils an CollectionViews gebunden: AnzuzeigendeInterpreten, AnzuzeigendeAlben und AnzuzeigendeMusiktitel.

    Die relevanten Filerlogiken der CollectionViews sehen so aus:

    VB.NET-Quellcode

    1. Private Function AnzuzeigendeAlben_Filter(obj As Object) As Boolean
    3. Dim Objekt As MP3FileInfo = DirectCast(obj, MP3FileInfo)
    4. Dim IstErlaubt = Not Objekt.Stream
    5. If Not IstErlaubt Then Return IstErlaubt
    6. If Not String.IsNullOrEmpty(TextFilter) Then
    7. IstErlaubt = Objekt.Interpret.ToLower.Contains(TextFilter) OrElse Objekt.Album.ToLower.Contains(TextFilter)
    8. If Not IstErlaubt Then Return IstErlaubt
    9. ElseIf AusgewählterInterpret IsNot Nothing Then
    10. If Objekt.Interpret = _AusgewählterInterpret.Interpret Then
    11. IstErlaubt = True
    12. Else
    13. IstErlaubt = False
    14. End If
    15. If Not IstErlaubt Then Return IstErlaubt
    16. End If
    17. For Each selGenre In AnzuzeigendeGenres.Where(Function(g) Not g.IstSelektiert)
    18. If Objekt.Genre = selGenre.Genre Then Return False
    19. Next
    20. Return IstErlaubt
    21. End Function
    23. Private Function AnzuzeigendeMusiktitel_Filter(obj As Object) As Boolean
    25. Dim Objekt As MP3FileInfo = DirectCast(obj, MP3FileInfo)
    26. Dim IstErlaubt = Not Objekt.Stream
    27. If Not IstErlaubt Then Return IstErlaubt
    28. If Not String.IsNullOrEmpty(TextFilter) Then
    29. IstErlaubt = Objekt.Interpret.ToLower.Contains(TextFilter) OrElse Objekt.Album.ToLower.Contains(TextFilter) OrElse Objekt.Musiktitel.ToLower.Contains(TextFilter)
    31. ElseIf AusgewähltesAlbum IsNot Nothing AndAlso AusgewählterInterpret Is Nothing Then
    32. If Objekt.Album = _AusgewähltesAlbum.Album Then
    33. IstErlaubt = True
    34. Else
    35. IstErlaubt = False
    36. End If
    38. ElseIf AusgewähltesAlbum Is Nothing AndAlso AusgewählterInterpret IsNot Nothing Then
    39. If Objekt.Interpret = _AusgewählterInterpret.Interpret Then
    40. IstErlaubt = True
    41. Else
    42. IstErlaubt = False
    43. End If
    45. ElseIf AusgewähltesAlbum IsNot Nothing AndAlso AusgewählterInterpret IsNot Nothing Then
    46. If Objekt.Album = _AusgewähltesAlbum.Album Then
    47. IstErlaubt = True
    48. Else
    49. IstErlaubt = False
    50. End If
    52. End If
    53. If Not IstErlaubt Then Return IstErlaubt
    54. For Each selGenre In AnzuzeigendeGenres.Where(Function(g) Not g.IstSelektiert)
    55. If Objekt.Genre = selGenre.Genre Then Return False
    56. Next
    57. Return IstErlaubt
    58. End Function


    Diese Properties sind an die .SelectedItem der Listboxen gebunden:

    VB.NET-Quellcode

    1. Public Property AusgewählterInterpret As MP3FileInfo
    2. Get
    3. Return _AusgewählterInterpret
    4. End Get
    5. Set(value As MP3FileInfo)
    6. If value IsNot Nothing Then
    7. AusgewähltesAlbum = Nothing
    8. _AusgewählterInterpret = value
    9. RaisePropertyChanged()
    10. AnzuzeigendeAlben.Refresh()
    11. AnzuzeigendeMusiktitel.Refresh()
    12. End If
    13. End Set
    14. End Property
    16. Public Property AusgewähltesAlbum As MP3FileInfo
    17. Get
    18. Return _AusgewähltesAlbum
    19. End Get
    20. Set(value As MP3FileInfo)
    21. If value IsNot Nothing Then
    22. _AusgewähltesAlbum = value
    23. RaisePropertyChanged()
    24. AnzuzeigendeMusiktitel.Refresh()
    25. MessageBox.Show("Album geändert")
    26. End If
    27. End Set
    28. End Property


    Nun sollen, wenn ein Interpret in der Listbox ausgewählt wird, alle Alben und Musiktitel des jeweiligen Interpreten in den entsprechenden anderen Listboxen angezeigt werden.
    Und dazu: Wenn ein Album ausgewählt wird, alle Musiktitel...

    Funktioniert auch, beim ersten Mal. Aber wenn ich ein Album aussuche und dann wieder einen Interpret, scheint die Listbox mit den Musiktiteln nicht sichtbar aktualisiert zu werden... Die der Alben aber schon. Ich glaube da kommt sich gegenseitig was in die Quere...

    So, ich hoffe, ich habe alle relevanten Codestellen gepostet, aber ich habe mit Sicherheit was vergessen. Zur Sicherheit lade ich das Projekt mal hoch, dann kann man auch ausprobieren, was ich meine... Der relevante Code ist in der SucheViewModel.vb
    Dateien

    Hallo

    kafffee schrieb:

    scheint die Listbox mit den Musiktiteln nicht sichtbar aktualisiert zu werden... Die der Alben aber schon


    Weil du die Titelliste ja auch nicht aktualisierst.
    Beim auswählen eines Interpreten setzt du mit ​AusgewähltesAlbum = Nothing zwar korrekt auf null (Nothing) aber das ignorierst du schlicht weil du im Setter (hier dein Setter)

    VB.NET-Quellcode

    1. ​
    2. If value IsNot Nothing Then
    3. AusgewählterInterpret = Nothing
    4. _AusgewähltesAlbum = value
    5. RaisePropertyChanged()
    6. AnzuzeigendeMusiktitel.Refresh()
    7. MessageBox.Show("Album geändert")
    8. End If

    Einfach drübersprichst, der Code wird ja nur ausgeführt wenn das Album NICHT Nothing ist.

    VB.NET-Quellcode

    1. ​
    2. AusgewählterInterpret = Nothing
    3. If value IsNot Nothing Then
    4. _AusgewähltesAlbum = value
    5. RaisePropertyChanged()
    6. End If
    7. AnzuzeigendeMusiktitel.Refresh()
    8. MessageBox.Show("Album geändert")


    So sollte es klappen. Das solltest du aber gemerkt haben da du keine Messagebox angezeigt bekommen hast.
    Ein kleiner Tipp am Rande, lass das mit den Messageboxen (Abgesehen davon das du die ja gar nicht verwenden dürftest bei MVVM (aber das ist wieder ein anderes Thema)) und verwende lieber ​Debug.WriteLine("Album geändert") für sowas. Das musst du dann bei "Auslieferung" auch nicht mühsam auskommentieren oder löschen da der Compiler im Release drüberspringt.

    Grüße
    Sascha
    If _work = worktype.hard Then Me.Drink(Coffee)
    Seht euch auch meine Tutorialreihe <WPF Lernen/> an oder abonniert meinen YouTube Kanal.

    ## Bitte markiere einen Thread als "Erledigt" wenn deine Frage beantwortet wurde. ##

    @Nofear23m

    Hey Sascha. Erstmal nachträglich nen schönen 1. Advent. Sry dass ich erst jetzt schreib, ich hatte mal wieder nicht den Kopf zu programmieren.

    Irgendwie komm ich mit deinem Post nicht ganz klar. Du schreibst:

    Nofear23m schrieb:

    Beim auswählen eines Interpreten...


    Veränderst dann aber den Code von AusgewähltesAlbum

    Selbst wenn ich statt AusgewählterInterpret = Nothing AusgewähltesAlbum = Nothing nehme funktioniert das nicht wie ich es will.

    Mein Code den ich jetzt habe ist:

    VB.NET-Quellcode

    1. Public Property AusgewählterInterpret As MP3FileInfo
    2. Get
    3. Return _AusgewählterInterpret
    4. End Get
    5. Set(value As MP3FileInfo)
    6. AusgewähltesAlbum = Nothing
    7. If value IsNot Nothing Then
    8. _AusgewählterInterpret = value
    9. RaisePropertyChanged()
    10. End If
    11. AnzuzeigendeAlben.Refresh()
    12. AnzuzeigendeMusiktitel.Refresh()
    13. End Set
    14. End Property
    16. Public Property AusgewähltesAlbum As MP3FileInfo
    17. Get
    18. Return _AusgewähltesAlbum
    19. End Get
    20. Set(value As MP3FileInfo)
    21. If value IsNot Nothing Then
    22. _AusgewähltesAlbum = value
    23. RaisePropertyChanged()
    24. AnzuzeigendeMusiktitel.Refresh()
    25. End If
    26. End Set
    27. End Property


    Ich habe natürlich auch andere mir nur denkbare Konstellationen probiert aber da bekam ich dann Probleme mit Rekursion und StackOverFlow...

    Du hast bestimmt nur einen Flüchtigkeitsfehler gemacht...

    Grüsse aus Kirchheim :)

    Lese nochmals meinen Post GENAU.

    kafffee schrieb:

    Veränderst dann aber den Code von AusgewähltesAlbum

    Richtig, aber im Code des Setters von ​AusgewähltesAlbum veränderst du nunmal den Interpreten mit ​AusgewählterInterpret = Nothing

    Änderst du einfach den Code wie von mir beschreiben funktioniert es doch. Zumindest bei mir. Ansonsten musst du Bescheid geben was genau denn nun nicht so funktioniert wie du es willst.

    Grüße
    Sascha
    If _work = worktype.hard Then Me.Drink(Coffee)
    Seht euch auch meine Tutorialreihe <WPF Lernen/> an oder abonniert meinen YouTube Kanal.

    ## Bitte markiere einen Thread als "Erledigt" wenn deine Frage beantwortet wurde. ##

    Nofear23m schrieb:

    Änderst du einfach den Code wie von mir beschreiben funktioniert es doch. Zumindest bei mir. Ansonsten musst du Bescheid geben was genau denn nun nicht so funktioniert wie du es willst.


    Yep ich glaube da ham wir uns falsch verstanden. Hier mal was genau ich meine:

    (1) Starte das Programm.
    (2) Wähle den Interpreten DMX aus:
    -Es erscheint in den Alben "Great Depression" und in den Titeln "Calling Up" und "Trina Moe"
    >>>>so soll es auch sein.
    (3) Klicke nun in den Alben auf "Great Depression"
    >>>>> Die Titel bleiben unverändert... so soll es ebenfalls sein.
    (4) Klicke nun in den Interpreten auf "Eminem".
    >>>>>>> Die Alben werden korrekt angezeigt ("Marshall Mathers LP"), aber die Titel zeigen immer noch "Calling Up" und "Trina Moe" an. >>>> was natürlich falsch ist.

    Warum also ist das so? Direkt nach dem Programmstart geht's doch? Wenn ich die gleiche Änderung in der AusgewählterInterpret mache, gibt es leider einen Stack Overflow...

    Zur Sicherheit mal der Code so, wie ich dich verstanden habe:

    VB.NET-Quellcode

    1. ​Public Property AusgewählterInterpret As MP3FileInfo
    2. Get
    3. Return _AusgewählterInterpret
    4. End Get
    5. Set(value As MP3FileInfo)
    6. If value IsNot Nothing Then
    7. AusgewähltesAlbum = Nothing
    8. _AusgewählterInterpret = value
    9. RaisePropertyChanged()
    11. AnzuzeigendeAlben.Refresh()
    12. AnzuzeigendeMusiktitel.Refresh()
    13. End If
    14. End Set
    15. End Property
    17. Public Property AusgewähltesAlbum As MP3FileInfo
    18. Get
    19. Return _AusgewähltesAlbum
    20. End Get
    21. Set(value As MP3FileInfo)
    22. AusgewählterInterpret = Nothing
    23. If value IsNot Nothing Then
    24. _AusgewähltesAlbum = value
    25. RaisePropertyChanged()
    26. End If
    27. AnzuzeigendeMusiktitel.Refresh()
    28. End Set
    29. End Property

    Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von „kafffee“ ()

    OK, jetzt sehe ich was du meinst.

    Die StackOverflowException bekommst du weil der Code ja im Kreis läuft.

    Bemerkung: Ich weis es wurde dir schon mehrmals gesagt soweit ich weis aber wenn du deine Variablen, Methode und Properties schon in Deutsch benamst dann bitte lass wenigstens so sachen wie Umlaute, das kann unter umständen (gerne unter XAML) zu erheblichen Problemen führen.

    Ändere den Code wie folgt:

    VB.NET-Quellcode

    1. ​
    2. Public Property AusgewählterInterpret As MP3FileInfo
    3. Get
    4. Return _AusgewählterInterpret
    5. End Get
    6. Set(value As MP3FileInfo)
    7. _AusgewähltesAlbum = Nothing
    8. If value IsNot Nothing Then
    10. _AusgewählterInterpret = value
    11. RaisePropertyChanged()
    12. AnzuzeigendeAlben.Refresh()
    13. AnzuzeigendeMusiktitel.Refresh()
    14. End If
    15. End Set
    16. End Property
    18. Public Property AusgewähltesAlbum As MP3FileInfo
    19. Get
    20. Return _AusgewähltesAlbum
    21. End Get
    22. Set(value As MP3FileInfo)
    23. _AusgewählterInterpret = Nothing
    24. If value IsNot Nothing Then
    25. _AusgewähltesAlbum = value
    26. RaisePropertyChanged()
    27. End If
    28. AnzuzeigendeMusiktitel.Refresh()
    29. Debug.WriteLine("Album geändert")
    30. End Set
    31. End Property


    Grüße und gute N8
    Sascha
    If _work = worktype.hard Then Me.Drink(Coffee)
    Seht euch auch meine Tutorialreihe <WPF Lernen/> an oder abonniert meinen YouTube Kanal.

    ## Bitte markiere einen Thread als "Erledigt" wenn deine Frage beantwortet wurde. ##

    Yep schon besser. Aber wenn ich jetzt z.B. zuerst DMX auswähle, dann auf Great Depression klicke und dann auf Eminem klicke (also auf den Interpreten, der noch nicht ausgewählt ist, dann erscheinen wieder alle Musiktitel, und nicht nur die von Eminem (Stan), also wie wenn überhaupt kein Interpret ausgewählt ist. Klar, denn wenn wir das Album ändern wird ja _AusgewählterInterpret wieder auf Nothing gesetzt und somit wird der Code unter If value IsNot Nothing nicht ausgeführt. Das kann man bestimmt irgendwie beheben, aber meine Kreativität bzw. Vorstellungskraft stösst da an ihre Grenzen.

    Ich habs mir jetzt noch eine Weile angeschaut und das Einzige was mir noch dazu einfällt ist:
    Ich habe bemerkt, dass der ausgewählte Interpret bei Klick auf ein Album natürlich nicht deselektiert wird und andersherum. Was also, wenn man statt _AusgewählterInterpret = Nothing bzw. _AusgewähltesAlbum = Nothing auf Nummer sicher geht, und anstatt dessen die jeweiligen SelectedIndex die man natürlich vorher auch gebunden hat auf -1 setzt, so dass auch im UI die Items deselektiert werden. Aber da wird man dann wahrscheinlich wieder in die Rekursion kommen...

    Fällt dir noch was dazu ein?

    Edit:
    Ahhhhh jetzt ja... Kommando zurück... Das _AusgewählterInterpret = Nothing brauchen wir nicht, also weggelassen und schwups gehts...

    Dieser Beitrag wurde bereits 4 mal editiert, zuletzt von „kafffee“ ()

Facebook
Telefonspende