Improved filesystem efficiency

Previously, `sheets.print()` would query the filesystem every time it
was invoked. This was inelegant, because it is called multiple times
every time `cheat` is executed. Thus, unnecessary calls were being made
out to the filesystem.

Now the result of that function is being buffered into a module variable
when it is executed the first time, and served from there thereafter. I
broke the "functional" paradigm to a degree by doing this, but it wasn't
worth the complexity of implementing proper memoization (decorators,
etc) for such a trivial case.

Bumped the version number accordingly.
This commit is contained in:
Chris Lane
2014-05-25 21:55:25 -04:00
parent f128167311
commit afcaaafbe5
3 changed files with 15 additions and 4 deletions

View File

@ -2,6 +2,12 @@ from cheat import cheatsheets
from cheat.utils import *
import os
# @kludge: it breaks the functional paradigm to a degree, but declaring this
# var here (versus within get()) gives us a "poor man's" memoization on the
# call to get(). This, in turn, spares us from having to call out to the
# filesystem more than once.
cheats = {}
def default_path():
""" Returns the default cheatsheet path """
@ -35,10 +41,15 @@ def default_path():
return default_sheets_dir
# @todo: memoize result
def get():
""" Assembles a dictionary of cheatsheets as name => file-path """
cheats = {}
# if we've already reached out to the filesystem, just return the result
# from memory
if cheats:
return cheats
# otherwise, scan the filesystem
for cheat_dir in reversed(paths()):
cheats.update(
dict([