回答編集履歴

2

致命的な誤字の修正

2017/09/27 13:56

投稿

miyabi-sun
miyabi-sun

スコア21158

test CHANGED
@@ -50,7 +50,7 @@
50
50
 
51
51
  本当にファイルサイズを気にするなら、まずは改行コードとインデント全部取り払って1行にまとめた方がいい。
52
52
 
53
- 手作業でそのレベルの最適化出来る?絶対無だよ。
53
+ 手作業でそのレベルの最適化出来る?絶対無だよ。
54
54
 
55
55
 
56
56
 

1

ツッコミ追加

2017/09/27 13:56

投稿

miyabi-sun
miyabi-sun

スコア21158

test CHANGED
@@ -1,43 +1,83 @@
1
+ ごめん、最初に一言突っ込ませて
2
+
3
+
4
+
1
5
  > とある外注より上がってきたjsファイルを改修しているのですが
2
6
 
3
7
 
4
8
 
5
- 外注の人がメンテしないんかい!
9
+ お前がメンテするんかい!
6
10
 
7
- が間違ってるかって、保守する気のないコードを他人がせっせと直そうとてるところだね。
11
+ 作った人メンテするべきでしょ!明らかに間違ってるょ!
8
12
 
9
13
 
10
14
 
11
- 質問者さんの立場としてアドバイスするよ。
15
+ ---
12
16
 
13
17
 
14
18
 
15
- もし後から他人が触るようなコードの場合、保守性を考慮したコードでなければ納品物として認めない
19
+ さて、質問者さん場としてアドバイスするよ
16
20
 
21
+
22
+
23
+ 複数人の作業者でプロジェクトを作るのであれば、
24
+
25
+ 後から他人が触るようなコードの場合、保守性を考慮したコードでなければ納品物として認めない。
26
+
17
- と言うふうな取り決めを先にするべきだったかも
27
+ と言うふうな取り決めをするべきだろうね
28
+
29
+
30
+
31
+ 特に新規開発と保守するメンバーが違うなら、なおさら保守するメンバーを守るような取り決めをしておくべき。
32
+
33
+ まぁ、どっちに向きすぎても駄目だからバランスは重要だけどね。
34
+
35
+
36
+
37
+ > この方がファイルも軽くなるしスマートだ、変えるつもりはないとの意見でした。
38
+
39
+ > 確かに、何度も同じ変数を使用する、コードも長いjsなので、ぱっと見綺麗です。
40
+
41
+
42
+
43
+ その意見は中途半端、それを今から証明しよう。
18
44
 
19
45
 
20
46
 
21
47
  後からminifyやuglifyみたいなツールをかければ最小かつ難読化されたコードを誰でも何時でも簡単に変換出来る。
22
48
 
23
- gulpやgruntみたいなワンコマンドで出来るようなタスクランナーツールもある)
49
+ Gulpのよう登録さえしておけばワンコマンドで出来るようなタスクランナーツールもある)
24
50
 
25
- 本当にファイルサイズ気にするなら、まずは改行コードとインデント全部取り払って1行にまとめた方がいい。
51
+ 本当にファイルサイズ気にするなら、まずは改行コードとインデント全部取り払って1行にまとめた方がいい。
26
52
 
27
53
  手作業でそのレベルの最適化出来る?絶対無料だよ。
28
54
 
29
55
 
30
56
 
31
- 更に簡素な記述にこだわりたいなら、CoffeeScriptやLiveScript等の記述量が半分以下になるAltJSを使った上で、先程紹介したGulpやGruntでJSに変換かければいい
57
+ もし簡素な記述にこだわりたいなら、記述量が半分以下になるAltJSも検討すべき
32
58
 
33
- そうすればついでにminifyuglifyもかけられるから手間は殆ど変わらない。
59
+ CoffeeScriptLiveScriptとった言語が代表例だね
34
60
 
35
- (まぁこの辺極論極論返すよう話になっちゃってるけど)
61
+ 素のJSでやってる限りAltJSに絶対敵わない、徹底きてい。
36
62
 
37
63
 
38
64
 
65
+ そして変数a, b, c...ab!?
66
+
67
+ 状態変数はあればあるだけ思考の邪魔になるから、そんなに沢山用意するのは明らかにスマートじゃない。
68
+
69
+ [Lodash](https://lodash.com/)や[Ramda](http://ramdajs.com/docs/)のような関数型プログラミング用のライブラリも併用してみたら?
70
+
39
- ちらにせよ、素のJSで頑張っ変数名を省略したりするのは努力の方向間違えてと思うよ。
71
+ 不要な状態変数めっゃ減るか慣れと思考邪魔されずにスイスイ読み書き出来になるよ。
40
72
 
41
73
 
42
74
 
75
+ ---
76
+
77
+
78
+
79
+ まぁ、ツールを上手く使うっていうネタがあるよって話だね。
80
+
43
- こちらから出せ主張
81
+ JSだけで頑張って変数名を省略したりするのは限界る。
82
+
83
+ 現に保守担当の質問者さんが被害を受けているわけだし、こういったアプローチで提案してみてはどうかな?