Meine erste App :)

dbaf14 · 24.05.2010, 04:51

So Leute hab mich mal hingesetzt und es in einem Zug vollbracht!!

hab zwar bis jetzt weder ein Buch gelesen noch mich langfristig irgendwo im Netz eingelesen aber es funktioniert!!
Das ist ein Schaltplan den ein kumpel (DJ) von mir für seine Lichttechnik gut gebrauchen kann!

Ich weiß es ist total uneffizient und durcheinander geschrieben aber dennoch wollte ich mal erfragen wie ich es mir hätte einfacher machen können ?
v----- lol sogar nen schreibfehler

hab mal Alles hochgeladen^^

>>>Schaltplan-APK<<<

>>>Schaltplan - Source<<<

..und ein Danke an die wenigen , die mir wenigstens ein paar Hinweise gegeben haben ^_^

mfg dbaf14

badboy_ · 24.05.2010, 08:16

ohne jetzt Java- geschweige denn Android-Kenntnisse zu haben, kann ich aus meiner Programmiererfahrung sagen: DRY! (Don't repeat yourself!)

Du hast da 24 Mal dieses "findViewById(R.id.ButtonXX)...".
Das kann man bestimmt als einfache Schleife verpacken und spart sich somit 23 der 24 Codeschnippsel.

Fr4gg0r · 24.05.2010, 12:52

Zitat:

Zitat von badboy_

ohne jetzt Java- geschweige denn Android-Kenntnisse zu haben, kann ich aus meiner Programmiererfahrung sagen: DRY! (Don't repeat yourself!)

Du hast da 24 Mal dieses "findViewById(R.id.ButtonXX)...".
Das kann man bestimmt als einfache Schleife verpacken und spart sich somit 23 der 24 Codeschnippsel.

Nein, das ist nicht dynamisch ansprechbar.

Btw:
Du packst jedes findViewById(); in einen Block, und packst diesen Block dann noch in den darübergestellten Block.

Die ganzen Klammern sind echt absolut sinnlos.
Das zweite:
Ich würde nicht tausend verschiedene onClickListeneres erstellen, sondern nur einen (was bestimmt auch performanter ist):

Code:

OnClickListener onclick = new OnClickListener(){
   public void onClick(View v){
      int id = v.getId();
      switch(id){
      case R.id.button1: //mach iwas break;
      case R.id.button2:   ... break;
.
.
.
      }
   }
};
findViewById(R.id.button1).setOnClickListener(onclick);
.
.
.

Was das Programm macht hab ich jetzt aber nicht verstanden.

dbaf14 · 24.05.2010, 19:11

Das sind binomische zahlen ,und damit er da im Stress nicht ewig überlegen muss ist so ne app denke ich mal ganz praktisch!

Klar hätte man das noch in zig anderen Varianten schreiben können. .aber so weit bin ich halt noch nicht. :-)

Danke für die Tipps!

Mfg

edit: cool danke . mit nur einem OnClickListener ist das zudem viel übersichtlicher.

Fr4gg0r · 26.05.2010, 00:17

So, habs jetzt auch geschafft die App aufm Emu zu installiere - lmao, hatte die ganze zeit mit adb push gekämpft, statt install zu nehmen.
Jedes mal einen Intent starten und somit auch immer alles neuzeichnen ist stark suboptimal mMn.
Stattdessen solltest du bei jedem Klick lediglich die Zahlen unten ändern...

Scogit · 27.05.2010, 00:48

Zitat:

Zitat von Fr4gg0r

Ich würde nicht tausend verschiedene onClickListeneres erstellen, sondern nur einen (was bestimmt auch performanter ist):

Da wäre ich mir gar nicht mal so sicher bei 3-4 Button's bestimmt aber je mehr unterschiedliche OnClick-Events du hast desto unübersichtlicher wird der Listener und du hast jedes mal toten Code ob das jetzt was macht weis ich nicht kann ich in Java leider noch nicht so beurteilen... hast du mal ein Profiling gemacht?

Eines ist jedoch klar schön ist keine beiden Lösungen.

Fr4gg0r · 27.05.2010, 01:11

Ja hast recht, schneller ist es wohl 30 verschiedene Listener zu haben.
Einen zu haben sollte jedoch weniger Speicher brauchen denk ich mal und auch weniger Initialisierungszeit benötigen?

the_alien · 27.05.2010, 10:51

Hätte mir den Code jetzt auch angeguckt, allerdings kann man den von Rapidshare nichtmehr runterladen...

MichaelS · 27.05.2010, 12:58

saubere programmierung ist das aber nicht 20 mal den gleichen listener auf zu setzen... man sollte seine methoden logisch aufbauen, dass man sie so oft wie möglich und logisch machbar ist benutzen kann. ich würde an der stelle allein wegen dem code style und der übersichtlichkeit auch nur 1 listener nutzen und den strukturiert ordentlich aufbauen... vorallem, wenn nur ne zahl angepasst wird also bitte.. naja jeder wie er mag

bei mir würde sowas nicht durchgehen =)

