Custom Cell: UIImage teilweise falsch

danyball · 20. Januar 2015, 18:08

Hi!

In meiner Tabelle hab ich Custom Cells. Dabei nutze ich die Technik mit "dequeueReusableCellWithIdentifier". Klappt soweit auch super.

Nun trat aber 1x folgendes Problem auf:
Von ca. 20 Zeilen hatten 3 Zeilen das gleiche Bild, obwohl es eig. andere Bilder zeigen sollte.

Ich habe ein UIImage bzw. eine eigene Unterklasse auf der Cell. Die eigene Klasse dient dazu, das Image aus dem Internet zu laden.
Das Bild setz ich dann so:

Quellcode

[cell.mypicture initWithUserId:[anyarray valueForKeyPath:@"userid"][indexPath.row] ];

Was innerhalb dieser Methode passiert würd ich als Fehlerquelle ausschließen. Vermute da eher ein Problem mit dem "Wiederverwenden" der Cell.

Muss ich evtl. bevor ich das Bild setze ein "Defaultbild" setzen?

Danke Jungs!!
DAniel.

little_pixel · 20. Januar 2015, 18:17

Hallo,

das Reuse bezieht sich auf "Art" von Zeilen, die Du erstellst und nicht an einer bestimmten Position.
D.h. Du mußt den Titel, Bildchen etc. dann natürlich dennoch setzen.

Viele Grüße

macmoonshine · 21. Januar 2015, 08:11

danyball schrieb:

Was innerhalb dieser Methode passiert würd ich als Fehlerquelle ausschließen.

Naja, der Name der Methode oder zumindest deren Aufruf ist schonmal eindeutig ein Fehler.

danyball · 21. Januar 2015, 08:14

macmoonshine schrieb:

Naja, der Name der Methode oder zumindest deren Aufruf ist schonmal eindeutig ein Fehler.

Oh ok. Schreibst du mir auch warum?
Und ob das dann evtl. schon der Bug ist?!

thx!

gritsch · 21. Januar 2015, 08:18

es ist sicher kein UIImage sondern eventuell ein UIImageView.
das datenladen gehört nicht in die view (da liegt auch dein fehler).
in den datasources sollten die daten schnell geliefert werden (also nicht erst aus dem netz laden).
und ja, wie bereits gesagt deutet bereits der name der methode darauf hin dass du nicht ganz weist was du tust... sorry...

macmoonshine · 21. Januar 2015, 08:21

danyball schrieb:

Oh ok. Schreibst du mir auch warum?

initWithUserId: Init-Methoden haben in Objective-C eine besondere Bedeutung.

danyball schrieb:

Und ob das dann evtl. schon der Bug ist?!

Das glaube ich zwar nicht, lässt aber vermuten, dass du dich nicht an Standards hältst, und das ist führt in der Regel zu fehlerhaften Programmen.

Vermutlich liegt die Ursache deines Problems an einer fehlerhaften Implementierung von tableView:cellForRowAtIndexPath: (s. Beitrag von little_pixel).

danyball · 21. Januar 2015, 09:31

Ok ich verstehe. Sicherlich entstehen bei mir einige Probleme, da ich noch nicht viel mit der OOP Art des Programmierens gearbeitet habe.

Aus Fehlern lernt man bzw. ich

Daher bin ich dankbar für eure Hinweise.

@macmoonshine : "init...." : ja ok, das ist ungünstig.

@gritsch: Gute Erklärung - leuchtet ein, danke! Da liegt sicher der Fehler.

Danke soweit!

danyball · 21. Januar 2015, 10:58

Eine Frage dazu noch:

Das grundsätzliche Vorgehen zum Laden von Bildern in cellForRowAtIndexPath: finde ich in einigen Beispielen wieder. Was nicht unbedingt heißt das es Richtig ist. Aber ganz falsch kann es auch nicht sein.

Z.b. blog.mugunthkumar.com/coding/i…nails-using-mknetworkkit/

Oder wie würdet Ihr vorgehen um Bilder zu laden und in Cells darzustellen?

gritsch · 21. Januar 2015, 11:07

