【每日一题】请对这8行代码Code Review
阿里技术【每日一题】
为了提升开发者的技能、有机会入职阿里工作，阿里技术开设了【每日一题】栏目，由阿里巴巴专家们出题，增进开发者的技术知识，并通过这些题目开放阿里巴巴的工作机会，让更多开发者加入阿里大家庭中。
今日题目（1月17日）请对这8行代码CR 
下方为代码大图：
同学们可以将自己对该问题的答案和讨论留言在评论区。
出题团队
出题人： 柏何（蚂蚁国际持续交付和会员开放平台质量负责人）
团队介绍： 我们的使命是“让开发和测试不再受环境问题的困扰”，多快好省的提供各种环境支持， 更好的丰富测试场景的多样化能力。
团队招聘信息： 测试开发专家 点击此处查看招聘信息并投递简历
展开
收起
7
条回答
写回答
写回答
-
-
-
1.注释应该写上开发人，日期，功能描述； 2.这种环境的判断就不该写到代码里面，可以用配置； 3.httpGetRequest.getUrl()出现了三次，可以只用使用变量； 4.缺乏异常处理；
2020-01-19 11:10:34赞同 展开评论 打赏 -
答案的Q8，难道空不应该默认生产环境吗？这个函数的目标就是在生产环境进行替换，开发环境不需要处理。估计因为在测试环境使用HTTPS比较麻烦。
2020-01-17 16:23:29赞同 展开评论 打赏 -
-
技术架构师 阿里云开发者社区技术专家博主 CSDN签约专栏技术博主 掘金签约技术博主 云安全联盟专家 众多开源代码库Commiter专业专业
2020-01-17 15:19:57赞同 展开评论 打赏 -
本期答案：
通过这道题目，想跟大家聊聊【如何有效做Code Review】。 这是8行代码提出的21个问题，具体回答请点击此处详细查看。 提示：这个问题的答案有4000+字，建议先收藏，找一个合适阅读的时间仔细看看。
2020-01-17 14:42:36赞同 展开评论 打赏
滑动查看更多
版权声明：本文内容由阿里云实名注册用户自发贡献，版权归原作者所有，阿里云开发者社区不拥有其著作权，亦不承担相应法律责任。具体规则请查看《阿里云开发者社区用户服务协议》和《阿里云开发者社区知识产权保护指引》。如果您发现本社区中有涉嫌抄袭的内容，填写侵权投诉表单进行举报，一经查实，本社区将立刻删除涉嫌侵权内容。
相关问答
问答排行榜
最热
最新
推荐问答
