使用Java 8流处理嵌套的if/else语句

Question

我是java 8的新手，并且有创建对象时需要进行一些检查的情况。 我当然在使用这个流，并且在完成这个过程中经历了艰难的时间。

输入是一个包含键/值对的HashMap对象，输出应该在下面。

| userrole | userid | username | output | 
|------------|--------|----------|----------| 
| "" (blank) | 111 | amathews | 111  | 
| ""   |  | amathews | amathews | 
| Admin  | 111 | amathews | 111  | 
| Admin  | 111 | ""  | 111  | 
| Admin  |  | amathews | Admin | 

这是如何确定userid> userrole>用户名。

每个HashMap对象都将包含userrole/username/userid作为键和其值以及其他键/值对。 我们将有大量嵌套的if/else语句来完成此任务的Java之前的版本。

我们将有一堆嵌套的if/else语句来完成此前的Java版本中的任务。

这是我到目前为止的代码。

map.entrySet().stream() 
     .filter(e -> e.getValue() instanceof String || e.getValue() instanceof Integer) 
     .filter(e -> e.getKey().contains("userrole") || e.getKey().contains("userid") || e.getKey().contains("username")) 
     .map(e -> e.getValue()) 
     .collect(Collectors.toList()); 

我知道我在流中写入map函数的方式也不正确。 如何在java 8中完成此操作？我不知道如何在这里添加嵌套的if/else部分。

请有人帮我在这里我卡住了，无法继续。谢谢。

编辑：对不起，如果我没有明确说明问题。这里是代码snippiet。

public List<UserAction> getUserActionList(Map<String, String> map) 
    { 
     String userRole = map.get("userrole"); 
     String userName = map.get("username"); 
     String userId = map.get("userid"); 

     String output = null; 
     // if userrole, userid and username are not null/empty, then output is userid 
     if(!checkForNullEmpty(userRole) && !checkForNullEmpty(userId) && !checkForNullEmpty(userName)) 
      output = userId; 
     // if userrole and userid are null/empty and username is not empty/null, then output is username 
     else if(checkForNullEmpty(userRole) && checkForNullEmpty(userId) && !checkForNullEmpty(userName)) 
      output = userName; 
     // if userid and username are null/empty and userrole is not empty/null, then output is userrole 
     else if(!checkForNullEmpty(userRole) && checkForNullEmpty(userId) && checkForNullEmpty(userName)) 
      output = userRole; 

     List<UserAction> udList = new ArrayList<>(); 
     // Add the map and output into a UserAction object 
     udList.add(new UserAction(map, output)); 

     return udList; 

    } 

我已经在这里按照表格处理了三个条件。所以这必须重构为使用Java 8流。希望现在有意义。

Answer 1

如果值的至少一个保证，你可以重构它是这样的：

public List<UserAction> getUserActionList(Map<String, String> map) { 
    return Stream.of("userid", "username", "userrole") 
     .map(map::get) 
     .filter(s -> !checkForNullEmpty(s)) 
     .limit(1) 
     .map(output -> new UserAction(map, output)) 
     .collect(Collectors.toList()); 
} 

如果不保证至少有一个值将非空，这是一个有点丑陋，但不是太糟糕了：

public List<UserAction> getUserActionList(Map<String, String> map) { 
    return Stream.of("userid", "username", "userrole") 
     .map(map::get) 
     .filter(s -> !checkForNullEmpty(s)) 
     .limit(1) 
     .map(output -> new UserAction(map, output)) 
     .map(Collections::singletonList) 
     .findFirst() 
     .orElseGet(() -> Arrays.asList(new UserAction(map, null))); 
} 

Answer 2

这不是真的清楚你需要完成的任务，但在一般情况下，你需要在你的if语句来写，你可以用filter()方法从Stream API做的一切。然后，在map()方法中，您将具有需要使用数据完成的确切逻辑（例如将其转换为其他类型或获取所需的值）。使用collect()方法来创建来自Stream的结果，例如，列表，设置，地图，单个对象或其他任何东西。例如：

map.entrySet().stream() 
       .filter(e -> { 
        // filter the data here, so if isStrOrInt or containsUserData is false - we will not have it in map() method 
        boolean isStrOrInt = e.getValue() instanceof String || e.getValue() instanceof Integer; 
        boolean containsUserData = e.getKey().contains("userrole") || e.getKey().contains("userid") || e.getKey().contains("username"); 
        return isStrOrInt && containsUserData; 
       }) 
       .map(e -> { 
        if (e.getKey().contains("userrole")) { 
         // do something 
        } 
        // some more logic here 
        return e.getValue(); 
       }) 
       .collect(Collectors.toList()); 
       // or e.g. .reduce((value1, value2) -> value1 + value2); 

如果你需要创建在最后一个对象，你可能需要reduce()方法。我建议您查看reduction operations，了解有关Stream API的一般信息，了解它们的工作原理。