就目前而言,这个问题不适合我们的问答形式。我们希望答案得到事实、引用或专业知识的支持,但这个问题可能会引起辩论、争论、投票或扩展讨论。如果您觉得这个问题可以改进并可能重新打开,visit the help center为指导。
8年前关闭。
锁定。这个问题及其答案是locked因为这个问题是题外话,但具有历史意义。它目前不接受新的答案或互动。
我正在尝试建立 PHP CodeSniffer 的想法在我们的持续集成服务器上,以提高我们代码库的质量。阅读文档后,我对规范化和执行我们的编码标准的想法感到非常兴奋。但是,我对我们产品的实际改进感到疑惑。我很清楚嗅探器只检测违反定义的编码标准,但是干净、一致的代码库提供什么类型的好处?用 10 万多行代码重构一个项目以符合 PEAR 标准是否值得付出额外的努力?
对于那些不熟悉 PHP CodeSniffer 或一般代码气味的人,这里是一个示例输出:
FILE: /path/to/code/myfile.php
FOUND 5 ERROR(S) AFFECTING 2 LINE(S)
--
2 | ERROR | Missing file doc comment
20 | ERROR | PHP keywords must be lowercase; expected "false" but found "FALSE"
47 | ERROR | Line not indented correctly; expected 4 spaces but found 1
51 | ERROR | Missing function doc comment
88 | ERROR | Line not indented correctly; expected 9 spaces but found 6
最佳答案
首先,我是 PHP_CodeSniffer 的维护者,所以我显然对这个领域有偏见。但在我作为 PHP 开发人员的 10 年里,我也研究过一些大型代码库,所以我希望我能提供一些具体的理由来说明为什么编码标准是一件好事。我可以写一个关于这个主题的博客系列,但我只会给你一个关于 PHP_CodeSniffer 如何产生的小故事,这样你就可以理解这个工具为我解决的问题。
我从事过一些大型 CMS 项目。第一个背后有一堆代码和一个相对较小的开发团队。我们没有标准。但我们没有真正的问题。团队很小,在一起呆了很长时间。我们彼此都习惯了。
然后我们建立了一个新的CMS。我们从几个开发人员开始。当时我是一个只有两个开发人员的团队的一员。同样,编码标准没有给我们带来任何问题。我和另一个开发人员来自相同的背景,并且已经制定了一些我们遵循的指导方针。那时我们不需要 PHPCS。
但是那个团队一次培养了一个开发人员,最终达到了 12 名全职开发人员,而且来来去去的人也不少。有些来自旧的 CMS,有些来自公司外部。所有人都有不同的背景和不同的发展方式。很明显谁写了什么代码,因为风格是如此不同。每当你处理一些复杂的事情时,你首先必须适应他们的风格,因为这不是你习惯看到代码的方式。这就像第一次阅读莎士比亚。您需要习惯它,然后才能以自然的速度阅读。
对于开发人员来说,不得不停下来找出另一种编码风格的额外时间纯粹是浪费时间。当您因间距、缩进和支架位置而陷入困境时,这是一个想法溜走的机会。归根结底,这些事情都无关紧要。但是让我告诉你,如果它们导致开发人员中断他们的流程,它们就很重要。所以我们需要一种方法来让他们避开障碍,让开发人员做他们最擅长的事情。
与此同时,我们更多地研究了 JavaScript。一种新语言,风格通常被抛出窗外。代码是从示例站点复制/粘贴并混合在一起的。当学习用一种新语言开发复杂的代码时,找到一种方法让我们的 JS 看起来类似于我们的 PHP 是有意义的。我们可以稍后将其最小化,但我们需要能够在语言之间快速切换,再次保持我们的流程。
所以 PHP_CodeSniffer 就是为了做到这一点而生的。它帮助开发人员使用相同的编码风格,使格式和其他诱饵问题完全摆脱困境。它允许你在某种程度上像对待 PHP 一样对待你的 JS。我使用它来检测特定于产品的气味,例如未翻译的字符串或开发人员未使用我们正确的类包含代码。我还将它用于特定于语言的气味,例如确保不会留下杀死 IE 的 JS 逗号。您可以随心所欲地使用它。它带有大量嗅探,可以使用 XML ruleset file 轻松合并在一起。 .你也可以自己写。您可以集成第 3 方工具,使其成为静态代码分析的一站式商店。您可以根据自己的喜好认真对待标准和代码异味。
PHP_CodeSniffer 与任何开发工具一样,应该适合您。你不为它工作。如果它产生太多您不关心的错误,请自定义标准以删除您不想要的错误,或者将错误转换为警告。但是,如果我的故事听起来像您正在经历或将来可能经历的事情,那么值得仔细研究 PHP_CodeSniffer,看看它是否可以帮助您。
我希望这能帮助您和其他人理解为什么编码标准对某些项目和开发人员非常重要。这不是关于细节。这是关于从导致开发人员失去焦点的事情列表中删除编码风格。
关于php - PHP CodeSniffer 有多大用处?代码标准执行的一般情况?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/982333/
我在使用omniauth/openid时遇到了一些麻烦。在尝试进行身份验证时,我在日志中发现了这一点:OpenID::FetchingError:Errorfetchinghttps://www.google.com/accounts/o8/.well-known/host-meta?hd=profiles.google.com%2Fmy_username:undefinedmethod`io'fornil:NilClass重要的是undefinedmethodio'fornil:NilClass来自openid/fetchers.rb,在下面的代码片段中:moduleNetclass
如何在buildr项目中使用Ruby?我在很多不同的项目中使用过Ruby、JRuby、Java和Clojure。我目前正在使用我的标准Ruby开发一个模拟应用程序,我想尝试使用Clojure后端(我确实喜欢功能代码)以及JRubygui和测试套件。我还可以看到在未来的不同项目中使用Scala作为后端。我想我要为我的项目尝试一下buildr(http://buildr.apache.org/),但我注意到buildr似乎没有设置为在项目中使用JRuby代码本身!这看起来有点傻,因为该工具旨在统一通用的JVM语言并且是在ruby中构建的。除了将输出的jar包含在一个独特的、仅限ruby
在rails源中:https://github.com/rails/rails/blob/master/activesupport/lib/active_support/lazy_load_hooks.rb可以看到以下内容@load_hooks=Hash.new{|h,k|h[k]=[]}在IRB中,它只是初始化一个空哈希。和做有什么区别@load_hooks=Hash.new 最佳答案 查看rubydocumentationforHashnew→new_hashclicktotogglesourcenew(obj)→new_has
这是在Ruby中设置默认值的常用方法:classQuietByDefaultdefinitialize(opts={})@verbose=opts[:verbose]endend这是一个容易落入的陷阱:classVerboseNoMatterWhatdefinitialize(opts={})@verbose=opts[:verbose]||trueendend正确的做法是:classVerboseByDefaultdefinitialize(opts={})@verbose=opts.include?(:verbose)?opts[:verbose]:trueendend编写Verb
我想在一个没有Sass引擎的类中使用Sass颜色函数。我已经在项目中使用了sassgem,所以我认为搭载会像以下一样简单:classRectangleincludeSass::Script::FunctionsdefcolorSass::Script::Color.new([0x82,0x39,0x06])enddefrender#hamlengineexecutedwithcontextofself#sothatwithintemlateicouldcall#%stop{offset:'0%',stop:{color:lighten(color)}}endend更新:参见上面的#re
我遵循了教程http://gettingstartedwithchef.com/,第1章。我的运行list是"run_list":["recipe[apt]","recipe[phpap]"]我的phpapRecipe默认Recipeinclude_recipe"apache2"include_recipe"build-essential"include_recipe"openssl"include_recipe"mysql::client"include_recipe"mysql::server"include_recipe"php"include_recipe"php::modul
我的主要目标是能够完全理解我正在使用的库/gem。我尝试在Github上从头到尾阅读源代码,但这真的很难。我认为更有趣、更温和的踏脚石就是在使用时阅读每个库/gem方法的源代码。例如,我想知道RubyonRails中的redirect_to方法是如何工作的:如何查找redirect_to方法的源代码?我知道在pry中我可以执行类似show-methodmethod的操作,但我如何才能对Rails框架中的方法执行此操作?您对我如何更好地理解Gem及其API有什么建议吗?仅仅阅读源代码似乎真的很难,尤其是对于框架。谢谢! 最佳答案 Ru
我在用Ruby执行简单任务时遇到了一件奇怪的事情。我只想用每个方法迭代字母表,但迭代在执行中先进行:alfawit=("a".."z")puts"That'sanalphabet:\n\n#{alfawit.each{|litera|putslitera}}"这段代码的结果是:(缩写)abc⋮xyzThat'sanalphabet:a..z知道为什么它会这样工作或者我做错了什么吗?提前致谢。 最佳答案 因为您的each调用被插入到在固定字符串之前执行的字符串文字中。此外,each返回一个Enumerable,实际上您甚至打印它。试试
我的假设是moduleAmoduleBendend和moduleA::Bend是一样的。我能够从thisblog找到解决方案,thisSOthread和andthisSOthread.为什么以及什么时候应该更喜欢紧凑语法A::B而不是另一个,因为它显然有一个缺点?我有一种直觉,它可能与性能有关,因为在更多命名空间中查找常量需要更多计算。但是我无法通过对普通类进行基准测试来验证这一点。 最佳答案 这两种写作方法经常被混淆。首先要说的是,据我所知,没有可衡量的性能差异。(在下面的书面示例中不断查找)最明显的区别,可能也是最著名的,是你的
几个月前,我读了一篇关于rubygem的博客文章,它可以通过阅读代码本身来确定编程语言。对于我的生活,我不记得博客或gem的名称。谷歌搜索“ruby编程语言猜测”及其变体也无济于事。有人碰巧知道相关gem的名称吗? 最佳答案 是这个吗:http://github.com/chrislo/sourceclassifier/tree/master 关于ruby-寻找通过阅读代码确定编程语言的rubygem?,我们在StackOverflow上找到一个类似的问题: