Create file ZIP in Kotlin

Question

I\'m trying to create a zip file in Kotlin. this is the code:

fun main(args: Array) {
var files: Array = arrayOf(\"/home/matte/th         


        

Answer 1

1) You are writing an empty byte array to the out for each line of an input file.

2) There is no need in BufferedReader because it is enough to read and write bytes instead of lines (which would lead the unpacked content not to be matched with the original).

3) All streams should be closed in the case of exceptions. Use method use like try-with-resources in java.

4) val instead var there possible

5) Don't use absolute paths except for the quick test snippets.

6) This snippet is not in idiomatic way for Kotlin (see the Todd's answer)

So this is how it should work (though in the Java way):

fun main(args: Array) {
    val files: Array = arrayOf("/home/matte/theres_no_place.png", "/home/matte/vladstudio_the_moon_and_the_ocean_1920x1440_signed.jpg")
    ZipOutputStream(BufferedOutputStream(FileOutputStream("/home/matte/Desktop/test.zip"))).use { out ->
        val data = ByteArray(1024)
        for (file in files) {
            FileInputStream(file).use { fi ->
                BufferedInputStream(fi).use { origin ->
                    val entry = ZipEntry(file)
                    out.putNextEntry(entry)
                    while (true) {
                        val readBytes = origin.read(data)
                        if (readBytes == -1) {
                            break
                        }
                        out.write(data, 0, readBytes)
                    }
                }
            }
        }
    }
}

EDIT: I've ran this snippet with my files and it worked OK.