Ascii Bild aus einem input file nur kommt nix im outputfile an??????

[pisco]

Hey Leute
Meine Aufgabe ist es ein Ascii Bild einzulesen und diesen dann in einem Outputfile zu speichern. Nur es kommt bei mir nix im outputfile an...

Also das wär das Ascii Bild amal:

In

read 18
.....X....X......................X....X.....
....XXX..XXX....................XXX..XXX....
....XXX..XXX....................XXX..XXX....
....XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX....
....XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX....
..XXXXXXXXXXXX................XXXXXXXXXXXX..
.XXXXXXXXXXXXXX..............XXXXXXXXXXXXXX.
.XXXXXXXXXXXXXX..............XXXXXXXXXXXXXX.
.XXXXXXXXXXXXXX..............XXXXXXXXXXXXXX.

Out

.....X....X......................X....X.....
....XXX..XXX....................XXX..XXX....
....XXX..XXX....................XXX..XXX....
....XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX....
....XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXX....................XXXXXXXX....
....XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX....
..XXXXXXXXXXXX................XXXXXXXXXXXX..
.XXXXXXXXXXXXXX..............XXXXXXXXXXXXXX.
.XXXXXXXXXXXXXX..............XXXXXXXXXXXXXX.
.XXXXXXXXXXXXXX..............XXXXXXXXXXXXXX.
44 18

und hier ist mein geschriebener Source:

import java.lang.*;
import java.util.*;
//importieren der Bibliotheken

public class AsciiShop3{//so heißt meine Klasse
	public static void main (String[] args){//das muss immer da stehen
		
		int laengebild = 0; 
		int anzahllines = 1;
		int output1=0, output2=0;
		boolean mistake = false;
		
		Scanner sc = new Scanner(System.in);
			
		String read=sc.next();
		if (read.equals("read")) { 
			
			if (sc.hasNextInt()) {
				laengebild = sc.nextInt();//hier lese ich die Länge des Bildes aus
				String firstline=sc.next();
				
				StringBuffer bild = new StringBuffer(firstline);
	//			StringBuffer fertig = new StringBuffer(zusammen);
				
				while (sc.hasNext() && mistake) {
					String zeile= sc.next();
					
					if (zeile.equals("decode")) {
						mistake=true;
					}
					/*WICHTIG*/
					//Hier kommen meine Überprüfungsroutinen für die Befehle rein/
					
										
					else {
						String afterline= zeile;//hier werden die reslichen Zeilen ausgelesen
						
						anzahllines++;//die Anzahl der Zeilen erhöht sich immer um 1ns bei jedem durchlauf 
						bild.append(afterline);//und hier werdens angehängt
						
						//Routine um das Bild auf seine Längenangaben zu verifizieren
						if (firstline.length() != afterline.length()) {
							System.out.println("INPUT MISMATCH");
							mistake =true;
						}
						//-----------------------------------------------------------
						if (sc.hasNext()==false){
							
							if (anzahllines!=laengebild) {
								System.out.println("INPUT MISMATCH");
								mistake = true;
							}
							
							else {
		
								while (output1<=bild.length()-firstline.length()) {
									
									String unverschluesselt=bild.toString();
										
										//uneverschlüsselt ist dann das bild das wieder in einen String umgewandelt wurde und dann
										//ausgegeben wird
										System.out.println(unverschluesselt);
										
										output1=output1+firstline.length();
									}
								System.out.println (firstline.length() + " " + anzahllines);
								mistake = true;
							
							}
						}
					}
				}
			}
		}
	
	
	
	
	/*Ende*/			
	}
}

Also compilierungsfehler hab ich keinen.

Ich würde mich freuen wenn ihr mir helfen könntet...

Natürlich freu ich mich über jeden Comment der mich weiterbringt und mir hilft meine Coding Skills noch weiter zu verbessern

greetz und vielen dank euch schon mal im voraus

pisco

 

[Developer_X]

Erstmal konkret:
Du willst ein ASCII-Bild (also den Text oben) in die Console eingeben, und das Programm soll das ganze speichern, richtig?

