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) 時にはあなたが望む「答え」だけでなく、「意見」などが寄せられる場合もあります。
528 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 15:59:08.90 ID:DxAsZ/F5.net] 1個くらいならバレないんじゃない?
529 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 16:05:16.91 ID:
] [ここ壊れてます]
530 名前:3bO9ZAmE.net mailto: Object.prototype.hogeにバーカって書いとけばいいのか [] [ここ壊れてます]
531 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 18:15:57.76 ID:LT2C5Mf1.net] コンソールに出すの? アラートで出すの? アラートで出たら、めちゃくちゃ怖いかもww
532 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 19:06:13.75 ID:qrHSyLm3.net] prototype汚染という考え方も古くなってきた感はあるが… しかし、風呂敷を広すぎではないかね 元々はprototuype汚染の話ではなかったわけで段々と本題がずれてきているように思う
533 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 19:27:07.34 ID:qrHSyLm3.net] >>489-491 画面を再描画するならサーバサイドの処理があるはずだが、それはJavaScriptで実装すべき案件なのかね フォームで値を受け渡してサーバサイドで処理、セッションを使う等、いくつか対策があるはずだが jsを使うなら再描画させずに動的にDOMを書きかえれば済むが、マルチポスト先を読む限りではこの質問者にはハードルが高そうだな https://teratail.com/questions/37462 https://teratail.com/questions/37489
534 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 20:02:15.97 ID:G+eNnn4N.net] >>500 少なくとも文法上は大差ないのだろう。MDNにも説明はある。 > 全ての関数は実際には Function オブジェクトです。 > Function コンストラクタか関数宣言か関数式か > https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Functions_and_function_scope new Functionや非トップレベル関数(strict modeでは禁止)も含めて書いてあるから余計に分かりにくくなっているが、 要するにどうでもよさそうだ。 現実的にはコンストラクタだけ前方参照できてもプロトタイプが用意できていないから意味がない。 だから関数式の方が似合っている気はする。 ちなみに汚染については変数名か関数名かだけでどちらも汚染する。 また、デバッグ時にこんなところが関数表示されたところで嬉しくない。 どのみちコンストラクタが呼ばれればコールスタックには載るし、ブレークポイントも当てられる。 お前らは異様に文法にこだわることを止めて、もっとコードを書いた方がいい。 指摘が間抜けなのは、コードを書いてないからだ。(実践的ではない) ○○の問題がある(キリッというのは正しいが、それが重要なことなのか、わりとどうでもいいことなのか、区別が出来ていない。 >>520 > マルチポスト先 よく見つけたなw というかこの手のモラルがない奴、急に増えたよな?何か理由があると思うのだが。 昔から居たけど、マルチポスト先が増えたから発覚しやすくなっただけか?
535 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 20:08:51.44 ID:RNr8lFlW.net] >>499 どうもありがとうございます。おかげで気がついてしまいました。 concatはArrayを受け取ってArrayを返すつもりで>>487 では書きましたが、 よく考えると今回拡張したA(以下単にA)を引数に受け取りたいですし、 返す結果もよく考えるとAであったほうが嬉しいかもしれません。 実装の手間という面だけでも尻込みしつつあったんですが、 いまや設計の方針と言う面でも相当きな臭いというか。 concatの返す値が決められなくて不気味。使うときに不安。 仮にsliceやspliceなどのもまとめて全部Aを返すように統一したとしても、 最終的にはArrayとAの二系統が混在する世界になっちゃうわけで不気味。 >>501 > 完璧なエミュレーションに拘らないのであれば妥協も必要 そのつもりだったんですがいざ用意してみると、 欲が出るわ粗が目立つわ方針で不安になるわで散々でした。 今後は生のArrayのみを運用していくつもりです(結論)。 >>504 ありがとうございます。Groovyはぐぐったことすら無いので分かりません。 > そういう複雑極まりない方向へ進むのは、最初から考え方がおかしいのでは? はい。完全にそのとおりです。
536 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 21:47:53.40 ID:qrHSyLm3.net] >>522 > 今後は生のArrayのみを運用していくつもりです(結論)。 それはそれで一つの答えだが、やや狭い世界になっていると思う > 最終的にはArrayとAの二系統が混在する世界になっちゃうわけで不気味。 2つの世界が存在することは問題なくてそれが息をするように自然に使えるかどうかが問題ではないだろうか mew Array と new A を行き来する世界はよくないが、それぞれの世界で完結する処理を書けるのならそれでいい どちらかといえば、設計指針を決めかねて迷っているのだと思うが… 基本的には new A は A だけで閉じた世界(new Array を返してはならない)であるべきなので A#concat は new A を返すべきだと思う 逆に A#push は汎用的なので特別な事をせずとも new A を返す このように汎用的な関数を基本的には使って非汎用関数に対しては Polyfill を作る要領でコードを書いていけばそれなりのものができるだろう 別の設計指針としては new Map を new ArrayMap 的な設計にして作り直しても面白いかもしれない new Map は関数でsetter/getterを作るので new Array よりは互換コードを書きやすいはずだ 機会があれば、是非新しい世界を作ることに挑戦してみてほしい
537 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 21:58:16.74 ID:MdFWcZSW.net] こういう人に読ます気がない文章は目の前で破り捨てるに限る
538 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 22:12:29.26 ID:AlU+uWXE.net] ES6だとプロトタイプ汚染がないのかと思ったけどもしかして新機能がない? 結局 Object.defineProperty(...){........} と書かんといけんのか
539 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 22:18:01.79 ID:G+eNnn4N.net] この程度で躓くのならもっと長いMDNは当然無理で、池沼のままなのもむべなるかな
540 名前:523 mailto:sage [2016/06/10(金) 22:51:59.24 ID:qrHSyLm3.net] 読みにくくてすまん どういうわけが改行コードが一部消えているな…
541 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 23:08:39.56 ID:+uuPJDCE.net] *∧ ∧:・。゚*。 : ・ 。 ゚ *★. (*・ω・*) ゚ *。* *〇 〇………………………** 祝☆jQuery 3.0 リリース https://blog.jquery.com/2016/06/09/jquery-3-0-final-released/ **………………………………* + JavaScript & jQuery 質問用スレッド vol.6 + [無断転載禁止]©2ch.net echo.2ch.net/test/read.cgi/hp/1465566635/
542 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 23:10:44.42 ID:+uuPJDCE.net] >>509 > > function Klass() {} という書き方が古くなったとか > > 聞いたこと無いし、 > グローバルネームスペースを汚染する弊害について聞いたことないのか なにいってんだおめぇ? var Klass = function() {} という書き方をしても グローバルネームスペース汚染してるだろw function hoge() { function Klass() {} } ↑Klassがグローバルネームスペース汚染ってどういうこと?w
543 名前:デフォルトの名無しさん mailto:sage [2016/06/10(金) 23:13:14.16 ID:+uuPJDCE.net] >>524 同じこと思ったw
544 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 00:01:17.78 ID:MQjYfq34.net] ライブラリスレが復活してた JavaScript ライブラリ総合質問所 vol.5 [無断転載禁止]©2ch.net echo.2ch.net/test/read.cgi/hp/1465399470/
545 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 08:41:10.92 ID:MQjYfq34.net] jQuery推しが酷いと思ったらテンプレ改変されていたのか
546 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 13:03:48.93 ID:n8nif7qi.net] 今更かよ ここ乗っ取られスレだよ
547 名前:デフォルトの名無しさん [2016/06/11(土) 13:06:25.18 ID:ijG8u0pc.net] java script初心者です。ご教授願います m(_ _)m 初めてpromiseで簡単な非同期処理を書いたんですが、一応動くもののどうにもコードがしっくりきません もっとシンプルにできたり、適切な書き方があるところがあったら教えてください 内容は単語queryに対応する画像検索結果がローカルストレージになかったら外部APIに問い合わせるというものです
548 名前:534 mailto:sage [2016/06/11(土) 13:09:15.52 ID:ijG8u0pc.net] function getCachedData(query) { return new Promise(function(resolve, reject){ chrome.storage.local.get(query, function(cachedJson){ if (isExist(cachedJson)) { resolve(cachedJson[query]); } else { reject(); } }); }); } function getImageData(query) { getCachedData(query) .catch(function(){ return fetch(window.googleCustomSearchApi + query) .then(function(response) { return response.json(); }).then(function(json){ chrome.storage.local.set({[query]: json}); return json; }); }).then(function(json) { createPopUp(json.items[0]); positionPopUp(); }).catch(function(err) { console.error(err); }); }
549 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 13:14:41.27 ID:ijG8u0pc.net] インデント潰れてしまった・・・ preタグで再投稿 <pre> function getImageData(query) { getCachedData(query) .catch(function(){ return fetch(window.googleCustomSearchApi + query) .then(function(response) { return response.json(); }).then(function(json){ chrome.storage.local.set({[query]: json}); return json; }); }).then(function(json) { createPopUp(json.items[0]); positionPopUp(); }).catch(function(err) { console.error(err); }); }</pre>
550 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 13:21:07.32 ID:e5AOazPt.net] >>534 この記事が参考になるかな gao-tec.seesaa.net/article/427643074.html
551 名前:デフォルトの名無しさん mailto:sage [2016/06/11(土) 13:33:33.96 ID:ijG8u0pc.net] >>537 ありがとう 3回読んできます
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)) が、一番手軽で早い。