the_alien · 27.05.2010, 13:47

Ich würde ja gerne mitreden anhand des codes, aber den hab ich nicht. Aber zu den Listenern:
Wenn man 20 mal per Copy & Paste einen Listener anhängt ist das schlecht. Wenn man genau ein Listener Objekt erstellt und das überall dranhängt und dann per switch case irgendetwas abfragt, ist das genauso schlecht. Solche Listener sehe ich hier auf der Arbeit zu Hauf und der Code macht ständig Probleme und man weiss nie was wofür ist.
Dann sollte man eine Methode haben die eben jenen Listener, der sonst 20 mal deklariert wird, beinhaltet und ein neues Objekt davon zurück gibt. Alles andere würde bei mir nicht durchgehen

24.05.2010, 04:51	#1 (permalink)
dbaf14 Android-Hilfe.de Mitglied Modell: SH-12C (NTT Docomo AQUOS PHONE) Registriert seit: 11.07.2009 Beiträge: 126 Abgegebene Danke: 17 Erhielt 9 Danke für 9 Beiträge	Meine erste App :) So Leute hab mich mal hingesetzt und es in einem Zug vollbracht!! hab zwar bis jetzt weder ein Buch gelesen noch mich langfristig irgendwo im Netz eingelesen aber es funktioniert!! Das ist ein Schaltplan den ein kumpel (DJ) von mir für seine Lichttechnik gut gebrauchen kann! Ich weiß es ist total uneffizient und durcheinander geschrieben aber dennoch wollte ich mal erfragen wie ich es mir hätte einfacher machen können ? v----- lol sogar nen schreibfehler hab mal Alles hochgeladen^^ >>>Schaltplan-APK<<< >>>Schaltplan - Source<<< ..und ein Danke an die wenigen , die mir wenigstens ein paar Hinweise gegeben haben ^_^ mfg dbaf14 __________________ will GUNBOUND fertig für ANDROID entwickelt auf meinem GALAXY haben! PETITION :www.petitionspot.com/petitions/gunboundforandroid YOUTUBE : GUNBOUND - ANDROID

24.05.2010, 08:16	#2 (permalink)
badboy_ Neuer Benutzer Modell: Nexus One Registriert seit: 23.05.2010 Beiträge: 19 Abgegebene Danke: 1 Erhielt 2 Danke für 2 Beiträge	AW: meine erste app ^_^ ohne jetzt Java- geschweige denn Android-Kenntnisse zu haben, kann ich aus meiner Programmiererfahrung sagen: DRY! (Don't repeat yourself!) Du hast da 24 Mal dieses "findViewById(R.id.ButtonXX)...". Das kann man bestimmt als einfache Schleife verpacken und spart sich somit 23 der 24 Codeschnippsel.

24.05.2010, 19:11	#4 (permalink)
dbaf14 Android-Hilfe.de Mitglied Modell: SH-12C (NTT Docomo AQUOS PHONE) Registriert seit: 11.07.2009 Beiträge: 126 Abgegebene Danke: 17 Erhielt 9 Danke für 9 Beiträge	AW: meine erste app ^_^ Das sind binomische zahlen ,und damit er da im Stress nicht ewig überlegen muss ist so ne app denke ich mal ganz praktisch! Klar hätte man das noch in zig anderen Varianten schreiben können. .aber so weit bin ich halt noch nicht. :-) Danke für die Tipps! Mfg edit: cool danke . mit nur einem OnClickListener ist das zudem viel übersichtlicher. Geändert von dbaf14 (24.05.2010 um 22:11 Uhr)

