回答編集履歴
2
致命的な誤字の修正
test
CHANGED
@@ -50,7 +50,7 @@
|
|
50
50
|
|
51
51
|
本当にファイルサイズを気にするなら、まずは改行コードとインデント全部取り払って1行にまとめた方がいい。
|
52
52
|
|
53
|
-
手作業でそのレベルの最適化出来る?絶対無
|
53
|
+
手作業でそのレベルの最適化出来る?絶対無理だよ。
|
54
54
|
|
55
55
|
|
56
56
|
|
1
ツッコミ追加
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
|
-
(
|
49
|
+
(Gulpのような登録さえしておけばワンコマンドで出来るようなタスクランナーツールもある)
|
24
50
|
|
25
|
-
本当にファイルサイズ気にするなら、まずは改行コードとインデント全部取り払って1行にまとめた方がいい。
|
51
|
+
本当にファイルサイズを気にするなら、まずは改行コードとインデント全部取り払って1行にまとめた方がいい。
|
26
52
|
|
27
53
|
手作業でそのレベルの最適化出来る?絶対無料だよ。
|
28
54
|
|
29
55
|
|
30
56
|
|
31
|
-
|
57
|
+
もし簡素な記述にこだわりたいなら、記述量が半分以下になるAltJSも検討すべき。
|
32
58
|
|
33
|
-
|
59
|
+
CoffeeScriptやLiveScriptといった言語が代表例だね。
|
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
|
-
|
71
|
+
不要な状態変数めっちゃ減るから慣れてくると思考の邪魔をされずにスイスイ読み書き出来るようになるよ。
|
40
72
|
|
41
73
|
|
42
74
|
|
75
|
+
---
|
76
|
+
|
77
|
+
|
78
|
+
|
79
|
+
まぁ、ツールを上手く使うっていうネタがあるよって話だね。
|
80
|
+
|
43
|
-
|
81
|
+
素のJSだけで頑張って変数名を省略したりするのは限界がある。
|
82
|
+
|
83
|
+
現に保守担当の質問者さんが被害を受けているわけだし、こういったアプローチで提案してみてはどうかな?
|