1.Frage : Zeilen Weise eintippen, oder gleich rein?
2.Frage : War das allesß

 

[pisco]

hey erstmal danke für deine Antwort....

nein nicht ganz Developer_X. Ich hab ein inputfile.txt und will das ganze in einem outputfile dann ausgeben

hab das beim compilieren immer so gemacht:

java MyProgramm <input.txt> output.txt //ich glaub so wars

greetz

 

[cr33p]

Dann mach das doch mit dem BufferedReader/Writer.

BufferedReader reader = new BufferedReader(new FileReader(new File(pfadZurDatei)));
BufferedWriter writer = new BufferedWriter(new FileWriter(new File(pfadZurDatei)));

while(reader.ready())
{
irgendeinString += reader.readLine() + "\n";
}

Jetzt hast du in der Variable "irgendeinString" das komplette ASCII Bild Zeile für Zeile drin.
Falls du es jetzt noch auf eine Datei schreiben willst, einfach writer.write(irgendeinString);

 

[Wortraum]

Beeindruckende Einrückungstiefe. Dazu ein kleiner Tip: Sie ist ein Indiz für schlechten Quelltext; die Aufteilung in mehrere Methoden kann bereits Abhilfe schaffen. In Deinem Fall kannst Du die while-Schleife auch überall dort mit break verlassen, wo Du stattdessen die Abbruchvariable mistake setzt, so daß Du Dir alle else-Zweige sparen kannst – das macht es zwar noch nicht schön, aber wenigstens ein bißchen geschminkt und gekämmt.

Außerdem solltest Du Dich an Java-Konventionen halten und Variablennamen entsprechend vergeben (laengeBild, anzahlLines). Persönlich ganz grausam empfinde ich die Vermischung von englischen und deutschen Namen (laengebild, zeile, aber afterline und mistake), und noch ein Quentchen grausamer ist die Vermischung innerhalb eines Namens (anzahllines).

Und dann noch ein dritter Hinweis: etwas weniger Leerzeilen täten der Lesbarkeit gut.

In Zeile 11 setzt Du mistake auf false, somit geht er erst gar nicht in die while-Schleife in Zeile 25. Ob das der einzige Fehler ist, weiß ich nicht, aber er fiel mir als erstes auf.

 

[Developer_X]

Schau dir mal das an, ist ein kleines Programm von mir, hilft dir bestimmt weiter:

package codes;

import java.io.*;
import java.util.*;

/**
 * 
 * @author Kevin Riehl
 *
 */

public class ASCII_Pic_Reader 
{
//	Attribute
	Scanner scanner;
	String[] image = {""}; // unser image
	String read;
	String data_type = ".img"; // Datentyp kannst du .img nennen wenn du willst.
	String directory = "images"; // Ordner in den die Dateien >>abgelegt<< werden.
	
//	Constructor
	public ASCII_Pic_Reader()
	{
		// 0. Initialisiere den Scanner.
		scanner = new Scanner(System.in);
		
		while(true)
		{
			System.out.println("Was möchten sie tun?");
			System.out.println("create - Erstellen Sie ein Bild.");
			System.out.println("read - Lesen Sie ein Bild.");
			
			String read = scanner.next();
			
			if(read.equals("create"))
				create(scanner);
			else if(read.equals("read"))
				read(scanner);
			
			image = new String[1];
			image[0] = "";
		}
	}

// 	 Methoden
	
	/**
	 * Mit dieser Methode kann man Bilder erstellen.
	 */
	public void create(Scanner sc)
	{
		// 1. Fordere das Bild an.
		System.out.println("Bitte geben Sie den Bild Code (ASCII) ein : ");
		System.out.println("(Info: Wenn Sie fertig sind, geben Sie einfach >>save<< ein,");
		System.out.println("und drücken ENTER.)");
		System.out.println("");
		
		// 2. Lese das Bild ein.
		for(;;) // Unendlich lange, bis man speichern will, 
				// sollte man das Bild eintippen.
		{
			read = scanner.next(); // Eingetippter Text wird eingelesen
			
			if(read.equals("save")) // Falls der Text >>save<< entspricht,
									   // Soll die Schleife beendet werden, 
									   // und es soll gespeichert werden.
			{
				save(scanner);
				break;
			}
			else // Andernfalls wird die Zeile einfach zum Array hinzugefügt.
				addZile(read);
			
			read = ""; // read wird wieder geleert, für den neuen Durchlauf der Schleife.
		}
	}
	
