Author Dusty Phillips Reviewers

[
 143 
]
self.temp_directory = Path("unzipped-{}".format(
zipname[:-4]))
def process_zip(self):
self.unzip_files()
self.process_files()
self.zip_files()
def unzip_files(self):
self.temp_directory.mkdir()
with zipfile.ZipFile(self.zipname) as zip:
zip.extractall(str(self.temp_directory))
def zip_files(self):
with zipfile.ZipFile(self.zipname, 'w') as file:
for filename in self.temp_directory.iterdir():
file.write(str(filename), filename.name)
shutil.rmtree(str(self.temp_directory))
We changed the 
filename
property to 
zipname
to avoid confusion with the 
filename
local variables inside the various methods. This helps make the code
more readable even though it isn't actually a change in design.
We also dropped the two parameters to 
__init__
(
search_string
and 
replace_
string
) that were specific to 
ZipReplace
. Then we renamed the 
zip_find_replace
method to 
process_zip
and made it call an (as yet undefined) 
process_files
method instead of 
find_replace
; these name changes help demonstrate the more 
generalized nature of our new class. Notice that we have removed the 
find_replace
method altogether; that code is specific to 
ZipReplace
and has no business here.
This new 
ZipProcessor
class doesn't actually define a 
process_files
method;
so if we ran it directly, it would raise an exception. Because it isn't meant to run 
directly, we removed the main call at the bottom of the original script.
Now, before we move on to our image processing app, let's fix up our original 
zipsearch
class to make use of this parent class:
from zip_processor import ZipProcessor
import sys
import os
class ZipReplace(ZipProcessor):
def __init__(self, filename, search_string,
replace_string):
super().__init__(filename)
www.it-ebooks.info

---

When to Use Object-oriented Programming