Message291838
After more thought on the subject, I think the n-queens demo should be left as is. The demo is principally about the class and not about being a command-line tool. Also, it is reasonable and appropriate to use sys.argv directly for such a simple API. How to do so is a legitimate example itself. Not all uses of sys.argv need argparse.
In other words, I don't think the proposed changes makes anyone's life better and in the some ways makes the example less approachable for teaching purposes (my area of expertise).
The example works fine as-is. Changing it seems like unnecessary code churn and already eating some of our time.
Thank you for the idea. It never hurts to look around for ways to improve code. Though I don't think there was a new win in this case, don't let that discourage you. Please continue to look for improvements :-) |
|
Date |
User |
Action |
Args |
2017-04-18 16:42:16 | rhettinger | set | recipients:
+ rhettinger, terry.reedy, eric.smith, r.david.murray, paka, louielu |
2017-04-18 16:42:16 | rhettinger | set | messageid: <1492533736.85.0.745305861765.issue30031@psf.upfronthosting.co.za> |
2017-04-18 16:42:16 | rhettinger | link | issue30031 messages |
2017-04-18 16:42:16 | rhettinger | create | |
|