---
# System prepended metadata

title: コードレビュー 査定CSV関連

---

# コードレビュー 査定CSV関連

変更ファイル群が不明なので目についた所をちらほら

## api（ファイル名不明）
62行目〜 行末コメント /* ~ */
行末コメントは上記の形式ではなく // を利用してください。
　理由：/* */ はブロックコメントで、該当箇所を大きく /* */ で囲うとコメントアウトできなくなる


## mypage.js
全般
- card[n]のnを定数で管理するなど、コードを読んでどのindexが何を表しているかわかるようにしてください
    ```
    card[9], //price
    card[10], //number
    => card[IDX_PRICE], card[IDX_NUMBER] など
    ```
    - 多少処理かかってもよければ連想配列に変換するケースもありますが、CSV処理するだけではあまりやらないかも
    - 上記のようにINDEX管理をしておくと、列が増えたなどの話がわかりやすいです
- 文字列結合 `str + str` ではなく変数展開を使った方が可読性があがります
    ```
    文字列結合　'<td class="total">' + condition_displayed + ' ' + price + '</td>'
    変数展開  `<td class="total">${condition_displayed} ${price}</td>'`
    ```

20行目 `if (this[0].length > 0) {`
this = csvをeachで回しており、0行目スキップされているのでこの判定いらないような？

21行目〜27行目　変数名
camelCaseとsnake_caseがFucntion名と混在。基本はcamelCaseですかね

41行目 `} else { <br>*3`
空tdなのであれば`<br>`いらないような？

64行目 `const cardConditions`
let conditionsおよび conditions = は不要。 return [[]]でOK
return行とわけたいのであれば `const conditions = [['normal'...]]`

124行目 `convertCondition` 内
return & breakはreturnだけでいいです。
コード規約としてswitchに必ずbreakを入れておくというルールならOK

★144行目などの `if (card[x].trim()) { return [ + card[x] + ];` 
returnするときにtrimしないでも大丈夫そうですか？



