search

fabianonline / geotweeter-for-android

Collaborative project to rebuild the geotweeter as android app. Discontinued since Twitter decided that third-party clients are "evil".
1 stars 0 forks source link

REST-API wird bei jedem Orientierungswechsel neu angefragt. #51

Closed el-emka closed 12 years ago

el-emka commented 12 years ago

Das ist m.E. unnötiger Overhead, da die Streams ja laufen sollten. Könnten wir den Status evtl. im übergebenen Bundle "savedInstanceState" speichern. (Hab noch nie mit den Dingern gearbeitet. Irgendwas müssen die ja können und dem Namen nach könnte es passen)

Gleiches gilt auch für die Registrierung beim GCM-Kram. Das könnte auch die Sache mit den doppelten Accounts elegant lösen. Im Moment mache ich das eher grobschlächtig.

fabianonline commented 12 years ago

Ist ein Duplikat von #3. Da hier mehr Text steht, mache ich #3 mal zu.

fabianonline commented 12 years ago

Nein. Das Bundle ist speziell defaür gedacht, Informationen über einen kompletten Neustart der App hinaus zu speichern. Beispiel dafür wäre ein halb geschriebener Tweet. Der sollte, wenn die NewTweetActivity geschlossen wird, in dem Bundle gesichert werden, damit er dann wieder angezeigt werden kann, wenn man den Geotweeter (und im speziellen die NewTweetActvity) wieder startet.

fabianonline commented 12 years ago

Doch noch nicht fertig. Wie ist das mit deinen (@el-emka) Doppelaccount-Checks? Können die damit auch leben?

el-emka commented 12 years ago

Kann grad nicht hundertprozentig folgen. (bin am ernten) Womit können die leben?

el-emka commented 12 years ago

Es sieht jedenfalls so aus, als könnten sie damit leben.

fabianonline commented 12 years ago

Frage war halt, ob der von dir zurechtgefrickelte Code, der doppelte Account vermeiden soll, nicht auch evtl. auf isRunning umgebaut werden sollte.

el-emka commented 12 years ago

Ohne akut nochmal in den Code geguckt zu haben, meine ich, dass du das en passant gemacht hast.

fabianonline commented 12 years ago

Öhm... Habe ich? :D

el-emka commented 12 years ago

Die Dublettenkontrolle könnte ich jetzt sogar wieder rausschmeißen, weil du den Teil in so einem !isRunning-Block mit untergebracht hattest.

fabianonline commented 12 years ago

Großartig. Dann mach das doch einfach mal bei Gelegenheit. Ich wiedereröffne dir dieses Issue so lange einfach mal. ;-)