- リファクタリングをただのコード修正と思ってる人へ
410 ：デフォルトの名無しさん[sage]：2014/08/03(日) 07:51:54.04 ID:KHwMExMo - >>408
あー、主に非技術者がやる作業だね。
|
- + JavaScript の質問用スレッド vol.117 +
72 ：デフォルトの名無しさん[sage]：2014/08/03(日) 08:06:05.05 ID:KHwMExMo - >>70
可読性が悪くなっただね。中二病コードだなｗ
絶対に真似したらダメだよ。
だいたいタイプ数を減らそうとか考えるのは意味がない。
どんだけタイプ速度遅いんだよって話。
重要なのは読みやすさ、意味の理解の明確さ
一般的に書くのは一回で終わるが、読むのは何回も行われる。
コードレビューの時、修正する時、バグが見つかった時。
君がこれなんで動くのかさっぱりわからないと言ってる通り、この書き方は意味が明確ではない。
意味が明確で無いのは、たとえ動くことがわかってもなんでこんなことしてるの？ って
思われるようなコードはダメということ。
「callはthisを変更する時に使うもの」というJavaScriptプログラマなら
だれでも知っている知識を用いればthisを変えたいんだなと意味が明確になる。
|
- スレ立てるまでもない質問はここで 137匹目
771 ：デフォルトの名無しさん[sage]：2014/08/03(日) 09:18:26.41 ID:KHwMExMo - >>766
それはlint使ってるからそうなるってだけだよ。
コードを良くするのが目的なのに、
lintに通すことが目的になっちゃってるよね？
|
- スレ立てるまでもない質問はここで 137匹目
774 ：デフォルトの名無しさん[sage]：2014/08/03(日) 10:17:53.09 ID:KHwMExMo - >>772
lintは重要なものだけに使えばいいってだけだよ。
どっちでもいいことは存在する。
たとえば改行を入れるかどうとかね。
|
- Subversion r14
990 ：デフォルトの名無しさん[sage]：2014/08/03(日) 10:32:04.22 ID:KHwMExMo - >>986
そんなことより、BELってて文字コードがあるんだぜ？
エディタではこの文字が出てきたら音を鳴らさないといけない。
|
- リファクタリングをただのコード修正と思ってる人へ
414 ：デフォルトの名無しさん[sage]：2014/08/03(日) 10:42:19.36 ID:KHwMExMo - まあありがちやねｗ
だから規約を作った人は
必ずコードレビューをしなくてはいけない。
|