1 名前:デフォルトの名無しさん mailto:sage [2016/05/22(日) 19:51:33.37 ID:WwOYSBmy.net] JavaScript を自ら学ぶ人のための質問スレッドです。 >>2-4 のテンプレを読んだ上で質問してください。次スレは>>950 が>>2 のテンプレ案(本スレで改善案があれば考慮)を元に立ててください ■質問を書く上で (1) 煽り、コード制作依頼等、人を不快にさせる投稿はご遠慮下さい。公序良俗を守った応対を心がけてください。 (2) 他の人に迷惑をかけるスクリプトの質問はご遠慮ください。 (ブラクラ、[戻る], [閉じる], [クリック] の妨害、画面占有など) (3) 質問者及び議論を行う人はメール欄を空欄にし、名前にレス番を入れることを強く推奨します。回答者はなりすましを判断できませんので、なりすましが現れても自己責任となります。 (4) 常に自発的に調べる心構えを持ってください。 具体的には「自分で調べてから質問する」「回答をもらってわからない単語があればGoogle検索してみる」など。 わからない内容を代わりに調べてくれる回答者をお望みの方は余所で質問してください。 (5) 出来るだけ一般的な用語を使用してください。脳内オレオレ用語は混乱の元です。 (6) 出来るだけサンプルコードを掲示してください。言葉による説明は行き違いが生まれる場合があります。 ※必ず「問題の事象が再現されること」を確認してください。 必要な部分だけ切り出したつもりで現象が再現できていなかったケアレスミスがしばしば見られます。 (7) サンプルコードに HTML が含まれる場合は validator.w3.org/ で [Check] してみてください。 (8) 質問を具体的かつ詳細に書くと回答を得られやすいです。>>2 の質問テンプレートを活用してみてください。 (9) 時にはあなたが望む「答え」だけでなく、「意見」などが寄せられる場合もあります。
552 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 13:39:59.04 ID:n8nif7qi.net] >>534-535 2chはHTMLタグ使えないからjsFiddleみたいな投稿サイトを使うといいよ 前まではテンプレにあったんだが 一見ネストが深いように見えるけど結構シンプルに纏まってると思う fetchの部分を外に出せば視認性がよくなるんじゃないのって程度 しいて言えばgetImageData内でPopUp関数を呼び出してるのがしっくりこない getImageData自身もPromiseインスタンスを返してresolve(json)するのがいい
553 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 13:59:44.48 ID:ijG8u0pc.net] >>539 今日だけ師匠と呼ばせてください 自分でもそこはなんか気持ちわるかったんです promiseチェインがcatchで始まった挙句、頭でっかちで でも外に出すというのはどうやるんでしょう・・・ fetchからの処理をまとめた関数(返り値は最終的なjsonデータのpromise)を外に定義して、それを呼び出す感じですか? そしてgetImageDataがPromiseインスタンスを返すように書き換えるには、ええと・・・(思考フリーズ)
554 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 14:41:46.28 ID:n8nif7qi.net] >>540 getCachedDataと同じ書き方をfetchとgetImagedataでやるだけだよ
555 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 15:16:00.71 ID:NAulAg4n.net] >>539 > 一見ネストが深いように見えるけど結構シンプルに纏まってると思う え? ちょっとまてよ?
556 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 15:45:12.80 ID:NAulAg4n.net] >>540 これgetImageDataの中でキャッシュあるか調べてるからいけないんだよ。 一般論として、キャッシュというのは存在を意識しないようにコードにするべき。 関数の役割ってのが書いてないし、これだけじゃ読み取れないから難しいんだが、 そのgetImageDataっていうのは画像があれば、ポップアップウインドウを表示するんだろう? であれば、getImageData() ではなくて showImageData() とかいう名前にするべき。 そして、getCachedData()ではなく、こっちをgetImageData()にするべき。 getImageData()っていうのは単にImageDataを返すだけ、その中でキャッシュを使っていようが 使っていまいが関係ないし、別の仕組みのキャッシュを使うかもしれない。がgetImageData()を 使う人からはそんなこと知ったことじゃない >>535 のコードは、chrome.storage.local を使っている箇所が、getCachedDataと getImageDataの2つに分かれているだろ? 関数の役目が明確に分離されていないからそうなる。 非同期の書き方の問題の前にまずやることがあるってことさ。 あと、スペース1個でインデントは見にくいからやめろw 最低でも2個だ。
557 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 15:47:35.86 ID:NAulAg4n.net] で、これが修正コード例だ。 やりたいならばchrome.storage.local.getをPromise化する事もできるがね。 function getImageData(query) { return new Promise(function(resolve, reject) { chrome.storage.local.get(query, function(cachedJson) { if (isExist(cachedJson)) { resolve(cachedJson[query]); return; }
558 名前: fetch(window.googleCustomSearchApi + query) .then(function(response) { var json = response.json() chrome.storage.local.set({[query]: json}); resolve(json); }) .catch(function(err) { reject(err); }) }); }); } function showImageData(query) { getImageData(query) .then(function(json) { createPopUp(json.items[0]); positionPopUp(); }) .catch(function(err) { console.error(err); }); } [] [ここ壊れてます]
559 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 15:56:05.66 ID:NAulAg4n.net] で、どうしてもgetImageData()の中身をPromise化したいならこんな感じかね? 動かせる環境がないので、これで正しく動いているかはしらない。 元のコードに合わせてisExistがfalseならreject()しているが、 nullを返して、データがnullだったらfetchしたほうがいいような気もするが。 function getImageData(query) { return new Promise(function(resolve, reject) { chrome.storage.local.get(query, function(cachedJson) { if (isExist(cachedJson)) { resolve(cachedJson[query]); } else { reject(); } }).cache(function()) { return fetch(window.googleCustomSearchApi + query) }).then(function(response) { var json = response.json() chrome.storage.local.set({[query]: json}); return json; }); }
560 名前:534 mailto:sage [2016/06/11(土) 16:03:43.18 ID:ijG8u0pc.net] >>543 自分でああでもないこうでもないと考えてコードを修正していたら返信遅れました か ら く ちwwww と思って訂正コードと自作スパゲッティを見比べたらグウの音も出ませんでした この整理されたしっくり来る感じ 勉強になります あとは・・ エラーハンドリングについてわからないところがあります!
561 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:07:35.34 ID:ZhHlBSFM.net] >>537 良い記事だ。 が、前から疑問なのだが、何故お前らはネストにこだわるのだ? 俺は展開して書いていて、特に問題を感じない。 要求が「脳筋無限ループの例ぐらい単純な構造で」「非同期処理を実現する」 「処理は上から下へ」「ネストが深くならない」ならば、これでいいと思うのだが。 好きずきはあると思うが、このような場合にでも例としても出てこないのは謎なんだが。 function Main(){ console.log("a"); setTimeout(Main_1,2000); function Main_1(){ console.log("b"); setTimeout(Main_2,1000); } function Main_2(){ console.log("c"); } } Main();
562 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:12:31.73 ID:NAulAg4n.net] >>545 は本当に動くかどうかわからない。 実はcacheのあとにthenをつなげたことないんでねw まあ関数の中を試行錯誤すりゃできるだろうと思うけど。 最悪>>544 でいいし。 "何をする" 関数か?(クラスか?)ってことをよく考えることが重要。 動くようにコードを配置するんじゃなくて、何をする関数か?を定義して それに合わない所はたとえコードが増えても外にだすべきだし、 入れるべきだと思えば、どうにかして持ってくる。 もちろん場合によっては、"何をする" の定義を変えてもいいけど はっきり定義させることが重要。 例えば、このgetImageData()はchrome.storage.local を使うことが前提になってるけど、 いろんなキャッシュの仕組みを入れ替えられるようにする関数(というかクラスになるだろうけど)を定義すれば getImageData()の中からはchrome.storage.localが消えて、キャッシュの仕組みを 入れ替えられるだけのことをするクラスができあがるわけ。
563 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:15:59.14 ID:NAulAg4n.net] >>547 > が、前から疑問なのだが、何故お前らはネストにこだわるのだ? やりかけで覚えておかなければならないことが増えるからだよ。 Aが終わってからBをして、Bが終わってからCをする。というやり方よりも Aの途中でBをして、Bをしているその途中でCをするほうが疲れるから。 Cが終わったら、Bはどうなったんだっけ? そしてAはどうなったんだっけ?って 戻って行かないといけないからね。 言い換えると可読性の問題だからだ。 非同期とかそういうところが問題になってるんじゃない。 やりかけの作業を脳に貯めておくと言うところが問題なんだよ。
564 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:20:02.55 ID:NAulAg4n.net] >>547 あと、そのコードの問題は「名前」が増える。 名前が増えるっていうのも、いちいち記憶して おかなければならないことだから面倒 限りある記憶領域は、記憶する価値のある物、 汎用的でどこでも使えることだけに絞る必要がある。 覚える価値がないものは覚えなくていい。 そのコードは関数の
565 名前:シ前が増えているだけだが、 クロージャーじゃなくなっているから、変数の名前も増えるだろうね あと処理があちこちに飛ぶから、コードが長くなったらスクロールする回数も増えるよw もちろん関数が長ければ分けるが少なければその場に埋め込むという意味だ。 [] [ここ壊れてます]
566 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:26:52.64 ID:ZhHlBSFM.net] >>549 そうか?ぶった切られているだけで、実は同じだろ。追加部分を見やすいように編集しておいた。 同期的に「放り投げて終わり」にしたいという気持ちも分かるが、無理にネスト/Promise化するより見やすくないか? 脳内 s/setTimeout...function(){// で全く問題ないはずだが。 function Main(){ console.log("a"); setTimeout(Main_1,2000); // 追加 function Main_1(){ // 追加 console.log("b"); setTimeout(Main_2,1000); // 追加 } // 追加 function Main_2(){ // 追加 console.log("c"); } // 追加 } Main();
567 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:29:29.39 ID:tWgkOxEq.net] もうそれはasync/await、もしくはgenerator/yieldを使うしかないよ。
568 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:35:45.96 ID:NAulAg4n.net] >>551 サンプルコードでやってるからだめなんだよw お前、頭の中で、このコードはsetTimeoutしかない。Main以外の名前は存在しない。 しかもそれぞれ短いコードで呼び出す順番に並んでいるいう「記憶」に頼ってコード読んでるだろ? それはお前だから出来ること。(短いから俺でも出来るけどさw) 実際に業務ではそういう「記憶」には頼れない。コードはもっと長いし 汎用的じゃないものは名前も覚えられない。(何日も経ったら忘れる) 汎用的でないものには、適切な名前をつけることも難しい。 仮にそのコードのまま、記憶に頼れないように書き換えてみようか? さっきと同じように気楽に読めるかい? どこでどう呼び出しが行われてるか探すだろ? function Kjads(){ console.log("a"); function cgrd(){ // あrtが console.log("c"); } // あrtが badf(fgaz); // あrtが function fgaz(){ // あrtが console.log("b"); asdfa(cgrd); // あrtが } // あrtが } Kjads();
569 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:36:17.69 ID:ZhHlBSFM.net] >>550 「名前」が増えるのは事実だけど、関数内関数で隠蔽するし、 そもそも処理を必ず「上から下に」書くのでその点は問題にならない。 つまり、このように展開した場合、その関数は直前のsetTimeoutからしか呼ばれない。 したがって、覚える必要はないし、 処理があちこちに飛ぶこともない。(
570 名前:ただこれはそう書く、ということだが) ちなみにクロージャーにはなっている。 > もちろん関数が長ければ分けるが少なければその場に埋め込むという意味だ。 いや、結構糞長い奴もそこに書くよね? $.ajaxとかその場にだらだら全部書くほうが普通じゃないか? [] [ここ壊れてます]
571 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:39:28.40 ID:NAulAg4n.net] >>554 もうそろそろ>>553 をみて、記憶に頼ってるなって自覚したはずw > ちなみにクロージャーにはなっている。 なってない function Main(){ var v = 1 setTimeout(function() { console.log(v); }, 1000); } ↓ エラーになる function Main(){ var v = 1 setTimeout(Main_1, 1000); } function Main_1(){ console.log(v); } 変数vを渡すために、名前が増えるwww
572 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:39:52.50 ID:ZhHlBSFM.net] >>553 ああ、ちょっと遅れたが、それは既に書いたように、俺内コーディングルール 処理を必ず「上から下に」書く で対応している。というか、そもそも意図的に可読性を落とす必要はないだろ。 「上から下に」書けるところでは「上から下に」書くのが常識だろ。 名前もこの手の場合は XXX_1, XXX_2, ..... と順に付けていく。おかしな名前を付けたりすることはない。
573 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:40:41.06 ID:NAulAg4n.net] >>554 > $.ajaxとかその場にだらだら全部書くほうが普通じゃないか? 下手なやつが多いだけ。 俺は関数に分けるべき理由が生まれれば当然分ける。
574 名前:534 mailto:sage [2016/06/11(土) 16:40:58.22 ID:ijG8u0pc.net] >>544 の綺麗なコードを見る前に、API問い合わせの部分を関数に分離しようと思って、fetch用の追加のエラーハンドリングも加えてこう書いてしまったんですね(お蔵入りですが、このなかで疑問が沸いたので・・) function googleCustomSearch(query) { return fetch(window.googleCustomSearchApi + query) .then(handleNetworkError) .then(response => response.json()) .then(json => { chrome.storage.local.set({[query]: json}); return json; }) .catch(err => { // 【ここが疑問】 }); } function handleNetworkError(response) { if (!response.ok) { throw new Error(reponse.status, response.statusText); } return response; }
575 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:44:00.74 ID:NAulAg4n.net] >>556 > 処理を必ず「上から下に」書く > > で対応している。というか、そもそも意図的に可読性を落とす必要はないだろ。 > 「上から下に」書けるところでは「上から下に」書くのが常識だろ。 ほらなw お前の常識w 何処かで、function hoge() {} じゃなくて var hoge = function() {} って 書けって言っていたやつが居るが、 (ちなみに俺はその必要がないならばfunction hoge()って書け派) これは上から下に書けない。 hoge(); var hoge = function() {} hage() var hage = function() {} まあ 「上から下に」"書けるところでは" って書いているところからもわかるように 書けない場合があると自覚してるんだろうがw その書ける所と書けない所、ここは書ける所だという前提を知ってるのも「記憶」だよねw
576 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:44:38.09 ID:ZhHlBSFM.net] >>555 何が言いたいのか分からんが、これでいいか? function Main(){ var count = 0; console.log('a: '+count++); setTimeout(Main_1,2000); function Main_1(){ console.log('b: '+count++); setTimeout(Main_2,1000); } function Main_2(){ console.log('c: '+count++); } } Main();
577 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:46:15.25 ID:NAulAg4n.net] >>560 勝手にコード書き換えんなよw function Main(){ var count = 0; console.log('a: '+count++); setTimeout(Main_1,2000); function Main_1(){ var v = 1; console.log('b: '+count++); setTimeout(Main_2,1000); } function Main_2(){ console.log('c: '+count++); console.log(v); // エラーになる } } Main(); それともお前、ローカル変数は作るな。 全部クラス変数にしろとか言う派なのか?www
578 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:50:41.71 ID:NAulAg4n.net] >>558 いやw 何が疑問なのかさっぱりわからないんだがwww
579 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:51:15.94 ID:ijG8u0pc.net] googleCustomSearchの呼出元にhandleNetworkErrorで起きたエラーを伝えるためには、 1.最後のcatchのなかでもう一度new Error(err)を投げる 2.Promise.reject(error)を返す 3.そもそも書き方が悪い、 >>544 みたいにfetchをnew Promiseでラップして、reject(err)してください どれが適切なんでしょうか
580 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 16:54:15.22 ID:ZhHlBSFM.net] >>561 いや勝手に書き換えたのはお前だろ。 >>555 のコードでは var v=1 は Mainの中じゃないか。 Main_1の中で新しい変数が必要なら、 ・Main_2をMain_1内に移動する(ネストが増える) ・v を Main内で定義しておく のどちらかだろ。俺なら後者的アプローチを取ることが多いと思う。 ただ、途中で変数が必要なときも管理しなくていいというのは確かにメリットかもな。
581 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 17:02:05.55 ID:ZhHlBSFM.net] >>559 それはクラスの時の話だ。 クラスの場合は、コンストラクタだけ呼べても意味がないだろ。 Hoge.prototype.XXX = function(){...}; の記述が「順序が必要」なので、どのみちその記述以降じゃないと new Hoge() 出来ないんだよ。 誰も普通の関数 hoge を全部 var hoge = function(){} と書けとは言ってない。 お前が話について来れてないだけ。 まあ分かった、お前が「文法的に読み込める方が善」というのならそれでいい。 で、他の連中もそうなのか?
582 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 17:09:12.67 ID:ZhHlBSFM.net] >>557 ちと細かいがこれにも言っておくと、 > 俺は関数に分けるべき理由が生まれれば当然分ける。 彼等は「関数に分けるべき理由」が分からないのではなくて、その必要がないという判断でそこに書いているんだよ。 下手なわけではない。 通常、$.ajaxの場合はそこでしか使われない関数になるから、ネスト派ならそこにだらだら書く方が自然なんだ。 ただ、「ネストが深くなる」のが嫌いなくせに、「ネストでしか書かない」のが俺には疑問なんだ。
583 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 17:31:06.27 ID:aOjkg+66.net] >>523 毎度ありがとうございます。 > 機会があれば、是非新しい世界を作ることに挑戦してみてほしい 今回のことで色々勉強させてもらいましたので、 次に生かせたいと思います! 色々ありがとうございまし!
584 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 17:31:44.01 ID:aOjkg+66.net] おっw ×まし! ○ました!
585 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 18:02:04.85 ID:NAulAg4n.net] >>566 > 下手なわけではない。 > 通常、$.ajaxの場合はそこでしか使われない関数になるから、ネスト派ならそこにだらだら書く方が自然なんだ。 > ただ、「ネストが深くなる」のが嫌いなくせに、「ネストでしか書かない」のが俺には疑問なんだ。 <
586 名前:br> いや、俺は$.ajaxでDeferred使うし、やっぱり下手なだけじゃね? [] [ここ壊れてます]
587 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 18:05:04.82 ID:NAulAg4n.net] >>563 > どれが適切なんでしょうか どれか迷ったら、一番楽な方法を選べばいいよw 何でもそうだけど原則として楽になるように作られてるわけなんだしさ。 しばらくPromiseは触ってないから忘れたが、throwで任意のエラーオブジェクトでも 投げればいいんじゃね?
588 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 18:07:10.39 ID:NAulAg4n.net] >>565 > クラスの場合は、コンストラクタだけ呼べても意味がないだろ。 意味あるぞ? function NantokaManager(params) { this.params = params; this.run = function() {} } var manager = new NantokaManager; manager.run() とかさ。
589 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 18:08:34.97 ID:NAulAg4n.net] >>565 > 誰も普通の関数 hoge を全部 var hoge = function(){} と書けとは言ってない。 > お前が話について来れてないだけ。 俺もそんな話はしていない。 ついてこいよw
590 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 19:28:03.24 ID:ZhHlBSFM.net] >>571-572 すまんが意味が分からんし、お前の主張も分からん。 その場合の駄目なコードとそちらの主張を対で出せるか?(以下例のように) 一応こちらの意見をもう一度言っておくと、 function hoge(){} // OK。通常の関数はこちら。 var Hoge = function(){}; // コンストラクタの場合はこっちの方がよさそう となる。 そのコードが「共通プロトタイプを持たない場合」に限定しているつもりなら、勝手にどうぞでしかない。
591 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 19:33:03.15 ID:NAulAg4n.net] >>573 え? お前コードは必ずプロトタイプを持つものっていう前提なの? どうやらお前は限定しているようだが、 俺は、どんな限定もしていない。
592 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 19:40:17.64 ID:ZhHlBSFM.net] >>574 何が言いたいのか分からん。 そちらが説明しきれないのなら、相互理解は不能だから終わりでいい。
593 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 21:27:53.17 ID:NAulAg4n.net] >>575 お前がこう言ってきたんだろ? > クラスの場合は、コンストラクタだけ呼べても意味がないだろ。 だから俺は、コンストラクタだけ呼べても意味がある場合を言ったまで。 そしたらお前が「俺は共通プロトタイプをもった場合に限定してるんだ」って いい出したんだろw 俺はお前と違って何も限定していない。 コンストラクタだけ呼べても意味がある場合は 実際に存在している。
594 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 22:29:21.64 ID:ZhHlBSFM.net] >>576 なるほど、お前は人殺し君か。 どおりで話が通じないわけだ。
595 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 00:46:04.68 ID:hCgFiHr7.net] 多分人殺しくんとは俺の事だろうが、 >>576 は俺ではないし、正直、ID:ZhHlBSFMは頑張って色々回答してるけど実務レベルではないわ。 処理は上から下へ流れるように、なんて、飼い犬が死んでも言わんレベルのたわ言じゃん。 相互に呼び出し合う事のある関数なんかどういう順序で記述するんだろ。 イベンハンドラやら、タイマーのハンドラは、どの順序なのかな。 子供のお題目にしか聞こえん。 逆に何年目か教えてほしいわw 何度かこういう言い方してるけど、ああいう悔しがり方とキレ方する奴が具体的にすら言えないってことは、ほんとに経験無いガクセーさんなんだろうけど。
596 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 00:51:30.61 ID:CJz8oq6+.net] 後学のためにID真っ赤にしてる方々の書いた素晴らしいスクリプトを見せてくださいよ
597 名前:576 mailto:sage [2016/06/12(日) 00:51:41.20 ID:AMLyjVO9.net] >>578 お前のことだったんかw いきなりなんか言われて意味不明だったわ。 えーと何の話だっけ? 同期命令が使える言語で、 sleep(1000) console.log(1) sleep(1000) console.log(2) sleep(1000) console.log(3) と書きたくなるようなコードを、ネストして書くだけでも苦痛なのに、これを関数に分解して function Main_1() { sleep(1000) console.log(1) } function Main_2() { sleep(1000) console.log(2) } function Main_3() { sleep(1000) console.log(3) } とかやりたくないわけでw あと関数なのに、大文字から始めってる時点で、経験不足だってことがわかるよ。 念の為に言っておくと、上のMain_1とか言うのは俺がいい出した名前じゃないからねw
598 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 00:52:45.85 ID:AMLyjVO9.net] >>579 俺は何回も書いてるよ。 >>546 とか読んでみな。(俺が>>546 という意味ではない)
599 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 01:26:35.68 ID:npk74fIw.net] はいはい自演乙。 自演じゃないならレベルの高い>>578-581 の間で色々解決できるだろうから頑張れ。
600 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 01:33:50.33 ID:AMLyjVO9.net] > 自演じゃないならレベルの高い>>578-581 の間で色々解決できるだろうから頑張れ。 自演じゃないぞw 解決できるだろうけど、その前に何も解決すべき 課題が出てないじゃないかw そして、俺とお前は他人だろ? その理屈ならいろいろ解決できるだろうな。 何か言ってみろ。俺は手伝わないがwww
601 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 01:40:01.79 ID:m2Zib5nl.net] あ?なんだこのゴミみたいな木っ端は 最低でも1000行以上あるプログラムとして成立してるもんもってこいや
602 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 01:43:07.89 ID:npk74fIw.net] >>583 じゃあとりあえず>>578 の疑問に答えてやれよ > 相互に呼び出し合う事のある関数なんかどういう順序で記述するんだろ。 > イベンハンドラやら、タイマーのハンドラは、どの順序なのかな。 厳密にはこれは質問ではないから、「俺がどう書くか」の予想でもいいぞ。 とにかくお前らで色々話をしてみろ。 面白そうなら俺も加わるし、他も加わってくるだろう。 どうでもよければ無視されるだけだ。
603 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 02:01:39.12 ID:AMLyjVO9.net] >>585 どうせ質問に答えても自演じゃないって認めないんだろw まあ質問じゃないとお前もわかってるようだから、それにレスしろと いうならば、レスするのは何の問題もない。 >>578 > 相互に呼び出し合う事のある関数なんかどういう順序で記述するんだろ。 > イベンハンドラやら、タイマーのハンドラは、どの順序なのかな。 全くだなw 必ずしも上から書けるわけがない。 関数を呼び出される順番に書くとか言っているやつは馬鹿じゃないだおろうかね どういう順番で書くかは、ばらばらで人によってはアルファベット順ってこともあるだろう。 だから必要もないのに小さな関数に分けるなんて論外。 一つの関数にし、ネストがいやだからPromiseというのが使われるようになった。 この件に関しては、俺は>>578 と同じ意見だろうな
604 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 02:36:10.54 ID:npk74fIw.net] >>586 > この件に関しては、俺は>>578 と同じ意見だろうな そりゃ同一人物だからなw まあ、人殺し君も多投するタイプだから、待ってればレスが返ってくるはずだ。 そして本当に別人なら、色々意見交換すれば相違点が発見できるだろう。 それは俺関係なしにいいことだから、やってみることだね。
605 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 02:56:48.44 ID:hCgFiHr7.net] >>587 同一人物とかよくわからんな。 俺が連投するのは暇だからで、まともに過ごしてる時はそれなりにしか書かんけど。 10日間際は忙しいからな。 まさか本気で同一人物だと思ってたら笑えるww
606 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 03:17:12.70 ID:hCgFiHr7.net] >>586 うむ。アルファベット順ならまだマシだけど。 台帳に書いてある順とかちょっとびっくりする会社もあるからな。どことは言わんけど。 多分業務経験ない奴は笑うだけだが、 実務やってれば「NDA結ばないとリバースエンジニアリングすら難しい」という人間minifierだったりする。 必要に応じて関数は分けるべきというより、むしろ、関数は射影的であるべきだと思う。 コールバック関数のネストを「関数と関数を繋いでる」と単純に誤解して悪手だと思ってほしくないのが、 本来は関数と「関数を引数に取る関数」を定義して、「関数を引数にとる関数」に関数を食わせるが為の手段を逆に見た結果、「コールバック」って姿に見えてるだけ。 だから、yieldとジェネレーターを使おうが、Promiseを使おうが、本来は何一つ定義が変わってる訳でも、処理が変わってる訳でも無い。 関数に関数を食わせて値を取り出してる。 ・水を冷やして出来た物を、回転する鉋で削って、皿に落とし入れたもの、 ・皿で受け止めた、回転する鉋で削られた、冷やされた水、 ・回転する鉋で水を冷やして出来た塊を削り、皿に落としたもの は全部主題が同じもの。 水を冷やす、という関数と水を、値として「氷」と扱おうが、 かき氷機を「『水を冷やした物』削り機」と展開しようが、とう扱うかは使い方次第。 そこに、ニーズ外の要件なんて無いの。扱いにくいとかどうでもいいんよ。 また、undefinedの時と同じ勘違いしてるけどさ。 水→機械A→中間生成物→機械B→製品、 というシーケンスであらわすのが正しいのか、 ((水→機械A)→機械B)→製品、とフローで合わすのかは瑣末すぎるほどの差。 しかし、言われてるのは俺じゃないが、関数を大文字から始めると経験不足なんだwww C20年目かもしれんけどな、そういう人はww
607 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 03:21:29.67 ID:hCgFiHr7.net] こいつ、言われたくないことを人に言って安全圏に逃げたいだけなんだろうなぁ。ユング的なシャドウをもう少し悪く歪めたものが見えるわ。 voidとか、岡部健みたいな感じの破綻者に対して憧れてる人間、くらい性根がさもしい。
608 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 09:01:52.07 ID:AMLyjVO9.net] >>587 だから同一人物じゃねーよw
609 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 09:04:46.11 ID:AMLyjVO9.net] >>589 > しかし、言われてるのは俺じゃないが、関数を大文字から始めると経験不足なんだwww JavaScriptのコーディング規約はいくつもあるが、どれも関数は小文字で始める。クラスは大文字。 JavaScriptの経験があるなら、これに従ってるはずなので 見た瞬間気持ち悪さを感じるものだよw
610 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 09:10:26.75 ID:AMLyjVO9.net] >>589 > だから、yieldとジェネレーターを使おうが、Promiseを使おうが、本来は何一つ定義が変わってる訳でも、処理が変わってる訳でも無い。 処理が変わってるわけでもない・・・ならば 読みやすい方がいいだろ? 逆に考えれば、どんなに読みづらく書いても 処理が変わっていないというコードにすることはできるだろう? 読みづらくても処理が同じならいいと言うのか? そうではないだろう。 コードっていうのは書いて終わりじゃない。修正するもの。 むしろ読むほうが多いので、書くよりも読むほうが大事。 だから「可読性」という言葉がある。「可書性」という言葉はない。 > そこに、ニーズ外の要件なんて無いの。扱いにくいとかどうでもいいんよ。 俺はいまだかつて「時間的に早く書けて修正も簡単で、メンテナンスにかかるコストが低い」 という要件がなかったことはないぞw どんなプロジェクトでも安く仕上げることは、暗黙的に含まれてる要件だ。
611 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 13:15:07.43 ID:npk74fIw.net] 別人であればもっと議論が深まるはずだよな。 みんな見てくれてるから、せいぜい頑張れ。
612 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 13:19:20.40 ID:AMLyjVO9.net] どうあっても別人と認めたくないようだ(笑)
613 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 13:20:50.61 ID:AMLyjVO9.net] あ、なるほど。話が進むと今度は 一人だから意図的に会話できる。 それが自作自演の証拠って言うつもりだなw
614 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 15:04:26.78 ID:npk74fIw.net] いやきみ(ら)にとっては俺はただの馬鹿なんだろ。 だったら俺を引っ張り出す方向のレスを付けること自体おかしいだろ。 ガン無視してくれよw いずれにしても、君らが「別人」かどうかは皆が判断することであって、 俺が「自演」主張しても、逆に君らが「別人」主張しても意味無いんだよ。 どちらがキチガイかは皆が判断してくれる。 そして一つアドバイスをするなら、君のキャラなら>>592 は 「え?お前偉そうなこと言っている割には『大文字』のルールも知らないんだwww」 と煽るべきだった。キャラを変えると自然な自演に見えない。 君も煽りで相手を引き出すキャラなんだし、 このレスなら人殺し君のキャラなら確実に食いついてくるし。 本当に別人なら、君らが会話すれば君らにとっても得る物があるはず。だからもっとやればいい。 いちいち俺に対してキョロキョロしたり、 或いはここに来て会話を終わらせる方向にキャラ変更するのは不自然だ。 はい続きをどうぞ。
615 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 16:27:25.35 ID:9HFtYPb0.net] その人殺し君とかいうのに論破されちゃって 悔しくて大ハッスルしてるの?
616 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 16:54:18.02 ID:hCgFiHr7.net] >>592 そりゃ、javascriptの規約はいくらでもあるだろうけど、 javascriptの規約しか知らないどころか、ありえないような口ぶりで話してたからねw なんかからトランスパイルするなら、こうなるだろうな、って考えたりしないんだ、って感じ。 >>593 もちろんそう。 しかし、読みやすいのと書きやすいのと、定義しやすいのはそれぞれ別の概念だから、そういう意味では可読性に相当する単語は、独立性とか、依存度とか、抽象度とか 見方次第でいくらでも変わるものじゃないか? 安くあげる事より、致命的な不具合を出さない事の方が求められるプロジェクトしかやってないからな。 正直、安くなくても売れるから。 安くないほうが売れるとも言えるレベルで。 >>597 お前ほんと何と戦ってんの?
617 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 19:20:47.80 ID:npk74fIw.net] >>599 > わざとこの態度だからなぁ。 (>>327 ) 煽っていくスタイルプリーズ
618 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 19:39:15.05 ID:AMLyjVO9.net] > なんかからトランスパイルするなら、こうなるだろうな、 なんでそんなコード書くの?www
619 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 19:40:36.87 ID:z4XaqaSO.net] function f(a) { let t = a; t[0] = 1; console.log(a); console.log(t); } let a = [0]; f(a); これaも[1]になっちゃうんですが ES2015とか新しい書き方で配列をコピーする方法ありませんか?
620 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 19:41:46.51 ID:AMLyjVO9.net] >>597 > どちらがキチガイかは皆が判断してくれる。 お前がキチガイだって俺は判断したよwww
621 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 22:24:47.44 ID:QKWa7och.net] 芝生くっつけるやつにろくなのはいないってのは間違いなさそうだ
622 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 22:34:59.19 ID:npk74fIw.net] >>601 馬鹿にはちゃんと安価付けろ。気づいてもらえないぞ。 てかお前もちゃんと煽れよ。 >>599 >>601 は>>599 宛だぞ。 おーい。煽られてるぞー。
623 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 22:46:30.04 ID:xguVhByI.net] そりゃ別人なんだから、煽りもするわwww
624 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 23:31:28.41 ID:GjxFekK/.net] >>602 concat
625 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 23:34:06.35 ID:xguVhByI.net] shallowコピーの場合はconcatでいいよね。 最初は?ってなるけど、知ってしまえば簡単に覚えられるし。 なんらかのdeepコピーの場合はライブラリ使うしか無いかな?
626 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 23:54:06.37 ID:npk74fIw.net] いやシャローコピーは slice() だと思うが。
627 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 23:56:00.26 ID:hCgFiHr7.net] >>605 お前、脳みその代わりにババロア詰まってて、しかも微妙に機能してんじゃねえの? 一番煽られてんのはお前だと思うんだけどw 恥ずかしくねえのかな。。
628 名前:デフォルトの名無しさん mailto:sage [2016/06/12(日) 23:57:34.48 ID:hCgFiHr7.net] >>608 JSON.parse(JSON.stringify(obj)) が、一番手軽で早い。
629 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 00:05:59.69 ID:6Ot446qT.net] >>611 関数コピーできない。 日付がおかしくなる。 undefinedが消える。 使えないねw
630 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 00:15:43.00 ID:rTAmnA9g.net] >>612 値をコピーするなら十分でしょ。 関数をコピーとか必要性がわからんし、 undefinedが消えるって、元からundefinedだったんだから、消えてないでしょ。 undefinedが消えるってのは、undefinedで無くなるときだよ。 お前の頭が使えねえなぁ。
631 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 00:15:53.62 ID:+G6yi6vX.net] let の宣言場所について質問させてください for文の中でのみ使う変数を let で宣言しようとしたら、forの中と外どちらで宣言したほうがいいんでしょうか
632 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 00:32:34.63 ID:6Ot446qT.net] >>613 > 値をコピーするなら十分でしょ。 勝手に決めるな。 日付がおかしくなる問題は解決できてないし、 undefinedが消えるっていうのは、キーが無くなるんだよ。 まったく使えないって言ってるだろ
633 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 00:40:16.01 ID:6Ot446qT.net] >>614 どうするのが一番可読性が良いかを考えれば自ずと答えは出る。 可読性を高くするには、頭の中で覚えておくコードを少なくするということでもある。 だからより小さい範囲、つまりfor文の中で書くのが良い。 ただし、for文の行でいろいろ書くなよ?w for (let i = 0, l = array.length; i < l; i++) for文の行でやるのは多くてもこれだけ(インデックス用の変数と終了条件)だ for (let i = 0, l = array.length, foo, bar, baz; i < l; i++) なんてfor文とは関係ない変数を宣言しないように。 言い換えると、変数に代入しないletはやってはいけないってこと。 let v = 0; // OK let v; // NG またletではなくconstを使ったほうが良いぞ constは再代入が不可能になる。つまり変数に入れた値を変えることが出来ない。 それでプログラミングが出来るのか?って思うかもしれないができるんだ。 関数型プログラミングの考え方を取り入れると殆どconstでよくなる。
634 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 01:17:56.81 ID:+G6yi6vX.net] >>616 むむむ・・・ インデックス用の変数でも終了条件でもないです 後で見たときに何をしているか分かりやすいように、計算途中の値に名前をつけて一時変数に入れていました forの試行毎に新しい計算値が入ります
635 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 01:49:05.39 ID:6Ot446qT.net] >>617 つまりこういうことでしょう? for(let i = 0; i < 10; i++) { let value = i * 10 console.log(value); } 当然forの中だよ。中でしか使わないんだから。 そしてこれはconstに置き換えられる。 for(let i = 0; i < 10; i++) { const value = i * 10 console.log(value); } ループ毎に新しく作られるからだ。 このconst valueをforの外に置くことは出来ない。
636 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 02:07:05.34 ID:LLEcmK37.net] es2015で()=>をつかえばthisの挙動を変更できるのに なんでjavascriptでdeep copyができないのですか? }
637 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 02:13:16.73 ID:+G6yi6vX.net] >>618 ありがとうございます! forの中に引っ越します varに慣れていたので繰り返し処理のなかで宣言するのに違和感があったんですよね
638 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 02:41:54.70 ID:6Ot446qT.net] 俺はvarのときからforの中に書いていたけどね。 そこだけでしか使わないし、別に文法エラーでもなんでも無いんだから。
639 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 06:18:58.38 ID:o3uO7eJP.net] >>615 キーが無くなるのと、キーに対応する値にundefinedが入ってるのは同値だろ。 だから、undefinedを代入してる!みたいな勘違いしたのかな?頭おかしいのかなw 日付がおかしくなるって具体的にはどう言うこと?エポックがズレるみたいな話では無くて、型が誤解されるみたいな話なら苦笑いだな。 >>616 変数に代入しないlet、では無いんじゃねえの? 仮宣言みたいなもんで。
640 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 07:23:07.35 ID:+G6yi6vX.net] スコープ先頭での let a; がダメとなると、スコープ後半
641 名前:になってから代入する値が確定するケースではどうしてるんでしょう? コード修正時にtemporal dead zoneによるミスを起こし易いので、スコープ途中でlet宣言する場合は、明示的に新たにブロックスコープを作ったほうがいいと書いてあるのをどこかで読んだんですが、 最後まで使う変数にそれをするとスコープのためだけにネストが深くなってしまって気持ち悪いです [] [ここ壊れてます]
642 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 07:33:47.84 ID:ZIxcBfsq.net] >>623 let a=nullで宣言しておいたら?
643 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 09:10:57.92 ID:6Ot446qT.net] >>622 > キーが無くなるのと、キーに対応する値にundefinedが入ってるのは同値だろ。 違うぞw keysで出力されるもの見てみwww
644 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 09:47:02.30 ID:uHiiHky6.net] 普通にブロック内に書けばいいんじゃないの? わざわざブロック使えなんていうなら、関数全体で同じことしてるvarだって先頭で全部明示しろってなると思うんだけど
645 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 09:57:57.31 ID:CfEi32hJ.net] >>612 が指摘している問題はおそらく、こういうこと var obj = {fn: function fn () {}, date: new Date, undefined: undefined}; console.log(obj); console.log(JSON.stringify(obj)); // "{"date":"2016-06-13T00:45:07.522Z"}" console.log(JSON.parse(JSON.stringify(obj))); // {"date":"2016-06-13T00:45:07.522Z"} var array = [function fn () {}, new Date, undefined]; console.log(array); console.log(JSON.stringify(array)); // "[null,"2016-06-13T00:48:58.483Z",null]" console.log(JSON.parse(JSON.stringify(array))); // [null, "2016-06-13T00:52:23.947Z", null]
646 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 10:10:14.80 ID:CfEi32hJ.net] JSON.stringify の一番の問題は new Object, new Array 以外の Object 型を扱えない事 new Date なら String 型に変換されるだけだから new Date し直す方法がある(ただし、参照は失われるので参照比較できなくなる)が、それ以外はどうしようもない function Person (name) { this.name = String(name); } var obj = {person: new Person('太郎')}; var obj2 = JSON.parse(JSON.stringify(obj)); console.log(obj); // {person: Person {name: "太郎"}} console.log(obj.person instanceof Person); // true console.log(JSON.stringify(obj)); // {"person":{"name":"太郎"}} console.log(obj2); // // {person: Object {name: "太郎"}} console.log(obj2.person instanceof Person); // false 今のところ、deep copy する標準関数はないので Object.assign を使って自前で deep copy 関数を作るのが妥当
647 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 10:57:57.06 ID:ZIxcBfsq.net] >>625 あのさ、keysで定義されていてもその中身はundefinedなんだろ? なら、データとしての扱われ方として、同じだと思うんだけど。 キーがある事が大切であれば、キーを列挙したもの、というデータがあるべきだと思うんだけどどうなの? >>628 あーなるほど、そういうことか。 参照を失わせるために、そういう事するんだから、であればそのオブジェクトの.toJSONを定義してやるべきだろうね。
648 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 11:55:17.37 ID:CfEi32hJ.net] >>629 > あのさ、keysで定義されていてもその中身はundefinedなんだろ? 横から口を出すが、hasOwnPropety の結果が異なるから同じではない 「undefinedを代入する設計が筋が悪い」といわれればその通りだが、「同じ」ではない 実際、プロパティの存在チェックなら obj.prop !== undefined よりも obj.hasOwnProperty('prop') を使うべきだ var obj = {undefined:undefined}, obj2 = {}; console.log(obj.hasOwnProperty('undefined')); // true console.log(obj2.hasOwnProperty('undefined')); // false > 参照を失わせるために、そういう事するんだから、であればそのオブジェクトの.toJSONを定義してやるべきだろうね。 プリミティブデータの倉庫として使うなら JSON.stringify で良いが、new Object, new Array 以外は参照を失わせるべきではないのでは? >>628 でいえば、new Person('太郎') を {name: '太郎'} に書き換えるメリットがない
649 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 12:22:56.73 ID:CfEi32hJ.net] > 参照を失わせるために、そういう事するんだから、であればそのオブジェクトの.toJSONを定義してやるべきだろうね。 ひょっとして、new Person する為の拡張構文をJSON に追加してやるといいたかったのだろうか 私が deep copy するのは構造を書き換えたいが、元の構造も残したい場合だからデータそのものは元のままでいい new Person('太郎') はたった一人の太郎であり、同名の太郎を作ってもらっては困る 同名オブジェクトを作ると taro === person の比較が出来ず、person instanceof Person && taro.name === person.name のようにダックタイピング的に比較しなければならない こうなるともう new Person を定義した意味がなく、{name: '太郎'} を生成したのと大して変わらない
650 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 13:52:54.74 ID:KnYKOJ/M.net] >>630 あー、わからんでもない。 undefinedを値として使う、ってニーズがあれは、それもまた真なんだろうな。 そうなると、assignでもなくオブジェクト毎にちゃんとcloneを実装すべきって感じか。 >>631 ちょっとまって、それはdeepcopyでは無いんじゃないの? ただの参照の張替えじゃないか。 比較するのであれば、それはequals的なメソッドを作るか、諦めて、stringifyした物同士が同じになるかを比較すべきだろ。
651 名前:デフォルトの名無しさん mailto:sage [2016/06/13(月) 17:10:28.33 ID:CILZ2bdj.net] >>632 {}, [] は deep copy する [{person: new Person('太郎')}, {person: new Person('一朗')}] 逆に new Person を deep copy したいと思う状況が分からない new Date も関数も期待通りに動かないわけで汎用性が大きく犠牲になっていると思うが…
652 名前:デフォルトの名無しさん [2016/06/13(月) 17:51:17.99 ID:u+uz+gNL.net] ボタンをクリックしたいのですがどう書けばいいかわかりません>_< ソースはこれです↓ i.imgur.com/IQLz1Kp.jpg