Frage zum BufferedWriter

[Guest]

Hallo,

ich arbeite im Moment viel mit Textdateien und habe unter anderem eine Funktion, die ein Textfile einliest, bearbeitet und in eine Zieldatei schreibt. Dazu nehme ich einen BufferedWriter:

BufferedWriter output = new BufferedWriter(new FileWriter(new File("C:/test.txt")));

output.write("blabla");
output.close();

Das Ganze funktioniert soweit schon ganz gut. Jetzt habe ich aber folgendes Problem. Es wird nur etwas in das Zielfile geschrieben, wenn es genug ist. Also wenn es ein langer Text ist, funktioniert es. Bei ein paar Zeilen funktioniert es nicht.

Muss man dem BufferedWriter vor dem Schließen irgendwie noch mitteilen, dass er den kompletten Inhalt nochmal in das File schreiben soll?

 

[The_S]

Da gibt es den Aufruf

bw.flush()

der den Inhalt in die Datei schreibt. Aber das ist unnormal, dass es nicht geht. Weil normalerweiße ruft close() auch ein flush() auf. Sollte also keinen Unterschied machen wenn du einfach nur den Stream closed ???:L . Hast du evtl. noch irgendwo anderen Code oder machst sonst noch was, was den BW stören könnte?

 

[thE_29]

Nur so aus Frage!

Windows und Java erlauben das hier

C:/test.txt

Aber schreibt doch bitte C:\\test.txt hin und net die Linux Windows mischvariante...

 

[L-ectron-X]

Du kannst vor dem Schließen des Streams den Stream leeren.

output.flush();

 

[thE_29]

Das hat hobbit vor 6 min auch schon geschrieben 🙂

 

[L-ectron-X]

Sorry, kam Telefonat dazwischen...

 

[Guest]

Das zusätzliche Flush hat leider nichts bewirkt. Ich nehme auch eher an, dass ich da etwas durcheinander gebracht habe. Ich lese aus der ersten Datei einige Werte aus und bearbeite anhand dieser Werte eine weitere Datei. Ich kann nicht verlangen, dass sich das hier jemand komplett anschaut, wäre über Hilfe jedoch dankbar.

	public static void main(String[] args) 
	{
		System.out.println("--- Start splitting --- " + (new Date()).toString());
		
		SimpleDateFormat df = new SimpleDateFormat("ddMMyyyy");
		int errors = 0;
		String rootDirectory = "d:/zzz/";
		
		
		try
		{	
			// Get partin filename
			String entries1[] = new File(rootDirectory + "partin/").list();
			String partinFilename = rootDirectory + "partin/" + entries1[0];
			
			// Get pricat filename
			String entries2[] = new File(rootDirectory + "pricat/").list();
			String pricatFilename = rootDirectory + "pricat/" + entries2[0];
			
			
			// Variables
			String outputFolder = rootDirectory + "output/";
			
			File partinInputFile          = new File(partinFilename);
			File pricatInputFile		  = new File(pricatFilename);
			
			FileReader partinInputStream  = new FileReader(partinInputFile);
			BufferedReader partin	      = new BufferedReader(partinInputStream);
			
			BufferedWriter pricatOutput	  = null;		
			
			String partinRow = "";
			String pricatRow = "";
			String header = "";
			 
			String partinBpCd = "";
			String partinOdsCd = "";
			String partinCustNb = "";
			
			String pricatBpCd = "";
			String pricatOdsCd = "";
			String pricatCustNb = "";
			
			boolean headerSet = false;
			
			// Process pricat file for every partin entry
			while ((partinRow = partin.readLine()) != null) 
			{
				if (!partinRow.substring(0, 5).trim().equals("."))
				{
					partinBpCd = partinRow.substring(0, 5).trim();
					partinOdsCd = partinRow.substring(5, 10).trim();
					partinCustNb = partinRow.substring(60, 75).trim();
					
					System.out.println("---");
					System.out.println("BpCd:   " + partinBpCd);
					System.out.println("OdsCd:  " + partinOdsCd);
					System.out.println("CustNb: " + partinCustNb);
					System.out.println("---");

					FileReader pricatInputStream = new FileReader(pricatInputFile);
					BufferedReader pricat = new BufferedReader(pricatInputStream);
					int rowcount = 0;
					
					while ((pricatRow = pricat.readLine()) != null) 
					{
						// check rowtype
						if (pricatRow.substring(40, 46).equals("LINHDR"))
						{
							header = pricatRow;
							headerSet = false;
						}
						else if (pricatRow.substring(40, 46).equals("LINMDC"))
						{
							pricatBpCd = pricatRow.substring(0, 5).trim();
							pricatOdsCd = pricatRow.substring(5, 10).trim();
							pricatCustNb = pricatRow.substring(95, 110).trim();

							
							// Check if partin & pricat data is equal
							if ((pricatBpCd.equals(partinBpCd))
								&& (pricatOdsCd.equals(partinOdsCd))
								&& (pricatCustNb.equals(partinCustNb)))
							{
								String outputFilename = outputFolder 
													  + pricatBpCd 
													  + pricatOdsCd 
													  + pricatCustNb 
													  + df.format(new Date());
								
								// Delete old output file and create a new one
								if (rowcount == 0)
								{
									if (new File(outputFilename).exists())
										new File(outputFilename).delete();
									
									File outputFile = new File(outputFilename);
									pricatOutput = new BufferedWriter(new FileWriter(outputFile));
								}

								
								// Write header
								if (!headerSet)
								{
									pricatOutput.write(header + "\n");
									headerSet = true;
								}
								
								// Write line
								pricatOutput.write(pricatRow + "\n");
								rowcount++;
							}
							
						}
						else if (!pricatRow.substring(40, 46).equals("MSGTRL"))
						{
							errors++;
						}
					}
					
					// Close pricat input stream					
					pricat.close();
					
				}
			}
			
			// Close partin input stream
			partin.close();
			
			// Close pricat output stream
			pricatOutput.close();	
			
		}
		catch (IOException e)
		{
			System.out.println(e);
		}
		

		System.out.println("--- End splitting --- " + (new Date()).toString());
		System.out.println("Errors: " + errors);
	}

 

[Leroy42]

normalerweiße ruft close() auch ein flush() auf. Sollte also keinen Unterschied machen

Genau aus dem Grund kann ich mir nur vorstellen, daß das close() aus irgendeinem Grund nicht erreicht
wird.

Darum schreibe anstatt:

		try {
			BufferedWriter bw = new BufferedWriter(new FileWriter("erna42"));
			bw.write("Krimskrams");
			//...
			bw.close();
		} catch (IOException e) {
			System.err.println(e);
		}

lieber

		BufferedWriter bw = null;
		try {
			bw = new BufferedWriter(new FileWriter("erna42"));
			bw.write("Krimskrams");
			//...
		} catch (IOException e) {
			System.err.println(e);
		} finally {
			if (bw != null)
				try {bw.close();} catch (IOException e1) {}
		}

Ist zwar etwas nervig, aber sauberer.

 

[Guest]

Es kann so einfach sein 🙂 Das close wird nur einmal erreicht, da ich mit der Klammerung der Schleifen durcheinander gekommen bin.

Danke für die schnelle Hilfe.