Make all the optional things available through a builder parameter.

[shelajev]

We have a number of optional features that are wanted to include/exclude:

• Preserving timestamps Add option for (disabling) preserving timestamps for any ZipUtil methods #19
• Charset support Add charset support (java 7) #29, could you support chinese character #30

I would even go further and implemented a destination archive (#17) through a parameters map.

Would be cool to simplify an api to somehting like:

ZipUtil.addEntry(File, ZipEntrySource, ZipUtilParams.preserveTimestamp().charset(charset).output(destFile)); 

Then by default all simple methods would be inplace and with no parametrization, however we won't need to pollute the API with all the variations of charsets, timestamps, existense/lack of destination). Methods would be simpler.

What do you think? Other features that we want to add later could be added there as well, so we won't need to change public api there.

[reinra]

A more generic solution:

ZipUtil.create(File).charset(Charset).preserveTimestamp(true).addEntry(ZipEntrySource).addEntries(ZipEntrySource...).addDirectory(File).addArchive(File).execute()

addDirectory() and addArchive() should optionally take additional charset to override the output one.

[shelajev]

Transformations, addArchive() and addDirectory() are not there yet.

I added charset (#29, #30) , timestamps (part of #19), optional destination (alternatively, it'll be inplace).
Also, Zips is not final and mostly uses instance methods, so it will be more extensible than ZipUtil (#23).

[reinra]

Added many comments to the commits.

[shelajev]

Closing this issue, as other comments that @reinra left are worth having their own issues.