Vector Manipulieren.. Wie?

[JDownHD]

Hallooooo,

ich will Folgendes realisieren:
Ich habe eine plz.txt Datei, welche wie Folgt aussieht:

Hattingen, 45525
Hattingen, 45527
Bochum, 44785
.
.
.
usw.​

Ich möchte nun durch die Eingabe einer Stadt die zugehörigen Postleitzahlen ausgegeben bekommen.
Ich möchte also nicht Hattingen eingeben und die Ausgabe:

Hattingen, 45525
Hattingen, 45527​

erhalten sondern:

45525
45527​

also nur die Plz. Bei meinem bisherigen Code habe ich das Problem, dass ich nicht weiß wie ich die Isolierung von "Hattingen" und "45525" in zwei Vectoren bewerkstelligen soll, bzw. wie ich es sonst lösen sollte???:L Da Fehlt mir die Erfahrung. - Ich kann auch keinen vergleichbaren Code finden.

Nun also mein Code:

import java.io.*;
import java.util.*;
import javax.swing.*;

class Aufgabe9{
	public static void main(String args[]){
		String stadt = JOptionPane.showInputDialog("Bitte den Stadtnamen eingeben!");
		
		try{
			BufferedReader in = new BufferedReader(new FileReader("plz.txt"));
			Vector<String> v = new Vector<String>();
			String s = "";
			while((s = in.readLine()) != null)
				v.add(s);
			for(int i=0; i<v.size(); i++)
				if(stadt.equals(v.get(i)))
					System.out.println(v.get(i));
			in.close();
		}catch(FileNotFoundException e){
			System.out.println("Datei kann nicht gefunden werden");
		}catch (IOException e){
			e.printStackTrace();
		}
	}
}

Zudem soll eine Meldung ausgegeben werden sobald die Stadt nicht in "plz.txt" vorhanden ist und der Eingabestrom soll korrekt beendet werden.

 

[Milo]

Hi,

Du musst die einzelnen Zeilen in Deiner Datei zerlegen am Komma. Hier könnte Dir split() weiterhelfen. Diese Methode liefert Dir ein Array, bei dem das erste Element die Stadt und das zweite die Postleizahl ist.

String plz = "";
while((s = in.readLine()) != null) {
  String[] data = s.split(", "); // oder ein regulärer Ausdruck
  if (data[0].equals(stadt)) {
    plz = data[1];
    break;
  }
}
System.out.println(plz);

Gruß Micha

 

[JDownHD]

Hi,

Du musst die einzelnen Zeilen in Deiner Datei zerlegen am Komma. Hier könnte Dir split() weiterhelfen. Diese Methode liefert Dir ein Array, bei dem das erste Element die Stadt und das zweite die Postleizahl ist.

String plz = "";
while((s = in.readLine()) != null) {
  String[] data = s.split(", "); // oder ein regulärer Ausdruck
  if (data[0].equals(stadt)) {
    plz = data[1];
    break;
  }
}
System.out.println(plz);

Gruß Micha

SUPER!:toll: FETTES DANKE! So leicht hatte ich es mir nicht Vorgestellt Aber die Vectoren waren wohl der falsche Ansatz!

import java.io.*;
import java.util.*;
import javax.swing.*;

class Aufgabe9{
	public static void main(String args[]){
		String stadt = JOptionPane.showInputDialog("Bitte den Stadtnamen eingeben!");
		
		try{
			BufferedReader in = new BufferedReader(new FileReader("plz.txt"));
			String s = "";
			while((s = in.readLine()) != null){
				String[] data = s.split(", ");
			for(int i=0; i<data.length; i=+2)
				if(data[i].equals(stadt)){
					System.out.println(data[(i+1)]);
					break;
				}else
					System.out.println("Die Stadt konnte nicht gefunden werden!");
					break;
			}
			in.close();
		}catch(FileNotFoundException e){
			System.out.println("Datei kann nicht gefunden werden");
		}catch (IOException e){
			e.printStackTrace();
		}
		
		System.exit(0);
	}
}

Könnte mir einer noch sagen ob der Code so Optimal ist?
Und wofür die "IOException" gut ist

 

[Kasti]

Könnte mir einer noch sagen ob der Code so Optimal ist?
Und wofür die "IOException" gut ist

In a nutshell: Die wird geworfen, wenn ein Fehler beim Lesen aus der Datei auftritt.

 

[langhaar!]

Könnte mir einer noch sagen ob der Code so Optimal ist?

Was verstehst du unter Optimal?
Lesbarkeit? Wartbarkeit? Laufzeit?
Eine Software kann eine Vierlzahl von Qualitäten haben, die zum Teil widersprüchlich sind, und nur wenige davon (Laufzeit) können objektiv gemessen werden.

Abgesehen davon ist folgende Stelle Käse:

for(int i=0; i<data.length; i=+2)
                if(data[i].equals(stadt)){
                    System.out.println(data[(i+1)]);
                    break;
                }else
                    System.out.println("Die Stadt konnte nicht gefunden werden!");
                    break;
            }