	/**
	 * Mit dieser Methode kann man Bilder laden.
	 */
	public void read(Scanner sc)
	{
		// Abfrage des Names der Datei
		File f = new File(directory+"/");
		System.out.println("Folgende Dateien vorhanden. " + // \n ist für Zeilenumbrüche
				"\n Geben Sie die Nummer der Datei an,\n die sie gerne lesen möchten.");
		
		String[] files = f.list();
		for(int i = 0;i<files.length;i++)
			System.out.println(i+". "+files[i]);
		
		// Frage nummer ab
		int number = scanner.nextInt();
		
		// Lese Datei
		File img_file = new File(directory+"/"+files[number]);
		
		try 
		{
			BufferedReader br = new BufferedReader(new FileReader(img_file));
			int zeilen = 0;
			
			for(;br.readLine()!=null;zeilen++)
				;
			br.close();
			
			image = new String[zeilen];
			br = new BufferedReader(new FileReader(img_file));
			for(int i = 0;i<image.length;i++)
				image[i] = br.readLine();
		} 
		catch (Exception e) 
		{
			e.printStackTrace();
		}
		
		// Gebe Bild aus
		System.out.println("");
		System.out.println("Bild : "+files[number]);
		System.out.println("");
		
		for(int i = 0;i<image.length;i++)
			System.out.println(image[i]);
	}
	
	/**
	 * Fügt die neue eingegebene Zeile zum Array hinzu.
	 * @param s Neue Zeile
	 */
	public void addZile(String s)
	{
		String[] old_string_array = image;
		image = new String[old_string_array.length+1]; // Eine Zeile mehr
		
		for(int i = 0;i<old_string_array.length;i++) // Fügt alle alten Zeilen hinzu
			image[i] = old_string_array[i];
		
		image[old_string_array.length] = s; // Fügt neuestes hinzu
	}
	
	/**
	 * Speichert das Bild.
	 */
	public void save(Scanner scanner)
	{
		// Fordere Namen der Datei an.
		System.out.println("Bitte geben Sie den Namen des Bildes an.");
		
		// Lese den Namen ein.
		String name = scanner.next();
		
		// Erstelle die Datei.
		File img_file = new File(directory+"/"+name+data_type);
		
		File directo = new File(this.directory);
		if(directo.exists()==false) // Falls der Ordner wo die Bilder gespeichert sind,
									// Noch gar nicht existiert, soll er erst erstellt
									// werden.
			directo.mkdir();
		try 
		{
			boolean created = img_file.createNewFile(); // Datei wird erstellt
			// Bescheid geben
			if(created)
			{
				System.out.println("Datei wurde erfolgreich erstellt.");
				PrintWriter pw = new PrintWriter(new FileWriter(img_file)); // Erstelle
									// Printwriter, mit dem man in Dateien schreiben kann.
				for(int i = 0;i<image.length;i++)
					pw.println(image[i]); // Schreibe in die Datei
				pw.close(); // Schließe die Verbindung zur Datei, damit andere Programme
							// Darauf zurückgreifen können.
			}
			else
				System.out.println("Datei konnte nicht erstellt werden.");
		} 
		catch (Exception e) 
		{
			e.printStackTrace();
		}
		
		// Letzter Bescheid
		System.out.println("Bild Daten wurde erfolgreich gespeichert.");
		
		System.out.println("-------------------------");
	}
	
//	Main
	public static void main(String[]args)
	{
		new ASCII_Pic_Reader();
	}
}

Ich hoffe es hat dir geholfen?
Wenn ja, gern geschehen,
Developer_X