Zipper z single thread executor - ocena kodu

Witajcie!

Mój pierwszy jakikolwiek review przez kogokolwiek - nie zostawcie na mnie suchej nitki (i tak ostatnio moja wiara w siebie zniknęła gdzieś w niewyjaśnionych okolicznościach, więc bez różnicy).
Chciałbym tylko zauważyć, że moim celem było stworzenie zippera, który dodawałby pliki jeden po drugim, każdy na nowym wątku (jako, że ćwiczę Concurrency).

Source: https://bitbucket.org/Burdzi0/commandzip/src

Za mało kodu, żeby konkretnie ocenić.

Nic nie jest za mało.

1. Ta metoda main to nie może mieć wydzielonych mniejszych metod które opiszą co się gdzie dzieje zamiast tych komentarzy?
2. Magic numbers
3. W tym forze w main nie da rady jechać od 0? Ja wiem że potem porównujesz argument większy o 1, ale jakoś brzydko to wygląda
4. Co się stanie jeśli w klasie ZipFile, File będzie nullem?
5. Metoda zipIt patrz punkt 1.
6. Ten try catch wygląda tragicznie.
7. Tak samo klasa ZipStreamManager znajduje zastosowanie w punktach wyżej.
8. Czy jesteś w stanie do tego kodu dopisać unit testy?

@panryz Średnio potrafię dzielić metody na mniejsze. Co polecasz z try ... catch? Nie umiem pisać unit testów, w ogóle testów.