Move parsed options into a data object

[ericuldall]

First off, I really like this plugin. I've written argv parsing snippets more times than I like and they always come out different. This is a much needed piece of code.

One thing I think would be a good addition is moving all the parsed options into a data object. This would allow us to abstract away all the private data from the public data a user is trying to access.

Instead of

if( program.myoption ){ //do something with myoption}

We can do

if( program.has('myoption') ){ program.get('myoption') }
//also would allow us to easily get all options
console.log(program.get_options()); //returns an object of all parsed options

I haven't dug too deep into this packages code, so perhaps there's an implementation detail that makes storing data this way less than simple. If not, then I'd like to see it work this way.

[ericuldall]

I've implemented the proposed changes with some helper methods in a forked feature branch. Still need to write tests for the new commands.

https://github.com/ericuldall/commander.js/tree/data_overload

New features allow the following:

#!/usr/bin/env node
var program = require('commander');

program
  .version('0.0.1')
  .description('Test')
  .usage('[options] string')
  .option('-t, --type [value]', 'Type')
  .option('-f, --format', 'Format')
  .parse(process.argv);

var type = program.get('type'); //if exists return type, else returns false
var options = program.get(); //return all options, or empty object

var has = program.has('type', function(type){
    console.log(type); //will return if type exists
});
console.log(has); //boolean

var has_all = program.has_all(['type', 'format'], function(type, format){
    console.log(type, format); //will return if both exist
});
console.log(has_all); //boolean

}

[ericuldall]

Pull request sent and all tests passing.
Pull Request 515

[shadowspawn]

Closing this as covered by #183 and #933