Du machst in beiden Zeigen der Bedingung ein break. Das macht rein technisch keinen Sinn. Dann kannst du das break auch nach der Bedingung setzen. Dann brauchst du aber keine Schleife.
Inhaltlich hast du eine Stadt und eine PLZ. Du prüfst die Stadt. Wozu die Schleife? Der richtige Code wurde dir doch bereits gepostet.

 

[Spin]

Kurze Anmerkung: BREAK benutzt man nicht!!!!!!!!!!!!!!!!!!!!

Break wird lediglich bei switch statements verwendet und woanders sollten die nicht benutzt werden.
Ich bitte immer

boolean run = false;

zu benutzen. Also einen boolean, der die Schleife entsprechend beendet.

Ein Break mag in java akzeptable und jeder Java Nutzer weiss das der Break an die richtige Stelle zurückspringt, aber ....das erfordert dem Rechner viel Arbeit!

 

[Milo]

Hallo,

Kurze Anmerkung: BREAK benutzt man nicht!!!!!!!!!!!!!!!!!!!!

Ein Ausrufezeichen hätte es auch getan, um Deinen Standpunkt deutlich zu machen.

das erfordert dem Rechner viel Arbeit!

Inwiefern? Er kann beim break direkt aufhören und die Schleife verlassen. Wodurch entsteht Deiner Meinung nach die "viele Arbeit" (= Rechenzeit?).

Gruß Micha

 

[Bleiglanz]

 System.exit(0);

lange nicht gesehen, woher hast du das denn?

"The System.exit method forces termination of all threads in the Java virtual machine. This is drastic....System.exit should be reserved for a catastrophic error exit, or for cases when a program is intended for use as a utility in a command script that may depend on the program's exit code."

 

[Milo]

Hi Bleiglanz,

in der Doku klingt das weit weniger dramatisch.

Terminates the currently running Java Virtual Machine. The argument serves as a status code; by convention, a nonzero status code indicates abnormal termination.

Mit dem Argument 0 würde ich also ein reguläres Beenden der Anwendung vermuten.

Gruß Micha

 

[Timothy Truckle]

Mit dem Argument 0 würde ich also ein reguläres Beenden der Anwendung vermuten.

Das Problem damit ist, dass sich Methoden, die ein

System.exit()

einthalten nicht (einfach) testen lassen. Außerdem wird dann gar kein weitere Code des Prorgamms mehr ausgeführt, also beispielsweise werden Temp-Dateien, die mit

file.deleteOnExit()

gelöscht werden sollten, eben nicht gelöscht. Gleiches gilt für alle

finally

-Blöcke...

Also wenn schon

System.exit()

dann ausschließlich am Ende von

main

!

bye
TT

 

[Spin]

Hallo Milo,

entschuldige bitte meine etwas gewollte Aufmerksamkeit.
Ein Break Befehl gehört ist wie ein GOTO Befehl und wenn du dass runterbrichst in C oder Assembla, dann wirst du feststellen dass dass im micorsekunden langsamer ist als ein iteratives Abarbeiten von Programmiercode. Vielleicht ist dass jetzt nicht umbedingt für ein Code der 1000 Zeilen lang ist auschclaggebend und auch nicht für einen Compiler der doch alle solche Dinge versucht zu beschleunigen.

Aber zudem ist das Bad Practise bzw. auch Anti Pattern Break in for Schleifen zu benutzen. Ich habe schon viel Code gesehen der mit Breaks beschütet wurde, und nach spätestens 5 Breaks weisst du nicht mehr wo das Programm hinspringt. Klar mit einem Debugger, aber als Entwickler sollte man sich in die Pflicht nehmen, gleich gut wartbaren Code zu schreiben.

Und schau mal hier - Pipelining.
Das Problem ist für break, continue und goto Befehle in dem Bereich immer wieder Schwirigkeiten bereiten , die die CPU zum brennen bringen .

Klar auf dem Java Level hat man nichts mehr mit Maschinencode zu tun Auch nichts mehr mit Hardwareimplementierungen.

Als Einsteiger kann man sich auch gleich anschauen was man nicht in Java machen sollte
Anti-Patterns oder doch erst was man mit java so machen kann? mhhhh :bae:

Break wird an einigen Stellen verwendet und das passiert mir auch das eine oder andere mal, aber ein kleines Gefühl dafür zu bekommen es weniger zu verwenden ist schonmal ein guter Ansatz

 

[Milo]

Hi TT,

Temp-Dateien, die mit

file.deleteOnExit()

gelöscht werden
sollten, eben nicht gelöscht.

Okay, das habe ich nicht gewusst. Vielen Dank für Deinen Hinweis.

Micha

 

[Firephoenix]

Hallo Milo,

