search

rubocop / rubocop-jp

A place for RuboCop discussions in Japanese
55 stars 2 forks source link

ヒアドキュメント風の勘違いを訂正したい #24

Closed onk closed 7 years ago

onk commented 7 years ago 
foo = """
  bar
  baz
"""

を見かけたので、warning を出してあげても良いかなと思った。

けどこういう 3 年に 1 回ぐらいしか見かけないコードを対象に cop 作るの、無駄っぽくてあんまり好きじゃないんだよねぇ。。 別言語 (coffee, elixir) から来た人には便利かもしれない？

(lvasgn :foo
  (dstr
    (str "")
    (dstr
      (str "\n")
      (str "  bar\n")
      (str "  baz\n"))
    (str "")))
onk commented 7 years ago

既に Lint/ImplicitStringConcatenation が怒ってくれてました

hoge.rb:1:7: W: Lint/ImplicitStringConcatenation: Combine "" and "\n  bar\n  baz\n" into a single string literal, rather than using implicit string concatenation.
foo = """ ...
      ^^^
hoge.rb:1:9: W: Lint/ImplicitStringConcatenation: Combine "\n  bar\n  baz\n" and "" into a single string literal, rather than using implicit string concatenation.
foo = """ ...
        ^
pocke commented 7 years ago

Lint/ImplicitStringConcatenation が「意図しない文字列連結を怒る」Copだと思うので、たしかにこれが怒れば充分そうですね。 ありがとうございます 🙇