java吧 关注:1,254,440贴子:12,742,299

大佬们急求救,这段代码找至少五个缺陷,考试要用

只看楼主收藏回复

大佬们急求救,这段代码找至少五个缺陷,考试要用


IP属地:山东来自Android客户端1楼2022-06-23 09:42回复
    我找到就3个,URL没用,createfile返回值没处理,fw、bw、Con、st用了没关闭


    IP属地:山东来自Android客户端2楼2022-06-23 09:43
    回复
      这个是题干


      IP属地:山东来自Android客户端3楼2022-06-23 09:45
      回复
        4.用log打印日常
        5.异常范围尽量小
        6.不要用魔法数,比如你的jdbc那些参数
        7.


        IP属地:安徽来自Android客户端4楼2022-06-23 09:47
        收起回复
          1.异常


          IP属地:安徽来自Android客户端5楼2022-06-23 09:48
          回复
            用prestatement去替换 statement


            IP属地:安徽来自Android客户端6楼2022-06-23 09:49
            收起回复
              应该够考试用了


              IP属地:安徽来自Android客户端7楼2022-06-23 09:49
              收起回复
                函数体太长,功能耦合,难于扩展...


                IP属地:浙江8楼2022-06-23 09:51
                收起回复
                  用prestatement替换 statement防止sql注入


                  IP属地:浙江来自iPhone客户端11楼2022-06-23 10:38
                  收起回复
                    resource关闭链接
                    抛自定义异常
                    catch缩小异常范围
                    配置文件
                    preparestatement防止sql注入
                    应该这些吧


                    IP属地:安徽来自iPhone客户端12楼2022-06-23 10:42
                    回复
                      阿里编程规范扫一下,答案就出来了


                      IP属地:广东来自Android客户端13楼2022-06-23 10:55
                      回复
                        方法的() ( )


                        IP属地:北京来自Android客户端14楼2022-06-23 12:36
                        回复
                          读完数据怎么不顺序关闭,打算一直用直到报resultset关闭错误吗


                          IP属地:河南来自Android客户端16楼2022-06-23 14:18
                          回复
                            应该把数据库连接单独拿出来做一个方法来调用,减少耦合。


                            IP属地:广西来自Android客户端17楼2022-06-23 14:30
                            回复
                              建议把这代码上到idea跑一遍


                              IP属地:广东来自Android客户端18楼2022-06-23 15:07
                              回复