みんなのオブザイヤーオブザイヤーを募集中!今年のツイート今年のうちにまとめよう!
38
tokoya @tokoya
新人プログラマへ : ソースコード書く時に最適化なんて考えるな。おまえよりコンパイラの方が賢い。
tokoya @tokoya
新人プログラマへ : 冗長なソースコードに見えても気にするな。コンパイラが何とかするから見やすさ優先で書け。
tokoya @tokoya
新人プログラマへ : なんでも一行にまとめるな。どうせ最適化で消えるから一時変数使って分けて書け。じゃないとデバッグで苦しむ。
tokoya @tokoya
新人プログラマへ : if(変数名)という書き方は厳禁。比較式の中にはちゃんと比較する値を明示的に書け。条件演算子も厳禁。
tokoya @tokoya
新人プログラマへ : なんでもポインタで処理するな。配列はちゃんと[]付けて書け。出力されるオブジェクトは同じだ。
tokoya @tokoya
新人プログラマへ、と書いてるけど実際には自分に対する自戒だなこれ(笑)。

コメント

やがてsyuu1228になる @syuu1228 2010-10-05 17:36:29
どうせ、これ全部守ってるけどロジックがハチャメチャなコードが生産されて来るんだぜ
SODA Noriyuki @n_soda 2010-10-05 17:49:03
if (boolean変数名) なら全然OKでしょ、っていうか、普通はbooleanを比較しちゃ駄目(ただしbool「変数」どうしの等価/不等価比較を除く) RT @tokoy: 新人プログラマへ : if(変数名)という書き方は厳禁。比較式の中にはちゃんと比較する値を明示的に書
こんにちは、日本目光協会のほうからやってきました @anekos 2010-10-05 18:49:25
新人プログラマへ : こういうのを安易に信じるな。
ぼんぼ (医薬部外品) @tm_bonvo 2010-10-05 18:52:00
boolean同士の比較ならlogical-XOR使いたいところだけど、そういう演算子ないことが多いのよね。
kartis56 @kartis56 2010-10-05 18:56:45
>ロジックがハチャメチャ  そりゃ、コード書く前にロジック書かせないからですよ。新人相手ならイッパツコーディング不許可にして、ロジックレベルのレビューしなきゃ。最適化するのはロジックでやって、後はそのとおりにコーディングするように習慣づけないと、ドキュメントすら残さなくなりますぜ。
さいたま・ザ・VR Developer @saitamasaitama 2010-10-05 19:41:34
if(A==true)とか、個人的にはアフォっぽい書き方に見えるんだがなあ。 if(A==false)とif(A==true)が混在するくらいなら、if(A)またはif(!A)でいいと思うんだが。ifの中身は常にtrueさ。
うに @aetos382 2010-10-05 19:56:58
if((((((((((boolVar == true) == true) == true) == true) == true) == true) == true) == true) == true) == true) { ... }
yooydon (~!@#$%^&*) @yooydon 2010-10-05 20:15:27
新人プログラマへ : デバッガに頼る前にコード全体を理解しろ。判らなければレビューで学べ。(デバッガで流れを追えるのは分岐の1ケースだけだったりするため)
まひわり @Mahiwari_jpnn 2010-10-05 20:17:01
明確な論理型変数がある言語ならif(変数名)はアリ。そうじゃない言語は==とか使っとくのが無難。doubleの比較で==はやめておこう。
ta @ta1nakamura 2010-10-05 21:29:48
テストコードも書こう。ユニットテストがしやすいコードを書こう。
atty303 @atty303 2010-10-05 23:24:17
僕も基本的に if (anObject.hasChild == YES) は受け付けない。if (anObject.hasChild) のほうが読み易い。ただし、変数名から論理を想起できる場合ね。
DMスパマーに乗っ取られました @master_q 2010-10-05 23:26:20
ユニットテストできないコードもあるんダゼ
inazakira @inazakira 2010-10-05 23:37:34
何をもって「冗長」とするかは人によって違うのよね。
Briareos@残弾数はいつも────── @briareos 2010-10-06 00:24:00
ぶっちゃけ「奇を衒ったコード」とか「趣味的な書き方をしたコード」は、ひんしゅくを買うだけにしかならないので、正直おすすめできない。てか、そういうコードでコードレビューやったら、突っ込まれまくりで恥かくだけですから。マジな話。
UЯAƧ @SARU_tw 2010-10-06 00:53:17
ifで比較するときはtrueじゃなくてfalseで比較すべきって教わった。 理由は、false=0で、true=(!false)だから。
師匠@ハイパーメディアデベロッパー @sea_show 2010-10-06 00:53:34
これを貼れと言われた気がした - 私がソフトウェア技術者をやめた理由 http://d.hatena.ne.jp/elm200/20100924/1285330288
山太夫 @_lushy 2010-10-06 01:18:45
if(変数名)の件は、新人向けならいいのでは?「冗長な~」という前提があるし、逆に覚えてバグ出されるよりは列挙されたほうがマシと言う意味で。
うてん。 @uten00 2010-10-06 01:55:39
if( 変数名 ) はたまにしちゃう。言語によっては想定しない結果になるからやっぱ条件式入れたほうがいいよね。
kartis56 @kartis56 2010-10-06 02:03:23
if(変数名) がいけない理由は、変数の型がbooleanかどうかによるでしょう。もっというと処理系によってはfalseが0とは限らないし、その程度書いたって誰も損しない。
金子篤史 @libra55net 2010-10-06 02:31:05
配列は[]付けろってのは賛同しかねる。
ぼうくん @VoQn 2010-10-06 03:25:33
1.ポインタが利用できる言語ならポインタ使いこなせてる方が腕はつくんじゃないの
ぼうくん @VoQn 2010-10-06 03:26:15
2. Javaで if("hoge" == hogeString){} なんてしようなら鬼軍曹からゲンコツくらうよね。要するにメソッドやメンバがboolean参照できるものなら素直に使った方がスマートでしょう
うてん。 @uten00 2010-10-06 09:06:59
他の人が見てわかるよーなソースって意味ではポインタの多用やboolean判定を期待する条件式をどこまで使っていいかって悩みはあるかもね。
倉瀬美都 @clausemitz 2010-10-06 09:28:29
いつの時代の新人プログラマーへのアドバイスなんだ? いずれも昔の参考書(それもとっくに絶版になった)で見た記憶があるものばかりだな。
mar @marrrrrrrrr 2010-10-06 10:55:50
新人プログラマへ:ちょっとはググれ
kartis56 @kartis56 2010-10-06 15:53:59
>Javaで if("hoge" == hogeString){} 定数を左辺値にしたらコンパイルエラーでしょ?
@holypp 2010-10-10 07:10:26
う〜ん、if(変数名)はふさわしいケースがたくさんあると思う。許されない職場には行きたくないな。それより(略)
ログインして広告を非表示にする
ログインして広告を非表示にする