Zadanie dotyczące plików tekstowych - problem z podmianą zawartości (JAVA)

Question

Hej wszystkim ponownie! Zacząłem pisać nowy program i znowu popełniam błędy. Mam problem z podmianą zawartości pliku tekstowego, bardziej szczegółowo chodzi o to że mam napisać funkcję która będzie szyfrowała zawartość pliku tekstowego ( coś na zasadzie szyfru cezara). Niestety kiedy wywołuję funkcję zawartość pliku która powinna zostać podmieniona znika, a plik zostaje pusty. Popełniam gdzieś błąd tylko nie wiem gdzie, prawdopodobnie napisałem coś bezsensowego :P. Oto treść tego co mam zrobić oraz kod który napisałem:

4. Stworzyć dwie funkcje:

void szyfruj(String nazwaWe, int przesun)

void deszyfruj(String nazwaWe, int przesun)

Funkcja szyfruj dokonuje szyfrowania pliku, którego nazwa podana została jako pierwszy parametr. Szyfrowanie polega na zamianie każdej litery na znak przesunięty o wartość podaną drugim parametrem np. dla przesunięcia równego 2 literka ’a’ powinna zostać zastąpiona literką ’c’, literka ’z’ literką ’b’ itp. Wynikiem działania funkcji ma być plik o nazwie utworzonej na podstawie nazwy pliku wejściowego poprzez dołączenie znaku ’ ’ np. dla pliku dane.txt zaszyfrowana postać powinna mieć nazwę dane.txt. Funkcja deszyfruj powinna deszyfrować plik (niekoniecznie ten sam) zaszyfrowany przez funkcję szyfruj.

package zadanie4;

import java.io.FileReader;
import java.io.IOException;
import java.io.FileWriter;

public class Zadanie4 {

    public static void szyfruj(String nazwaWe, int przesun) throws IOException
    {
        FileReader f = null;
        FileWriter fw = null;
        try
        {
            f = new FileReader("C:/pliki_java/" + nazwaWe);
            fw = new FileWriter("C:/pliki_java/" + nazwaWe);
            int AsciiCode;
            while((AsciiCode = f.read()) != -1)
            {
                AsciiCode += przesun;
                char NowyZnak = (char)AsciiCode;
                fw.write(NowyZnak);
            }                
        }
        finally
        {
            if(f != null)
            {
                f.close();
            }
        }
    }
    
    public static void deszyfruj(String nazwaWe, int przesun)
    {
    }
    
    public static void main(String[] args) throws IOException {
        
        szyfruj("text4.txt",2);
        
    }
    
}

Dziękuję za wszelką pomoc :)

Answer 1

Nieco namieszałeś.

1. Czy nazwa pliku we i wy mają być takie same? To po co coś dodawać?
2. W programie otwierasz dwa razy plik o tej samej nazwie - raz do odczytu i raz do zapisu. To na pewno nie zadziała, tak jakbyś chciał.
3. W finally zamykasz plik wejściowy. OK, jednak ważniejsze jest zamknięcie pliku wyjściowego, bo przed zamknięciem może nie być kompletny na dysku. Oczywiście jak program się kończy to wszystko jest zamykane, ale niekoniecznie na poziomie programu - czyli np. bufory w bibliotekach Javy mogą zostać nie zapisane.
4. Jednak daje się napisać w Javie kod nieprzenośny.

Comments

2 W programie otwierasz dwa razy plik o tej samej nazwie - raz do odczytu i raz do zapisu. To na pewno nie zadziała, tak jakbyś chciał.

dlaczego można nawet jednocześnie wczytać  litere z tekstu i zapisać inną w jej miejscu :)  

Answer 2

Zapoznałbym się dodatkowo z BufferedReader i BufferedWriter, na pewno program będzie działał wydajniej. Dodatkowo także przeczytaj sobie o metodzie flush() do zapisu.

Tak jak mój przedmówca wspomniał, musisz zamknąć obydwa strumienie danych - ten do zapisu i odczytu. Zapis i odczyt musi odbywać się także na innych plikach, chyba, że zaczytasz całą zawartość pliku do pamięci, przerobisz ją i następnie nadpiszesz istniejący plik.