entschuldige bitte meine etwas gewollte Aufmerksamkeit.
Ein Break Befehl gehört ist wie ein GOTO Befehl und wenn du dass runterbrichst in C oder Assembla, dann wirst du feststellen dass dass im micorsekunden langsamer ist als ein iteratives Abarbeiten von Programmiercode. Vielleicht ist dass jetzt nicht umbedingt für ein Code der 1000 Zeilen lang ist auschclaggebend und auch nicht für einen Compiler der doch alle solche Dinge versucht zu beschleunigen.

Aber zudem ist das Bad Practise bzw. auch Anti Pattern Break in for Schleifen zu benutzen. Ich habe schon viel Code gesehen der mit Breaks beschütet wurde, und nach spätestens 5 Breaks weisst du nicht mehr wo das Programm hinspringt. Klar mit einem Debugger, aber als Entwickler sollte man sich in die Pflicht nehmen, gleich gut wartbaren Code zu schreiben.

...

Bei dem Antipattern stimme ich dir teilweise zu, wenn mir allerdings jemand erzählt, dass er etwas benutzt/nicht benutzt weil der Code schneller läuft kann er gleich wieder gehen

Performance sollte niemals ein Diskussionargument sein vor lesbarem Code, optimiert wird erst wenn es nötig ist (oder am Ende des Produktionszyklus), und dann nur an ausgewählten Hotspots.

Die einzige Referenz die ich zu Performanceunterschieden von break / boolean-flags gefunden habe war diese hier: Are break-statements (in Java and .Net) as bad as taught to me? | Division by Zero (die Version mit break dort doppelt so schnell wie der boolean-flag), allerdings würde ich solche Benchmarks auch nicht an die Wand pinnen, da Microbenchmarks fast nie zuverlässig sind und das Ergebnis hier vermutlich sogar von der zugrunde liegenden Hardware abhängen dürfte.

Gruß

 

[bone2]

Aber dennnoch, break ist ein jmp vorwärts nach die schleife. boolean ist, variable setzen, jmp zum schleifenkopf, variable prüfen, jmp hinter schleife.
obwohl ich meine, das java aus while-do intern do-while macht, was dann nur boolean vergleich wäre und ende

gleich mal wieder raussuchen

edit: break = goto, boolean = 1 vergleich
http://www.java-forum.org/java-basics-anfaenger-themen/144802-break-sprungmarken-2.html#post967271

 

[JDownHD]

@langhaar!
also unter Optimal verstehe ich momentan eigentlich nur ob der Code schlüssig ist, oder ob so Sachen wie das break; weg gelassen werden können. Oder ob das Angesprochene System.exit();.
Ich bin Anfänger und lege noch keinen Wert auf spezifische Laufzeitauswirkungen von verschiedenen Programmiermöglichkeiten. Mich interessiert nur, ob der Code der Norm entspricht und ob man von der Lesbarkeit bzw. der Syntax Sachen verbessern kann oder nicht (z.B. die von dir Angesprochene for-Schleife).

@Spin
Ich habe nicht verstanden was du mit

boolean run = false;

meinst und wie ich es einsetzen soll.

@Bleiglanz

 System.exit(0);

habe ich aus unserem Skript
Also soll ich es nicht mehr verwenden?

Edit:
Ich habe den Code jetzt abgeändert:

import java.io.*;
import java.util.*;
import javax.swing.*;

class Aufgabe9{
	public static void main(String args[]){
		String stadt = JOptionPane.showInputDialog("Bitte den Stadtnamen eingeben!");
		
		try{
			BufferedReader in = new BufferedReader(new FileReader("plz.txt"));
			String s = "";
			boolean z = false;
			while((s = in.readLine()) != null){
				String[] data = s.split(", ");
				if(data[0].equals(stadt)){
					System.out.println(data[(1)]);
					z = true;
				}
			}
			if(z==false)
				System.out.println("Die Stadt konnte nicht gefunden werden!");

			in.close();
		}catch(FileNotFoundException e){
			System.out.println("Datei kann nicht gefunden werden");
		}catch (IOException e){
			System.out.println("Es ist ein Fehler beim lesen der Datei aufgetreten");
		}
	}
}

Ich weiß jetzt halt nur nicht ob ich den Eingabestrom korrekt beendet habe, also das von alleine passiert oder ich noch Code einfügen muss.

 

[bone2]

mit boolean statt break meint er so etwas:

        int i = 1;
        while(1 < 10){
            i++;
            
            if (i == 5) {
                break; // hässich
            }
        }

        boolean run = true;
        i=1;
        while(1 < 10 && run){
            i++;
            
            if (i == 5) {
                run = false; // schön
            }
        }

 

[JDownHD]

Also bei mir eher schwer zu Implementieren da:

String s = "";
			boolean z = false;
			boolean run = true;
			while((s = in.readLine()) != null && run){
				String[] data = s.split(", ");
				if(data[0].equals(stadt)){
					System.out.println(data[(1)]);
					z = true;
					run = false;
				}
			}

ich ja mehrere Werte will.
In diesem Fall würde er ja nach einem Treffer aufhören.