26.05.2010, 00:17	#5 (permalink)
Fr4gg0r Android Guru Modell: Samsung Galaxy S (I9000) Registriert seit: 21.12.2009 Beiträge: 2.572 Abgegebene Danke: 76 Erhielt 506 Danke für 295 Beiträge	AW: meine erste app ^_^ So, habs jetzt auch geschafft die App aufm Emu zu installiere - lmao, hatte die ganze zeit mit adb push gekämpft, statt install zu nehmen. Jedes mal einen Intent starten und somit auch immer alles neuzeichnen ist stark suboptimal mMn. Stattdessen solltest du bei jedem Klick lediglich die Zahlen unten ändern... __________________ Meine Apps: GTalk Widget free Filehosting mit direkt download links & ohne werbung

27.05.2010, 01:11	#7 (permalink)
Fr4gg0r Android Guru Modell: Samsung Galaxy S (I9000) Registriert seit: 21.12.2009 Beiträge: 2.572 Abgegebene Danke: 76 Erhielt 506 Danke für 295 Beiträge	AW: meine erste app ^_^ Ja hast recht, schneller ist es wohl 30 verschiedene Listener zu haben. Einen zu haben sollte jedoch weniger Speicher brauchen denk ich mal und auch weniger Initialisierungszeit benötigen? __________________ Meine Apps: GTalk Widget free Filehosting mit direkt download links & ohne werbung

27.05.2010, 10:51	#8 (permalink)
the_alien Android Guru Modell: T-Mobile G1, Nexus One, Samsung Galaxy Tab 10.1, Samsung Galaxy Nexus Registriert seit: 04.05.2009 Beiträge: 1.599 Abgegebene Danke: 128 Erhielt 210 Danke für 193 Beiträge	AW: meine erste app ^_^ Hätte mir den Code jetzt auch angeguckt, allerdings kann man den von Rapidshare nichtmehr runterladen...

27.05.2010, 12:58	#9 (permalink)
MichaelS Fortgeschrittenes Mitglied Registriert seit: 14.08.2009 Beiträge: 354 Abgegebene Danke: 23 Erhielt 49 Danke für 43 Beiträge	AW: Meine erste App :) saubere programmierung ist das aber nicht 20 mal den gleichen listener auf zu setzen... man sollte seine methoden logisch aufbauen, dass man sie so oft wie möglich und logisch machbar ist benutzen kann. ich würde an der stelle allein wegen dem code style und der übersichtlichkeit auch nur 1 listener nutzen und den strukturiert ordentlich aufbauen... vorallem, wenn nur ne zahl angepasst wird also bitte.. naja jeder wie er mag bei mir würde sowas nicht durchgehen =) __________________ http://android.fungamer.net/

27.05.2010, 13:47	#10 (permalink)
the_alien Android Guru Modell: T-Mobile G1, Nexus One, Samsung Galaxy Tab 10.1, Samsung Galaxy Nexus Registriert seit: 04.05.2009 Beiträge: 1.599 Abgegebene Danke: 128 Erhielt 210 Danke für 193 Beiträge	AW: Meine erste App :) Ich würde ja gerne mitreden anhand des codes, aber den hab ich nicht. Aber zu den Listenern: Wenn man 20 mal per Copy & Paste einen Listener anhängt ist das schlecht. Wenn man genau ein Listener Objekt erstellt und das überall dranhängt und dann per switch case irgendetwas abfragt, ist das genauso schlecht. Solche Listener sehe ich hier auf der Arbeit zu Hauf und der Code macht ständig Probleme und man weiss nie was wofür ist. Dann sollte man eine Methode haben die eben jenen Listener, der sonst 20 mal deklariert wird, beinhaltet und ein neues Objekt davon zurück gibt. Alles andere würde bei mir nicht durchgehen

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Ansicht
Linear-Darstellung Zur Hybrid-Darstellung wechseln Zur Baum-Darstellung wechseln

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Erste kostenpflichtige App - Gewerbeanmeldung?	Alarmania	Android App Entwicklung	15	04.04.2012 09:06
Erste Finanz App für den deutschen Finanzmarkt	MichaelS	Sonstige Apps & Widgets	66	06.07.2011 15:35
Meine erste offizielle App (beta testversion)	quiddix	Android App Entwicklung	5	25.05.2010 11:58
Das Milestone, meine erste Erfahrungen	Thurg	Motorola Milestone Forum	22	08.03.2010 20:02
wo ist meine app im market?	swordi	Android App Entwicklung	7	04.06.2009 13:02

Meine erste App :)

Ähnliche Themen