Simplify PdfFileSplitter challenge solution by removing calls to str()

somacdivad · somacdivad · commit 2c6d799c6819 · 2020-01-14T13:43:04.000-06:00
diff --git a/ch13-interact-with-pdf-files/3-challenge-PdfFileSplitter-class.py b/ch13-interact-with-pdf-files/3-challenge-PdfFileSplitter-class.py
@@ -11,7 +11,7 @@ class PdfFileSplitter:
 
     def __init__(self, pdf_path):
         # Open the PDF file with a new PdfFileReader instance
-        self.pdf_reader = PdfFileReader(str(pdf_path))
+        self.pdf_reader = PdfFileReader(pdf_path)
         # Initialize the .writer1 and .writer2 attributes to None
         self.writer1 = None
         self.writer2 = None
@@ -32,10 +32,10 @@ def split(self, breakpoint):
     def write(self, filename):
         """Write both PdfFileWriter instances to files"""
         # Write the first file to <filename>_1.pdf
-        with Path(str(filename) + "_1.pdf").open(mode="wb") as output_file:
+        with Path(filename + "_1.pdf").open(mode="wb") as output_file:
             self.writer1.write(output_file)
         # Write the second file to <filename>_2.pdf
-        with Path(str(filename) + "_2.pdf").open(mode="wb") as output_file:
+        with Path(filename + "_2.pdf").open(mode="wb") as output_file:
             self.writer2.write(output_file)
 
 
