アカウント名:
パスワード:
ソースコードをざっと眺めました.第一印象としては- ソースコードが冗長.長い!- コメントが英語で書かれているが文法の誤りが多い点が気になりました
たとえばhttp://ja.softether.org/@api/deki/files/679/=sourcecode.png [softether.org]に例示されているたった20行のソースコードでもIPV4_GET_FLAGS(ip) & 0x01 を直接評価する行もあれば,その後 last_packet 変数に代入する行もあったりで全体的に冗長です
プロトコルやパケットは実装側のことも考えてデザインされているので,そのデザインを正しく理解できれば,多分ソースコードは今の半分ぐらいの行数で簡潔に実装出来ると思います
この人の書いたコードって、短いけど読みにくそう。
半分とか大きく出たよなと思うわ。流石にじゃあやってみなと言いたいレベル。コメント削ってほら半分とかじゃないこと願うよ。
>半分とか大きく出たよなと思うわ。流石にじゃあやってみなと言いたいレベル。
当然、ソースコードを読んだ上で発言してるのですよね。
# 出る杭を叩くだけのACの多さに辟易する今日このごろ。
ソースコードにケチをつけるだけの人を「出る杭」とは褒め過ぎだと思う。
少なくとも例に出ていた20行のコードを見ても、#2522684で書かれている通り、『同一のものを複数(この場合は2つ)の表現で記述』していますから、『冗長だ』『分かり辛い』と思われても仕方がないかと。# それ以外にも、直値を出している時点で危険な気がしますけど…。
#2522684のannoymouse cowardさんがどのようなコードを書かれるか知らないのですが、該当箇所については、より読みやすい事が期待できる気がします。# ソースをまだ見てませんので、半分になるかどうかは不明ですが。
# 短くすると読みづらいと言う妄想を捨てると幸せになれるかも知れませんよ。# もちろん長ければ良いと言うものでもありません。# 何にせよ『分かりやすくて読みやすいのが一番』です。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
犯人はmoriwaka -- Anonymous Coward
ソースコードを読んでみた (スコア:5, 興味深い)
ソースコードをざっと眺めました.第一印象としては
- ソースコードが冗長.長い!
- コメントが英語で書かれているが文法の誤りが多い
点が気になりました
たとえば
http://ja.softether.org/@api/deki/files/679/=sourcecode.png [softether.org]
に例示されているたった20行のソースコードでも
IPV4_GET_FLAGS(ip) & 0x01 を直接評価する行もあれば,その後 last_packet 変数に代入する行もあったりで
全体的に冗長です
プロトコルやパケットは実装側のことも考えてデザインされているので,
そのデザインを正しく理解できれば,多分ソースコードは今の半分ぐらいの行数で簡潔に実装出来ると思います
Re:ソースコードを読んでみた (スコア:0)
この人の書いたコードって、短いけど読みにくそう。
Re: (スコア:0)
半分とか大きく出たよなと思うわ。流石にじゃあやってみなと言いたいレベル。
コメント削ってほら半分とかじゃないこと願うよ。
Re: (スコア:0)
>半分とか大きく出たよなと思うわ。流石にじゃあやってみなと言いたいレベル。
当然、ソースコードを読んだ上で発言してるのですよね。
# 出る杭を叩くだけのACの多さに辟易する今日このごろ。
Re: (スコア:0)
ソースコードにケチをつけるだけの人を「出る杭」とは褒め過ぎだと思う。
Re: (スコア:0)
少なくとも例に出ていた20行のコードを見ても、#2522684で書かれている通り、
『同一のものを複数(この場合は2つ)の表現で記述』していますから、『冗長だ』
『分かり辛い』と思われても仕方がないかと。
# それ以外にも、直値を出している時点で危険な気がしますけど…。
#2522684のannoymouse cowardさんがどのようなコードを書かれるか知らない
のですが、該当箇所については、より読みやすい事が期待できる気がします。
# ソースをまだ見てませんので、半分になるかどうかは不明ですが。
# 短くすると読みづらいと言う妄想を捨てると幸せになれるかも知れませんよ。
# もちろん長ければ良いと言うものでもありません。
# 何にせよ『分かりやすくて読みやすいのが一番』です。