BufferedReader und String-Array

[The_S]

Diese Zeile Code funktioniert nicht bzw. bleibt die Zeile leer

line[zeilenUser] = UserNamesB.readLine();
System.out.println(line[zeilenUser])

wenn ich dagegen

System.out.println(UserNamesB.readLine());

mache, bekomme ich den richtigen Wert!

Das ganze Steht in einer while-Schleife (Ja ich erhöhe den Wert zeilenUser :wink: ). Woran kann das liegen?

[edit] Lässt sich ganz normal Complimieren und Exception bekomme ich auch keine ???:L

[edit2] Falls es sich jemand nicht denken kann :bae: :line ist ein StringArray, UserNamesB ist ein BufferedReader

 

[meez]

Zeig mal die ganze Funktion...

 

[The_S]

Bin grad dabei ein Programm zu schreiben, für das ich alle Benutzernamen brauche. Soweit bin ich bis jetzt (funzt nur unter XP):

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

public class Zugriff extends JFrame implements ActionListener, ItemListener {
	
	int zeilenUser = 0;
	boolean schonDa = false;
	boolean nochNamen = false;
	String Verzeichnis = System.getProperty("java.io.tmpdir");
	String[] line = new String[100];
	File pfad = new File(Verzeichnis + "\\usernames.txt");
	JPanel pane = new JPanel();
	JComboBox Benutzernamen = new JComboBox();
	
	public Zugriff() {
		
		super("Zugriffsverwaltung");
		setSize(100, 100);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		Benutzernamen.addItemListener(this);
		pane.add(Benutzernamen);
		setContentPane(pane);
	}
	
	void writeUsersTxt() {
		
		JOptionPane.showMessageDialog(null,
		"Erstelle Benutzerdatei, OK um den Vorgang zu beginnen",
		"INFO01",
		JOptionPane.INFORMATION_MESSAGE);
		try {
			Runtime.getRuntime().exec("cmd /c start wmic /OUTPUT:\"" + Verzeichnis + "\\usernames.txt\" useraccount get caption");
		}
		catch (Exception e) {
			JOptionPane.showMessageDialog(null,
			"Konnte die Benutzerdatei nicht speichern",
			"ERROR01",
			JOptionPane.ERROR_MESSAGE);
		}
	}
	
	void usernamesDa() {
		
		JOptionPane.showMessageDialog(null,
		"Lese Benutzerdatei zu Testzwecken, OK um den Vorgang zu beginnen",
		"INFO02",
		JOptionPane.INFORMATION_MESSAGE);
		while (schonDa == false) {
			try {
				if (pfad.exists() == true) {
					FileReader schonDaTest = new FileReader(pfad);
					BufferedReader schonDaTestB = new BufferedReader(schonDaTest);
					if (schonDaTestB.ready()) {
						schonDa = true;
					}
				}
			}
			catch (Exception e) {
				JOptionPane.showMessageDialog(null,
				"Fehler beim überprüfen der Benutzerdaten",
				"ERROR02",
				JOptionPane.ERROR_MESSAGE);
			}
		}
	}
	
	void writeUsersCombo() {
		
		JOptionPane.showMessageDialog(null,
		"Lese Informationen aus der Benutzerdatei, OK um den Vorgang zu beginnen",
		"INFO03",
		JOptionPane.INFORMATION_MESSAGE);
		try {
			FileReader UserNames = new FileReader(pfad);
			BufferedReader UserNamesB = new BufferedReader(UserNames);
			while (true) {
				line[zeilenUser] = UserNamesB.readLine();
				zeilenUser++;
			}
		}
		catch (Exception e) {
		}
		for (int i = 2; nochNamen == true; i = i + 2) {
			if (line[i].equals("")) {
				nochNamen = false;
			}
			else {
				Benutzernamen.addItem(line[i]);
			}
		}
	}
	
	public static void main(String[] args) {
		
		Zugriff started = new Zugriff();
		started.writeUsersTxt();
		started.usernamesDa();
		started.writeUsersCombo();
		started.setVisible(true);
	}
	
	public void actionPerformed(ActionEvent evt1) {
	}
	
	public void itemStateChanged(ItemEvent evt2) {
	}
}

Der Fehler liegt in writeUsersCombo. Komischerweise funktioniert das hier aber:

import java.io.*;

public class zugriff { 
    
   public static void main(String[] args) { 
       
       	String Verzeichnis = System.getProperty("java.io.tmpdir");

      try { 
         Runtime.getRuntime().exec("cmd /c start wmic /OUTPUT:\"" + Verzeichnis + "\\usernames.txt\" useraccount get caption"); 
      } 
      catch (Exception e) { 
      } 
      boolean j = false;
      File pfad = new File(Verzeichnis + "\\usernames.txt");
      while (j == false) {
      	try {
      		if (pfad.exists() == false) {
      		}
      		else {
      			FileReader test = new FileReader(pfad);
				BufferedReader testR = new BufferedReader(test);
				if (testR.ready()) {
      				j = true;
      			}
      		}
      	}
      	catch (Exception e) {
      		System.out.println("j");
      	}
      }
      int i = 0; 
      String[] line = new String[100]; 
      boolean nochNamen = true; 
      try { 
         FileReader name = new FileReader(pfad); 
         BufferedReader nameR = new BufferedReader(name); 
         while (true) { 
            line[i] = nameR.readLine(); 
            i++; 
         } 
      } 
      catch (Exception e) { 
      } 
      for (int z = 2; nochNamen == true; z = z + 2) { 
         if (line[z].equals("")) { 
            nochNamen = false; 
         } 
         else { 
            System.out.println(line[z]); 
         } 
      } 
   } 
}

@ KSG9|plak ich beachte deine Verbesserungsvorschläge schon noch, möchte aber es erstmal so zum laufen bekommen :wink:

[edit] Ich weiß, dass man das schöner machen könnte, das kommt dann später :bae:

 

[foobar]

So sollte es gehen:

String buffer = null;
while ( (buffer = UserNamesB.readLine()) != null )
{
   line[zeilenUser] = buffer;
   System.out.println(line[zeilenUser])
}

 

[meez]

Sollte auch schon worher gehen...(Er arbeitet mit der Exception)
Aber da stellen sich mir allgemein die Nackenhaare auf...
Warum so kompliziert?
Machs so, und lass den "vorinitalisierten" Array weg:

 void writeUsersCombo() {
       
    try {
       JOptionPane.showMessageDialog(null,
       "Lese Informationen aus der Benutzerdatei, OK um den Vorgang zu beginnen",
       "INFO03",
       JOptionPane.INFORMATION_MESSAGE)
       BufferedReader UserNamesB = new BufferedReader(new FileInputStream(new File(pfad)));
       for (String buffer;(buffer = UserNamesB.readLine()) !=null;Benutzernamen.addItem(buffer));
       UserNamesB .close();
     } catch (IOException e) {
        e.printStackTrace();
     }
}

 

[The_S]

Sollte auch schon worher gehen

Tuts aber leider nicht und ich hab keine Ahnung warum :cry:

Aber da stellen sich mir allgemein die Nackenhaare auf...

Wie gesagt, wenn es erstmal funktioniert kümmere ich mich um die Kosmetik! :bae:

So muss jetzt erstma das von foobar austesten ...

 

[The_S]

Das von foobar funktioniert auch nicht

 

[The_S]

Sehr komisch, hab meinen alten, funktionierenten Code so abgeändert, dass er genau so ausschaut wie mein Alter und siehe Da es funktioniert! Warum weiß ich nicht, aber es funktioniert!