ls.py doesn't seem to know if its UI layer or logic layer.
I'd personally rather have a clear logic layer - something that takes
the needed parameters and provides an iterator over [semi-]structured
data.
then the command can be
for thing in [dols]:
self.output.write("%s%s\n" % (thing[0], thing[1]))
This would make this useful for GUI's as well.
I think a good home for this would be tree.py, rather than a new python
module, as it is very tree specific code (in fact, in refactoring terms,
the first parameter 'tree' stands out like a ForeignMethod).
Just a couple of comments.
ls.py doesn't seem to know if its UI layer or logic layer.
I'd personally rather have a clear logic layer - something that takes
the needed parameters and provides an iterator over [semi-]structured
data.
then the command can be output. write(" %s%s\n" % (thing[0], thing[1]))
for thing in [dols]:
self.
This would make this useful for GUI's as well.
I think a good home for this would be tree.py, rather than a new python
module, as it is very tree specific code (in fact, in refactoring terms,
the first parameter 'tree' stands out like a ForeignMethod).
-Rob