6
9

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 1 year has passed since last update.

【JavaScript】コーディング規約を選定した話

Last updated at Posted at 2023-06-07

※こちらは2019年に弊社ブログで掲載していた内容と同様の内容の記事になります。

初めまして、MtFormです。
今回はVue.jsのプロジェクトのコーディング規約を定めるにあたっていろいろ迷ったお話です。

コーディング規約とは(ざっくり概念)

プログラムって、同じ動作をする場合でも異なる書き方ができますよね。
例えば、

let a,b,c;

と書くか、

let a;
let b;
let c;

と書くか。

このあたりは個人の好みで書き方が分かれてしまうので、複数人でコードを書くとバラバラの記法になって、バグの原因になったりします。
そこで、コードの書き方を統一するためのルールを定めることにしました、これがコーディング規約です。

メジャーなJSコーディング規約

コーディング規約を1から全部決めるのは大変なので、大手の企業が公開しているコーディング規約を使わせていただきます。(場合によってはカスタマイズします)
今回は3つの候補に絞りました。

JavaScript Standard Style

・2015年に提唱された。ESLintのサブセットである(カスタマイズ済のESLint)
・行末のセミコロン禁止のため、人によっては受け入れられない。
※セミコロンを必須とした「JavaScript Semi-Standard Style」もある。

Google JavaScript Style Guide
・ESLintの候補として表示されるほど標準的なスタイル
・AngularやGWTなどを開発しているGoogle製

Airbnb JavaScript Style Guide
・ESLintで候補として表示されるほど標準的なスタイル
・GitHub上で最も人気のあるスタイル(starが90kで第1位、第2位は22.7k:2019年11月現在)
・オープンな議論を通じてスタイルが決定している。
・スタイルについてパフォーマンスに関するエビデンスがある。

選ばれたのはAirbnbでした

どのスタイルガイドもESLint(JSの構文チェッカー)に対応しているのですが、パフォーマンスに対するエビデンスがあるということで、Airbnb JavaScript Style Guide を利用することにしました。

今回のプロジェクトはVue-CLIで作られていたので、導入は次のコマンドでOK
$ yarn add @vue/eslint-config-airbnb -dev
これでESLintの設定がvue.jsとAirbnb JavaScript Style Guideに対応してくれました。

構文チェックは次のコマンド
$ yarn lint --no-fix
試してみると50個くらいエラーが検出されて焦りましたが、--no-fixオプションを外して実行するとlint側で勝手にコードを修正してくれるのでエラーは5個くらいまで減り、
残りも日本語のリファレンスを見ながら修正できました。

苦戦したポイント

個人的に引っかかった部分は次の2点。

comma-dangle

(https://mitsuruog.github.io/javascript-style-guide/#commas--dangling)

配列の末尾には、カンマが必須らしい!
末尾カンマはJSONだとエラーになるから気持ち悪いなあと思いつつリファレンスを読んでみると、
・末尾コンマはbabelが自動的に消してくれる
・gitの差分がきれいになる
とのこと。
確かに、差分はこっちのほうが断然わかりやすい!
スタイルガイドはgit管理の部分まで視野に入ってるんですね。

BAD

const array = [
  apple,
  banana,
  chocolate
];

GOOD

const array = [
  apple,
  banana,
  chocolate,
];
operator-linebreak

(https://mitsuruog.github.io/javascript-style-guide/#variables--linebreak)
代入で=の前後に改行を入れてはいけない。
でも、そこを直したらmax-lenの100文字に引っかかってエラーになるんだけど……。
という板挟みの状況に陥ったわけですが、リファレンスにはちゃんと書いてありました。

そういう場合は、「値を丸括弧で囲むこと」だそうです。

BAD

const text =
  'superLongText';

GOOD

const text = (
  'superLongText'
);
結論

Airbnb JavaScript Style Guideは、リファレンスが丁寧!!
以上です、ありがとうございました。

参考サイト

・JavaScriptのスタイルガイドまとめ(おすすめ4選)
https://qiita.com/takeharu/items/dee0972e5f39bfd4d7c8
・JavaScript Standard Styleのススメ
https://arui.tech/recommendation-of-standard-js/
・セミコロンは省略?人気スタイルガイドに学ぶJavaScriptのコーディング規則
https://www.webprofessional.jp/why-use-javascript-style-guide/
・JavaScriptの有名なコーディングスタンダードまとめ
https://blog.htmlhifive.com/2017/09/12/javascript-style-guide/
・ESLintをAtomに導入し、Reactの構文にも対応したAirbnbのJSスタイルガイドを使う
https://mae.chab.in/archives/2874

6
9
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
6
9

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?