Kleiner Texteditor mit MD5-Check

[Spitfire777]

Hi,

ich habe vor, mir einen kleinen Texteditor zu basteln. In diesen moechte ich das Feature einbauen, dass bei bei geaendertem Text der Speichern-Button aktiv wird und wenn der Text gespeichert wurde/der Inhalt nicht veraendert wurde, der Button inaktiv wird.

Da es ja aus Performance-Gruenden Quatsch ist, den kompletten Text zwischenzuspeichern und ihn bei jedem Tastendruck mit der jetzigen Version zu vergleichen, will ich das ganze mit MD5-Hashs loesen.
Bei jedem Tastendruck soll der Anfangshash mit dem aktuellen Hash verglichen werden.

Leider geht das ganze programmiertechnisch nicht so fix wie z.B. in PHP ( md5("abc") ), also hab ich mir folgendes zusammengebastelt:

So bekomme ich einen MD5-Hash:

    /**
     * Returns a string MD5-Hash for a given string
     */
    public String getMD5ForString(String string) {

        try {
            MessageDigest digest = java.security.MessageDigest.getInstance("MD5");
            digest.update(string.getBytes());
            byte[] hash = digest.digest();

            StringBuffer hexString = new StringBuffer();

            for (int i = 0; i < hash.length; i++) {
                hexString.append(Integer.toHexString(0xFF & hash[i]));
            }
            
            return hexString.toString();
        }
        catch (NoSuchAlgorithmException e) {
            System.err.println(e);
            System.exit(0);
            return null;
        }
    }

So wird der MD5-Wert des Ursprungsinhaltes geholt:

    public void loadFile(String path) throws FileNotFoundException, IOException {
        file = new File(path);

        if (!file.exists() || !file.isFile()) throw new FileNotFoundException();

        fileReader = new FileReader(file);
        bufferedReader = new BufferedReader(fileReader);

        String line = bufferedReader.readLine();
        tfContent.append(line);

        while (line != null) {
            tfContent.append("\n" + line);
            line = bufferedReader.readLine();
        }

        fileContentHash = getMD5ForString(tfContent.getText());

        bufferedReader.close();
    }

So ueberpruefe ich, ob der Anfangs-MD5-Wert derselbe wie der jetzige ist.
tfContent ist ein JTextArea

    public boolean contentChanged() {
        if (fileContentHash.equals(getMD5ForString(tfContent.getText())))
            return true;
        else
            return false;
    }

Und schlussendlich in ein Key-Typed-Event eingebunden:

    private void tfContentKeyTyped(java.awt.event.KeyEvent evt) {                                   
        if (contentChanged())
            btSave.setEnabled(true);
        else 
            btSave.setEnabled(false);
    }

Jetzt ist halt das Problem, dass das ganze nicht funktioniert. Mal aktiviert er den Button, mal nicht. Hat auf jeden Fall was mit den Hashwerten zutun!

 

[madboy]

Ohne eine Lösung für dein Problem zu haben:

Warum machst du das nicht ganz einfach und setzt ein Flag wenn der Benutzer was ändert?

boolean userHasChangedText

Alternativ kannst du den Button auch gleich aktivieren/deaktivieren, statt ein Flag zu setzen. Dann hast du keine Probleme mit Hashes und musst auch keinen Text vergleichen.

 

[Spitfire777]

hi,

schon richtig, aber ich wills so haben, dass wenn der text wieder im ursprungszustand ist, der speicherbutton wieder inaktiv wird.
mit deiner methode wuerde der speicherbutton wieder aktiv bleiben, wenn ich an den text ein zeichen anhaengen wuerde und es wieder wegnehmen wuerde.

 

[madboy]

Ok, das stimmt. Du schreibst, dass es manchmal funktioniert, manchmal aber nicht. Stimmen denn die Hashes? Prüfe auch mal, was zum Zeitpunkt der Hashberechnung in der JTextArea steht (getText() ). Könnte sein, dass getText() bei keyTyped() noch nicht immer den ganzen geänderten Text liefert. Zum Beispiel bei DEL, BACKSPACE oder anderen Tasten.

 

[Der Müde Joe]

Deine MD5 Methode liefert falsche Werte. Die for-Schleife liefert bei Werten kleiner als 16 falsche Werte. Sprich wenn zb 0x0F da ist kommt F anstatt 0F in deinen MD5 (Kann man zB mit "hallo" testen".

Entwerder du benutzt sowas wie org.apache.commons.codec.binary.Hex oder addedst ein 0 wenn <16 oder benutzt sowas:

private static char[] CHARS = "0123456789abcdef".toCharArray();

public static String getHex(byte b) {
    char[] result = new char[2];
    result[0] = CHARS[(b & 0xF0) >>> 4];
    result[1] = CHARS[(b & 0x0F)];
	return String.valueOf(result);
}