Frage zu public static

[seux]

Hallo,
ich habe eine Klasse mit unter anderem einer Hilfsvariable, die ich als public static deklariert hab. Dieser Variablen wird auch ein Wert zugewiesen. In einer der Methoden dieser Klasse erzeuge ich eine Instanz von einer zweiten Klasse und rufe eine ihrer Methoden auf. Diese Methode nutzt ebenfalls meine Hilfsvariable, allerdings hat sie keinen Wert mehr. Sobald ich die Methode wieder verlassen hab, ist mein alter Wert wieder da.

woran liegt das?

gruß seux

 

[njans]

Magst du vielleicht ein bisschen Code reinstellen, so dass ich/wir besser nachvollziehen können, was du meinst?

 

[seux]

public class LZW
{
	
	//...
	
	public static String g_fileExtension = new String();
	
	public void compress(File file)
	{
             //Hier ist g_fileExtension noch verfügbar
             FileHeader fh = new FileHeader(comFile);
	         fh.writeHeader();
             //Hier auch wieder
    }

public class FileHeader {
        //...
        public void writeHeader()
	    {
             //Hier ist g_fileExtension nicht verfügbar
		     if(LZW.g_fileExtension == null);
		     {
			    System.out.println("Something unusual happend. No file Extension...");
		     }
                     //...
         }
}

 

[njans]

Was heißt für dich " Hier ist g_fileExtension nicht verfügbar" ?
Mit new String() machst du nicht anderes als String s = "";

Was passiert da und was würdest du erwarten, dass passiert?

 

[Schandro]

hast du die Klasse LZW nicht importiert oder was? Bitte eine sinnvolle Fehlermeldung/Compilererror/Stack Trace posten...

btw.: static ist "böse", versuch es zu vermeiden.

 

[Landei]

Schreibe mal [c]public static final String g_fileExtension = "";[/c]. Entweder beschwert sich dann der Compiler, dass die Variable irgendwo geändert wird (also eventuell auf null gesetzt wird), oder es geht wirklich etwas "seltsames" vor (dass z.B. aus irgendwelchen Gründen nicht alle Klassen neu compiliert wurden oder so).

 

[seux]

Was heißt für dich " Hier ist g_fileExtension nicht verfügbar" ?

Ich bekomm meine Fehlermeldung "Something unusual happend. No file Extension..." ausgegeben.

Ich habe die Variable mal im Debugger verfolgt, und beim eintreten in die Funktion writeHeader ändert sich ihr Wert auf <error(s)_during_the_evaluation>

Danach ist wieder der alte Wert vorhanden. Es wird auch nur zweimal auf die Variable zugegriffen. Einmal, wenn ich ihr einen Wert zuweise (klappt auch), und einmal in der obigen Methode

Ich habe es auch wie Landei beschrieben hat probiert, und die Zeile, in der ich g_fileExtension einen Wert zuweise auskommentiert. Es kommt trotzdem die Fehlermeldung

 

[seux]

hast du die Klasse LZW nicht importiert oder was?

Müsste mir Eclipse dann nicht nen Fehler anzeigen, bzw gar nicht erst kompilieren?

 

[njans]

[STRIKE]Welche IDE verwendest du ? Eclipse, Netbeans o.ä.?[/STRIKE] Eclipse also
Es wäre vielleicht mal sinnvoll, die kompilierten Datein (.class) zu löschen und komplett neu zu kompilieren.
Der Fehler ist mir noch nie unter gekommen. Vielleicht hilft ja Landeis Vorschlag.

Eclipse hebt Fehler hervor, sofern sie zur Kompilirzeit ersichtlich sind.

 

[seux]

[STRIKE]Welche IDE verwendest du ? Eclipse, Netbeans o.ä.?[/STRIKE] Eclipse also
Es wäre vielleicht mal sinnvoll, die kompilierten Datein (.class) zu löschen und komplett neu zu kompilieren.
Der Fehler ist mir noch nie unter gekommen. Vielleicht hilft ja Landeis Vorschlag.

Eclipse hebt Fehler hervor, sofern sie zur Kompilirzeit ersichtlich sind.

[STRIKE]Ich benutze Eclipse.[/STRIKE] Das löschen der class Dateien hat nichts gebracht. Landeis vorschlag ebenfalls nicht.

Eclipse bietet mir sogar Autovervollständigung an, nachdem ich LZW. eingebe

 

[Landei]

Kann auch sein, dass du verschiedene Versionen der Klasse hast (z.B. irgendwann mal von einem Package ins andere verschoben, und es geistern noch alte Class-Files rum). Falls du eine IDE benutzt, versuche mal ein "clean" oder so.

 

[njans]

Kann auch sein, dass du verschiedene Versionen der Klasse hast (z.B. irgendwann mal von einem Package ins andere verschoben, und es geistern noch alte Class-Files rum). Falls du eine IDE benutzt, versuche mal ein "clean" oder so.

Er benutzt, wie er selbst sagt, Eclipse. Landei, leg dir mal ne Brille zu

Clean findet man unter dem Reiter Project -> Clean (in der Menübar von Eclipse)

 

[seux]

Clean findet man unter dem Reiter Project -> Clean (in der Menübar von Eclipse)

Hat nichts gebracht

 

[njans]

Von wie viel SoruceCode reden wir hier eigentlich?
Ist das ein großes Projekt oder ist das relativ handlich?

Kannst du ein kleines Beispiel zum Ausführen geben, welches dieses Problem wirft ?

 

[seux]

Ist nicht so viel SourceCode, nur drei Dateien. Ich werde ihn eben hochladen. Der Code ist aber nicht sehr sauber

Zum ausführen müsst ihr den Namen einer Datei (am besten mit .mip Endung, andernfalls weiß ich nicht, ob es sich rekonstruieren lässt) bei den Argumenten mit hinzufügen.

LZW.java

import java.io.File;
import java.io.RandomAccessFile;
import java.io.IOException;

public class LZW
{
	