lokalen placeholder anzeigen und das bild nebnläufig laden. wenn das fertig ist, kann man es anzeigen (falls noch benötigt).

macmoonshine · 21. Januar 2015, 11:16

Hier findest es dazu ein Beispiel.

Fortrackz · 21. Januar 2015, 11:49

macmoonshine schrieb:

Hier findest es dazu ein Beispiel.

Ich benutze oft

Quellcode

dispatch_async()

um Bilder nachzuladen. Gibt es da einen Nachteil im Vergleich zu NSURLRequest?

macmoonshine · 21. Januar 2015, 11:56

Fortrackz schrieb:

Gibt es da einen Nachteil im Vergleich zu NSURLRequest?

Meines Wissens nutzt das URL-Ladesystem die Wartemechanismen des Betriebssystems. Wenn der Server mit dem Nachliefern von Daten nicht nachkommen kann, kann der Thread währenddessen etwas Sinnvolleres machen als Warten. Eine synchrone Anfrage blockiert hingegen immer den Thread, in dem sie läuft.

Fortrackz · 22. Januar 2015, 09:44

macmoonshine schrieb:

Fortrackz schrieb:

Gibt es da einen Nachteil im Vergleich zu NSURLRequest?

Meines Wissens nutzt das URL-Ladesystem die Wartemechanismen des Betriebssystems. Wenn der Server mit dem Nachliefern von Daten nicht nachkommen kann, kann der Thread währenddessen etwas Sinnvolleres machen als Warten. Eine synchrone Anfrage blockiert hingegen immer den Thread, in dem sie läuft.

Okay, danke!

danyball · 22. Januar 2015, 19:00

macmoonshine schrieb:

Hier findest es dazu ein Beispiel.

Super, danke dir!

Bezugnehmend darauf hab ich nur noch ein Problem:
Ich würde gern die Subclass des UIImageViews weiter nutzen. Nur kann ich die im Bsp. genannte Technik dann nicht nutzen, da ich das Image in meiner Subclass direkt mit self.image setze.
Oder kann ich da auch irgendwie feststellen, ob die Cell, auf der sich die imageview befindet, sich im sichtbaren Bereich der Tabelle befindet?

macmoonshine · 22. Januar 2015, 22:54

Natürlich: visibleCells, indexPathsForVisibleRows oder cellForRowAtIndexPath:

danyball · 23. Januar 2015, 13:58

Achso ok. Dann übergeb ich meiner (ungünstig benannten) Methode den aktuellen IndexPath der Cell?

In etwa so?!:

Quellcode

[cell.mypicture loadingWithUserId:[anyarray valueForKeyPath:@"userid"][indexPath.row] andPath: indexPath ];

Und kann dann innerhalb der Methode mit einen der 3 von Dir vorgeschlagenen Methoden prüfen ob die Cell nach dem Laden des Bildes sich noch im sichtbaren Bereich befindet. Richtig?

Ich danke Dir!!.....

gritsch · 23. Januar 2015, 14:03

nein, du lädst weder im image noch im imageview bilder aus dem netz. das hat da drin nichts verloren.

macmoonshine · 23. Januar 2015, 14:18

Das oben verlinkte Beispielprojekt behandelt auch den Fall, dass der Nutzer die Tabelle während des Ladens scrollt.

danyball · 23. Januar 2015, 18:17

gritsch schrieb:

nein, du lädst weder im image noch im imageview bilder aus dem netz. das hat da drin nichts verloren.

Das recht verbreitete Framework github.com/rs/SDWebImage macht das auch so wenn ich das richtig gesehen hab. Ist evtl.. nicht ganz MVC-Konform, scheint aber recht gut zu funktionieren.

gritsch · 23. Januar 2015, 18:20

das macht in dem fall aber keinen sinn, denn die cell und damit das imageview werden wiederverwertet.
also wenn du weiterscrollst wird die alte cell von oben wieder hergenommen und ein neues bild gesetzt.
wenn dann noch geladen wird... etc...

Quellcode

Quellcode

Quellcode

Ähnliche Themen