循环体内包含了大量没有必要在循环中处理的语句或获取数据库连接或进行不必要的try-catch操作

最新推荐文章于 2024-07-22 16:04:38 发布

Rubenyu

最新推荐文章于 2024-07-22 16:04:38 发布

阅读量1.5k

点赞数

CC 4.0 BY-SA版权

分类专栏： alibaba java 文章标签：数据库 null string file class sql

本文链接：https://blog.youkuaiyun.com/Rubenyu/article/details/6596815

java 同时被 2 个专栏收录

52 篇文章

订阅专栏

alibaba

24 篇文章

订阅专栏

本文分析了在循环中常见的几种不当做法，并提供了改进措施。包括将条件判断移出循环、避免在循环内频繁获取数据库连接、减少不必要的IO操作等。通过这些优化，可以显著提高程序运行效率。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

错误案例

public class CheckCycle
{
    public void cycleInfo()
    {
        String[] Str = null;

        //获取各个部件的通知地址
        Str = outNotify.getResult();
        StringBuffer sb = null;
        BufferedReader in = null;

        Connection conn = null;
        APLResultSet rs = null;
        String sql= "";
        File file = null；

        for (int i = 0; i < Str.length; i++)
        {
           //错误1 
           if( null != Str)
           {
               System.out.print(Str[i]);
           }

          //错误2
           conn = DB.getConnection();    //打开数据库连接

           //错误3
           try
           {
               //错误4
               in = new BufferedReader(new FileReader(file));  //读取文件内容

               sb = new StringBuffer();
               ……
           }
           catch (IOException ex)
           {
               Throw ex;
           }
        }
    }
}

错误分析

错误1：判断完全可以在for外部进行而不是在循环体内部。

错误2：不要在循环中获取数据库连接。

错误3：循环体内进行不必要的try-catch操作,完全可以放在循环外边。

错误4：减少在循环中不必要的IO操作。

正确用法

public class CheckCycle
{
    public void cycleInfo()
    {
        String[] Str = null;

        //获取各个部件的通知地址
         Str = outNotify.getResult();
        StringBuffer sb = null;
        BufferedReader in = null;

        Connection conn = null;
        APLResultSet rs = null;
        String sql= "";
        File file = null；

         if( null != Str)  //判断移到循环体外面
         {
          try
          {
            conn = DB.getConnection();//打开数据库连接
              in = new BufferedReader(new FileReader(file)); //读取文件内容
              for (int i = 0; i < Str.length; i++)
            {
                System.out.print(Str[i]);
                sb = new StringBuffer();
                ……
              }
          }
          catch (IOException ex)
          {
            Throw ex;
          }
        }
    }
}