首页
学习
活动
专区
圈层
工具
发布
首页
学习
活动
专区
圈层
工具
MCP广场
社区首页 >问答首页 >有什么建议可以改进我的实现吗?

有什么建议可以改进我的实现吗?
EN

Stack Overflow用户
提问于 2019-04-26 09:48:22
回答 2查看 67关注 0票数 1

这是我下面代码的一部分。当我使用for循环(注释掉)时,它可以工作。当我使用这个流的时候--它有点工作,但是在我从来没有预料到的方法上表现不好。

代码语言:javascript
运行
复制
        Spravochnik spr = new Spravochnik();
        ResultSetMetaData rsmeta = rs.getMetaData();
        List<String> columnNames = new ArrayList<>();
        List<List<String>> valuesInRows = new ArrayList<List<String>>();
        int columnCount = rsmeta.getColumnCount();

        IntStream.range(0, rsmeta.getColumnCount()).forEach(i -> {
            try {
                columnNames.add(rsmeta.getColumnName(i));
            } catch (SQLException e) {
                e.printStackTrace();
            }
        });

//      for(int column = 1; column <= columnCount; column++){
//          columnNames.add(rsmeta.getColumnName(column));
//      }

我想我的实现是错的。这里有什么改进的建议吗?谢谢!

EN

回答 2

Stack Overflow用户

回答已采纳

发布于 2019-04-26 10:03:26

您必须小心循环或流中包含的数字和不包含的数字。

注意,第一个数字总是包含在内的,因为您从1循环,所以您也希望从1启动Stream

代码语言:javascript
运行
复制
IntStream.range(1, rsmeta.getColumnCount()).forEach(i -> {
    // ...
});

最后,这里使用Stream的好处是值得怀疑的。实际上,根本没有的好处。我强烈建议在这个用例中坚持使用for-loop

代码语言:javascript
运行
复制
for (int column = 1; column <= columnCount; column++) {
    columnNames.add(rsmeta.getColumnName(column));
}
票数 1
EN

Stack Overflow用户

发布于 2019-04-26 10:02:06

您可以使用IntStream.range(1, rsmeta.getColumnCount()+1),这将给出从1开始到总计数的列号。

票数 1
EN
页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接:

https://stackoverflow.com/questions/55864975

复制
相关文章

相似问题

领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档