単一行のステートメントとグッドプラクティス

私は最近、多くの人が眉をひそめるかもしれないと知っている習慣を身につけましたが、最終的には、単一の（時には）反復的な方法の構造ではなく、グローバルなコード構造に目を向けるのに役立ちます：次のような1行のステートメント

textBox1.Text = "Something!"; textBox2.Text = "Another thing!"; textBox3.Text = "Yet another thing!";

とは対照的に

textBox1.Text = "Something!";
textBox2.Text = "Another thing!";
textBox3.Text = "Yet another thing!";

コードの全体的な「美しさ」を維持し、プログラム構造を簡単に追跡できるようにするために、繰り返し行うタスクでそれを使用しますが、それは良い習慣ではないことを認めます。私は実際にそれを頻繁に使用するので、これについてあなたの考えを知りたいと思います。また、私のコードを保守しなければならない人は誰でもこのアプローチに問題があると思いますか？

あなたにとっても、コードを読む他の人にとっても、読みやすさは大きく損なわれると私は本当に思っています。それはあなたの心の中に積極的にあるので、それを最初に書くとき、それはすべて理にかなっています。コードをスキャンして変数や関数がどこにあるかを確認するときは異なります...自分のコードをスキャンするあなた自身の能力を破壊しています。それには、何も巨大でなく、悪いを超えて他の誰場合は、これまで、あなたのコードを読む必要があります。

また、コードの読み方についても考えてください。常に上から下にスクロールします。あなたのメソッドはこれに適合せず、コード読み取りで最もuい問題の1つをもたらします。水平スクロール。それがコードを読むのをどれだけ難しくするかを過小評価しないでください。水平方向にスクロールすることも、人々を水平方向にスクロールさせることもありません。ほとんどどんな状況でも、それは非常に不自然です。

また、問題がコード入力の繰り返しである場合は、Ctrl-Cを忘れないでください。あなたのサンプルコードから、すべてを手動で入力する方が効率的かもしれませんが、たくさんの行を何度もコピーする必要がある場合、1行目と新しい行をコピーするのと同じくらい効率的であるように思えますx回、変更を加えますが、タイプミスをする可能性は低くなります。

ああ、タイプミス！そのようなコードの可読性を損なうと、50個の変数宣言のどれを間違って設定したかを見つけるのが悪夢になります。現在、ほとんどのコンパイラーは行番号と列番号でエラーを出しますが、行でエラーを見つけることは列を見つけるよりもはるかに簡単です。

また、1行に1つのステートメントを使用することで、サイドバイサイドdiffで何が変更されたかを簡単に確認できます。

この例ではこれを示していませんが、1行で複数のステートメントをグループ化することには別の問題があります。1行にある5つのステートメントの1つが例外をスローした場合はどうなりますか？

スタックトレースには「EBlah at line N」と表示されますが、これらの5つのステートメントのどれが例外をスローしたかわかりません。

（同じことが、どんな種類の過度に長いステートメントでも起こります。）

1行につき1ステートメントは、広く使用されているコーディングスタイルです。結果として、将来あなたのコードを見る開発者のほとんどは、1行に複数のステートメントが表示された場合に勝つでしょう。ある方法で何かを見ることに慣れている場合、別の方法でそれを見るのは混乱を招く可能性があります。

このため、まれな状況を除き、これに反対することをお勧めします。

私はこれを25年前に、低いクロック速度で動作する小さなマイクロでインタープリター言語を使用してこれを行いました。

私は今、それを考えてひるむ（それは正当な理由で行われたが）。

残念ながら、このようなコードは読みにくく、したがって維持するのが困難です。

構文的には、実際に問題はありません。チームのコーディングスタイルに大きく依存します。

私が見たほとんどのコード（標準のc ++ヘッダー内のコードを含む）はこのようにして行われるため、最初の方法を使用します。

textBox1.Text = "Something!";
textBox2.Text = "Another thing!";
textBox3.Text = "Yet another thing!";

これは本当に珍しいコーディングスタイルです。

代わりに、空行を使用してコードの論理部分を区切ることをお勧めします。

右に行きすぎると、複数の行と同じくらい多くの問題が発生する可能性があります。

何十ものフィールドを持ついくつかのsqlステートメントを処理する必要がありました。通常、行ごとに1つを配置しますが、いくつかの機会に、3または4を行に統合しました。これは、開発中に数回上下にスクロールしなければならないときの良いアイデアのようです。

このコードに戻って後悔しています。余分な行があっても、それほど多くの問題が発生するわけではないので、通常はクリーンアップします。

また、私のコードを保守しなければならない人は誰でもこのアプローチに問題があると思いますか？

1分間手をかざした後、お気に入りのIDE Regex機能を使用して、読み取り不能なコードをすべて1行に1つのステートメントに自動的に分離します。

2番目のアプローチがどれほど読みやすいかを理解するには、示した例を簡単に確認するだけで十分です。

目を永久に水平方向に動かすことなく、垂直方向のページフローを簡単に追跡できます。

あなたの例を見てください：コードはすべてText異なるtextBoxオブジェクトのプロパティに関するものであり、値として文字列が含まれていることがすぐにわかります。とても簡単です。

私は個人的にそのようなスタイルを使用しません。要約する

長所

• スクロールするコード行が少ない
• コードを意味的にグループ化して、「多くの割り当てもの」を表現するために使用できます。しかし、煩わしい場合は、いつでもそのようなブロックを関数にリファクタリングできます。

短所

• 読みにくく、一般的にコードを水平方向に読みやすくなります（スキミング、目の動きがない、...）
• diffはマージを含め、悪夢になりやすい
• 変更がより難しい（コピー＆貼り付け、コメントアウト、...）
• 多くのIDEでは、個々の式ではなく行で動作するため、デバッグが問題になる場合があります。