This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author rhettinger
Recipients eric.smith, louielu, paka, r.david.murray, rhettinger, terry.reedy
Date 2017-04-18.16:42:16
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1492533736.85.0.745305861765.issue30031@psf.upfronthosting.co.za>
In-reply-to
Content
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 :-)
History
Date User Action Args
2017-04-18 16:42:16rhettingersetrecipients: + rhettinger, terry.reedy, eric.smith, r.david.murray, paka, louielu
2017-04-18 16:42:16rhettingersetmessageid: <1492533736.85.0.745305861765.issue30031@psf.upfronthosting.co.za>
2017-04-18 16:42:16rhettingerlinkissue30031 messages
2017-04-18 16:42:16rhettingercreate