Abhilfe von Spaghetti-Code!?!

aquarium1974

Aktives Mitglied
Hallo zusammen,

ich beschäftige mich gerade mit Android und hab mir ein kleines Tool zusammengebaut. Das funktioniert auch soweit alles wie es soll, aber da ich nicht so der Java-Freak bin und es mit Klasen und Objekten habe wird mein Code mehr und mehr Spaghetti-artig.

Kann mir jemand sagen, wie man sowas vernünftig zerlegt?
Das Problem was ich habe ist, daß ich oben im Code Variablen definiere und auf diese zugreife und dann unten weiter in buttonSenden.OnClickListener() nochmal drauf zugreifen will, dort aber alles noch einmal holen muss. Dort ist es leider sonst unbekannt. Es funktioniert so, ist aber ziemlich unelegant denk ich.

Kann mir einer verraten, wie ich das vernünftig zerlege, so das ich mal objektorientiert programmiere und nicht mit diesem Spaghetti-Code?
Danke.

Aquarium1974

Java:
public void onCreate(Bundle savedInstanceState) {

		super.onCreate(savedInstanceState);
		setContentView(R.layout.main);
		final Test meinString = new Test();
	
		
		auslesenKonfiguration();
		textOut = (TextView) findViewById(R.id.strasseort);
		locationManager = (LocationManager) getSystemService(LOCATION_SERVICE);
		geocoder = new Geocoder(this);
		

		Location lastLocation = locationManager
				.getLastKnownLocation(LocationManager.GPS_PROVIDER);
		if (lastLocation != null)
			onLocationChanged(lastLocation);

		// Fokus auf erste Textview legen, damit das Softkeyboard nicht beim
		// Programmstart aufpoppt
		TextView textview = (TextView) findViewById(R.id.textview);
		textview.setFocusableInTouchMode(true);
		textview.requestFocus();

		Button buttonJetztDatum = (Button) findViewById(R.id.ButtonJetztDatum);
		Button buttonJetztUhrzeit = (Button) findViewById(R.id.ButtonJetztUhrzeit);
		Button buttonGPS = (Button) findViewById(R.id.GPS);
		Button buttonSenden = (Button) findViewById(R.id.buttonSenden);
		Button buttonBeweisfoto = (Button) findViewById(R.id.buttonBeweisfoto);

		Spinner spinner1 = (Spinner) findViewById(R.id.spinnerfahrzeugtyp);
		ArrayAdapter<CharSequence> adapter1 = ArrayAdapter.createFromResource(
				this, R.array.arrFahrzeugtyp,
				android.R.layout.simple_spinner_item);
		adapter1.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
		spinner1.setAdapter(adapter1);
		spinner1.setOnItemSelectedListener(new MyOnItemSelectedListener());

		Spinner spinner2 = (Spinner) findViewById(R.id.spinnerordnungsamt);
		ArrayAdapter<CharSequence> adapter2 = ArrayAdapter.createFromResource(
				this, R.array.arrOrdnungsamt,
				android.R.layout.simple_spinner_item);
		adapter2.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
		spinner2.setAdapter(adapter2);
		// spinner2.setOnItemSelectedListener(new MyOnItemSelectedListener2());

		Spinner spinner3 = (Spinner) findViewById(R.id.spinnerfabrikat);
		ArrayAdapter<CharSequence> adapter3 = ArrayAdapter
				.createFromResource(this, R.array.arrFabrikat,
						android.R.layout.simple_spinner_item);
		adapter3.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
		spinner3.setAdapter(adapter3);

		Spinner spinner4 = (Spinner) findViewById(R.id.spinnervergehen);
		ArrayAdapter<CharSequence> adapter4 = ArrayAdapter
				.createFromResource(this, R.array.arrVergehen,
						android.R.layout.simple_spinner_item);
		adapter4.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
		spinner4.setAdapter(adapter4);

		buttonJetztDatum.setOnClickListener(new OnClickListener() {
			@Override
			public void onClick(View v) {
				// Toast.makeText(Test1Activity.this, "Datum",
				// Toast.LENGTH_SHORT)
				// .show();

				EditText t = (EditText) findViewById(R.id.datum);
				t = (EditText) findViewById(R.id.datum);

				SimpleDateFormat dateFormat = new SimpleDateFormat("dd.MM.yyyy");
				Date date = new Date(System.currentTimeMillis());
				final String datetime = dateFormat.format(date);
				t.setText(datetime);
			
			}
		});



buttonSenden.setOnClickListener(new OnClickListener() {
			@Override
			public void onClick(View v) {

				EditText datum = (EditText) findViewById(R.id.datum);
				String StrJetztDatum = datum.getText().toString();

				EditText uhrzeit = (EditText) findViewById(R.id.uhrzeit);
				String StrJetztUhrzeit = uhrzeit.getText().toString();

				EditText strasseort = (EditText) findViewById(R.id.strasseort);
				String strStrasseOrt = strasseort.getText().toString();
				GPSActivity gps = new GPSActivity();

				final EditText kennzeichen = (EditText) findViewById(R.id.kennzeichen);
				String StrKennzeichen = kennzeichen.getText().toString();
				String StrKennzeichen2;
				StrKennzeichen2 = StrKennzeichen.toUpperCase();

                                ...........

                     }
 
Zuletzt bearbeitet:

Evil-Devil

Top Contributor
Du könntest den ein oder anderen redundanten Codeteil in eine Funktion auslagern. Z.B. deinen Spinner Code.

Java:
Spinner spinner1 = (Spinner) findViewById(R.id.spinnerfahrzeugtyp);
        ArrayAdapter<CharSequence> adapter1 = ArrayAdapter.createFromResource(
                this, R.array.arrFahrzeugtyp,
                android.R.layout.simple_spinner_item);
        adapter1.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
        spinner1.setAdapter(adapter1);
        spinner1.setOnItemSelectedListener(new MyOnItemSelectedListener());

Bau dir hier eine Hilfsfunktion die den Krams erzeugt. Denn oben sind deine Spinner soweit ich das richtig gesehen habe bis auf die Daten immer gleich.

Java:
private Spinner createSpinnerByViewId(TypDef viewId, ResourceType rtype, boolean listener) {
	Spinner spinner = new Spinner(); //?
	ArrayAdapter<CharSequence> adapter = ArrayAdapter.createFromRessource(
		this, rtype, android.R.layout.simple_spinner_item);
	adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
	spinner.setAdapter(adapter);
	if (listener) {
		spinner.setOnItemSelectedListener(new MyOnItemSelectedListener())
	}
	return spinner;
}

Wie das genau aussehen muss, wirst du denke ich besser wissen. Mir ist nicht bekannt ob und wie man so einen Spinner in Android instanziert, aber ich denke als Anregung genügt das um doppelten Code zu reduzieren. Damit hättest du dann schon einiges weniger an Spaghetti ;)

[edit]zu langsam ... -.-"[/edit]
 

maestr0

Bekanntes Mitglied
Ich finde es z.b. übersichtlicher die Widgets (Buttons etc..) schon unterhalb der Klasse zu deklarieren,aber ist geschmackssache.

Java:
public class Abhilfe{

Button buttonJetztDatum,buttonJetztUhrzeit.....;

public void onCreate(Bundle savedInstanceState) {

buttonJetztDatum= new Button();
buttonJetztUhrzeit= new Button();


}
}

Außerderm könntest du deine Spinner noch umbenennen,die Namen Spinner1.... mögen für dich nun eindeutige Namen sein,jedoch jemand,der deinen Quelltext sieht müsste sich erstmal mühsam damit auseinandersetzen,welcher Spinner nun für was zuständig ist.

Ansonsten wie oben schon gesagt,manchmal kann man Spaghetti-Code beim Programmieren nicht verhindern :)

Edit:Mist beim ersten war Tomate_Salat schneller ;(
 

aquarium1974

Aktives Mitglied
Es gibt in diesem Fall keine Abhilfe das ist normal beim Programmieren. Aber warum musst du die "Buttonsenden" nocheinmal deklarieren? Bekommst du da ne NullPointerException?

Die Methode sieht die Variable nicht, daher dachte ich ich müßte sie global definieren oder mit Parameterübergabe oder sowas.
Ich hatte dan versucht sie irgendwie global sichtbar zu machen aber das hatte irgendwie nicht hingehauen.

Der Fehler war '..variable not declared...'. Kann jetzt leider nicht reinschauen, bin im Büro.
 

aquarium1974

Aktives Mitglied
Danke für die Antworten.

Also wenn keiner die Händer überm Kopf zusammenschlägt kanns ja soo schlim doch nich sein....
Wundert mich eigentlich, aber egal.

Ich werde die Anregungen dann heute abend mal einbauen.

Vielen Dank.

Gruss

Aquarium1974
 

HimBromBeere

Top Contributor
Kannste größtenteils so lassen, man hat schon viel Schlimmeres gesehen. Oftmals lassen sich (v.a. wenn´s um GUI geht) Programmteile nicht unbegrenzt weit vereinfachen, ohne zu viel zu zerheckseln. Den Hinweis von Evil-Devil möchte ich dir trotzdem ans Herz legen. Außerdem könntest du mal über MVC nachdenken, sofern das Projekt noch ein wenig größer werden soll.
 

maestr0

Bekanntes Mitglied
Wobei ich noch eine Sache dazu sagen möchte :)
Das was "Tomate_Salat" und ich gesagt haben,mit dem Membervariablen anlegen,mag zwar der Übersichtlichkeit dienen,jedoch sollte man in der Programmierung den Scope von Variablen so klein wie möglich halten.
Da du die Variable textOut auch nur an einer Stelle verwendest,kannst du sie ruhig als lokale Variable stehen lassen.
Aber je nach Übersichtlichkeit ist es nicht unbedingt falsch eine Membervariable dafür zu deklarieren.
 
T

Tomate_Salat

Gast
textOut auch nur an einer Stelle verwendest

Das Feld mit der id R.id.strasseort wird später nochmals verwendet.

Ich bin mir nicht sicher, wie teuer es ist, einen View über findView... zu finden. Aber ich händel das generell so, dass alles was außerhalb der onCreate/onCreateView gebraucht wird, als MemberVariable angelegt wird. Ansonsten verliert man imho schnell mal den überblick und verursacht Redundants.
 
Ähnliche Java Themen
  Titel Forum Antworten Datum
J Erste Schritte Vorsicht Spaghetti-Code! Wie verschachtel ich meine If-Anweisung??? Java Basics - Anfänger-Themen 4
T Kann mir jemand wörtlich erklären, was in dem Code genau passiert? Java Basics - Anfänger-Themen 1
Ü Dead Code im Programm? Java Basics - Anfänger-Themen 13
I QR code in Java selber generieren Java Basics - Anfänger-Themen 5
terashy VS Code Project run error Java Basics - Anfänger-Themen 10
JaZuDemNo Code Erklärung Java Basics - Anfänger-Themen 3
M Connect-4-Code analysieren Java Basics - Anfänger-Themen 2
N BMI Rechner Was haltet ihr von dem Code habt ihr Verbesserungsvorschläge weil design teschnisch ist das nicht das geilste würde das gerne überarbeiten Java Basics - Anfänger-Themen 12
W In alten Code zurück- und dort wieder zurechtfinden? Java Basics - Anfänger-Themen 17
T code so schreiben das er von sich selber anpasst (code soll die anzahl aller bustaben bestimmen) Java Basics - Anfänger-Themen 16
J Frage zu einem "Taschenrechner" code Java Basics - Anfänger-Themen 9
T Fehlercode bei code der das Alter ausrechnet Java Basics - Anfänger-Themen 2
T Text einlesen code was kommt dahin? Java Basics - Anfänger-Themen 1
jhfjeh Strukturgramm in code Java Basics - Anfänger-Themen 11
D Tipps zum Code Java Basics - Anfänger-Themen 24
W Java-Code mit Array Java Basics - Anfänger-Themen 14
W Java-Code Java Basics - Anfänger-Themen 2
W Java code- TicTac toe Java Basics - Anfänger-Themen 51
W Java-code Java Basics - Anfänger-Themen 8
W Java-code Java Basics - Anfänger-Themen 9
W Java-Code erklären Java Basics - Anfänger-Themen 6
ohneInformatik; For Schleife. Was macht dieser Code?? Java Basics - Anfänger-Themen 5
Say Fehlenden Code finden in einer while-Schleife? Java Basics - Anfänger-Themen 11
Say 2-DIM Array Code lesen und verstehen Java Basics - Anfänger-Themen 5
Say Stelle in Code herausfinden, wie geht man vor? Java Basics - Anfänger-Themen 12
Say do-While Code Ausführung Java Basics - Anfänger-Themen 3
W Rückfrage zur Programmgestaltung (clean code) Java Basics - Anfänger-Themen 12
M intelliJ auf neuem PC, plötzlich kein Code Java Basics - Anfänger-Themen 3
Pinhg Sound in Greenfoot Code einbinden Java Basics - Anfänger-Themen 2
C Java boolean Code läuft nicht Java Basics - Anfänger-Themen 5
I Code für Bezahlsystem (auch bei Offline Aktivität) Java Basics - Anfänger-Themen 7
J Größter gemeinsamer Teiler: mein Code Java Basics - Anfänger-Themen 6
B Den Dateipfad einer Java Datei durch Code in Selbiger finden? Java Basics - Anfänger-Themen 10
A Wie könnte man diesen Code kürzer machen ? Java Basics - Anfänger-Themen 7
J Frage zu meinem Code (OOP) Java Basics - Anfänger-Themen 4
Alen123 Warum funktioniert mein Code nicht? Java Basics - Anfänger-Themen 64
Max246Sch Frage zu Währungsrechner Code Java Basics - Anfänger-Themen 2
S Hilfe bei Umänderung von Java Code Java Basics - Anfänger-Themen 16
I Code wird nicht ausgeführt Java Basics - Anfänger-Themen 2
K Wie kann man diesen Code schnell und effizient interpretieren (Man hat nur 4 Minuten) Java Basics - Anfänger-Themen 3
R ISBN-10-Code überprüfen Java Basics - Anfänger-Themen 7
I Bitte um Hilfe zu unterstehenden Code Java Basics - Anfänger-Themen 6
I Interface von einer EJB Klasse, um Code zu reduzieren Java Basics - Anfänger-Themen 1
I HTML Code säubern Java Basics - Anfänger-Themen 4
B Brauche Hilfe zu einem Code Java Basics - Anfänger-Themen 5
Temsky34 Problem mit dem Code Java Basics - Anfänger-Themen 17
N Fehler im Code (Aufgabe für Anfänger) Java Basics - Anfänger-Themen 11
N Java-Code abwärtskompatibel machen Java Basics - Anfänger-Themen 4
J Erste Schritte Was mache ich in meinem Code falsch. Java Basics - Anfänger-Themen 3
Ameise04 Variablen Inhalt einer Variable im Code verwenden? Java Basics - Anfänger-Themen 9
S Compiler-Fehler Nicht adressierbarer Code ( Non-addressable code ) Java Basics - Anfänger-Themen 5
Aemulit Java Schaltjahr berechnen Code Java Basics - Anfänger-Themen 7
A Code Problem Java Basics - Anfänger-Themen 6
C Fehler im Code Java Basics - Anfänger-Themen 10
A Zu einem bestimmten Ort im Code springen Java Basics - Anfänger-Themen 11
L Ist der Code richtig Java Basics - Anfänger-Themen 3
josfe1234 code vereinfachen Java Basics - Anfänger-Themen 15
nonickatall Ausführbarkeit von Code testen bzw. Remote Debugging Java Basics - Anfänger-Themen 4
F Frage betreff Programm mit dem man C++-Code in JAVA-Code übersetzen lassen kann Java Basics - Anfänger-Themen 2
S Fehler bei Code mit SubStrings für mich nicht auffindbar. Java Basics - Anfänger-Themen 4
G Programm Code Java Basics - Anfänger-Themen 5
C Code zusammenfassen Java Basics - Anfänger-Themen 5
I Erklärung zum Java Code Java Basics - Anfänger-Themen 2
T Programmablaufsplaninterpretation in Code umformen Java Basics - Anfänger-Themen 1
dieter000 Kurze Frage kann mir ejmand kurz diesen Code erklären, bzw wie man die zeilen erklärt und so Java Basics - Anfänger-Themen 1
AlexVo String zu Java Anweisung getString("*** java code ***") Java Basics - Anfänger-Themen 19
M ISBN-Code Java Basics - Anfänger-Themen 26
B Zeitgleiches Arbeiten am Code mit mehreren Personen? Java Basics - Anfänger-Themen 7
S Wie kann ich bei diesem Code erreichen, das als Ergebnis hier 15 herauskommt? Java Basics - Anfänger-Themen 23
N Kann man den Code vereinfachen? Java Basics - Anfänger-Themen 25
marcooooo Code erklären Java Basics - Anfänger-Themen 28
marcooooo Code erklären Java Basics - Anfänger-Themen 4
S Advent of Code Day4 Java Basics - Anfänger-Themen 4
B Nach eingefügtem Code erkennt Compiler keine Instanzvar und meldet SyntaxError Java Basics - Anfänger-Themen 2
Gaudimagspam Caesars Code entziffern in Java Java Basics - Anfänger-Themen 8
Lukasbsc Wie kann ich meinen Code optimieren? Java Basics - Anfänger-Themen 4
NeoLexx equals()-Methode Verständnis Frage anhand Code Beispiel Java Basics - Anfänger-Themen 22
I Input/Output Code wird doppelt ausgeführt Java Basics - Anfänger-Themen 3
T Main startet nicht bei vorgegebenen Code Java Basics - Anfänger-Themen 41
B Frage zum Code verständnis im Resultat Java Basics - Anfänger-Themen 10
J Fehler im Code, aber ich weiß nicht wieso! Java Basics - Anfänger-Themen 6
S Mehrere Probleme im Code Java Basics - Anfänger-Themen 7
M Code nur für Cracks? Crack the Passwort Übung Java Basics - Anfänger-Themen 7
parrot Code entferneJedeZweiteZiffer Java Basics - Anfänger-Themen 6
G Code kürzen Java Basics - Anfänger-Themen 5
Bluedaishi Source Code Signieren Java Basics - Anfänger-Themen 22
L Best Practice Code Refactoring für Methoden mit fast gleicher Aufbau Java Basics - Anfänger-Themen 6
L Best Practice Code refactern Java Basics - Anfänger-Themen 30
G code kürzen Java Basics - Anfänger-Themen 16
A Code umschreiben Java Basics - Anfänger-Themen 6
Torsten.E JavaFX mit Visual Studio Code verwenden Java Basics - Anfänger-Themen 1
C Beispiel-Code mit Pair wird nicht compiliert. Java Basics - Anfänger-Themen 8
X Reverse algorithm engineering (Java code) Java Basics - Anfänger-Themen 6
T Bufferedwriter code Nullpointerexception Java Basics - Anfänger-Themen 4
V Switch Methode macht Code kaputt Java Basics - Anfänger-Themen 18
R Was muss ich an meinem Code ändern? Java Basics - Anfänger-Themen 2
S Bewertet meinen Code - Part 1 Java Basics - Anfänger-Themen 8
M Java Code Verständnis Java Basics - Anfänger-Themen 4
M Java Code Verständnis Java Basics - Anfänger-Themen 2
J Ich brauche Hilfe bei einem Code (Variablen speichern) Java Basics - Anfänger-Themen 29

Ähnliche Java Themen

Neue Themen


Oben