查看: 2849| 回复: 17
跳转到指定楼层
上一主题 下一主题
收起左侧

[Leetcode] 问 代码风格检测的必要性

全局:

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看附件。没有帐号?注册账号

x
最近发现lintcode有一个功能是代码风格检测的功能,比如if后面要加空格,每个符号前后都要加空格,if后必须用{}括起来,变量声明一个占一行等等。我很好奇是否有必要这么做?很多时候看leetcode很多答案有都是尽量求简,恨不得把答案写成一行最好。但是我觉得代码不应该是在可读性好和清楚的基础上做简化?

先谢谢各位解答

评分

参与人数 1大米 +3 收起 理由
Jonathan张 + 3 给你点个赞!

查看全部评分


上一篇:leetcode combination sum 时间复杂度到底应该怎么答?
下一篇:Leetcode glob matching 扩展
推荐
NikoToretto 2017-6-19 07:42:20 | 只看该作者
全局:
过量的简化其实不是好习惯。if后接一行语句不用{}括起来多不安全,感觉就像是ML不戴Condom。

评分

参与人数 1大米 +10 收起 理由
tommylin + 10 可哥真牛逼

查看全部评分

回复

使用道具 举报

全局:
lintcode 现在还有这么个功能了,感觉还是不错的啊。上了班感觉大家要是coding style都比较好,读代码就没那么费力了。。。毕竟几乎不可能一个人永远维护一部分代码,极度简洁风格独特不利于合作。除非你不想跟别人分享你的牛逼代码哈哈哈哈。

不过每个变量声明一个一行不是很重要吧,个人觉得显示声明类型和有一个可读的变量名更加重要一些。楼上老兄这个比喻真是服了哈哈哈,人才啊
回复

使用道具 举报

推荐
newgod2500 2017-6-20 08:52:34 | 只看该作者
全局:
因为不是Java选手,所以没用过Lintcode.. 所以就只讨论Leetcode上面的。

感觉很多高票答案在coding style上面都过不了关.... 就冲使用brackets这点...虽然只是业余/刷面经,但是加了brackets真的提高了易读性。 其余的例如 nums[i++] > nums[x]这种....我宁愿分两行写。

真正的简洁并不是体现在行数上。
回复

使用道具 举报

🔗
dukecat0613 2017-6-19 06:50:20 | 只看该作者
全局:
coding style 我觉得很重要, 给人很profession的感觉. 因为我为某开源项目contribute的时候, coding style如果不对, 别谈code review了,jenkins自动检测都过不了。
回复

使用道具 举报

🔗
dukecat0613 2017-6-19 07:48:35 | 只看该作者
全局:
NikoToretto 发表于 2017-6-19 07:42
过量的简化其实不是好习惯。if后接一行语句不用{}括起来多不安全,感觉就像是ML不戴Condom。

兄弟 你这个比喻 我服 哈哈哈哈哈哈哈哈哈哈哈

评分

参与人数 1大米 +10 收起 理由
NikoToretto + 10 送10米不成敬意

查看全部评分

回复

使用道具 举报

🔗
NikoToretto 2017-6-19 07:50:01 | 只看该作者
全局:
dukecat0613 发表于 2017-6-19 07:48
兄弟 你这个比喻 我服 哈哈哈哈哈哈哈哈哈哈哈

在此谢过了
回复

使用道具 举报

🔗
 楼主| why1992 2017-6-20 01:49:56 | 只看该作者
全局:
NikoToretto 发表于 2017-6-19 07:42
过量的简化其实不是好习惯。if后接一行语句不用{}括起来多不安全,感觉就像是ML不戴Condom。

666666,学习了

评分

参与人数 1大米 +10 收起 理由
NikoToretto + 10 回答的很好!

查看全部评分

回复

使用道具 举报

🔗
 楼主| why1992 2017-6-20 01:52:29 | 只看该作者
全局:
sheepmiemies 发表于 2017-6-19 08:08
lintcode 现在还有这么个功能了,感觉还是不错的啊。上了班感觉大家要是coding style都比较好,读代码就没 ...

好的,明白了,还是应该以可读性好作为标准
回复

使用道具 举报

🔗
krizalio001 2017-6-20 02:46:38 | 只看该作者
全局:
NikoToretto 发表于 2017-6-19 07:42
过量的简化其实不是好习惯。if后接一行语句不用{}括起来多不安全,感觉就像是ML不戴Condom。

别的不服,就服你

评分

参与人数 1大米 +10 收起 理由
NikoToretto + 10 回答的很好!

查看全部评分

回复

使用道具 举报

🔗
JustSoSo 2017-6-20 07:09:33 | 只看该作者
全局:
NikoToretto 发表于 2017-6-19 07:42
过量的简化其实不是好习惯。if后接一行语句不用{}括起来多不安全,感觉就像是ML不戴Condom。

墙都不服就服你 LOL

评分

参与人数 1大米 +10 收起 理由
NikoToretto + 10 回答的很好!

查看全部评分

回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号
隐私提醒:
  • ☑ 禁止发布广告,拉群,贴个人联系方式:找人请去🔗同学同事飞友,拉群请去🔗拉群结伴,广告请去🔗跳蚤市场,和 🔗租房广告|找室友
  • ☑ 论坛内容在发帖 30 分钟内可以编辑,过后则不能删帖。为防止被骚扰甚至人肉,不要公开留微信等联系方式,如有需求请以论坛私信方式发送。
  • ☑ 干货版块可免费使用 🔗超级匿名:面经(美国面经、中国面经、数科面经、PM面经),抖包袱(美国、中国)和录取汇报、定位选校版
  • ☑ 查阅全站 🔗各种匿名方法

本版积分规则

>
快速回复 返回顶部 返回列表