Convert PDF numbering to indexing from 0

Author: netotz

README.md

@@ -36,8 +36,6 @@ I'm using a Windows executable generated with [`PyInstaller`](https://github.com
 Arguments `-s` and `-p` can be repeated as many times as needed, but both have to be repeated the same number of times.
 So if a file needs to be split into 3 new files, 3 file names will need to be specified, each one with `-s`, and also 3 page ranges with `-p`, respectively.
 
-The page numbers start in 0.
-
 ### Examples
 
 Assume this is the folder, and you're using the executable:
@@ -51,7 +49,7 @@ folder/
 Following the previous example, running this inside `folder/`:
 
 ```txt
-pdf-splitter -s DEGREE-BACH -s IDCARD -p 0 1 -p 2 2 1234.pdf
+pdf-splitter -s DEGREE-BACH -s IDCARD -p 1 2 -p 3 3 1234.pdf
 ```
 
 Will result in:

pdf-splitter/main.py

@@ -20,14 +20,26 @@
     required=True,
     type=int,
     nargs=2,
-    help='first and last page numbers to split from the input file'
+    help='range of first and last page numbers to split from the input file, according to file numbering (first page is page 1)'
 )
 
 args = parser.parse_args()
-print(args)
 if len(args.page_numbers) != len(args.split_name):
     raise argparse.ArgumentTypeError('specify the same number of page ranges as the number of splits.')
 
 inputfile = InputFile(args.filepath)
-splits = [SplitFile(f'{name}.pdf', pages) for name, pages in zip(args.split_name, args.page_numbers)]
+splits = [
+    SplitFile(f'{inputfile.id}-{name}.pdf', pages)
+    for name, pages in zip(
+        args.split_name,
+        # PDF objects number pages starting in 0
+        ((first - 1, last - 1) for first, last in args.page_numbers)
+    )
+]
+
+# info in console
+print(f'Splitting {inputfile.name} into {len(splits)} new files...', flush=True)
 split_pdf(inputfile, splits)
+print(f'{len(splits)} files created:', flush=True)
+for split in splits:
+    print(f' - {split.name}')