Datei nicht erstellen

[selim]

Hallo,

irgendwie ich stehe ich auf dem schlauch....

Ich will die Datei nur erstellen wenn ich auch Positionen habe, wie kann ich das machen?

try {
f1 = new FileWriter(DateiName);
for (int i = 0; i < list.size(); i++) 
{
if (i == 0) {
f1.write(Header + "\r\n");
}
f1.write("ADR;");
f1.write(list.get(i) + "\r\n");
								
f1.close();
}
}

Danke schon mal!

 

[truesoul]

Was heißt Position?
Wenn du Admin bist oder was?

If(isAdmin){
erstelle File
} else {
du darfst das nicht.
}

Erkläre mal bitte was du mit Position meinst

 

[selim]

Sorry, hätte etwas mehr details dazu geben sollen.

Ich erstelle jeden Tag paar listen, bei manchen abfragen bekomme ich keinen Wert zurück (positionen), diese Listen will ich nicht erzeugen, da ich nur einen kopfsatz (konstane) erzeuge pro Liste, also würde ich müll erzeugen und das nicht mein ziel.

 

[Final_Striker]

Ich will die Datei nur erstellen wenn ich auch Positionen habe, wie kann ich das machen?

In dem du das vorher mit einer if-Bedingung überprüfst.

 

[truesoul]

If Bedingung?!

 

[selim]

Hat leider nicht funktioniert mit der If anweisung.
Kann ich irgendwie das erzeugen abbrechen?

 

[Gast2]

f1 = new FileWriter(DateiName);

Durch die Zeile wird schon die Datei erstellt. Du musst also wie gesagt vorher schon prüfen ob du überhaupt was schreiben willst.

 

[selim]

Wie müsste denn die If anweisung aussehen?

ResultSet rSet = stmt.executeQuery(sqlQuery);
			while (rSet.next())

				list.add(rSet.getString("COM") + ";"
						+ rSet.getString("BL") + ";;"
						+ rSet.getString("Name") + ";"
						+ rSet.getString("Strasse") + ";"
						+ rSet.getString("PLZ") + ";" + rSet.getString("Ort")
						+ ";;" + rSet.getString("Land")
						+ ";;;;;;;;;;;;;;;;;;;;;" +
						rSet.getString("DEP") +
						";;;;;;;"
						+ "XX");
			stmt.close();
			conn.close();

			try {
				f1 = new FileWriter(DateiName);
				for (int i = 0; i < list.size(); i++) 
					{
					
					
					if (i == 0) {
						f1.write(Header + "\r\n");
					}
					f1.write("ADR;");
					f1.write(list.get(i) + "\r\n");
								
				f1.close();
					}
				
			}
			
			catch (IOException e) 
			{
				System.out.println("Fehler beim Erstellen der Datei");
			}

			}
		catch (SQLException e) 
		{
			System.out.println("Fehler bei Tabellenabfrage" + e);
			return;
		}

	}

 

[selim]

Wie kann ich denn prüfen ob ich einen Satz per query gefunden habe?

f1 = new FileWriter(DateiName);

Durch die Zeile wird schon die Datei erstellt. Du musst also wie gesagt vorher schon prüfen ob du überhaupt was schreiben willst.

 

[Gast2]

Wie wärs denn mit

list.size() > 0

?

 

[selim]

try {
				
				if (list.size() > 0 ) 
				{
				f1 = new FileWriter(DateiName);
				}
				for (int i = 0; i < list.size(); i++) {

					if (i == 0) {
						f1.write(Header + "\r\n");
					}
					f1.write("ADR;");
					f1.write(list.get(i) + "\r\n");

					f1.close();
				}

Wenn ich es so mache brechen mir die anderen nach dem ersten durchlauf ab. wieso?

 

[Gast2]

Ja so passts natürlich nicht. Spiel mal durch was passiert wenn

list.size() == 0

ist. Da bekommst du ne NullPointerException.

 

[selim]

Verstehe ich leider gerade nicht mehr, stehe auf dem schlauch.

 

[Gast2]

Ne vergiss es, mein Hinweis war nicht der richtige.

Was steht in list? Was passiert genau? Was sollte stattdessen passieren?

 

[tagedieb]

try {
				
				if (list.size() > 0 ) 
				{
				f1 = new FileWriter(DateiName);
				}
				for (int i = 0; i < list.size(); i++) {

					if (i == 0) {
						f1.write(Header + "\r\n");
					}
					f1.write("ADR;");
					f1.write(list.get(i) + "\r\n");

					f1.close();
				}

Wenn ich es so mache brechen mir die anderen nach dem ersten durchlauf ab. wieso?

Weil du das File nach der ersten Position mit

f1.close()

schliesst. Das hat bestimmt schon vorher nicht funktioniert! Du solltest mal deinen Code ueberdenken und die Bedingungen, das Oeffnen und Schliessen der Datei sowie das schreiben der Positionen an die richtige stelle setzen. Mit scheint es doch etwas kreuz und quer. Etwas pseudo code koennte dir helfen die Uebersicht zu behalten:

if list > 0
  open file
  write header
  for 0 to list.size
     write position
  end for
  close file
end if

Daraus ergibt sich folgender Java Code:

if (list.size() > 0 ) {
  try {
    f1 = new FileWriter(DateiName);
    f1.write(Header + "\r\n");
    for (int i = 0; i < list.size(); i++) {
      f1.write("ADR;");
      f1.write(list.get(i) + "\r\n");
    }
  } finally {
    if (f1 != null) {
      f1.close();
    }
  }
}

 

[selim]

DANKE, auf finally bin ich nicht gekommen mit close!!!!

Weil du das File nach der ersten Position mit

f1.close()

schliesst. Das hat bestimmt schon vorher nicht funktioniert! Du solltest mal deinen Code ueberdenken und die Bedingungen, das Oeffnen und Schliessen der Datei sowie das schreiben der Positionen an die richtige stelle setzen. Mit scheint es doch etwas kreuz und quer. Etwas pseudo code koennte dir helfen die Uebersicht zu behalten:

if list > 0
  open file
  write header
  for 0 to list.size
     write position
  end for
  close file
end if

Daraus ergibt sich folgender Java Code:

if (list.size() > 0 ) {
  try {
    f1 = new FileWriter(DateiName);
    f1.write(Header + "\r\n");
    for (int i = 0; i < list.size(); i++) {
      f1.write("ADR;");
      f1.write(list.get(i) + "\r\n");
    }
  } finally {
    if (f1 != null) {
      f1.close();
    }
  }
}

 

[tagedieb]

Resources, Stream und Files sollten immer geschlossen werden.

Siehe best practices Java Practices -> Finally and catch