Converting string to string[] on Method.Invoke()

Question

I have a Dictionary<string, Delegate> commands with my delegate taking two strings and a string[] as params.

I'm entering a command in a Console Application:

:adduser username [email protected] Firstname Middlename Lastname

where everything is converted to an array of strings (size 6). I then do

list[3] = string.Join(" ", input, 3, 3);
list.RemoveRange(4, list.Count - 4);

to convert Firstname, Middlename, and Lastname into a single string, so that my array looks like this:

input[0]: :adduser

input[1]: :username

input[2]: :[email protected]

input[3]: :Firstname Middlename Lastname

I then try to call commands[input[0]].Method.Invoke(this, list.Skip(1).ToArray<object>());, but it results in an ArgumentException with the message

Object of type 'System.String' cannot be converted to type 'System.String[]'.

I'm a bit lost, as I don't really know what's wrong. I'd appreaciate any help!

Alois Kraus · Accepted Answer · 2016-04-16 23:53:10Z

1

You perhaps did mean

commands[input[0]].Method.Invoke(this, new object[] { list[0], list[1], list.Skip(2).ToArray()});

That should match your delegates signature which expect 3 arguments. Your error message indicates that your list is passed as first argument. The reason is that you must pass the other arguments as an object array where the first item is arg1, the second is arg2, ....

answered Apr 16, 2016 at 23:53

Alois Kraus

13.6k2 gold badges45 silver badges69 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

D Stanley · Accepted Answer · 2016-04-16 23:43:49Z

1

Method.Invoke takes an array of parameters after the instance. When you have to pass an array as a single parameter, you need to wrap it in an object[]. Otherwise it gets interpreted as a set of parameters to the delegate:

commands[input[0]].Method.Invoke(this, new object[] {list.Skip(1).ToArray()});

edited Apr 16, 2016 at 23:43

answered Apr 16, 2016 at 23:36

D Stanley

153k12 gold badges190 silver badges257 bronze badges

Comments

Toxantron · Accepted Answer · 2016-04-17 00:09:50Z

1

Your problem is in list[3] = string.Join(" ", input, 3, 3);. If your signature is in fact (string user, string mail, string[] names) you can not call it with 3 strings.

My proposal:

list[3] = input.Skip(3).Take(3).ToArray();

PS: Regarding design I do not recommend recycling the list object but rather create a dedicated object that carries the arguments.

answered Apr 17, 2016 at 0:09

Toxantron

2,40615 silver badges24 bronze badges

Collectives™ on Stack Overflow

Converting string to string[] on Method.Invoke()

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related