Hi Marko,
thank you very much for your quick reply.
On 13.05.2016 00:01, Marko Rauhamaa wrote:
[...]
- Is this a good approach
As an approach, that looks ok. The code looks a bit too lenient, though,
which can easily surprise and baffle the user. I think it would be
better to fail:
========================================================================
class AlreadyExistsError(Exception):
pass
[...]
Sure, I'll have exceptions in mind for the final implementation.
[...]
For example, why do you need a key? Couldn't you simply pass the task
master class as an argument?
The idea behind this is, to be able to select classes by giving a parameter on the command-line. So at some point a translation from
a given "key" to its actual class has to happen, I guess.
Regards,
Dirk
--
https://mail.python.org/mailman/listinfo/python-list