16/05/21 07:46:43.16 XRbWBdK9.net
>>922
速くするための最適化と
早い段階での最適化を近藤していませんか?
952:デフォルトの名無しさん
16/05/21 07:49:03.87 +h4mdsTD.net
>>916
＞どうせID名が変われば、変数名も変わる。
うそだろ・・・？
仮に100箇所で $('#search_form')してたとして
'#search_form'→'#search_form1'に変えた時に
100箇所直さなくていいように
var $form = $('#search_form'); するんだろ・・・
普通の「固定値」なら100箇所じゃなくても2～3箇所でも使ってるところがあった時点で
変数に入れるだろ
953:デフォルトの名無しさん
16/05/21 08:11:20.66 EoA45sOs.net
話がキャッシュとか最適化とか保守性とかごっちゃになってるけどさ
複数箇所で使われてる固定値のjQueryオブジェクトなら
広いスコープで変数に入れて(擬似)定数にしといたほうが保守性もいいしキャッシュも効いて一石二鳥だよね？何か問題ある？
↓
でも、それってJSのクロージャの仕組みを考えるとメモリーリークとか危なくね？
ってことでしょ？
実際メモリーリークは起きるの？
954:デフォルトの名無しさん
16/05/21 08:35:24.16 FusMkK2d.net
フロントエンジニアって程度が低いらしいけどここまで酷いの？
955:デフォルトの名無しさん
16/05/21 08:37:16.66 86JuHo21.net
>>936
ローカル変数がキャッシュではないと思ってるのはお前だけ
(1) 関数A、関数Bで要素Cへ1回ずつ参照する
(2) 関数Aで要素Cに2回参照する
(1), (2) の参照コストの総計(2回)は変わらない
関数Aの中でローカル変数を使って要素Cへの参照回数を1回に抑えるのは意味がある
956:デフォルトの名無しさん
16/05/21 10:26:05.39 q2Zh6d9K.net
>>942
> var $form = $('#search_form'); するんだろ・・・
それをやっていいのは、ローカル変数だけだ。
広いスコープで$formとあっても、それがなにか区別付かない。
それは、"保守性を下げる"
今はキャッシュの話をしている。
ローカル変数だとキャッシュとしての効果は
ローカルだけにとどまる。
そしてjQueryを使えばメソッドチェーンが使えるので
ローカルで変数に入れる必要はない。
何度も言ったことだ
957:デフォルトの名無しさん
16/05/21 10:27:55.15 q2Zh6d9K.net
>>941
> 速くするための最適化と
> 早い段階での最適化を近藤していませんか?
速くする最適化でも
軽くする最適化でも
○○する最適化でも
どれも「最適化」だ。
これらの「最適化」を早いうちからするなって言うのが、
「早すぎる最適化は諸悪の根源である」という言葉だ。
958:デフォルトの名無しさん
16/05/21 10:29:34.18 q2Zh6d9K.net
>>945
> ローカル変数がキャッシュではないと思ってるのはお前だけ
ローカル変数がキャッシュではないと言ったことは
一度もない。
jQueryはメソッドチェーンを使えるから、
ローカル変数にキャッシュしないくてよいという話だ。
あとローカル変数ってわかってるか？
別の関数では共有できないぞｗ
959:デフォルトの名無しさん
16/05/21 10:31:56.21 q2Zh6d9K.net
あと、いまはjQueryオブジェクトの話をしている。
$(セレクタ)のコストがかかるから「いったん変数に入れておくキャッシュ」の話だ。
忘れるなよ？ 別の話にすり替えるな。
960:デフォルトの名無しさん
16/05/21 11:12:30.21 hclWowah.net
キャッシュの結論としてはやらないほうがいいんだろうな
innerHTMLとかinputのvalueとかキャッシュしても
他の場所から実体を書き換えられたら整合性狂うからね
そこまでケアしてあげるとなると費用対効果が合わないと思う
961:デフォルトの名無しさん
16/05/21 11:18:11.26 q2Zh6d9K.net
>>950
> キャッシュの結論としてはやらないほうがいいんだろうな
その通り。
というとどっかの馬鹿が変数に入れるなということなのか！？とか言い出すんだよな（苦笑）
「変数に入れるとキャッシュの効果もありまぁす」じゃねーんだよｗ
楽な方法をとった結果、たまたまそうなる話なんかどうでもいい。
キャッシュを理由に変数に入れておくべきか？とか言ってるから
キャッシュを理由に変数に入れるなんてことは、
「入れるべきもの」ではなく逆に「入れないべきもの」であって
必要になってからやれという話だと言ってるのに。
962:デフォルトの名無しさん
16/05/21 12:03:04.42 oKW2aYRF.net
整合性がどうのこーのって、実装の問題だろ。
問題をゴチャゴチャにするなよ。
馬鹿大杉だろ。スレ読む気も起きんが。
963:デフォルトの名無しさん
16/05/21 12:13:36.28 rd5jwdZc.net
>>950-951
一切更新しない気なのか？
964:デフォルトの名無しさん
16/05/21 12:43:50.40 ATvI0r3K.net
誰か病院紹介してやれよ
965:デフォルトの名無しさん
16/05/21 12:49:10.28 HhzhPZLm.net
お前が行ってる所でいいんじゃないかな
966:デフォルトの名無しさん
16/05/21 12:49:16.18 q2Zh6d9K.net
>>953
キャッシュを更新しても無駄だぞｗ
だからキャッシュを使わない方がいい。
967:デフォルトの名無しさん
16/05/21 12:51:52.14 q2Zh6d9K.net
>>952
整合性を保つには、シンプルな実装が一番いい。
だから、必要もないのに余計なキャッシュなんか
するべきじゃないって話をしているんだよ。
968:デフォルトの名無しさん
16/05/21 13:11:46.29 3KjPC0qs.net
このスｒ流れが早すぎてなんかの質問をしたから確認に来たんだけど流れが早すぎてどこで質問したか何の質問したか忘れる
969:デフォルトの名無しさん
16/05/21 13:26:02.02 XOGjEe4x.net
>>948
> ローカル変数がキャッシュではないと言ったことは一度もない。
おまえが>>936で「"ローカル" 変数に入れることで見やすくすることはキャッシュとは別の話だって言ったよね？」といったことに対してキャッシュとして機能する事例(>>945)で反論しただけ
日本語が読めない人の相手は本当に疲れるな
970:デフォルトの名無しさん
16/05/21 13:29:04.39 XOGjEe4x.net
>>947
> これらの「最適化」を早いうちからするなって言うのが、
> 「早すぎる最適化は諸悪の根源である」という言葉だ。
その通りだが、逆に言えば最後に最適化してもいいので、それは最適化テクニックを否定する格言ではない
Charles Cook は「時期尚早な最適化」と称しているが、「しかし逆にシステムレベルのソフトウェアを設計するときは、性能問題を常に念頭に置くべきだ」ともいっている
jquery.jsやjQuery pluginはそれに近い立場にあるので常に最適化に気をつかうべきだろう
971:デフォルトの名無しさん
16/05/21 13:51:27.17 XRbWBdK9.net
プログラム全体の処理速度に大きく影響するレベルの本格的な最適化の話と、ちょっとしたローカル変数に値を代入する話をごちゃ混ぜにしているのがおかしいだろ。クヌース先生も呆れている。
972:デフォルトの名無しさん
16/05/21 14:02:20.57 XOGjEe4x.net
>>937-939
ドヤ顔で披露した割には最後のコードも問題があるな
> $('#hoge tbody:first-child tr td').addClass('bar').each(function() {
> console.log(this.firstChild.data);
> });
子孫セレクタを使っているのでコストが重く、ネストされたtable要素でおかしくなる
addClass, each で二重走査してるのが無駄
forEach 系で実装可能なのは改めて教えてもらわずとも分かっている
繰り返し処理するのにわざわざ関数呼び出しするのが煩わしいだけ
>>940
説明用変数はキャッシュとしても機能するので同じ事
そもそも、俺は説明用の為だけに変数を定義することはしない
同じオブジェクトを2回以上参照するから変数を定義するだけだ
それはキャッシュでもあり、説明用変数でもある
人間は document.getElementById('hoge').tBodies[0].rows[i].cells[j]; のような長い名前を何度も見て同じものと即座に認識することは出来ない
これが1回だけ参照するなら変数にはしない
2回以上登場するから同じ名前を何度も読まなくて済むように、同じオブジェクトを何度も参照しなくてすむように変数を定義する
973:デフォルトの名無しさん
16/05/21 14:16:15.05 +6m5wOVw.net
ＩＤコロコロするやつのレスなんて信憑性0
974:デフォルトの名無しさん
16/05/21 14:16:40.41 q2Zh6d9K.net
>>959
> おまえが>>936で「"ローカル" 変数に入れることで見やすくすることはキャッシュとは別の話だって
> 言ったよね？」といったことに対してキャッシュとして機能する事例(>>945)で反論しただけ
だから、それをするなって>>951で書いたんだよｗ
> というとどっかの馬鹿が変数に入れるなということなのか！？とか言い出すんだよな（苦笑）
> 「変数に入れるとキャッシュの効果もありまぁす」じゃねーんだよｗ
> 楽な方法をとった結果、たまたまそうなる話なんかどうでもいい。
やっぱりという感想しかねーよｗ
最初の>>898の話は「検索コストがかかるという理由で変数に入れたほうがいいですか？」だ。
スタート地点は「変数に入れない状態」その状態から初めて「検索コスト」だけを理由に
「変数に入れたほうがいいか」って質問だろ。
だからそれはやるなって俺は言ってるんだが、
お前の反論（？）は、（別の理由で）「変数に入れた状態」から初めて
「キャッシュの効果もありまぁす」じゃねーかｗ
全く議論になってないんだよ。
975:デフォルトの名無しさん
16/05/21 14:18:39.39 q2Zh6d9K.net
>>962
> 子孫セレクタを使っているのでコストが重く、ネストされたtable要素でおかしくなる
ネストされた～は単に > を付け忘れただけ。
で、いきなりコストの話をすんな。
「早すぎる最適化は諸悪の根源」だ。
それで問題がある場合に限って、最適化すればいいだろ。
HTMLが十分にシンプルであれば、コストが重いといっても
この程度じゃ1回あたり10nsも差は出ないから。
976:デフォルトの名無しさん
16/05/21 14:21:45.75 q2Zh6d9K.net
>>960
> Charles Cook は「時期尚早な最適化」と称しているが、「しかし逆にシステムレベルのソフトウェアを設計するときは、性能問題を常に念頭に置くべきだ」ともいっている
念頭に置くのと、実際に行動するのは別の話。
この2つを区別できてないやつが多いんだよなｗ
念頭に置いて、問題なるんじゃね？って思って最初から最適化してしまうのが
「時期尚早な最適化」
念頭に置いた上で、問題が起きてから改善すればいい話だと俺は言ってる。
977:デフォルトの名無しさん
16/05/21 14:40:58.65 q2Zh6d9K.net
少し言葉が足りなかったら補足。
念頭に置いた上で、問題が起きてから改善すればいい話だと俺は言ってる。
ここで、念頭に置くというのは、問題が起きてから改善すればいいが
その改善がしづらくなるような設計にはしないということ。
あとから簡単に問題を改善できることならば、
あとから問題を改善すればいい話。
念頭に置くというのは、最初に改善してしまうのではなくて、
改善しにくい方向にはしないということ。
978:デフォルトの名無しさん
16/05/21 15:33:15.09 gfKVobxg.net
src属性の特定の数字を取得した後そのリンクを自動でクリックさせるuserjsを書きたいのですがどう記述して良いのかさっぱりわかりません。
979:デフォルトの名無しさん
16/05/21 16:34:58.52 FusMkK2d.net
じゃ諦めましょ
980:デフォルトの名無しさん
16/05/21 16:35:02.87 KBCgSbCY.net
>>968
取得はドム原理主義者得意の引っこ抜き。
発火タイミングはカーソルクリックや当たったら。
読み込み後発動発火。
スクロールで表示されたら発火(その場合、表示中の現在位置を取得が必要)
981:デフォルトの名無しさん
16/05/21 16:39:21.48 qqwYRUOU.net
>>968
どういう事？contentに探したいものを含むaタグに、clickイベントを発火させればいいのでは？
キャッシュ論が続いているから思うけど、なるほどnodeのスレでjs使いがバカにされる訳だな。
domアクセスを最低限にしたいからキャッシュさせるさせない、セレクタの結果を保持するってその発想自体気が触れてる。
だから、変数とpost内容の不一致とかわけわからん類の障害出すんだろ。
中身になるべきものを持っといて、全部jsで書け。初期コンテンツだけ置いとけ。readyでもいで差し替えろ。
そうすりゃ差分なんか自分が知ってるんだから、どう更新すべきか自明だろ。
982:デフォルトの名無しさん
16/05/21 17:02:31.50 q2Zh6d9K.net
> 中身になるべきものを持っといて、全部jsで書け。初期コンテンツだけ置いとけ。readyでもいで差し替えろ。
> そうすりゃ差分なんか自分が知ってるんだから、どう更新すべきか自明だろ。
その説明じゃ誰にも通じないだろうなｗ
全部jsで書けとか言ってるから、逆に面倒になってるだけだろう。
983:デフォルトの名無しさん
16/05/21 18:57:20.03 qpbh63BA.net
>>964
するな、はおまえの勝手な持論
DOMのキャッシュだけで「最適化が時期早々」の結論に持って行く流れがおかしい
必要なタイミングで最適化すれば問題はないのだから
そして、その時に様々な最適化の手法を知っているべきであって「早すぎる最適化」を理由に最適化テクニックを覚える機会を失うのは本末転倒
タイミングは重要だが、テクニックとしては知っておくべき
>>965
おまえは何かといえば「早すぎる最適化」しかいわないな
それはミスした時の言い訳に使われるものじゃないぞ
コストよりもセレクタが正しく機能しない事の方がに問題
984:デフォルトの名無しさん
16/05/21 22:48:21.57 q2Zh6d9K.net
>>973
> コストよりもセレクタが正しく機能しない事の方がに問題
だからそう言ってるだろ。
問題になってない検索コストのために最適化することで
セレクタが正しく機能しない（可能性がある）
正しく機能させるために追加で考えなくちゃいけなくなるので
早すぎる最適化は何も良いことがない。
985:デフォルトの名無しさん
16/05/21 22:53:03.77 q2Zh6d9K.net
>>973
> 必要なタイミングで最適化すれば問題はないのだから
だからそう言ってるだろ。
> jQueryのセレクタはid以外を指定する場合、検索のコストがかかるから
> できるだけキャッシュしたほうがいいってきいたんだけど、
とか言ってるから、それは必要なタイミングで最適化するべきことであって
通常は、できるだけキャッシュなんてことをする必要はない。と言ってる。
誰も変数に入れることが最適化のテクニックじゃないとか言ってない。
これを「最適化のテクニックなんだ。ほー。知らなかったー」とか思ってるのはお前だけ。
こっちも元レス人も最適化のテクニックだと知った上で、
元レスの人「できるだけやったほうが良い？」
俺「できるだけやらないほうが良い」
という会話をしてる。
お前は議論の内容から数歩遅れてるんだよｗ
986:デフォルトの名無しさん
16/05/22 07:40:36.92 LUJui5ls.net
初心者ですが、ためしにある.jsファイルとか
書いてみて、動作確認しようにも連携するファイルがまだできていないので、本当にできているのかいないのか、検討もつきません。
１つ目のファイルが動作確認で動くのかもわからないまま、他の連携ファイルを書くことは疑心暗鬼が増大しすぎて出来なくなってしまいます。
自分が書いたJavaScriptは動くのかを確認する方法がありましたら教えてください。
987:NAS6 ◆n3AmnVhjwc
16/05/22 08:35:04.93 KRZX8LQR.net
検証用のテストコードを作ればいいのに、なんでそうしないの？
988:デフォルトの名無しさん
16/05/22 08:57:55.90 JfPxy53K.net
>>974
> 問題になってない検索コストのために最適化することで
> セレクタが正しく機能しない（可能性がある）
おまえが>>938セレクタの問題点を何も理解してないことはわかったのでもういい
989:デフォルトの名無しさん
16/05/22 11:41:58.58 Tdpnvle1.net
>>974
> これを「最適化のテクニックなんだ。ほー。知らなかったー」とか思ってるのはお前だけ。
おまえは読解力を身に着けた方がいいんじゃね？
> 元レスの人「できるだけやったほうが良い？」
> 俺「できるだけやらないほうが良い」
アホか
「必要なタイミングで最適化した方がいい」が正だろ
やるかやらないかはその時に判断するもので「やらない」の結論を出すのは時期早々なんだよ
「早すぎる最適化」を持ち出してその結論なるんじゃ、おまえは「早すぎる最適化」の何を理解したんだ？といわれても仕方ないぞ
990:デフォルトの名無しさん
16/05/22 11:42:40.25 7nFY/y9y.net
そろそろ死ねよクソカス
991:デフォルトの名無しさん
16/05/22 12:12:36.77 +7XP2PmG.net
>>977
検証用テストコードの存在を知らなかったからです。
少しくぐってみます。ありがとう。
992:デフォルトの名無しさん
16/05/22 12:32:37.42 WwOYSBmy.net
>>979
はぁ？
お前日本語読解力ないなｗ
できる限りやったほうが良い。の問いにYESって答えたら
できるかぎりやってしまうだろｗ
だからNOだよ。やらないほうが良い。
で、お前は反論するの？
しないなら黙ってろよｗ
993:デフォルトの名無しさん
16/05/22 12:34:37.15 WwOYSBmy.net
> やるかやらないかはその時に判断するもので「やらない」の結論を出すのは時期早々なんだよ
結論出さなくても、どちらかを書かないといけないだろｗ
結論が出てない状態なら、最適化をやらない方を選べって言ってるんだよ。
早すぎる最適化は諸悪の根源だからだ。
994:デフォルトの名無しさん
16/05/22 12:48:36.51 XAOP4iva.net
無駄に長文はkenokabeを彷彿とさせるからﾔﾒﾛ
995:デフォルトの名無しさん
16/05/22 12:53:36.96 CHy0uo/r.net
NG突っ込んどけ
996:デフォルトの名無しさん
16/05/22 16:25:24.06 5k8BO1OF.net
>>976-977
スタブのような、テストに都合のよい、返り値を返す関数を作る
関数から、10が返ってきたと想定する
a = 10;
ここで、テストでa、つまり10を使う
JSにも、JavaのJUnitみたいな、テスティング・フレームワークがあるのでは？
997:デフォルトの名無しさん
16/05/22 16:43:20.80 X0ga1YPw.net
「早すぎる最適化」は初めから全ての最適化を施すと保守性が著しく落ちるから何よりも先にプロトタイプを完成させた後にボトルネックとなるコードを探して最適化しましょう、という理論。
一般的には速度よりも厳密性、堅牢性、安全性、前方互換性、後方互換性あたりがより重要視されるからね。
ただし、これは保守性が下がらなければ最適化を施してもいいという事でもある。
代表的なのでは array.length を変数にキャッシュするテクニックがあるね。
ループ回数が10万回とか、著しく大きな回数参照される場合に大きな速度差として現れる。
>>945は参照回数の重要性を諭してくれているけど、ようするにローカル変数かグローバル変数かはどうでもよくて参照回数が多いところを最適化すべきなんだよね。
ボトルネックが初めから経験的についてわかっていれば、「早すぎる最適化」理論のタイミングを待たずして最適化を施すことも出来る。
結論としては>>898への答えは「経験的にボトルネックと分かる場所には即座について最適化しましょう。そうでなければ、プロトタイプを作ってからボトルネックを探しましょう。」
蛇足だけど、厳密性を上げることで結果的に速度的な最適化が施される場合もあって>>962のセレクタの指摘がそれ。
'#hoge>tbody:first-child>tr>td' を指定する事で>>927と等価になり、>>938では別の要素を拾ってしまう危険性がある。
厳密性を上げることで保守性が下がるわけでもなし、修正すべき箇所だと思うよ。
998:デフォルトの名無しさん
16/05/22 17:36:54.53 FTGg1fS8.net
一昨日から蛇足続きですけどそれは…
999:デフォルトの名無しさん
16/05/22 17:45:35.33 Y6WruITb.net
array.lengthを変数に入れるなんて言うのは最適化のテクニックなのか？
誰でもやるぞ。
1000:デフォルトの名無しさん
16/05/22 17:53:12.31 tpad688i.net
すごーいつづきはぜひぶろぐかなんかひらいてかいたほうがみんなみられていいとおもうー();
1001:デフォルトの名無しさん
16/05/22 18:09:24.31 WwOYSBmy.net
>>987
> 結論としては>>898への答えは「経験的にボトルネックと分かる場所には即座について最適化しましょう。
> そうでなければ、プロトタイプを作ってからボトルネックを探しましょう。」
何汎用的な話にしてるんだよｗ わざとらしいな。今はjQueryのセレクタの話だろ。
>>898の話は経験的にボトルネックと成りえない場所だから、
それを「できるだけキャッシュしたほうがいい」と言ってることに対して
経験的にボトルネックにならないから「できるだけしないほうがいい」と言ってるんだが。
1002:デフォルトの名無しさん
16/05/22 18:11:58.37 WwOYSBmy.net
>>989
今はarray.lengthの話はしてない
1003:デフォルトの名無しさん
16/05/22 18:14:27.64 BAs222Jl.net
うるせーから埋める
1004:デフォルトの名無しさん
16/05/22 18:15:11.70 E7LBxgcJ.net
あ
1005:デフォルトの名無しさん
16/05/22 18:15:40.12 nPHdXwiF.net
あ
1006:デフォルトの名無しさん
16/05/22 18:15:56.89 0PCmMNra.net
あ
1007:デフォルトの名無しさん
16/05/22 18:16:13.55 5hn9uytB.net
あ
1008:デフォルトの名無しさん
16/05/22 18:16:30.05 2Z+XxbZ+.net
あ
1009:デフォルトの名無しさん
16/05/22 18:16:46.61 ax9WOfgG.net
あ
1010:デフォルトの名無しさん
16/05/22 18:17:03.03 yL49bMPd.net
あ
1011:1001
Over 1000 Thread.net
このスレッドは１０００を超えました。
もう書けないので、新しいスレッドを立ててくださいです。。。
life time: 74日 17時間 18分 27秒
1012:1002
Over 1000 Thread.net
２ちゃんねるの運営はプレミアム会員の皆さまに支えられています。
運営にご協力お願いいたします。
──────────
《プレミアム会員の主な特典》
★ ２ちゃんねる専用ブラウザからの広告除去
★ ２ちゃんねるの過去ログを取得
★ 書き込み規制の緩和
──────────
会員登録には個人情報は一切必要ありません。
月300円から匿名でご購入いただけます。
▼ プレミアム会員登録はこちら ▼
URLﾘﾝｸ(premium.2ch.net)
URLﾘﾝｸ(pink-chan-store.myshopify.com)
1013:過去ログ ★
[過去ログ]
■ このスレッドは過去ログ倉庫に格納されています