のくす牧場
コンテンツ
牧場内検索
カウンタ
総計:127,096,545人
昨日:no data人
今日:
最近の注目
人気の最安値情報

    元スレ+ JavaScript & jQuery 質問用スレッド vol.5 +

    JavaScript覧 / PC版 /
    スレッド評価: スレッド評価について
    みんなの評価 :
    タグ : 追加: タグについて ※前スレ・次スレは、スレ番号だけ登録。駄スレにはタグつけず、スレ評価を。荒らしタグにはタグで対抗せず、タグ減点を。
    ←前へ 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 次へ→ / 要望・削除依頼は掲示板へ / 管理情報はtwitter

    801 = :

    手続き型プログラミングとしてはこれ以上は減らしようがない気がする
    関数型プログラミングに変えれば、多少は変わるけど、所詮はコーディングスタイルの違いで明確な優劣はない
    for文はforEachよりも速く、forEachは関数スコープで区切ることが出来るぐらいか
    for文でもlet文を使えばスコープを区切ることが可能だけど

    803 = :

    >>800
    > ライブラリがあれば短くかけるのは当然としてライブラリを使わないから冗長で無駄ってのは子のスレでしか聞かないなー

    元のレスにjQuery使ってって書いてあるのに、使わないで冗長に書くっていうのは
    目的を果たしていない上に無駄でしか無いなw

    804 = :

    >>803
    >>793

    806 = :

    とりあえず中間のつもりだけど、これ以上やっても
    あまり良くならないのでこれで最後にするかも。
    http://jsfiddle.net/eoxzjqwu/14/

    for文の中で無意味にやっていた変数の宣言の排除
    多すぎて分かりづらい変数の削除
    処理の改善(++k して --k するのにはワロタw)
    jQuery化
     jQueryのremove()への置き換えと不要になったremove()の削除
     即時関数なくしてインデントを下げる
    79行 => 56行(不自然な行減らしはしてないつもり)

    などを行った。

    807 = :

    あと ++v から v++ に変更(笑)

    C++を知らんのか?
    普通は変数の後ろに++やで?

    808 = :

    一応、案だけ書いておくか。

    colSpanの設定は、attr('colspan', function() {・・・}) あたりを使えば
    1~2行減らせるんじゃないかと思ってる。rowSpanも同じく。

    そしてrowSpanの方は、インデックス(i, j)を使ってforでぐるぐる回すんじゃなくて
    セレクタtd:nth-childを使って縦のcellを取ってくれば、
    colspanとほぼ同じコードで良くなって、3行程減らせるんじゃないかと思ってる。

    もしこれによってcolspanとrowspanのコードを共通化できれば半分になるから
    40行程度まで減らせるかもしれない。

    809 = :

    やばいw 理屈の上ではcolspanとrowspanのコードは同じでいいはずだよなぁ
    ぐらいの感覚で>>808書いて、なんも考えずにコピペして適当に置き換えたら動いたw
    これ本当に共通化できるよ。

    http://jsfiddle.net/eoxzjqwu/16/

    810 = :

    ここまでかな。
    http://jsfiddle.net/eoxzjqwu/22/

    attrを使う案はうまく行かなかった。
    あとせっかく減らしたインデントが共通関数のために深くなってしまった。残念。
    最終的に 79行 => 42行

    811 = :

    せっかくの減らしたインデントがまた深くなったのが悔しいので
    jQueryプラグインにしたよw

    行数も39行まで減った(無理やり空行削れば34行)
    http://jsfiddle.net/eoxzjqwu/29/

    812 = :

    >>790はパフォーマンス寄りにチューンしてる感じ
    いかにも手続き型言語らしい書き方だな

    813 = :

    >>807
    > 普通は変数の後ろに++やで?
    何その固定観念
    君はもう少し、頭を柔らかくした方がいいよ

    814 = :

    C++の名前から後置インクリメントがベターと記憶したのか
    深く考えずに定型コードをそのまま使うタイプだな

    816 = :

    俺も基本的に後++だなぁ
    もちろん使い分けるけど

    817 = :

    >>812
    > >>790はパフォーマンス寄りにチューンしてる感じ
    可読性を落としてまでやることじゃない。

    818 = :

    >>814
    深く考えたら、++i なんてしねーよw

    819 = :

    >>798
    > for文の中でvar宣言すると多重定義になるからそうしてるんだろう
    JavaScriptで変数の多重定義は違反にはならない。

    820 = :

    そもそもfor文内で宣言した変数はその1ループごとに破棄されるから多重扱いですらないのでは

    821 = :

    >>820
    まあ何のために、一行を長くして可読性を下げてまで
    for内でやらなくていい変数定義をしたのか理由がわからんね。
    深く考えてないんだろうけどw

    そもそも元のコードは変数多すぎ。
    それに縦と横でアルゴリズムが大きく変わるはずがないことに
    気づかなかったのかねぇ。おかげで2倍のコードを読む羽目になった。

    アルゴリズム上、これlengthっていらなくね?って思ったらやっぱりそうだった。
    元のコードは縦のcellリストを作らずにループでやってたから終了条件として必要だったんだろうな。
    http://jsfiddle.net/eoxzjqwu/30/
    無駄なコードを削って37行に。気づけば元のコードの半分以下になってるな。
    修正後のvarで定義している変数の数はたった3つ。

    822 = :

    >>772

    770だよ

    俺は初見だ!
    なんの議論にも加わってねーよ!www

    823 = :

    さらに、縦と横のコードを両方共セレクタに置き換えることで
    汎用化してblock呼び出しも削った。
    http://jsfiddle.net/eoxzjqwu/35/ (30行)

    分割代入とか使えば、もう少し可読性上がるけど
    IEで動かないのであえて使ってないw

    824 = :

    >>818
    評価値を使わないならどちらでも変わらない

    825 = :

    >>817
    お前がおまえのコードを書く分には好きにすればいい
    他人のコードにケチをつけるほどの理由はない

    826 = :

    >>825
    コードレビューっていうのは、コードにケチを付けることだよw
    人格批判じゃないんだから気にすんなよw

    827 = :

    動いてはいるのだけど、明らかに無駄な処理過ぎるので突っ込まれる前に直した。
    http://jsfiddle.net/eoxzjqwu/36/

    828 = :

    >>826
    「一長一短」という言葉があってだな
    おまえの中の優先順位と他人の優先順位が同じとは限らない
    更にいえば、「おまえの中では可読性が低い」というだけ
    手続き型プログラミングとしてはあのぐらいは当たり前にある

    829 = :

    えとな、これはJavaScriptだ。

    言ってる意味わかるか?

    831 = :

    >>819
    知ってるけど、多重定義を避けたい気持ちは理解できる

    while (i--) {
    var obj = array[i];
    console.log(obj.a, obj.b);
    }

    このコードの var obj の行がとても気持ち悪い
    これは感覚であって理屈じゃないんだよね
    まあ、変数の代入に先行して初期化処理があるべき、といえなくもないけど

    832 = :

    >>831
    varで書いてメリットは、あとでletに置き換えやすいということ。

    コードには、そうする意図があるわけで、
    その位置にvarを書いておけば、whileの外では使わないものである
    という意図が明確になる。たとえ使えたとしてもだ。

    833 = :

    >>831
    いやそれって多重定義にならんよな?
    てか何言語出身よ?
    Cならそれ普通だし、JavaScriptならそれ以前だし。

    834 = :

    はい、自己満足で解決

    835 = :

    >>828
    それで、おまえの中の優先順位ってなんだよw

    優先順位を満たしていると言う根拠も示せず
    あとづけで言っても説得力ねーぞ?

    836 = :

    732氏は凄いなあ

    837 = :

    >>836

    >>732はできるかな?って思ってやってみただけだけどね。
    処理的には属性つけて更新して削除してとかやってるからなんか無駄だし、
    縦にはそのままでは対応できない。頑張ってもあの程度の量で出来るかどうかわからない。
    >>827が思いの外シンプルに修正できたので、通常はこっちをおすすめする。

    それはそれとして、こういうのに挑戦すると楽しいし技術力も上がるよ。
    人によっては動けばいいが最優先になってる人もいるけど、
    そういう人のコードは無駄が多いし、いつもその程度でやめるてるから技術力はあがらない。
    JavaScript使っていても過去にやってた手続き型風の書き方のままになっちゃう。
    動けばいいでやめるからそうなるわけ。

    時間をかけて最初に書いたものをそれ以上に修正していくことで
    次からはそれを最初からかけるようになるよ。
    面白いネタを持ってきてくれた人に感謝だなw

    838 = :

    >>790は面目丸つぶれw

    >http://jsfiddle.net/eoxzjqwu/
    >
    > 件の人が「コードが長すぎて冗長だ」と煽り立てるのが目に浮かぶが、「簡潔で美しいコード」はコード量が少ないことを指すわけではない。
    > >>732のようなアルゴリズム上の無駄を美しくないと感じる人もいる。


    >>732>>790を修正したコード

    http://jsfiddle.net/eoxzjqwu/36/

    839 = :

    >>832
    そうだね
    >>790はletに置き換える事を意識している気配がある

    >>833
    仕様上、それが問題ない事は分かってるよ

    840 = :

    >>839
    >>790はletに置き換える事を意識している気配がある

    話の流れ無視すんなーw

    > for (var k = 0, cells = rows[j].cells, n = cells.length, cell, textContent, colSpan, nextCell; k < n; ++k) {
    > cell = cells[k];
    > textContent = cell.textContent;
    > colSpan = cell.colSpan;

    例えばこことか、forの行でcells.lengthをnに入れるのはまだわかるが、

    そのすぐ下の行で代入してる、cell、textContent、colSpanなんかを
    forの行でやる意味がない。

    for (var k = 0, n = cells.length; k < n; ++k) {
    var cell = cells[k];
    var textContent = cell.textContent;
    var colSpan = cell.colSpan;

    ↑こう書けばいいだろ。
    varの動作からしたら、適切じゃないかもしれないが、
    意図としてはこのとおりなんだし、あとからletに置き換えやすい。

    という流れだろうがw

    842 = :

    > 仕様上は問題ないけど、2回目はvar宣言を外したい

    いや、forの行でvarしても中でvarしても
    変数に対するvarは一回だけですが?

    2回目ってどこのことですかね?

    843 = :

    >>842
    k === 1 になった時

    844 = :

    >>843
    行番号言ってくれない?

    別々の行番号を2つ言えるはずだよね?

    845 = :

    関数で考えればわかるはずなんだが、

    ↓ このコードは実行できる。

    foo();
    function foo() { ・・・ }

    foo()を実行する段階では、まだfunction foo() {}は実行されてないじゃないか!?って
    思う人はいないだろう。

    理由は簡単で、実行時じゃなくてパースした後にfunction foo()は定義されるから。
    それと同じで、var a も、var aと書いた所でvarされるのではなくて、
    パースした後に定義されるんだけどな。

    ループで繰り返すたびに、varするわけじゃないんだよw
    もちろん代入自体は行われるけどね。

    846 = :

    >>844
    行番号は変わらない
    >>840の2,3,4行目を展開すればそうなる

    847 = :

    >>846

    >>845を読んでねw

    848 = :

    >>845
    だから、それは理解してる
    変数は関数呼び出しされた時点でundefinedで初期化される
    わかってるけど、コードをそのまま読むと変数を再宣言しているように見えてしまうんだよ
    letならブロックスコープ単位で初期化されるから再宣言はしないね
    何度もいうけど、感覚の話だから

    849 = :

    >>846
    > >>840の2,3,4行目を展開すればそうなる
    JavaScriptを勉強しようぜw

     function setColSpan (tableSection /* [... tableSection] */) {
       省略
       for (var k = 0, n = cells.length; k < n; ++k) {
         var cell = cells[k];
         var textContent = cell.textContent;
         var colSpan = cell.colSpan;

    っていうのは変数の巻き上げって処理が行われて、こう解釈されるんだぜ

     function setColSpan (tableSection /* [... tableSection] */) {
       var k, n, cell, textContent, colSpan;
       省略
       for (k = 0, n = cells.length; k < n; ++k) {
         cell = cells[k];
         textContent = cell.textContent;
         colSpan = cell.colSpan;

    この後で展開した所で、varは2回でてくることはない。

    850 = :

    >>831
    お前はJavascriptだけじゃなくプログラムを学び直すべき


    ←前へ 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 次へ→ / 要望・削除依頼は掲示板へ / 管理情報はtwitterで / JavaScript一覧へ
    スレッド評価: スレッド評価について
    みんなの評価 :
    タグ : 追加: タグについて ※前スレ・次スレは、スレ番号だけ登録。駄スレにはタグつけず、スレ評価を。荒らしタグにはタグで対抗せず、タグ減点を。

    類似してるかもしれないスレッド


    トップメニューへ / →のくす牧場書庫について