	private int dictionaryIndexToPrint = 0;		//Befindet sich das Teilwort schon im Wörterbuch, wird hier der Index gespeichert
	private boolean wasInDictionary = false;	//Wird für die Funktion isAlreadyInCodeTable benötigt
	
	public static String g_fileExtension = new String();
	
	public void compress(File file)
	{
		if(file.exists() != true)
		{
			System.out.println("Datei existiert nicht, programm wird beendet");
			return;
		}
		
		try{
			File comFile = new File(newFileName(file));		//Die neue Datei anlegen
			comFile.createNewFile();
			
			//FileHeader erstellen
			FileHeader fh = new FileHeader(comFile);
			fh.writeHeader();

			RandomAccessFile fileReader = new RandomAccessFile(file, "r");
			RandomAccessFile fileWriter = new RandomAccessFile(comFile, "rw");
			
			System.out.println("Die Originaldatei " + file.getName() + " hat eine groesse von " + fileReader.length() + " Bytes\n");
			
			short gelesenesByte;
			String puffer = new String();
			short index = 256;
			String[][] codeTable = new String[(int)fileReader.length()][2];	//Spalte zwei für den Index
			
			//Codetabelle mit "" füllen
			for(long i = 0; i < fileReader.length(); i++)
				codeTable[(int)i][0] = "";
			
			//LZW Tabelle
			System.out.println("Lesen\tCodetabelle\tAusgabe\t\tPuffer");
			
			for(long i = 0; i < fileReader.length(); i++)
			{
				gelesenesByte = (short)fileReader.read();	//Einlesen des nächsten zeichens
				
				if(i == 0)						//Im ersten durchgang wird noch nichts ins Wörterbuch geschrieben
				{
					puffer = Integer.toString(gelesenesByte);			//sondern nur das gelesene zeichen in den puffer übernommen
					
					System.out.printf("%5s %40s\n", String.valueOf(gelesenesByte), puffer);
					continue;
				}
				
				//Befindet sich das Teilwort bestehen aus puffer + gelesenesByte schon in der Codetabelle
				if(isAlreadyInCodeTable(codeTable, new String(puffer + Integer.toString(gelesenesByte)), i) == false)
				{	
					//Es befindet sich noch nicht in der Codetabelle
					codeTable[(int)i][0] = "" + puffer + gelesenesByte;
					codeTable[(int)i][1] = Integer.toString(index);
					//System.out.printf("%10s %10s\n", String.valueOf(codeTable[(int)i][0]), String.valueOf(codeTable[(int)i][1]));
					index++;
					
					//Den Puffer in die Datei schreiben
					//for(int j = 0; i < puffer.length(); j++)
						//fileWriter.write(puffer.charAt(j));
						
					//Ausgabe für die Tabelle
					if(wasInDictionary == false) 
					{
						System.out.printf("%5s %13s %11s %14s\n", String.valueOf(gelesenesByte), puffer + String.valueOf(gelesenesByte), puffer, String.valueOf(gelesenesByte));
						fileWriter.writeShort(Integer.parseInt(puffer));
					}
					else
					{
						System.out.printf("%5s %13s %11s %14s\n", String.valueOf(gelesenesByte), puffer + String.valueOf(gelesenesByte), dictionaryIndexToPrint, String.valueOf(gelesenesByte));
						fileWriter.writeShort(dictionaryIndexToPrint);
					}
					puffer = Integer.toString(gelesenesByte);
					wasInDictionary = false;
				}
				else
				{
					//Es befindet sich schon in der Codetabelle
					puffer += gelesenesByte;

				}
			}
			System.out.printf("EOF %42s", puffer);
			fileWriter.writeShort(Short.parseShort(puffer));
			
			System.out.println("\nDie komprimierte Datei hat eine groesse von " + fileWriter.length() + " Bytes");
			
			fileReader.close();
			fileWriter.close();
			
		} catch(IOException e) {
		}
	}
	
