## 執筆理由
変数名や関数名、api名は第三者から見てもその存在意義がわかるようにすべきという意識はあったなかでも、以下のようなミスをしてしまったので備忘録として

# 本題
[PR](https://github.com/weseek/growi/pull/2673)

>getHoge と命名する場合は、必ず return する。
>
言われてみれば当たり前だが、今回なんでそこに気づかなかったのか

## 作った関数
```javascript
getLinkTextPreview() {
    const linker = this.generateLink();
    const linkText = linker.generateMarkdownText();
    this.setState({ linkText });
}
```

Q. なぜgetという接頭辞をつけた？

A. (yusuketk) この関数の中で LinkTextPreview という情報を取得 (get) しているので、そうゆう処理を行ってる関数であることがひと目で分かるように、getという接頭辞をつけました！

Q. そのメソッドを呼び出す側は getLinkTextPreview()を呼び出せば 返り値にlinkTextPreviewが入ってると期待するんじゃないですか？

A. (yusuketk)...そうですね

## なぜ気づかなかったのか
普段getHogeという関数を作るとき、hogeという情報をreturnするのは当たり前のように書いていた。これは、その関数内に情報を取得する処理を書いたあとに、取得した情報はこのあとどうするの？という質問が自然と投げかけられ（場合によってはlint errorとして指摘される）、呼び出し元にわたすんですと答えるので、その答えの通り取得した情報をreturnする文を書きます。

今回はメソッド呼び出し側に取得した情報をreturnで返すのではなくstateにセットする処理であるため、本来はその時点でgetHogeという変数名は適さないと気づくべきであったが、関数内で行っていることを説明するためにつけていたgetという接頭辞をつける、という誤った認識があったのでそのままにしていた。

もしgetHogeを作るなら以下のようにすべき

```javascript
setLinkTextPreview(){
    const linkText = getLinkTextPreview();
    this.setState({ linkText });
}

getLinkTextPreview() {
    const linker = this.generateLink();
    return linkText = linker.generateMarkdownText();
}
```

## 結論
関数名は、その関数の中で何をやっているのかより、その関数を実行した暁には何ができるのか（実行した側がどのようなことを達成できるのか）、どのような目的を果たしてくれる関数なのかを明示すべき

---

## Page Navigation

- Canonical URL: https://tips.weseek.co.jp/今日のレビューから/関数名定義(getHoge)
- Permalink: https://tips.weseek.co.jp/5f4883eb0802bb00498f41d9
- Parent: [今日のレビューから](/5e38deb8e848c600487d69a0.md)
- Children: 0 total
- Total descendants: 0
- Siblings: 4 total
  - [Requestに適正な値が入ってなかったら400エラーを返す](/5e38f144e848c600487d69a5.md)
  - [routerとserviceの責務](/5f323bfc55d58b0048c43b00.md)
  - [それって車輪の再発明じゃない？](/5e425d62decd0a0049953aef.md)
  - [イテレータを利用してスマートな実装しよう](/5e3f9b11f5921400485be6ad.md)
- Last updated: 2020-08-28T04:18:59.855Z by deleted_at_1672138409189
- Full page listing (all children regardless of count): https://tips.weseek.co.jp/_api/v3/page-listing/children?id=5f4883eb0802bb00498f41d9