アカウント名:
パスワード:
ソースコードをざっと眺めました.第一印象としては- ソースコードが冗長.長い!- コメントが英語で書かれているが文法の誤りが多い点が気になりました
たとえばhttp://ja.softether.org/@api/deki/files/679/=sourcecode.png [softether.org]に例示されているたった20行のソースコードでもIPV4_GET_FLAGS(ip) & 0x01 を直接評価する行もあれば,その後 last_packet 変数に代入する行もあったりで全体的に冗長です
プロトコルやパケットは実装側のことも考えてデザインされているので,そのデザインを正しく理解できれば,多分ソースコードは今の半分ぐらいの行数で簡潔に実装出来ると思います
案の定ツリー荒れてますね…。
僕にもわかった!ここ冗長!みたいなコーディングの指摘1箇所でもって「多分ソースコードは今の半分ぐらいの行数で簡潔に実装出来ると思います」と豪語されたら、経験的にアチャーと感じる人たちは多いでしょうな。
そうですね。でも、もし高い確度で「出来ると思います」と言えるだけのソースに目を通していても、ここに例としてあげるのは、外からよく目立つ部分だけになるのでは。
だから、「指摘1箇所でもって(中略)豪語されたら」ではないかも。
・・・と考えましょうよ。
このツリー大本のIDはなんでも煽っていくスタイルの困った方なんで、そこも吟味しなさいな。
>このツリー大本のIDはなんでも煽っていくスタイルの困った方なんで、そこも吟味しなさいな。
結構大きくなったこのツリーだが、実際にコードを読んだうえで発言したのは、その大本のIDだけなのだが、このことをどう思うか?
× 実際にコードを読んだうえで発言したのは、その大本のIDだけ○ 実際にコードを読んだと言い張っているのは、その大本のIDだけ
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
あと、僕は馬鹿なことをするのは嫌いですよ (わざとやるとき以外は)。-- Larry Wall
ソースコードを読んでみた (スコア:5, 興味深い)
ソースコードをざっと眺めました.第一印象としては
- ソースコードが冗長.長い!
- コメントが英語で書かれているが文法の誤りが多い
点が気になりました
たとえば
http://ja.softether.org/@api/deki/files/679/=sourcecode.png [softether.org]
に例示されているたった20行のソースコードでも
IPV4_GET_FLAGS(ip) & 0x01 を直接評価する行もあれば,その後 last_packet 変数に代入する行もあったりで
全体的に冗長です
プロトコルやパケットは実装側のことも考えてデザインされているので,
そのデザインを正しく理解できれば,多分ソースコードは今の半分ぐらいの行数で簡潔に実装出来ると思います
Re: (スコア:0)
案の定ツリー荒れてますね…。
僕にもわかった!ここ冗長!みたいなコーディングの指摘1箇所でもって
「多分ソースコードは今の半分ぐらいの行数で簡潔に実装出来ると思います」
と豪語されたら、経験的にアチャーと感じる人たちは多いでしょうな。
Re:ソースコードを読んでみた (スコア:0)
そうですね。
でも、もし高い確度で「出来ると思います」と言えるだけのソースに目を通していても、
ここに例としてあげるのは、外からよく目立つ部分だけになるのでは。
だから、「指摘1箇所でもって(中略)豪語されたら」ではないかも。
・・・と考えましょうよ。
Re: (スコア:0)
このツリー大本のIDはなんでも煽っていくスタイルの困った方なんで、そこも吟味しなさいな。
Re: (スコア:0)
>このツリー大本のIDはなんでも煽っていくスタイルの困った方なんで、そこも吟味しなさいな。
結構大きくなったこのツリーだが、
実際にコードを読んだうえで発言したのは、
その大本のIDだけなのだが、
このことをどう思うか?
Re: (スコア:0)
× 実際にコードを読んだうえで発言したのは、その大本のIDだけ
○ 実際にコードを読んだと言い張っているのは、その大本のIDだけ