7
\$\begingroup\$

I have a form that submits to a servlet that has optional parameters. If they are empty they are just ignored. This is the template I am using:

if (!(param = (String)params.get("httpParam")).equals("")) {
   // Handle parameter
}

It doesn't quite feel right. It works, but I think it could be more readable. Any suggestions?

\$\endgroup\$
4
\$\begingroup\$

I suggest extract code for getting parameter to separate method (see extract method refactoring's tehnique):

param = getOptionalParameter("httpParam");
if (! param.isEmpty()) {
    // Handle parameter
}
\$\endgroup\$
3
\$\begingroup\$

You can populate a map containing all your parameters and than convert that map into POJO:

    HashMap map = new HashMap();
    Enumeration names = request.getParameterNames();
    while (names.hasMoreElements()) {
      String name = (String) names.nextElement();
      map.put(name, request.getParameterValues(name));
    }
    BeanUtils.populate(data, map);

Where 'data' is your POJO class with attributes corresponding to your parameter names.

\$\endgroup\$
  • 1
    \$\begingroup\$ How about request.getParameterMap() instead of while loop? \$\endgroup\$ – Alexander Vasiljev Dec 3 '11 at 11:26

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.