Cannot add numbers as options

[siochs]

Hi,
having

[Option('4', "ip4", Required = true, HelpText = "Target Ipv4")]
public string Ipv4 { get; set; }

Passing the option -4 to the exe makes the string Ipv4 null. Am I doing something wrong?
Thanks

[ajtruckle]

Your property is a string so it expects -4 “value”. So if you don’t provide the value it will be null by default.

…

Sent from my iPad On 27 Sep 2017, at 09:05, Simon Graf <notifications@github.com<mailto:notifications@github.com>> wrote: Hi, having [Option('4', "ip4", Required = true, HelpText = "Target Ipv4")] public string Ipv4 { get; set; }``` Passing the option -4 to the exe makes the string Ipv4 null. Am I doing something wrong? Thanks — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#485>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AHnYsyaptbcJ3UkHX8yocSRRmOOSDABcks5smgHXgaJpZM4PlXxF>.

[siochs]

Yes, but this is how I pass the arguments:
MyApp.exe -4 127.0.0.1 -u Username -p Password
--> string Ipv4 = null
MyApp.exe --ip4 127.0.0.1 -u Username -p Password
--> string Ipv4 = "127.0.0.1"
So is this a bug or am I overlooking something?

[nemec]

Hi @siochs the parser is interpreting -4 as a negative integer rather than an option. If you add another property [ValueAttribute(0)]public IEnumerable<string> Remaining { get; set;} to your options, you should see them collected there.

I don't believe it's possible to use integers as short options due to the ambiguity it would introduce. That said, this is clearly a poor user experience - it should warn or error if you have set an invalid short option.

[ericnewton76]

Hmmm, my thoughts are that this is a command line parsing library first, and then primitive types parser second

…

-4 should work just fine... since its declared as an option, the parser library should favor these first before interpreting as a negative integer.

On Fri, Sep 29, 2017 at 4:50 PM, Dan Nemec ***@***.***> wrote: Hi @siochs <https://github.com/siochs> the parser is interpreting -4 as a negative integer rather than an option. If you add another property [ValueAttribute(0)]public IEnumerable<string> Remaining { get; set;} to your options, you should see them collected there. I don't believe it's possible to use integers as short options due to the ambiguity it would introduce. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#485 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACG_zHQ7c9v36ynCn1mKW_6aJIMbKl8nks5snVgQgaJpZM4PlXxF> .

-- E.Newton Senior Software Craftsman Cell: 407-497-1429 EMail: ericnewton76@gmail.com

[siochs]

Okay, thanks. I was just confused, I will go with long options --ip4 --ip6 ...