目录
源代码排版
所有风格都又丑又难读,自己的除外。几乎人人都这样想。把“自己的除外”拿掉,他们或许是对的…
——Jerry Coffin(论缩排)
使用UTF-8
作为源文件的编码。
[link]
每个缩排层级使用两个空格。不要使用制表符。
[link]123456789# 差 - 四个空格def some_methoddo_somethingend# 好def some_methoddo_somethingend
使用 Unix 风格的换行符。(*BSD/Solaris/Linux/OS X 系统的用户不需担心,Windows 用户则要格外小心。)
[link]如果你使用 Git,可用下面这个配置来保护你的项目不被 Windows 的换行符干扰:
1$ git config --global core.autocrlf true
不要使用;
隔开语句与表达式。推论:一行一条语句。
[link]123456789101112# 差puts 'foobar'; # 不必要的分号puts 'foo'; puts 'bar' # 一行里有两个表达式# 好puts 'foobar'puts 'foo'puts 'bar'puts 'foo', 'bar' # 仅对 puts 适用
对于没有主体的类,倾向使用单行定义。
[link]123456789# 差class FooError < StandardErrorend# 勉强可以class FooError < StandardError; end# 好FooError = Class.new(StandardError)
定义方法时,避免单行写法。尽管这种写法有时颇为普遍,但其略显古怪的定义语法容易使人犯错。无论如何,至少保证单行写法的方法不应该拥有一个以上的表达式。
[link]12345678910111213141516# 差def too_much; something; something_else; end# 勉强可以 - 注意第一个 ; 是必选的def no_braces_method; body end# 勉强可以 - 注意第二个 ; 是可选的def no_braces_method; body; end# 勉强可以 - 语法正确,但没有 ; 使得可读性欠佳def some_method() body end# 好def some_methodbodyend这个规则的一个例外是空方法。
12# 好def no_op; end
操作符前后适当地添加空格,在逗号,
、冒号:
及分号;
之后。尽管 Ruby 解释器(大部分情况下)会忽略空格,但适量的空格可以增强代码的可读性。
[link]123sum = 1 + 2a, b = 1, 2class FooError < StandardError; end(对于操作符)唯一的例外是当使用指数操作符时:
12345# 差e = M * c ** 2# 好e = M * c**2(
、[
之后,]
、)
之前,不要添加任何空格。在{
前后,在}
之前添加空格。
[link]
|
|
{
与 }
需要额外说明,因为它们可以同时用在区块、哈希字面量及字符串插值中。
对于哈希字面量,有两种可被接受的风格。第一种风格更具可读性(在 Ruby 社区里似乎更为流行)。第二种风格的优点是,在视觉上使得区块与哈希字面量有所区分。无论你选择何种风格,务必在使用时保持连贯性。
|
|
对于插值表达式,括号内两端不要添加空格。
|
|
!
之后,不要添加任何空格。
[link]12345# 差! something# 好!something
范围的字面量语法中,不要添加任何空格。
[link]1234567# 差1 .. 3'a' ... 'z'# 好1..3'a'...'z'
把when
与case
缩排在同一层级。这是《Programming Ruby》与《The Ruby Programming Language》中早已确立的风格。
[link]1234567891011121314151617181920212223# 差casewhen song.name == 'Misty'puts 'Not again!'when song.duration > 120puts 'Too long!'when Time.now.hour > 21puts "It's too late"elsesong.playend# 好casewhen song.name == 'Misty'puts 'Not again!'when song.duration > 120puts 'Too long!'when Time.now.hour > 21puts "It's too late"elsesong.playend
当将一个条件表达式的结果赋值给一个变量时,保持分支缩排在同一层级。
[link]12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849# 差 - 非常费解kind = case yearwhen 1850..1889 then 'Blues'when 1890..1909 then 'Ragtime'when 1910..1929 then 'New Orleans Jazz'when 1930..1939 then 'Swing'when 1940..1950 then 'Bebop'else 'Jazz'endresult = if some_condcalc_somethingelsecalc_something_elseend# 好 - 结构清晰kind = case yearwhen 1850..1889 then 'Blues'when 1890..1909 then 'Ragtime'when 1910..1929 then 'New Orleans Jazz'when 1930..1939 then 'Swing'when 1940..1950 then 'Bebop'else 'Jazz'endresult = if some_condcalc_somethingelsecalc_something_elseend# 好 - 并且更好地利用行宽kind =case yearwhen 1850..1889 then 'Blues'when 1890..1909 then 'Ragtime'when 1910..1929 then 'New Orleans Jazz'when 1930..1939 then 'Swing'when 1940..1950 then 'Bebop'else 'Jazz'endresult =if some_condcalc_somethingelsecalc_something_elseend
在各个方法定义之间添加空行,并且将方法分成若干合乎逻辑的段落。
[link]1234567891011def some_methoddata = initialize(options)data.manipulate!data.resultenddef some_methodresultend
避免在方法调用的最后一个参数之后添加逗号,尤其当参数没有分布在同一行时。
[link]123456789101112# 差 - 尽管移动、新增、删除参数颇为方便,但仍不推荐这种写法some_method(size,count,color,)# 差some_method(size, count, color, )# 好some_method(size, count, color)
当给方法的参数赋予默认值时,在=
前后添加空格。
[link]123456789# 差def some_method(arg1=:default, arg2=nil, arg3=[])# 做一些事情end# 好def some_method(arg1 = :default, arg2 = nil, arg3 = [])# 做一些事情end尽管有几本 Ruby 书籍推荐使用第一种风格,但第二种在实践中更为常见(而且似乎更具可读性)。
避免在非必要的情形下使用续行符\
。在实践中,除了字符串拼接,避免在其他任何地方使用续行。
[link]12345678910# 差result = 1 - \2# 好 - 但仍然丑到爆result = 1 \- 2long_string = 'First part of the long string' \' and second part of the long string'
使用统一的风格进行多行链式方法调用。在 Ruby 社区中存在两种流行的风格:前置.
(风格 A)与后置.
(风格 B)。
[link](风格 A) 当多行链式方法调用需要另起一行继续时,将
.
放在第二行开头。1234567# 差 - 需要查看第一行才能理解第二行在做什么one.two.three.four# 好 - 立刻能够明白第二行在做什么one.two.three.four(风格 B) 将
.
放在第一行末尾,以表示当前表达式尚未结束。1234567# 差 - 需要查看第二行才能知道链式方法调用是否结束one.two.three.four# 好 - 立刻能够明白第二行还有其他方法调用one.two.three.four
两种风格各自优点查阅这里。
当方法调用参数过长时,将它们排列在多行并对齐。若对齐后长度超过行宽限制,将首个参数位置挪到下一行进行缩排也是可以接受的。
[link]12345678910111213141516171819202122232425262728293031# 初始(行太长了)def send_mail(source)Mailer.deliver(to: 'bob@example.com', from: 'us@example.com', subject: 'Important message', body: source.text)end# 差 - 双倍缩排def send_mail(source)Mailer.deliver(to: 'bob@example.com',from: 'us@example.com',subject: 'Important message',body: source.text)end# 好def send_mail(source)Mailer.deliver(to: 'bob@example.com',from: 'us@example.com',subject: 'Important message',body: source.text)end# 好 - 普通缩排def send_mail(source)Mailer.deliver(to: 'bob@example.com',from: 'us@example.com',subject: 'Important message',body: source.text)end
当构建数组时,若元素跨行,应当保持对齐。
[link]1234567891011121314# 差 - 没有对齐menu_item = ['Spam', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam','Baked beans', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam']# 好menu_item = ['Spam', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam','Baked beans', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam']# 好menu_item =['Spam', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam','Baked beans', 'Spam', 'Spam', 'Spam', 'Spam', 'Spam']
使用_
语法改善大数的数值字面量的可读性。
[link]12345# 差 - 有几个零?num = 1000000# 好 - 方便人脑理解num = 1_000_000
使用 [RDoc][rdoc] 及其惯例来编写 API 文档。注意,不要在注释与def
之间添加空行。
[link]
将单行长度控制在 80 个字符内。
[link]
避免行尾空格。
[link]
文件以空白行结束。
[link]
不要使用区块注释。它们不能被空白字符引导,且不如常规注释容易辨认。
[link]123456789# 差=begincomment lineanother comment line=end# 好# comment line# another comment line
语法
使用::
引用常量(包括类与模块)与构造器(比如Array()
、Nokogiri::HTML()
)。不要使用::
调用常规方法。
[link]123456789# 差SomeClass::some_methodsome_object::some_method# 好SomeClass.some_methodsome_object.some_methodSomeModule::SomeClass::SOME_CONSTSomeModule::SomeClass()
使用def
定义方法时,如果有参数则使用括号,如果无参数则省略括号。
[link]12345678910111213141516171819# 差def some_method()# 省略主体end# 好def some_method# 省略主体end# 差def some_method_with_parameters param1, param2# 省略主体end# 好def some_method_with_parameters(param1, param2)# 省略主体end
定义可选参数时,将可选参数放置在参数列表尾部。如果可选参数出现在列表头部,则此方法在调用时可能会产生预期之外的结果。
[link]1234567891011121314151617# 差def some_method(a = 1, b = 2, c, d)puts "#{a}, #{b}, #{c}, #{d}"endsome_method('w', 'x') # => '1, 2, w, x'some_method('w', 'x', 'y') # => 'w, 2, x, y'some_method('w', 'x', 'y', 'z') # => 'w, x, y, z'# 好def some_method(c, d, a = 1, b = 2)puts "#{a}, #{b}, #{c}, #{d}"endsome_method('w', 'x') # => '1, 2, w, x'some_method('w', 'x', 'y') # => 'y, 2, w, x'some_method('w', 'x', 'y', 'z') # => 'y, z, w, x'
定义变量时,避免并行赋值。但当右值为方法调用返回值,或是与*
操作符配合使用,或是交换两个变量的值,并行赋值也是可以接受的。并行赋值的可读性通常不如分开赋值。
[link]123456789101112131415161718192021222324252627282930# 差a, b, c, d = 'foo', 'bar', 'baz', 'foobar'# 好a = 'foo'b = 'bar'c = 'baz'd = 'foobar'# 好 - 交换两个变量的值a = 'foo'b = 'bar'a, b = b, aputs a # => 'bar'puts b # => 'foo'# 好 - 右值为方法调用返回值def multi_return[1, 2]endfirst, second = multi_return# 好 - 与 * 操作符配合使用first, *list = [1, 2, 3, 4] # first => 1, list => [2, 3, 4]hello_array = *'Hello' # => ["Hello"]a = *(1..3) # => [1, 2, 3]
除非必要,否则避免在并行赋值时使用单字符的_
变量。优先考虑前缀形式的下划线变量,而不是直接使用_
,因为前者可以提供一定的语义信息。但当赋值语句左侧出现带*
操作符的变量时,使用_
也是可以接受的。
[link]12345678910111213141516171819foo = 'one,two,three,four,five'# 差 - 可有可无,且无任何有用信息first, second, _ = foo.split(',')first, _, _ = foo.split(',')first, *_ = foo.split(',')# 好a, = foo.split(',')a, b, = foo.split(',')# 好 - 可有可无,但提供了额外信息first, _second = foo.split(',')first, _second, = foo.split(',')first, *_ending = foo.split(',')# 好 - 占位符,_ 担当最后一个元素*beginning, _ = foo.split(',')*beginning, something, _ = foo.split(',')
永远不要使用for
, 除非你很清楚为什么。大部分情况下,你应该使用迭代器。for
是由each
实现的,所以你绕弯了。另外,for
没有引入一个新的作用域 (each
有),因此在它内部定义的变量在外部仍是可见的。
[link]123456789101112131415arr = [1, 2, 3]# 差for elem in arr doputs elemend# 注意,elem 可在 for 循环外部被访问elem # => 3# 好arr.each { |elem| puts elem }# 注意,elem 不可在 each 块外部被访问elem # => NameError: undefined local variable or method `elem'
永远不要在多行if/unless
中使用then
。
[link]123456789# 差if some_condition then# 省略主体end# 好if some_condition# 省略主体end
在多行if/unless
中,总是把条件表达式与if/unless
放置在同一行。
[link]123456789101112# 差ifsome_conditiondo_somethingdo_something_elseend# 好if some_conditiondo_somethingdo_something_elseend
倾向使用三元操作符(?:
)而不是if/then/else/end
结构。前者更为常见且简练。
[link]12345# 差result = if some_condition then something else something_else end# 好result = some_condition ? something : something_else
三元操作符的每个分支只写一个表达式。即不要嵌套三元操作符。嵌套情况请使用if/else
结构。
[link]123456789# 差some_condition ? (nested_condition ? nested_something : nested_something_else) : something_else# 好if some_conditionnested_condition ? nested_something : nested_something_elseelsesomething_elseend
永远不要使用if x; ...
。使用三元操作符来替代。
[link]12345# 差result = if some_condition; something else something_else end# 好result = some_condition ? something : something_else
利用“if
与case
是表达式”的这个特性。
[link]1234567891011121314# 差if conditionresult = xelseresult = yend# 好result =if conditionxelseyend
在case
表达式中,单行情况使用when x then ...
语法。另一种语法when x: ...
在 Ruby 1.9 之后被移除了。
[link]
不要使用when x; ...
语法。参考前一条规则。
[link]
使用!
而不是not
。
[link]12345# 差 - 因为操作符的优先级,这里必须使用括号x = (not something)# 好x = !something
避免使用!!
。
[link]12345678910111213141516# 差x = 'test'# 令人费解的 nil 检查if !!x# 省略主体endx = false# 对于布尔变量,双重否定是多余的!!x # => false# 好x = 'test'unless x.nil?# 省略主体end
永远不要使用and
与or
关键字。使用&&
与||
来替代。
[link]1234567891011121314151617# 差# 布尔表达式if some_condition and some_other_conditiondo_somethingend# 流程控制document.saved? or document.save!# 好# 布尔表达式if some_condition && some_other_conditiondo_somethingend# 流程控制document.saved? || document.save!
避免使用多行三元操作符(?:
)。使用if/unless
来替代。
[link]
对于单行主体,倾向使用if/unless
修饰语法。另一种方法是使用流程控制&&/||
。
[link]12345678910# 差if some_conditiondo_somethingend# 好do_something if some_condition# 好 - 使用流程控制some_condition && do_something
避免在多行区块后使用if/unless
修饰语法。
[link]1234567891011# 差10.times do# 省略多行主体end if some_condition# 好if some_condition10.times do# 省略多行主体endend
避免使用嵌套if/unless/while/until
修饰语法。适当情况下,使用&&/||
来替代。
[link]12345# 差do_something if other_condition if some_condition# 好do_something if some_condition && other_condition
对于否定条件,倾向使用unless
而不是if
(或是使用流程控制||
)。
[link]1234567891011# 差do_something if !some_condition# 差do_something if not some_condition# 好do_something unless some_condition# 好some_condition || do_something
不要使用unless
与else
的组合。将它们改写成肯定条件。
[link]12345678910111213# 差unless success?puts 'failure'elseputs 'success'end# 好if success?puts 'success'elseputs 'failure'end
不要使用括号包裹if/unless/while/until
的条件表达式。
[link]123456789# 差if (x > 10)# 省略主体end# 好if x > 10# 省略主体end这个规则的一个例外是条件表达式中的安全赋值。
在多行while/until
中,不要使用while/until condition do
。
[link]1234567891011121314151617# 差while x > 5 do# 省略主体enduntil x > 5 do# 省略主体end# 好while x > 5# 省略主体enduntil x > 5# 省略主体end
对于单行主体,倾向使用while/until
修饰语法。
[link]1234567# 差while some_conditiondo_somethingend# 好do_something while some_condition
对于否定条件,倾向使用until
而不是while
。
[link]12345# 差do_something while !some_condition# 好do_something until some_condition
对于无限循环,使用Kernel#loop
而不是while/until
。
[link]12345678910111213# 差while truedo_somethingenduntil falsedo_somethingend# 好loop dodo_somethingend
对于后置条件循环语句,倾向使用Kernel#loop
与break
的组合,而不是begin/end/until
或begin/end/while
。
[link]123456789101112# 差beginputs valval += 1end while val < 0# 好loop doputs valval += 1break unless val < 0end
对于 DSL 的内部方法(比如 Rake、Rails、RSpec)、具有“关键字”特性的内部方法(比如attr_reader
、puts
)、以及属性存取方法,省略这些方法外围的括号。所有其他的方法调用则使用括号。
[link]123456789101112131415161718192021class Personattr_reader(:name, :age) # 差attr_reader :name, :age # 好# 省略主体endtemperance = Person.new 'Temperance', 30 # 差temperance = Person.new('Temperance', 30) # 好puts(temperance.age) # 差puts temperance.age # 好x = Math.sin y # 差x = Math.sin(y) # 好array.delete e # 差array.delete(e) # 好expect(bowling.score).to eq 0 # 差expect(bowling.score).to eq(0) # 好
对于可选参数的哈希,省略其外围的花括号。
[link]12345# 差user.set({ name: 'John', age: 45, permissions: { read: true } })# 好user.set(name: 'John', age: 45, permissions: { read: true })
对于 DSL 的内部方法调用,同时省略其外围的圆括号与花括号。
[link]1234567class Person < ActiveRecord::Base# 差validates(:name, { presence: true, length: { within: 1..10 } })# 好validates :name, presence: true, length: { within: 1..10 }end
无参调用方法时,省略括号。
[link]1234567891011# 差Kernel.exit!()2.even?()fork()'test'.upcase()# 好Kernel.exit!2.even?fork'test'.upcase
当被调用方法是当前区块中唯一操作时,倾向使用简短的传参语法。
[link]12345# 差names.map { |name| name.upcase }# 好names.map(&:upcase)
对于单行主体,倾向使用{...}
而不是do...end
。对于多行主体,避免使用{...}
。对于“流程控制”或“方法定义”(比如 Rakefile、其他 DSL 构成片段),总是使用do...end
。避免在链式方法调用中使用do...end
。
[link]1234567891011121314151617names = %w(Bozhidar Steve Sarah)# 差names.each do |name|puts nameend# 好names.each { |name| puts name }# 差names.select do |name|name.start_with?('S')end.map { |name| name.upcase }# 好names.select { |name| name.start_with?('S') }.map(&:upcase)某些人可能会争论在多行链式方法调用时使用
{...}
看起来还可以。但他们应该扪心自问——这样的代码真的可读吗?难道不能把区块内容提取出来放到小巧的方法里吗?
优先考虑使用显式区块参数,以避免某些情况下通过创建区块的手法来传递参数给其他区块。此规则对性能有所影响,因为区块会被转换为Proc
对象。
[link]12345678910111213141516171819require 'tempfile'# 差def with_tmp_dirDir.mktmpdir do |tmp_dir|Dir.chdir(tmp_dir) { |dir| yield dir } # 通过创建区块的手法来传递参数endend# 好def with_tmp_dir(&block)Dir.mktmpdir do |tmp_dir|Dir.chdir(tmp_dir, &block)endendwith_tmp_dir do |dir|puts "dir is accessible as a parameter and pwd is set: #{dir}"end
避免在不需要流程控制的情况下使用return
。
[link]123456789# 差def some_method(some_arr)return some_arr.sizeend# 好def some_method(some_arr)some_arr.sizeend
避免在不需要的情况下使用self
。(只有在调用self
的修改器时才需要)
[link]1234567891011121314151617# 差def ready?if self.last_reviewed_at > self.last_updated_atself.worker.update(self.content, self.options)self.status = :in_progressendself.status == :verifiedend# 好def ready?if last_reviewed_at > last_updated_atworker.update(content, options)self.status = :in_progressendstatus == :verifiedend
避免局部变量遮蔽方法调用,除非它们具有相同效果。
[link]1234567891011121314151617181920212223class Fooattr_accessor :options# 勉强可以def initialize(options)self.options = options# 此处 self.options 与 options 具有相同效果end# 差def do_something(options = {})unless options[:when] == :lateroutput(self.options[:message])endend# 好def do_something(params = {})unless params[:when] == :lateroutput(options[:message])endendend
不要在条件表达式中使用=
(赋值语句)的返回值,除非赋值语句包裹在括号之中。这种惯用法被称作条件表达式中的安全赋值。
[link]123456789101112131415161718# 差 - 会出现警告if v = array.grep(/foo/)do_something(v)...end# 好 - 尽管 Ruby 解释器仍会出现警告,但 RuboCop 不会if (v = array.grep(/foo/))do_something(v)...end# 好v = array.grep(/foo/)if vdo_something(v)...end
优先考虑简短的自我赋值语法。
[link]123456789101112131415# 差x = x + yx = x * yx = x**yx = x / yx = x || yx = x && y# 好x += yx *= yx **= yx /= yx ||= yx &&= y
当变量尚未初始化时,使用||=
对其进行初始化。
[link]12345678# 差name = name ? name : 'Bozhidar'# 差name = 'Bozhidar' unless name# 好 - 当且仅当 name 为 nil 或 false 时,设置 name 的值为 'Bozhidar'name ||= 'Bozhidar'
不要使用||=
对布尔变量进行初始化。
[link]12345# 差 - 设置 enabled 的值为 true,即使其原本的值是 falseenabled ||= true# 好enabled = true if enabled.nil?
使用&&=
预先检查变量是否存在,如果存在,则做相应动作。使用&&=
语法可以省去if
检查。
[link]12345678910111213141516# 差if somethingsomething = something.downcaseend# 差something = something ? something.downcase : nil# 勉强可以something = something.downcase if something# 好something = something && something.downcase# 更好something &&= something.downcase
避免使用case
语句等价操作符===
。从名称可知,这是case
语句隐式使用的操作符,在case
语句外的场合中使用,会产生难以理解的代码。
[link]123456789# 差Array === something(1..100) === 7/something/ === some_string# 好something.is_a?(Array)(1..100).include?(7)some_string =~ /something/
能使用==
就不要使用eql?
。提供更加严格比较的eql?
在实践中极少使用。
[link]123456# 差 - 对于字符串,eql? 与 == 具有相同效果'ruby'.eql? some_str# 好'ruby' == some_str1.0.eql? x # 当需要区别 Fixnum 1 与 Float 1.0 时,eql? 是具有意义的
避免使用 Perl 风格的特殊变量(比如$:
、$;
等)。它们看起来非常神秘,但除了单行脚本,其他情况并不鼓励使用。建议使用English
程序库提供的友好别名。
[link]123456# 差$:.unshift File.dirname(__FILE__)# 好require 'English'$LOAD_PATH.unshift File.dirname(__FILE__)
永远不要在方法名与左括号之间添加空格。
[link]12345# 差f (3 + 2) + 1# 好f(3 + 2) + 1
如果方法调用的第一个参数以左括号开头,则此方法调用应该使用括号,比如f((3 + 2) + 1)
。
[link]
运行 Ruby 解释器时,总是开启-w
选项来。如果你忘了某个上述某个规则,它就会警告你!
[link]
不要在方法中嵌套定义方法,使用 lambda 方法来替代。 嵌套定义产生的方法,事实上和外围方法处于同一作用域(比如类作用域)。此外,“嵌套方法”会在定义它的外围方法每次调用时被重新定义。
[link]1234567891011121314151617181920212223# 差def foo(x)def bar(y)# 省略主体endbar(x)end# 好 - 作用同前,但 bar 不会在 foo 每次调用时被重新定义def bar(y)# 省略主体enddef foo(x)bar(x)end# 好def foo(x)bar = ->(y) { ... }bar.call(x)end
对于单行区块,使用新的 lambda 字面量定义语法。对于多行区块,使用lambda
定义语法。
[link]123456789101112131415161718# 差l = lambda { |a, b| a + b }l.call(1, 2)# 好 - 但看起来怪怪的l = ->(a, b) dotmp = a * 7tmp * b / 50end# 好l = ->(a, b) { a + b }l.call(1, 2)l = lambda do |a, b|tmp = a * 7tmp * b / 50end
定义 lambda 方法时,如果有参数则使用括号。
[link]12345# 差l = ->x, y { something(x, y) }# 好l = ->(x, y) { something(x, y) }
定义 lambda 方法时,如果无参数则省略括号。
[link]12345# 差l = ->() { something }# 好l = -> { something }
倾向使用proc
而不是Proc.new
。
[link]12345# 差p = Proc.new { |n| puts n }# 好p = proc { |n| puts n }
对于 lambda 方法或代码块,倾向使用proc.call()
而不是proc[]
或proc.()
。
[link]1234567891011# 差 - 看上去像是枚举器的存取操作l = ->(v) { puts v }l[1]# 差 - 极少见的调用语法l = ->(v) { puts v }l.(1)# 好l = ->(v) { puts v }l.call(1)
未被使用的区块参数或局部变量,添加_
前缀或直接使用_
(尽管表意性略差)。这种做法可以抑制 Ruby 解释器或 RuboCop 等工具发出“变量尚未使用”的警告。
[link]1234567891011121314151617181920212223# 差result = hash.map { |k, v| v + 1 }def something(x)unused_var, used_var = something_else(x)# ...end# 好result = hash.map { |_k, v| v + 1 }def something(x)_unused_var, used_var = something_else(x)# ...end# 好result = hash.map { |_, v| v + 1 }def something(x)_, used_var = something_else(x)# ...end
使用$stdout/$stderr/$stdin
而不是STDOUT/STDERR/STDIN
。STDOUT/STDERR/STDIN
是常量,尽管在 Ruby 中允许给常量重新赋值(可能是重定向某些流),但解释器会发出警告。
[link]
使用warn
而不是$stderr.puts
。除了更加简练清晰外,warn
允许你在需要时通过设置解释器选项(使用-W0
将警告级别设置为 0)来抑制警告。
[link]
倾向使用sprintf
或其别名format
而不是相当晦涩的String#%
方法。
[link]123456789101112131415161718# 差'%d %d' % [20, 10]# => '20 10'# 好sprintf('%d %d', 20, 10)# => '20 10'# 好sprintf('%{first} %{second}', first: 20, second: 10)# => '20 10'format('%d %d', 20, 10)# => '20 10'# 好format('%{first} %{second}', first: 20, second: 10)# => '20 10'
倾向使用Array#join
而不是相当晦涩的带字符参数的Array#*
方法。
[link]1234567# 差%w(one two three) * ', '# => 'one, two, three'# 好%w(one two three).join(', ')# => 'one, two, three'
当你希望处理的变量类型是数组,但不太确定其是否真的是数组时,通过使用Array()
来替代显式的数组类型检查与转换。
[link]
|
|
通过使用范围或Comparable#between?
来替代复杂的比较逻辑。
[link]12345678# 差do_something if x >= 1000 && x <= 2000# 好do_something if (1000..2000).include?(x)# 好do_something if x.between?(1000, 2000)
倾向使用谓词方法而不是==
操作符。但数值比较除外。
[link]12345678910111213141516171819202122232425# 差if x % 2 == 0endif x % 2 == 1endif x == nilend# 好if x.even?endif x.odd?endif x.nil?endif x.zero?endif x == 0end
不做显式的non-nil
检查,除非检查对象是布尔变量。
[link]1234567891011# 差do_something if !something.nil?do_something if something != nil# 好do_something if something# 好 - 检查对象是布尔变量def value_set?!@some_boolean.nil?end
避免使用BEGIN
区块。
[link]
永远不要使用END
区块。使用Kernel#at_exit
来替代。
[link]12345# 差END { puts 'Goodbye!' }# 好at_exit { puts 'Goodbye!' }
避免使用 flip-flops 操作符。
[link]
流程控制中,避免使用嵌套条件。
[link]倾向使用防御从句进行非法数据断言。防御从句是指处于方法顶部的条件语句,其能尽早地退出方法。
12345678910111213141516171819# 差def compute_thing(thing)if thing[:foo]update_with_bar(thing)if thing[:foo][:bar]partial_compute(thing)elsere_compute(thing)endendend# 好def compute_thing(thing)return unless thing[:foo]update_with_bar(thing[:foo])return re_compute(thing) unless thing[:foo][:bar]partial_compute(thing)end循环中,倾向使用
next
而不是条件区块。123456789101112# 差[0, 1, 2, 3].each do |item|if item > 1puts itemendend# 好[0, 1, 2, 3].each do |item|next unless item > 1puts itemend
倾向使用map
而不是collect
,find
而不是detect
,select
而不是find_all
,reduce
而不是inject
以及size
而不是length
。这不是一个硬性要求,如果使用别名可以增强可读性,使用它也没关系。这些别名方法继承自 Smalltalk 语言,但在别的语言并不通用。鼓励使用select
而不是find_all
的理由是前者与reject
搭配起来一目了然。
[link]
不要使用count
作为size
的替代方案。除了Array
外,其他Enumerable
对象都需要通过枚举整个集合才可以确定数目。
[link]12345# 差some_hash.count# 好some_hash.size
倾向使用flat_map
而不是map + flatten
的组合。此规则并不适用于深度超过 2 层的数组。举例来说,如果users.first.songs == ['a', ['b','c']]
成立,则使用map + flatten
的组合而不是flat_map
。flat_map
只能平坦化一个层级,而flatten
能够平坦化任意多个层级。
[link]12345# 差all_songs = users.map(&:songs).flatten.uniq# 好all_songs = users.flat_map(&:songs).uniq
倾向使用reverse_each
而不是reverse.each
,因为某些混入Enumerable
模块的类可能会提供reverse_each
的高效版本。即使这些类没有提供专门特化的版本,继承自Enumerable
的通用版本至少能保证性能与reverse.each
相当。
[link]12345# 差array.reverse.each { ... }# 好array.reverse_each { ... }
命名
程序设计的真正难题是替事物命名及使缓存失效。
——Phil Karlton
标识符使用英文命名。
[link]12345678# 差 - 标识符使用非 ASCII 字符заплата = 1_000# 差 - 标识符使用拉丁文写法的保加利亚单词zaplata = 1_000# 好salary = 1_000
符号、方法、变量使用蛇底式小写(snake_case
)。
[link]123456789101112131415161718192021# 差:'some symbol':SomeSymbol:someSymbolsomeVar = 5def someMethod...enddef SomeMethod...end# 好:some_symboldef some_method...end
类与模块使用驼峰式大小写(CamelCase
)。(HTTP、RFC、XML 等首字母缩写应该仍旧保持大写形式)
[link]1234567891011121314151617181920212223242526272829# 差class Someclass...endclass Some_Class...endclass SomeXml...endclass XmlSomething...end# 好class SomeClass...endclass SomeXML...endclass XMLSomething...end
文件名使用蛇底式小写,如hello_world.rb
。
[link]
目录名使用蛇底式小写,如lib/hello_world/hello_world.rb
。
[link]
尽量使一个源文件中只有一个类或模块。文件名就是类名或模块名,但使用蛇底式小写而不是驼峰式大小写。
[link]
其他常量使用尖叫蛇底式大写(SCREAMING_SNAKE_CASE)。
[link]12345# 差SomeConst = 5# 好SOME_CONST = 5
谓词方法(返回布尔值的方法)的名字应当以问号结尾。(比如Array#empty?
)。不返回布尔值的方法不应以问号结尾。
[link]
谓词方法的名字应当避免使用is
、does
、can
等助动词作为前缀。这些助动词在实际场景中显得冗余,且与标准库的命名习惯(比如empty?
、include?
)很不一致。
[link]1234567891011121314151617181920212223242526272829# 差class Persondef is_tall?trueenddef can_play_basketball?falseenddef does_like_candy?trueendend# 好class Persondef tall?trueenddef basketball_player?falseenddef likes_candy?trueendend
具有潜在危险性的方法,当其存在对应安全版本的方法时,其名字应当以惊叹号结尾。(比如修改self
或参数值的方法、相对exit
方法不会在退出时运行 finalizers 执行清理工作的exit!
方法等)
[link]1234567891011121314151617181920# 差 - 没有对应安全版本的方法class Persondef update!endend# 好class Persondef updateendend# 好class Persondef update!enddef updateendend
尽量根据危险方法来定义对应安全版本的方法。
[link]123456789101112131415class Arraydef flatten_once!res = []each do |e|[*e].each { |f| res << f }endreplace(res)enddef flatten_oncedup.flatten_once!endend
当配合单行区块使用reduce
时,将参数命名为|a, e|
(accumulator/累加器,element/元素)。
[link]
当定义二元操作符时,将参数命名为other
(<<
与[]
例外,因为其语义与此不同)。
[link]123def +(other)# 省略主体end
注释
良好的代码自身就是最佳的文档。当你要添加一个注释时,扪心自问,“如何改善代码让它不需要注释?” 改善代码,再写相应文档使之更清楚。
——Steve McConnell
编写让人一目了然的代码然后忽略这一节的其它部分。我是认真的!
[link]
使用英文编写注释。
[link]
前导#
与注释文本之间应当添加一个空格。
[link]
避免无谓的注释。
[link]12# 差counter += 1 # Increments counter by one.
及时更新注释。过时的注释比没有注释还要糟糕。
[link]
好的代码就像是好的笑话 —— 它不需要解释。
——Russ Olsen
避免替烂代码编写注释。重构它们使其变得一目了然。(要么做,要么不做,不要只是试试看。——Yoda)
[link]
注解
注解应该直接写在相关代码之前那行。
[link]
注解关键字后面,跟着一个冒号及空格,接着是描述问题的文本。
[link]
如果需要用多行来描述问题,后续行要放在#
后面并缩排两个空格。
[link]12345def bar# FIXME: This has crashed occasionally since v3.2.1. It may# be related to the BarBazUtil upgrade.baz(:quux)end
当问题是显而易见时,任何文档都是多余的,注解应当放在有问题的那行末尾且不带任何多余说明。这个用法应该算是例外而不是规则。
[link]123def barsleep 100 # OPTIMIZEend
使用TODO
标记应当加入的特征与功能。
[link]
使用FIXME
标记需要修复的代码。
[link]
使用OPTIMIZE
标记可能引发性能问题的低效代码。
[link]
使用HACK
标记代码异味,即那些应当被重构的可疑编码习惯。
[link]
使用REVIEW
标记需要确认与编码意图是否一致的可疑代码。比如,REVIEW: Are we sure this is how the client does X currently?
。
[link]
适当情况下,可以自行定制其他注解关键字,但别忘记在项目的README
或类似文档中予以说明。
[link]
类与模块
在类定义中,使用一致的结构。
[link]12345678910111213141516171819202122232425262728293031323334353637383940class Person# 首先是 extend 与 includeextend SomeModuleinclude AnotherModule# 内部类CustomErrorKlass = Class.new(StandardError)# 接着是常量SOME_CONSTANT = 20# 接下来是属性宏attr_reader :name# 跟着是其他宏(如果有的话)validates :name# 公开的类方法接在下一行def self.some_methodend# 初始化方法在类方法和实例方法之间def initializeend# 跟着是公开的实例方法def some_methodend# 受保护及私有的方法等放在接近结尾的地方protecteddef some_protected_methodendprivatedef some_private_methodendend
如果嵌套类数目较多,进而导致外围类定义较长,则将它们从外围类中提取出来,分别放置在单独的以嵌套类命名的文件中,并将文件归类至以外围类命名的文件夹下。
[link]1234567891011121314151617181920212223242526272829303132333435# 差# foo.rbclass Fooclass Bar# 定义 30 多个方法endclass Car# 定义 20 多个方法end# 定义 30 多个方法end# 好# foo.rbclass Foo# 定义 30 多个方法end# foo/bar.rbclass Fooclass Bar# 定义 30 多个方法endend# foo/car.rbclass Fooclass Car# 定义 20 多个方法endend
定义只有类方法的数据类型时,倾向使用模块而不是类。只有当需要实例化时才使用类。
[link]1234567891011121314151617181920212223# 差class SomeClassdef self.some_method# 省略主体enddef self.some_other_method# 省略主体endend# 好module SomeModulemodule_functiondef some_method# 省略主体enddef some_other_method# 省略主体endend
当你想将模块的实例方法变成类方法时,倾向使用module_function
而不是extend self
。
[link]12345678910111213141516171819202122232425# 差module Utilitiesextend selfdef parse_something(string)# 做一些事情enddef other_utility_method(number, string)# 做一些事情endend# 好module Utilitiesmodule_functiondef parse_something(string)# 做一些事情enddef other_utility_method(number, string)# 做一些事情endend
总是替那些用以表示领域模型的类提供一个适当的to_s
方法。
[link]123456789101112class Personattr_reader :first_name, :last_namedef initialize(first_name, last_name)@first_name = first_name@last_name = last_nameenddef to_s"#{@first_name} #{@last_name}"endend
使用attr
系列方法来定义琐碎的存取器或修改器。
[link]12345678910111213141516171819202122232425# 差class Persondef initialize(first_name, last_name)@first_name = first_name@last_name = last_nameenddef first_name@first_nameenddef last_name@last_nameendend# 好class Personattr_reader :first_name, :last_namedef initialize(first_name, last_name)@first_name = first_name@last_name = last_nameendend
避免使用attr
。使用attr_reader
与attr_accessor
来替代。
[link]1234567# 差 - 创建单个存取方法(此方法在 Ruby 1.9 之后被移除了)attr :something, trueattr :one, :two, :three # 类似于 attr_reader# 好attr_accessor :somethingattr_reader :one, :two, :three
优先考虑使用Struct.new
。它替你定义了那些琐碎的访问器、构造器及比较操作符。
[link]12345678910111213# 好class Personattr_accessor :first_name, :last_namedef initialize(first_name, last_name)@first_name = first_name@last_name = last_nameendend# 更好Person = Struct.new(:first_name, :last_name) doend
不要扩展Struct.new
实例化后的对象。对它进行扩展不但引入了毫无意义的类层次,而且在此文件被多次引入时可能会产生奇怪的错误。
[link]123456# 差class Person < Struct.new(:first_name, :last_name)end# 好Person = Struct.new(:first_name, :last_name)
优先考虑通过工厂方法的方式创建某些具有特定意义的实例对象。
[link]12345class Persondef self.create(options_hash)# 省略主体endend-
123456789101112131415161718192021222324252627282930313233# 差class Animal# 抽象方法def speakendend# 继承父类class Duck < Animaldef speakputs 'Quack! Quack'endend# 继承父类class Dog < Animaldef speakputs 'Bau! Bau!'endend# 好class Duckdef speakputs 'Quack! Quack'endendclass Dogdef speakputs 'Bau! Bau!'endend
避免使用类变量(@@
)。类变量在继承方面存在令人生厌的行为。
[link]12345678910111213class Parent@@class_var = 'parent'def self.print_class_varputs @@class_varendendclass Child < Parent@@class_var = 'child'endParent.print_class_var # => 此处打印的结果为 'child'如你所见,在类的层次结构中所有类都会共享同一类变量。通常情况下,倾向使用类实例变量而不是类变量。
根据方法的目的与用途设置适当的可见级别(private
、protected
)。不要什么都不做就把所有方法设置为public
(默认值)。毕竟我们写的是 Ruby 而不是 Python。
[link]
把public
、protected
、private
与其作用的方法缩排在同一层级。且在其上下各留一行以强调此可见级别作用于之后的所有方法。
[link]123456789101112131415class SomeClassdef public_method# ...endprivatedef private_method# ...enddef another_private_method# ...endend
使用def self.method
定义类方法。这种做法使得在代码重构时,即使修改了类名也无需做多次修改。
[link]12345678910111213141516171819202122class TestClass# 差def TestClass.some_method# 省略主体end# 好def self.some_other_method# 省略主体end# 在需要定义多个类方法时,另一种便捷写法class << selfdef first_method# 省略主体enddef second_method_etc# 省略主体endendend
在类的词法作用域中定义方法别名时,倾向使用alias
。因为定义期间alias
与self
指向的都是词法作用域,除非明确说明,否则该别名所引用的方法不会在运行期间被改变,或是在任何子类中被修改。
[link]1234567class Westernerdef first_name@names.firstendalias given_name first_nameend因为
alias
与def
一样都是关键字,倾向使用裸字而不是符号或字符串。也就是说,使用alias foo bar
而不是alias :foo :bar
。另外需要了解 Ruby 是如何处理别名和继承的:别名所引用的原始方法是在定义期间被指定的,而不是运行期间。
12345class Fugitive < Westernerdef first_name'Nobody'endend在这个例子中,
Fugitive#given_name
仍然调用原先的Westerner#first_name
方法,而不是Fugitive#first_name
。如果想要覆写Fugitive#given_name
,必须在子类中重新定义。1234567class Fugitive < Westernerdef first_name'Nobody'endalias given_name first_nameend
在运行期间定义模块方法、类方法、单件方法的别名时,总是使用alias_method
。在上述情况下,使用alias
可能会导致预期之外的结果。
[link]123456789module Mononymousdef self.included(other)other.class_eval { alias_method :full_name, :given_name }endendclass Sting < Westernerinclude Mononymousend
异常
对于异常处理,倾向使用raise
而不是fail
。
[link]12345# 差fail SomeException, 'message'# 好raise SomeException, 'message'
不要在带双参数形式的raise
方法中显式指定RuntimeError
。
[link]12345# 差raise RuntimeError, 'message'# 好 - 默认就是 RuntimeErrorraise 'message'
倾向使用带异常类、消息的双参数形式调用raise
方法,而不是使用异常的实例。
[link]12345# 差 - 并无 raise SomeException.new('message') [, backtraces] 这种调用形式raise SomeException.new('message')# 好 - 与调用形式 raise SomeException [, 'message' [, backtraces]] 保持一致raise SomeException, 'message'
永远不要从ensure
区块返回。如果你显式地从ensure
区块返回,那么其所在的方法会如同永远不会发生异常般的返回。事实上,异常被默默地丢弃了。
[link]12345def fooraiseensurereturn 'very bad idea'end
尽可能隐式地使用begin/rescue/ensure/end
区块。
[link]123456789101112131415# 差def foobegin# 主逻辑rescue# 异常处理逻辑endend# 好def foo# 主逻辑rescue# 异常处理逻辑end
通过使用 contingency 方法(一个由 Avdi Grimm 创造的词)来减少begin/rescue/ensure/end
区块的使用。
[link]1234567891011121314151617181920212223# 差beginsomething_that_might_failrescue IOError# 处理 IOErrorendbeginsomething_else_that_might_failrescue IOError# 处理 IOErrorend# 好def with_io_error_handlingyieldrescue IOError# 处理 IOErrorendwith_io_error_handling { something_that_might_fail }with_io_error_handling { something_else_that_might_fail }
不要抑制异常。
[link]123456789# 差begin# 抛出异常rescue SomeError# 不做任何相关处理end# 差do_something rescue nil
避免使用rescue
修饰语法。
[link]123456789# 差 - 这里将会捕捉 StandardError 及其所有子孙类的异常read_file rescue handle_error($!)# 好 - 这里只会捕获 Errno::ENOENT 及其所有子孙类的异常def fooread_filerescue Errno::ENOENT => exhandle_error(ex)end
不要将异常处理作为流程控制使用。
[link]12345678910111213# 差beginn / drescue ZeroDivisionErrorputs 'Cannot divide by 0!'end# 好if d.zero?puts 'Cannot divide by 0!'elsen / dend
避免捕获Exception
。这种做法会同时将信号与exit
方法困住,导致你必须使用kill -9
来终止进程。
[link]123456789101112131415161718192021# 差 - 信号与 exit 方法产生的异常会被捕获(除了 kill -9)beginexitrescue Exceptionputs "you didn't really want to exit, right?"# 处理异常end# 好 - 没有指定具体异常的 rescue 子句默认捕获 StandardErrorbegin# 抛出异常rescue => e# 处理异常end# 好 - 指定具体异常 StandardErrorbegin# 抛出异常rescue StandardError => e# 处理异常end
把较具体的异常放在处理链的较上层,不然它们永远不会被执行。
[link]1234567891011121314151617# 差begin# 抛出异常rescue StandardError => e# 处理异常rescue IOError => e# 处理异常,但事实上永远不会被执行end# 好begin# 抛出异常rescue IOError => e# 处理异常rescue StandardError => e# 处理异常end
在ensure
区块释放程序的外部资源。
[link]12345678f = File.open('testfile')begin# .. 文件操作rescue# .. 处理异常ensuref.close if fend
在调用资源获取方法时,尽量使用具备自动清理功能的版本。
[link]123456789# 差 - 需要显式关闭文件描述符f = File.open('testfile')# ...f.close# 好 - 文件描述符会被自动关闭File.open('testfile') do |f|# ...end
倾向使用标准库中的异常类而不是引入新的类型。
[link]
集合
对于数组与哈希,倾向使用字面量语法来构建实例(除非你需要给构造器传递参数)。
[link]1234567# 差arr = Array.newhash = Hash.new# 好arr = []hash = {}
当创建一组元素为单词(没有空格或特殊字符)的数组时,倾向使用%w
而不是[]
。此规则只适用于数组元素有两个或以上的时候。
[link]12345# 差STATES = ['draft', 'open', 'closed']# 好STATES = %w(draft open closed)
当创建一组符号类型的数组(且不需要保持 Ruby 1.9 兼容性)时,倾向使用%i
。此规则只适用于数组元素有两个或以上的时候。
[link]12345# 差STATES = [:draft, :open, :closed]# 好STATES = %i(draft open closed)
避免在数组与哈希的字面量语法的最后一个元素之后添加逗号,尤其当元素没有分布在同一行时。
[link]123456789101112# 差 - 尽管移动、新增、删除元素颇为方便,但仍不推荐这种写法VALUES = [1001,2020,3333,]# 差VALUES = [1001, 2020, 3333, ]# 好VALUES = [1001, 2020, 3333]
避免在数组中创造巨大的间隔。
[link]12arr = []arr[100] = 1 # 现在你有一个很多 nil 的数组
当访问数组的首元素或尾元素时,倾向使用first
或last
而不是[0]
或[-1]
。
[link]
当处理的对象不存在重复元素时,使用Set
来替代Array
。Set
是实现了无序且无重复元素的集合类型。它兼具Array
的直观操作与Hash
的快速查找。
[link]
倾向使用符号而不是字符串作为哈希键。
[link]12345# 差hash = { 'one' => 1, 'two' => 2, 'three' => 3 }# 好hash = { one: 1, two: 2, three: 3 }
避免使用可变对象作为哈希键。
[link]
当哈希键为符号时,使用 Ruby 1.9 的字面量语法。
[link]12345# 差hash = { :one => 1, :two => 2, :three => 3 }# 好hash = { one: 1, two: 2, three: 3 }
当哈希键既有符号又有字符串时,不要使用 Ruby 1.9 的字面量语法。
[link]12345# 差{ a: 1, 'b' => 2 }# 好{ :a => 1, 'b' => 2 }
倾向使用Hash#key?
而不是Hash#has_key?
,使用Hash#value?
而不是Hash#has_value?
。Matz 在此提及,较长形式的方法将被废弃。
[link]1234567# 差hash.has_key?(:test)hash.has_value?(value)# 好hash.key?(:test)hash.value?(value)
倾向使用Hash#each_key
而不是Hash#keys.each
,使用Hash#each_value
而不是Hash#values.each
。
[link]123456789# 差hash.keys.each { |k| p k }hash.values.each { |v| p v }hash.each { |k, _v| p k }hash.each { |_k, v| p v }# 好hash.each_key { |k| p k }hash.each_value { |v| p v }
当处理应该存在的哈希键时,使用Hash#fetch
。
[link]12345678heroes = { batman: 'Bruce Wayne', superman: 'Clark Kent' }# 差 - 如果我们打错了哈希键,则难以发现这个错误heroes[:batman] # => 'Bruce Wayne'heroes[:supermann] # => nil# 好 - fetch 会抛出 KeyError 使这个错误显而易见heroes.fetch(:supermann)
当为哈希键的值提供默认值时,倾向使用Hash#fetch
而不是自定义逻辑。
[link]1234567batman = { name: 'Bruce Wayne', is_evil: false }# 差 - 如果仅仅使用 || 操作符,那么当值为假时,我们不会得到预期结果batman[:is_evil] || true # => true# 好 - fetch 在遇到假值时依然可以正确工作batman.fetch(:is_evil, true) # => false
当提供默认值的求值代码具有副作用或开销较大时,倾向使用Hash#fetch
的区块形式。
[link]1234567batman = { name: 'Bruce Wayne' }# 差 - 此形式会立即求值,如果调用多次,可能会影响程序的性能batman.fetch(:powers, obtain_batman_powers) # obtain_batman_powers 开销较大# 好 - 此形式会惰性求值,只有抛出 KeyError 时,才会产生开销batman.fetch(:powers) { obtain_batman_powers }
当需要一次性从哈希中获取多个键的值时,使用Hash#values_at
。
[link]123456# 差email = data['email']username = data['nickname']# 好email, username = data.values_at('email', 'nickname')
利用“Ruby 1.9 之后的哈希是有序的”的这个特性。
[link]
当遍历集合时,不要改动它。
[link]
当访问集合中的元素时,倾向使用对象所提供的方法进行访问,而不是直接调用对象属性上的[n]
方法。这种做法可以防止你在nil
对象上调用[]
。
[link]12345# 差Regexp.last_match[1]# 好Regexp.last_match(1)
当为集合提供存取器时,尽量支持索引值为nil
的访问形式。
[link]12345678910111213# 差def awesome_things@awesome_thingsend# 好def awesome_things(index = nil)if index && @awesome_things@awesome_things[index]else@awesome_thingsendend
数值
通过Integer
检查对象是否是数值类型,而不是Fixnum
或Bignum
。因为Fixnum
或Bignum
表达的数值大小存在范围限定。
[link]12345678timestamp = Time.now.to_i# 差timestamp.is_a? Fixnumtimestamp.is_a? Bignum# 好timestamp.is_a? Integer
字符串
倾向使用字符串插值或字符串格式化,而不是字符串拼接。
[link]12345678# 差email_with_name = user.name + ' <' + user.email + '>'# 好email_with_name = "#{user.name} <#{user.email}>"# 好email_with_name = format('%s <%s>', user.name, user.email)
使用统一的风格创建字符串字面量。在 Ruby 社区中存在两种流行的风格:默认单引号(风格 A)与默认双引号(风格 B)。
[link](风格 A) 当你不需要字符串插值或特殊字符(比如
\t
、\n
、'
)时,倾向使用单引号。12345# 差name = "Bozhidar"# 好name = 'Bozhidar'(风格 B) 除非字符串中包含双引号,或是你希望抑制转义字符,否则倾向使用双引号。
12345# 差name = 'Bozhidar'# 好name = "Bozhidar"
本指南使用第一种风格。
不要使用?x
字面量语法。在 Ruby 1.9 之后,?x
与'x'
(只包含单个字符的字符串)是等价的。
[link]12345# 差char = ?c# 好char = 'c'
不要忘记使用{}
包裹字符串插值中的实例变量或全局变量。
[link]1234567891011121314151617181920212223242526class Personattr_reader :first_name, :last_namedef initialize(first_name, last_name)@first_name = first_name@last_name = last_nameend# 差 - 语法正确,但略显笨拙def to_s"#@first_name #@last_name"end# 好def to_s"#{@first_name} #{@last_name}"endend$global = 0# 差puts "$global = #$global"# 好puts "$global = #{$global}"
在字符串插值中,不要显式调用Object#to_s
方法,Ruby 会自动调用它。
[link]12345# 差message = "This is the #{result.to_s}."# 好message = "This is the #{result}."
当你需要构造巨大的数据块时,避免使用String#+
,使用String#<<
来替代。String#<<
通过修改原始对象进行拼接工作,其比String#+
效率更高,因为后者需要产生一堆新的字符串对象。
[link]123456789101112131415# 差html = ''html += '<h1>Page title</h1>'paragraphs.each do |paragraph|html += "<p>#{paragraph}</p>"end# 好 - 并且效率更高html = ''html << '<h1>Page title</h1>'paragraphs.each do |paragraph|html << "<p>#{paragraph}</p>"end
当存在更快速、更专业的替代方案时,不要使用String#gsub
。
[link]12345678910url = 'http://example.com'str = 'lisp-case-rules'# 差url.gsub('http://', 'https://')str.gsub('-', '_')# 好url.sub('http://', 'https://')str.tr('-', '_')
heredocs 中的多行文本会保留各行的前导空白。因此做好如何缩排的规划。
[link]1234567code = <<-END.gsub(/^\s+\|/, '')|def test| some_method| other_method|endEND# => "def test\n some_method\n other_method\nend\n"
使用 Ruby 2.3 新增的<<~
操作符来缩排 heredocs 中的多行文本。
[link]1234567891011121314151617181920212223# 差 - 使用 Powerpack 程序库的 String#strip_margincode = <<-END.strip_margin('|')|def test| some_method| other_method|endEND# 差code = <<-ENDdef testsome_methodother_methodendEND# 好code = <<~ENDdef testsome_methodother_methodendEND
正则表达式
有些人在面对问题时,不经大脑便认为,“我知道,这里该用正则表达式”。现在他要面对两个问题了。
——Jamie Zawinski
如果只是在字符串中进行简单的文本搜索,不要使用正则表达式,比如string['text']
。
[link]
对于简单的构建操作,使用正则表达式作为索引即可。
[link]123match = string[/regexp/] # 获取匹配的内容first_group = string[/text(grp)/, 1] # 获取匹配的分组(grp)的内容string[/text (grp)/, 1] = 'replace' # string => 'text replace'
当你不需要分组结果时,使用非捕获组。
[link]12345# 差/(first|second)/# 好/(?:first|second)/
避免使用 Perl 风格的、用以代表最近的捕获组的特殊变量(比如$1
、$2
等)。使用Regexp.last_match(n)
来替代。
[link]12345678/(regexp)/ =~ string...# 差process $1# 好process Regexp.last_match(1)
避免使用数字来获取分组。因为很难明白它们代表的含义。使用命名分组来替代。
[link]123456789# 差/(regexp)/ =~ string...process Regexp.last_match(1)# 好/(?<meaningful_var>regexp)/ =~ string...process meaningful_var
在字符类别中,只有少数几个你需要特别关心的特殊字符:^
、-
、\
、]
,所以你不需要转义[]
中的.
与中括号。
[link]
小心使用^
与$
,它们匹配的是一行的开始与结束,而不是字符串的开始与结束。如果你想要匹配整个字符串,使用\A
与\z
。(注意,\Z
实为/\n?\z/
)
[link]123string = "some injection\nusername"string[/^username$/] # 匹配成功string[/\Ausername\z/] # 匹配失败
对于复杂的正则表达式,使用x
修饰符。这种做法不但可以提高可读性,而且允许你加入必要的注释。注意的是,空白字符会被忽略。
[link]1234567regexp = /start # some text\s # white space char(group) # first group(?:alt1|alt2) # some alternationend/x
对于复杂的替换,使用sub/gsub
与哈希或区块组合的调用形式。
[link]123words = 'foo bar'words.sub(/f/, 'f' => 'F') # => 'Foo bar'words.gsub(/\w+/) { |word| word.capitalize } # => 'Foo Bar'
百分号字面量
只有当字符串中同时存在插值与双引号,且是单行时,才使用%()
(%Q
的简写形式)。多行字符串,倾向使用 heredocs。
[link]1234567891011121314# 差 - 不存在插值%(<div class="text">Some text</div>)# 应当使用 '<div class="text">Some text</div>'# 差 - 不存在双引号%(This is #{quality} style)# 应当使用 "This is #{quality} style"# 差 - 多行字符串%(<div>\n<span class="big">#{exclamation}</span>\n</div>)# 应当使用 heredocs# 好 - 同时存在插值与双引号,且是单行字符串%(<tr><td class="name">#{name}</td>)
避免使用%q
,除非字符串同时存在'
与"
。优先考虑更具可读性的常规字符串,除非字符串中存在大量需要转义的字符。
[link]12345678910# 差name = %q(Bruce Wayne)time = %q(8 o'clock)question = %q("What did you say?")# 好name = 'Bruce Wayne'time = "8 o'clock"question = '"What did you say?"'quote = %q(<p class='quote'>"What did you say?"</p>)
只有当正则表达式中存在一个或以上的/
字符时,才使用%r
。
[link]123456# 差%r{\s+}# 好%r{^/(.*)$}%r{^/blog/2011/(.*)$}
除非调用的命令使用了反引号(这种情况并不多见),否则不要使用%x
。
[link]123456# 差date = %x(date)# 好date = `date`echo = %x(echo `date`)
避免使用%s
。倾向使用:"some string"
来创建含有空白字符的符号。
[link]
使用%
字面量语法时,倾向使用()
(%r
除外,因为圆括号在正则表达式中比较常用,此时可以使用{}
等其他形式来替代)。
[link]1234567# 差%w[one two three]%q{"Test's king!", John said.}# 好%w(one two three)%q("Test's king!", John said.)
元编程
避免无谓的元编程。
[link]
当编写程序库时,不要使核心类混乱(不要使用 monkey patch)。
[link]
对于class_eval
方法,倾向使用区块形式,而不是字符串插值形式。
[link]当使用字符串插值形式时,总是提供
__FILE__
及__LINE__
,以使你的调用栈看起来具有意义:1class_eval 'def use_relative_model_naming?; true; end', __FILE__, __LINE__倾向使用
define_method
而不是class_eval { def ... }
当使用class_eval
(或其他的eval
)的字符串插值形式时,添加一个注释区块来说明它是如何工作的(来自 Rails 代码中的技巧)。
[link]123456789101112131415# 摘录自 activesupport/lib/active_support/core_ext/string/output_safety.rbUNSAFE_STRING_METHODS.each do |unsafe_method|if 'String'.respond_to?(unsafe_method)class_eval <<-EOT, __FILE__, __LINE__ + 1def #{unsafe_method}(*params, &block) # def capitalize(*params, &block)to_str.#{unsafe_method}(*params, &block) # to_str.capitalize(*params, &block)end # enddef #{unsafe_method}!(*params) # def capitalize!(*params)@dirty = true # @dirty = truesuper # superend # endEOTendend
避免使用method_missing
。它会使你的调用栈变得凌乱;其方法不被罗列在#methods
中;拼错的方法可能会默默地工作(nukes.launch_state = false
)。优先考虑使用委托、代理、或是define_method
来替代。如果你必须使用method_missing
的话,务必做到以下几点:
[link]- 确保同时定义了
respond_to_missing?
。 - 仅仅捕获那些具有良好语义前缀的方法,像是
find_by_*
——让你的代码愈确定愈好。 - 在语句的最后调用
super
。 委托到确定的、非魔术的方法,比如:
12345678910111213141516171819# 差def method_missing?(meth, *params, &block)if /^find_by_(?<prop>.*)/ =~ meth# ... 一堆处理 find_by 的代码elsesuperendend# 好def method_missing?(meth, *params, &block)if /^find_by_(?<prop>.*)/ =~ methfind_by(prop, *params, &block)elsesuperendend# 最好的方式可能是在每个需要支持的属性被声明时,使用 define_method 定义对应的方法
- 确保同时定义了
倾向使用public_send
而不是send
,因为send
会无视private/protected
的可见性。
[link]123456789101112131415161718192021222324252627282930313233module Activatableextend ActiveSupport::Concernincluded dobefore_create :create_tokenendprivatedef reset_token...enddef create_token...enddef activate!...endendclass Organization < ActiveRecord::Baseinclude Activatableendlinux_organization = Organization.find(...)# 差 - 会破坏对象的封装性linux_organization.send(:reset_token)# 好 - 会抛出异常linux_organization.public_send(:reset_token)
倾向使用__send__
而不是send
,因为send
可能会被覆写。
[link]123456789101112require 'socket'u1 = UDPSocket.newu1.bind('127.0.0.1', 4913)u2 = UDPSocket.newu2.connect('127.0.0.1', 4913)# 这里不会调用 u2 的 sleep 方法,而是通过 UDP socket 发送一条消息u2.send :sleep, 0# 动态调用 u2 的某个方法u2.__send__ ...
其他
总是开启ruby -w
选项,以编写安全的代码。
[link]
避免使用哈希作为可选参数。这个方法是不是做太多事了?(对象构造器除外)
[link]
避免单个方法的长度超过 10 行(不计入空行)。理想上,大部分方法应当不超过 5 行。
[link]
避免参数列表数目多于三或四个。
[link]
如果你真的需要“全局”方法,将它们添加到Kernel
并设为私有。
[link]
使用模块实例变量而不是全局变量。
[link]1234567891011# 差$foo_bar = 1# 好module Fooclass << selfattr_accessor :barendendFoo.bar = 1
使用OptionParser
来解析复杂的命令行选项。使用ruby -s
来处理琐碎的命令行选项。
[link]
使用Time.now
而不是Time.new
来获取当前的系统时间。
[link]
使用函数式思维编写程序,避免副作用。
[link]
不要修改参数值,除非那就是这个方法的作用。
[link]
避免使用三层以上的嵌套区块。
[link]
保持一致性。在理想的世界里,遵循这些准则。
[link]
使用常识。
[link]