CodeGym/Pomoc z zadaniami Java/What's wrong?

Poziom 22

05.10.2023
286widoki
12komentarze

What's wrong?

Pytania dotyczące zadania

Rozwiązane

I think my solution is correct.

Stwórzmy plik złożony z różnych elementów.
Odczytuj nazwy plików z konsoli.
Każdy plik ma swoją nazwę: <someName>.partN.

Na przykład, Lion.avi.part1, Lion.avi.part2, ..., Lion.avi.part37.

Nazwy plików są podawane w kolejności losowej. Słowo "end" jest używane do zatrzymywania odczytu nazw plików.
W folderze, w którym znajdują się wszystkie pliki, utwórz plik bez rozszerzenia "part", tzn. bez ".<partN>".

Na przykład, Lion.avi.

Użyj bufora, aby skopiować wszystkie bajty z plików częściowych do utworzonego pliku.
Skopiuj pierwszy w odpowiedniej kolejności, najpierw pierwszą część, potem drugą, ..., w końcu - część ostatnią.
Zamknij strumienie.

Wymagania:

Program musi odczytywać nazwy plików z konsoli do momentu wprowadzenia słowa "end".
Utwórz strumień do zapisu do pliku bez sufiksu "part" (".<partN>") w folderze z wszystkimi plikami "part".
Skopiuj wszystkie bajty z plików *.partN do nowego pliku.
Do odczytu i zapisu należy używać bufora.
Strumienie pliku muszą zostać zamknięte.
Nie używaj zmiennych statycznych.

package pl.codegym.task.task18.task1825; import com.sun.org.apache.xpath.internal.objects.XString; import java.io.*; import java.util.*; /* Tworzenie pliku */ public class Solution { public static void main(String[] args) throws Exception{ BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in)); ArrayList<String> nazwy = new ArrayList<>(); while (true) { String nazwa = bufferedReader.readLine(); if (!nazwa.equals("end")){ nazwy.add(nazwa); }else break; } bufferedReader.close(); Collections.sort(nazwy); String nazwaDoFolderu = nazwy.get(0); String [] robocze = nazwaDoFolderu.split("\\.part"); String nazwaGlowna = robocze[0] + "."; FileOutputStream fileOutputStream = new FileOutputStream(nazwaGlowna, true); for (String s : nazwy ){ FileInputStream fileInputStream = new FileInputStream(s); BufferedInputStream bufferedInputStream = new BufferedInputStream(fileInputStream); int i; while((i = bufferedInputStream.read())!= -1){ fileOutputStream.write(i); } fileInputStream.close(); bufferedInputStream.close(); } fileOutputStream.close(); } }

Komentarze (12)

Popularne
Najnowsze
Najstarsze

Musisz się zalogować, aby dodać komentarz

Krystian

Poziom 29 , Poland, Poland

5 października 2023, 18:40

Linia 30, masz ustawioną flagę na append na true, czyli zapisujesz dane kolejnych plików do wyjściowego od początku pliku i tak w kółko

TheVirus

Poziom 22

5 października 2023, 18:59

Z tego co wiem, ustawienie append na true powoduje, że dane są dodawane do pliku i zapisywane na końcu, a nie na początku

Krystian

Poziom 29 , Poland, Poland

5 października 2023, 19:22

racja

Guadalupe Gagnon

Poziom 37 , Tampa, United States

5 października 2023, 19:46

When append is true this means that when you open an output stream a new output location will not be created if one already exist. In the case that one exists already then it continues adding to it. When append is set to false (which is default) a new location is created which effectively overwrites any previous data. Take for example if you had a file that said this: "some random large set of data" And you wanted to write "_small data" to that file. If you opened the output stream with append being false then the result would be: "_small data" Because the file is over written. If you open the output steam with append being true then the file is not over written and the new data will be written at the very end, so the result would be: "some random large set of data_small data"

Krystian

Poziom 29 , Poland, Poland

5 października 2023, 18:34

rozwiązanie

28 linia, kropka na końcu w nazwie pliku z nazwy Lion.avi.part34 wyjdzie Lion.avi. a powinno Lion.avi

TheVirus

Poziom 22

5 października 2023, 18:56

Thanks. Conditions confused me: "Na przykład, Lion.avi." There is "." at the end... Problem solved

Thomas

Poziom 41 , Bayreuth, Germany

5 października 2023, 16:39

rozwiązanie

I haven't tested your code but I'll write down what I think may be a problem. - Sorting is probably not working the way you expect. Try with part1, part2, part3 and part12 files - OutputStream is not buffered. Don't know if you're supposed to use a byte array as buffer or if wrapping in a BufferedOutputStream will do - try not to use the OutputStream in append mode (it'll append anyway as long as you won't close it). - line28... why adding a dot

TheVirus

Poziom 22

5 października 2023, 19:08

I've checked sorting before. It works correctly. Maybe i'm wrong, but InputStream is buffered, so it passed conditions. I was inspired by this article: https://codegym.cc/groups/posts/245-inputoutput-in-java-fileinputstream-fileoutputstream-and-bufferedinputstream-classes But you're right. Dot was the problem

Guadalupe Gagnon

Poziom 37 , Tampa, United States

5 października 2023, 19:59

The code above certainly does not sort properly. This is because Strings sort alphabetically and this task requires the part numbers to be sorted numerically. Take this code for example:

public class Solution {
    public static void main(String[] args) throws Exception{
        String[] temp = "1 18 5 22 4 100 57 47 589 3 9 147 56 14 87 25 36 586".split(" ");
        ArrayList<String> test = new ArrayList<>();

        for(String x : temp) test.add(x);

        System.out.println("Raw data in 'test': ");
        System.out.println(test);

        Collections.sort(test);

        System.out.println("\n\nData in 'test' after sorting: ");
        System.out.println(test);
    }
}

It puts a set of different numbers into an ArrayList. The first output shows the data that is in the list. It then sorts the list and outputs the results. here is the output: Raw data in 'test': [1, 18, 5, 22, 4, 100, 57, 47, 589, 3, 9, 147, 56, 14, 87, 25, 36, 586] Data in 'test' after sorting: [1, 100, 14, 147, 18, 22, 25, 3, 36, 4, 47, 5, 56, 57, 586, 589, 87, 9] Both the set before and after the sorting look random, but the "sorted" numbers (the bottom set) if you look are, in fact, sorted alphabetically. The numbers starting with 1 are in the front, followed by numbers starting with 2, then 3 etc This is how numbers are sorted alphabetically.

Thomas

Poziom 41 , Bayreuth, Germany

5 października 2023, 20:00

I checked sorting and it does not work as it should. Codegym seems not to check as thorough anymore than before. Have a look:

List<String> list = new ArrayList<>(Arrays.asList("1", "2", "4", "3", "9", "10", "11", "12", "20"));
Collections.sort(list);
list.forEach(System.out::println);

sorted output:

Buffering: I had a look at the link you posted. And yes, you did everything they told you there. However they spared out output buffering and just bufferd an input stream. Hava a look here. I used an array to buffer the output (but you also could use the BufferedOutputStream class). That example creates 1_000_000 bytes and saves them to disk (ssd here). First run no buffering, just an output stream and I stopped the time needed. In the second run I first save all bytes in an array (buffer) and then I save the array to disk. And the third run uses the BufferedOutputStream class Time needed: first run (unbuffered): 2153ms second run (buffered array): 10 ms third run (buffered class): 11ms

Thomas

Poziom 41 , Bayreuth, Germany

5 października 2023, 20:00

public class StreamTest {
    private static String TEST_FILE = "c:\\Temp\\out.txt";

    public static void main(String[] args) throws IOException {
        // no buffering
        try (OutputStream out = new FileOutputStream(TEST_FILE)) {
            long start = start();
            for (int i = 0; i < 1_000_000; i++) {
                out.write(ThreadLocalRandom.current().nextInt(0, 127));
            }
            System.out.println(stop(start));
        };

        // now simple buffering using an array
        byte[] b = new byte[1_000_000];
        try (OutputStream out = new FileOutputStream(TEST_FILE)) {
            long start = start();
            for (int i = 0; i < 1_000_000; i++) {
                b[i] = (byte) ThreadLocalRandom.current().nextInt(0, 127);
            }
            out.write(b);
            System.out.println(stop(start));
        };

        // buffering using class
        try (OutputStream out = new BufferedOutputStream(new FileOutputStream(TEST_FILE))) {
            long start = start();
            for (int i = 0; i < 1_000_000; i++) {
                out.write(ThreadLocalRandom.current().nextInt(0, 127));
            }
            System.out.println(stop(start));
        };
    }

    private static long start() {
        return System.currentTimeMillis();
    }

    private static long stop(long start) {
        return System.currentTimeMillis() - start;
    }
}

TheVirus

Poziom 22

6 października 2023, 18:38

Thanks! I understand it now