	private String newFileName(File file)
	{
		String s = file.getName();							// meinText.txt
		String path = file.getAbsolutePath();
		
		int indexPath = path.lastIndexOf(s);
		path = path.substring(0, indexPath);
		
		int index = s.lastIndexOf(".");
		g_fileExtension = s.substring(index +1, s.length());//txt
		s = s.substring(0, index) + ".mip";					//meinText.mip
		
		path += s;
		
		return path;
	}
	
	private boolean isAlreadyInCodeTable(String[][] ct, String s, long lCounter)
	{
		for(long i = 0; i < lCounter; i++)
		{
			if(ct[(int)i][0].equals(s))		
			{
				dictionaryIndexToPrint = Integer.parseInt(ct[(int)i][1]); 
				wasInDictionary = true;
				return true;
			}
		}
		
		return false;
	}
	
	private String origFileName(File file)
	{
		//Über den Header die Originalendung herauslesen
		FileHeader fh = new FileHeader(file);
		
		String ext = new String(fh.readHeader());
		
		String path = file.getAbsolutePath();
		
		int index = path.lastIndexOf(".");
		path = path.substring(0, index) + ext;
		
		
		return path;
	}
	
	
	public void decompress(File file)
	{
		if(file.exists() != true)
		{
			System.out.println("Datei existiert nicht, programm wird beendet");
			return;
		}
		
		try{
			
			File decomFile = new File(origFileName(file));		//Die neue Datei anlegen
			decomFile.createNewFile();
			
			RandomAccessFile fileReader = new RandomAccessFile(file, "r");
			RandomAccessFile fileWriter = new RandomAccessFile(decomFile, "rw");
			
			//String sAusgabe = "";		//In diesem String soll die Ausgabe gespeichert werden, die eigl in der Originaldatei stand
			
			short geleseneZweiByte;
			String letztesWort = "";
			String codeTable[][] = new String[(int)fileReader.length()][2]; 	//Zweite Spalte wieder für den Index
			int index = 256;
			
			//Codetabelle mit "" füllen
			for(long i = 0; i < fileReader.length(); i++)
				codeTable[(int)i][0] = "";
			
			for(long i = 0; i < fileReader.length(); i++)
			{
				geleseneZweiByte = (short)fileReader.readUnsignedShort();		//Zwei Byte einlesen
				
				if(i == 0)														//Im ersten Durchgang wird noch nichts im Wörterbuch geprüft
				{
					//sAusgabe += String.format("%C", geleseneZweiByte);
					fileWriter.writeBytes(String.format("%C", geleseneZweiByte));
					letztesWort = Short.toString(geleseneZweiByte);
					continue;
				}
				
				if(geleseneZweiByte <= 255)		//Befindet sich in der ASCII Tabelle und nicht im Wörterbuch
				{
					//sAusgabe += String.format("%C", geleseneZweiByte);
					fileWriter.writeBytes(String.format("%C", geleseneZweiByte));
					
					//Befindet sich letztesWort + geleseneZweiByte in der CodeTabelle
					if(isAlreadyInCodeTable(codeTable, new String(letztesWort + Short.toString(geleseneZweiByte)), i) == false)
					{
						//Befindet sich noch nicht in der Code Tabelle
						codeTable[(int)i][0] = letztesWort + Short.toString(geleseneZweiByte);
						codeTable[(int)i][1] = Integer.toString(index);
						index++;
					}
					
					
					continue;	//Verhindert die if() abfrage von > 255
				}
				
				if(geleseneZweiByte > 255)
				{
					//gesucht wird die Position von j, an der sich geleseneZwei Byte befindet
					for(int j = 0; j < i; j++)
					{
						if(codeTable[j][1] == Short.toString(geleseneZweiByte))
						{
							//sAusgabe += codeTable[j][0];
							fileWriter.writeBytes(codeTable[j][0]);
						}
					}
				
				}
			}
			
			
			//Den Originaltext ausgeben
			//System.out.println("Originaltext:\n" + sAusgabe);
			
			
		} catch(IOException e) {	
		}
		
	}

FileHeader.java

import java.io.File;
import java.io.IOException;
import java.io.RandomAccessFile;


public class FileHeader {
	
