When a method has too many parameters? [closed]

Answer 1

As far as web services go, I prefer to handle paramaters as a single object. While your data contract may change, your serice contract won't. This makes your services more future proof.

For everything else, I am for 3 parameters and ocassionally go as high as 5.

Answer 2

In my own experience, I've found that method signatures start getting confusing and hard to remember with more than 5 or 6 parameters. And once you get past 10 parameters it's just ridiculous.

These parameters really need to be combined into an object (or a small set of objects) which hold all the data. In the services I use, each service operation always takes a single Request object and returns a single Response object.

Answer 3

Well, I would suggest overloading the method so that you have simpler implementations of the method. This may be especially useful if many of the parameters rarely change and can be assigned defaults. However, if memory serves, I don't think you can overload a web service call (you have to use a distinct method name).

Another possible solution is to encapsulate the parameters into some kind of metadata class whose purpose is to hold the parameters. This would simplify the method signature. However, in some respects you're just offloading the problem to the parameter class. But, if the parameters can be catagorized into topics this technique may be employed by utilizing several metadata classes, each which would be included as a parameter to the web service. This should simplfy things and help you get your head around this monster.

Finally, it's hard to say without any details, but it certainly appears this code is a serious candidate for refactoring. I don't have a hard and fast rule on the number of parameters for a method other than to embrace the general principle of simplicity and readability.

Answer 4

Great Buddha!! Ninety-seven????

Good practise usually advises about max. six to eight. Of course, ymmv, and there may be a valid reason, from time to time, for a ninth. But 97??!!

A few thoughts ... are these simply data, or are decisions being made based on their values?

If many/most affect flow control you have an almost unmaintainable (even understandable, or testable) "design" (for small values of "design").

If they are simply data, can they be grouped into structures and pointers ot those structures passed?

Do you have any design documentation? Might that explain what is going on.

Oh, and, "Danger, Will Robinson" - anyone who will pass 97 parameters openly might also pass any number - not so obviously - as global variables.

P.s don't know how Eclipse works on Java, but with C/C++, if you put the paramaeters on separate lines

char DoEverything(
        int meaninglessParameterName1,
        char *meaninglessParameterName2,
        ....
        long double *meaninglessParameterName97)
        { return !NULL;}

Eclipse will actually identify the line with the bad parameter

Answer 5

If its more than 5-10 parameters, then create an object that takes the parameters instead, it could be a typed dataset, structure or whatever.

Instead of :

Mywebservice.CreateUser(Firstname, LastName, Age,Sex, Haircolor,AmountOfFingers,AmountOfTeeht,Childrens,,,,,,,,,,,,,and so on)

Do:

Dim MyUser as new UserObject
MyUser.Firstname="Stefan"
...and so on...
MyWebservice.CreateUser(UserObject)

Answer 6

Well, if you make it a JSON object, you can then wrap all 97 (or more) in that object and only send one object.