有没有更好的方法来处理向方法传递多个参数

Question

我发现自己做了很多以下事情：

/**
 * Redirect to a MVC controller&action
 * @param controller
 * @param action
 */
public void redirect(String controller, String action) {
    redirect(controller, action, new HashMap<String, String>());
}
/**
 * Redirect to a MVC controller&action with extra URL parameters
 * @param controller
 * @param action
 * @param data
 */
public void redirect(String controller, String action, Map<String, String> data) {
    String urlParameters = "";
    for(String key : data.keySet()) {
        urlParameters += "&" + key + "=" + data.get(key);
    }
    m_binder.putLocal("RedirectParams", "IdcService=MVC_FRONTCONTROLLER&controller="+controller+"&action="+action + urlParameters);
}

要调用第二种方法，我实际上需要创建一个哈希图来在其中添加数据，我想知道是否有更方便的方法来实现这一点？

如您所见，我需要知道键和值，因此可变参数不起作用（据我所知）。

我对所有想法持开放态度，包括使用反思。

Answer 1

为什么必须创建新地图？我想您可以传入 null，然后在第二个 redirect 方法中检查 null 映射。无论如何，检查 null 可能是个好主意。

Answer 2

如果问题是创建新地图很尴尬，对于小地图，您可以使用 guava ImmutbleMap.of() 方法，

ImmutableMap.of("key1", "value1", "key2", "value2");

Answer 3

我不久前为构建地图编写了这个便捷方法。它需要可变参数并成对地生成一个映射。对于在测试代码中创建简单的地图来说，这非常方便。您需要确保参数的数量正确，但我喜欢它，因为它使代码量更小。

@SuppressWarnings("unchecked")
public static <K, V> Map<K, V> mapOf(K key, V value, Object... morePairs) {
    Map<K, V> map = new HashMap<K, V>();
    map.put(key, value);
    for (int i=0; i<morePairs.length; i+=2) {
        map.put((K)morePairs[i], (V)morePairs[i+1]);
    }
    return map;
}

然后您可以使用以下方法创建一个映射：

Map<String, String> map = mapOf("One", "1", "Two", "2");

然而，这并不是每个人都喜欢的（因为缺乏类型安全），因此您可以更改实现以采用对：

Map<String, String> map = mapOf(pair("One", "1"), pair("Two", "2"));

将pair定义为静态方法，该方法创建一个包含两个的简单对象值，然后是 mapOf 将这些对转换为映射中的条目。

Answer 4

你所做的事情是正常的、常见的。另一种方法是允许某些参数为空，并在第二种方法中进行一些空检查。您必须决定哪种方法您认为不那么难看。

我见过一些 API，其中相同的方法至少有 10 个这样的参数减少版本，显然没有任何原因。当它像这样膨胀时，你的设计就有问题了。在那之前，你所拥有的对我来说看起来不错，尽管我不太喜欢 Map 实例化。

Answer 5

我想说这是一种非常常见的方法。

我目前能想到的唯一其他（不太）方便的方法是：

public void redirect(String[]... params)
{
   //build the query string
}

then call

redirect(new String[][]{{"a","1"}, {"b", "2"}} );
redirect();

注意，这使用了可变参数，但使用起来不太安全（您可以传递长度为 1 甚至 0 的数组）。