Does it really work like that?

Frage zur Aufgabe

Korrigiere den Fehler. Klassen und Interfaces

Java Core, Level 9, Lektion 11

Gelöst

1. Question: The task states the following:

try (
  FileOutputStream outputStream = new FileOutputStream(args[0]);
  InputStream is = Solution.class.getClassLoader().getResourceAsStream(args[1]);
) {
    ;
    byte[] b = new byte[is.available()];
    ....

I have set the second parameter to a valid file, but when I run this in IntelliJ, a NullPointerException is thrown on line 6 because the variable "is" is null - and null, of course, does not have an available() method. If I replace line 3 with the following, everything works fine:

InputStream is = new FileInputStream(args[1]);

But according to the task, the only error is the wrong if statement. Does the command "Solution.class.getClassLoader().getResourceAsStream(args[1])" really work? And if so, how? 2. Question: According to the task, the following line should be correct:

outputStream.write(is.read(b));

"is.read(b)" returns the number of bytes read as an integer. "outputStream.write()" writes this integer as a byte to the file - makes no sense to me, or am I missing something? Anyway, as expected, there is only one ascii character in my output file. Best regards.

Kommentare (6)

Beliebt
Neu
Alt

Du musst angemeldet sein, um einen Kommentar schreiben zu können

Thomas

Level 41 , Bayreuth, Germany

25 Mai 2023, 10:12

nützlich

Classloader is a mechanism in Java that allows the runtime to find classes and resources. This is not a trivial task for the JVM and can be complicated in times. To really explain this I would have to write quite a few lines. But in very short... getResource() and getResourceAsStream() are adressing the resource relatively. The class you invoked getResource() on is the source and you can move inside the package hierarchy. Example

package cc.codegym;

import java.io.InputStream;

public class Solution {
    public static void main(String[] args) throws Exception {
        try (InputStream in = Solution.class.getResourceAsStream("./test.txt")) {
            System.out.println(new String(in.readAllBytes()));
        }
    }
}

expects a file test.txt to exist in cc.codegym If you use the command line to compile your project, then you'd need to manually copy the test.txt to your target directory (where the compiler places the .class files). If you use an IDE then that one usually takes care of it and you place the test.txt in your src directory and the IDE copies it to the compiled files. new FileInputStream on the other hand uses other ways to find the file. It is more dependent your OSes filesystem and you usually use absolute addressing. Question 2: I do not really understand what you're asking? If the OutputStream is a byte stream then what is wrong about sending a byte to it? is.read() reads the current byte from the input stream. If nothing has been read before, then it is the very first byte in the file. If you invoke read() a second time, then you read the second byte in the file and so forth. That byte you pass to the output streams write method is widened to an int and the write method then ignores the first 24 bit and writes the last 8 bit (the passed byte) to the target.

TK87

Level 34 , Aachen, Germany

25 Mai 2023, 11:59

Hallo Thomas, vielen dank für die Erläuterung zu Punkt 1, das erklärt schon mal, warum die Ausnahme bei mir ausgelöst wurde. Bei Punkt 2 hast du mich scheinbar missverstanden. Beim Aufruf der Read-Methode wird hier der Bytebuffer "b" als Argument übergeben. Die Methode gibt also nicht etwa ein gelesenes Byte zurück, sondern die Anzahl der Bytes, die in den Buffer gelesen wurden. Wenn in der Quelldatei also z.B. 65 Zeichen stehen, wird so anstelle des Buffers nur "A" (Asciicode 65) in die Zieldatei geschrieben. Gruß auch Thomas

Thomas

Level 41 , Bayreuth, Germany

25 Mai 2023, 12:49

Ja, so funktioniert das. Der Sinn ist aber nicht, die Anzahl der gelesenen bytes in die zweite Datei zu schreiben, sondern die Möglichkeit zu haben, herauszufinden, wieviele bytes gelesen wurden, um dann exakt diese Zahl aus dem Array heraus in eine andere Datei zu schreiben. Beispiel:

public static void main(String[] args) throws IOException {

    try (InputStream in = Files.newInputStream(Paths.get("c:\\Temp\\in.txt"));
         OutputStream out = Files.newOutputStream(Paths.get("c:\\Temp\\out.txt"))) {

        byte[] b = new byte[2 * 1024];      // ein Puffer
        int len;                            // die Anzahl der gelesenen bytes

        // die Datei hat z.B. 5000 bytes, dann werden 2048 in den Puffer gelesen. 2048 wird in len
        // gespeichert. In der nächsten Zeile wird der Puffer von 0 bis len nach out gespeichert.
        // Im dritten Durchgang sind nur noch 4 bytes zu lesen. In len wird 4 gespeichert und in write
        // wird nicht das gesamte Array (2048 bytes) gespeichert, sondern nur die gelesenen bytes von
        // 0 bis 4
        while ((len = in.read(b)) != -1) {
            out.write(b, 0, len);
        }
    }
}

TK87

Level 34 , Aachen, Germany

25 Mai 2023, 13:09

Genau das meinte ich ja gerade - der Befehl ergibt so wie er ist keinen Sinn. Die Aufgabenstellung heisst "korrigiere logische Fehler". Wenn man dann aber die fehlerhafte Logik:

byte[] b = new byte[is.available()];
outputStream.write(is.read(b));

abändert in:

byte[] b = new byte[is.available()];
is.read(b);
outputStream.write(b);

wird angemerkt, lediglich die Logik im if-Statement wäre falsch.

Thomas

Level 41 , Bayreuth, Germany

25 Mai 2023, 13:42

Dann war wohl die Aufgabe, syntaktische Probleme zu korrigieren, nicht aber semantische Schwächen. Sei froh, daß dir das aufgefallen ist. Damit bist du schon ein gutes Stück weiter. Es gibt einige Tasks, die einem recht sinnfrei erscheinen. Das ist halt einfach Beispielcode, der nicht immer Sinnvolles vollbringt. Viel Spaß noch ;)

TK87

Level 34 , Aachen, Germany

25 Mai 2023, 13:49

Danke, werde ich haben ^^