Copy Excel file xslx encoding problems

Question

I am trying to copy an xslx file from a remote system within a jenkins groovy script. This xslx file is encoded with windows-1252, so I give this Charset to the FileReader and FileWriter:

OutputStreamWriter osw = new OutputStreamWriter(new FileOutputStream(file), Charset.forName("windows-1252"));

            BufferedWriter bufferedWriter = new BufferedWriter(osw);

            try (BufferedReader reader = new BufferedReader(new InputStreamReader(new SmbFileInputStream(sFile), Charset.forName("windows-1252")))) {
                int byteRead;

                while ((byteRead = reader.read()) != -1) {
                    bufferedWriter.write(byteRead);
                }

                bufferedWriter.flush();
            }

The result xslx is encoded in windows-1252 and has nrly the same content, but there were additional questionmarks added into it:

Can anybody tell me where those come from and how I can get my correct file content?

If you just want to copy a file 1:1 you don't need to care about encoding. You can copy bytes directly from SmbFileInputStream to FileOutputStream. stackoverflow.com/questions/43157/… — zett42
– zett42, Commented Mar 25 at 10:20
Thanks a lot, IOUtils.copy(new SmbFileInputStream(sFile),Files.newOutputStream(file.toPath())) did the job — Kaffetrinken
– Kaffetrinken, Commented Mar 25 at 11:24
Is there something wrong with using e.g. sh "cp $sFile $file" if all you need is a copy? — MaratC
– MaratC, Commented Mar 25 at 15:03

chubbsondubs · Accepted Answer · 2025-03-25 14:58:03Z

1

Copying files does NOT require interpretation of their contents. All files are bags of bytes regardless if they are binary or text. Bytes do not require you interpret them (ie using Charsets).

This makes it easy for you to perform this using just InputStream and OutputStream.

file.withOutputStream { out ->
   new SmbFileInputStream(sFile).withCloseable { smb ->
      out << smb
   }
}

This has been groovy'ed up so as to limit how much code you need to write, but it's essentially doing the exact same thing your code was doing, but only using InputStream and OutputStream. This also flushes and closes streams for you similar to try-resource statements.

answered Mar 25 at 14:58

chubbsondubs

39.4k25 gold badges110 silver badges143 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Copy Excel file xslx encoding problems

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related