	private final int EXT_MAX_LENGTH = 5;
	private double dVersion = Mip.VERSION;
	private char[] fileExtension = new char[EXT_MAX_LENGTH];
	private File file;
	
	public FileHeader(File f)
	{
		file = f;
	}
	
	public char[] readHeader()
	{
		try {
			RandomAccessFile fileReader = new RandomAccessFile(file, "r");
			dVersion = fileReader.readDouble();
			
			for(int i = 0; i < EXT_MAX_LENGTH; i++)
				fileExtension[i] = fileReader.readChar();
			
		} catch (IOException e) {
			e.printStackTrace();
		}
		
		return fileExtension;
	}
	
	public int sizeOfHeader()
	{
		return Double.SIZE + Character.SIZE;
	}
	
	public void writeHeader()
	{
		if(LZW.g_fileExtension == null);
		{
			System.out.println("Something unusual happend. No file Extension...");
		}
		
		if(LZW.g_fileExtension.length() > EXT_MAX_LENGTH )
		{
			System.out.println("The file extension is larger than 5....");
		}
		
		try {
			RandomAccessFile fileWriter = new RandomAccessFile(file, "rw");
			fileWriter.writeDouble(dVersion);
			
			for(int i = 0; i < LZW.g_fileExtension.length(); i++)
				fileWriter.writeChar(LZW.g_fileExtension.charAt(i));
			
		} catch (IOException e) {
			e.printStackTrace();
		}
	}
}

Mip.java

import java.io.File;
public class Mip {
	
	public static final double VERSION = 1.0;
	
	public static void main(String[] args)
	{
		LZW lzw = new LZW();
		
		//is .mip file or not
		int index = args[0].lastIndexOf(".");
		String Ext = args[0].substring(index, args[0].length());
		
		if(Ext.equals(".mip") == false)
			lzw.compress(new File(args[0]));
		else
			lzw.decompress(new File(args[0]));
	}
}

 

[njans]

Jetzt bräuchte ich noch eine mip Datei, am besten eben diese, welche du verwendest

 

[seux]

Weißt du wo ich die eben hochladen kann um dann den Link hier zu posten?

[EDIT] [STRIKE]Es würde aber schneller gehen, wenn du eben eine Testdatei erstellt, ein paar Buchstaben renschreibst und dann die txt datei übergibst. Das Programm müsste dir dann ne .mip datei auspucken.[/STRIKE]

Okay, müsste nun hochgeladen sein. Die endung .7z einfach gegen .mip austauschen

 

[njans]

Du kannst in jedem Beitrag auch einen Anhang hochladen
Auch beim Editieren.

 

[KrokoDiehl]

In der Methode

LZW.newFileName()

wird

g_fileExtension

geändert. Dies wird vor dem

FileHeader.writeHeader()

aufgerufen.
Kann es also sein dass bei

g_fileExtension = s.substring(index +1, s.length());//txt

null herauskommt?

 

[Landei]

[c]substring[/c] liefert nie [c]null[/c] zurück, sondern entweder einen String oder eine Exception.

 

[KrokoDiehl]

Okay, das stimmt natürlich (@ Landei). Dann wäre es aber ferner gut solche Catch-Blöcke wie

        } catch(IOException e) {
        }

zu vermeiden (@ seux) und zumindest während der Entwicklung die Ausnahme auszugeben.
Das null erklärts leider dennoch nicht.

 

[hüteüberhüte]

Vor jeder Objekterzeugung werden Werte an public static-Variablen zugewiesen, daran kann es also nicht liegen. Welche Fehlermeldung wird denn ausgegeben? Zwischen der ""- und der new String()-Zuweisung besteht auch ein Unterschied, denn bei ersterem kann auch ein schon vorhandenes Objekt (string pool) der Variable zugewiesen werden, bei zweiterem nicht.

 

[Sonecc]

if(LZW.g_fileExtension == null);

Das ; Am Ende der Zeile muss weg (Zeile ~42 in der Klasse FileHeader (Methode writeHeader() ))

 

[KrokoDiehl]

if(LZW.g_fileExtension == null);

Das ; Am Ende der Zeile muss weg (Zeile ~42 in der Klasse FileHeader (Methode writeHeader() ))

Doh!

 

[seux]

if(LZW.g_fileExtension == null);

Das ; Am Ende der Zeile muss weg (Zeile ~42 in der Klasse FileHeader (Methode writeHeader() ))

dafuq Kann ja mal passieren ^^

[STRIKE]Löst das Problem aber auch nicht [/STRIKE]

Ich teste nochmal, scheint tatsächlich daran gelegen zu haben

 

[Sonecc]

Es HAT daran gelegen

Übrigens, teste ich den Code grade und muss sagen, dass es schrecklich inperformant ist^^
Desweiteren gibts bei größeren Dateien (249 kB z.b.) eine NumberFormatException weil du versuchst einen viel zu großen Wert in einen Integer umzuwandeln

 

[seux]

Es HAT daran gelegen

Übrigens, teste ich den Code grade und muss sagen, dass es schrecklich inperformant ist^^
Desweiteren gibts bei größeren Dateien (249 kB z.b.) eine NumberFormatException weil du versuchst einen viel zu großen Wert in einen Integer umzuwandeln

Kritik ist herzlich willkommen. Aber dann auch bitte mit etwas konkretem. Was ist denn schrecklich inperformant? Wie würdest du es machen? Das mit den größeren Dateien hab ich mir auch schon gedacht, dass es für einen Int nicht reichen wird. Aber soweit bin ich ja noch nicht. Was schlägst du denn vor, damit das nicht auftritt? Ich hab ja schon vor, dass das Programm meine Datei auch komprimentiert und nicht wie jetzt vergrößert.

 

[Sonecc]

Ja, wegen dem vergrößern hatte ich es mal mit einer größeren Datei getestet um zu sehen, wie es dann aussieht.

Schrecklich inperformant ist es u.a. wegen den Konsolenausgaben, der Verwendung eines Arrays als "codetable" und dem damit verbundenen millionenfachen iterationen durch das Array. Da sollte sich eine bessere Datenstruktur anbieten.
Was sonst noch zu verbessern wäre, kann ich so nicht sagen, habs halt mal laufen lassen und für eine 249 kB Datei brauchts schon lockere 20-30 Sekunden bei meinem Rechner (der nicht langsam ist)

					if(wasInDictionary == false) 
					{
//						System.out.printf("%5s %13s %11s %14s\n", String.valueOf(gelesenesByte), puffer + String.valueOf(gelesenesByte), puffer, String.valueOf(gelesenesByte));
						fileWriter.writeShort(Integer.parseInt(puffer));
					}
					else
					{
						System.out.println("blubb");
//						System.out.printf("%5s %13s %11s %14s\n", String.valueOf(gelesenesByte), puffer + String.valueOf(gelesenesByte), dictionaryIndexToPrint, String.valueOf(gelesenesByte));
						fileWriter.writeShort(dictionaryIndexToPrint);
					}

Der erste Teil des If-Zweigs wird nie ausgeführt. "wasInDictionary" ist zu diesem Zeitpunkt immer true (wird in isAlreadyInCodeTable() entsprechend gesetzt. Wäre "wasInDictionary" zu diesem zeitpunkt false, würde die Methode den entsprechenden CodeTeil gar nicht erreichen

 

[seux]

Okay, danke erstmal. Die Konsolenausgaben kommen nachher wieder weg. Die sind nur mir eine Hilfe, damit ich auch nachvollziehen kann, dass es das tut, was es auch soll.
Das der int schon bei paar hundert kByte überläuft ist erstmal ziemlich doof. Welchen Datentyp sollte ich denn dafür verwenden. Schleißlich haben heutzutage Dateien schon mehrere 100 mb?

Den Hinweis mit dem Array für die Codetabelle behalte ich auch mal mit im Hinterkopf und werd mich demnächst mal nach was besserem dafür umschauen.

 

[Sonecc]

Was mir spontan einfällt um den Überlauf zu verhindern wäre, dass der Puffer nicht als ganzes reingeschrieben werden sollte, sondern einzeln.