エルス適切な「何もしない」構文は、私は他の文は、forループ内にネストされた場合は、次を作成した場合あなたが見ることができます、私は文が真であれば、何もしない方法を望みます。この場合、 "// Do Nothing"をコメントアウトするのは正しい構文ですか、それとも開いて閉じたカッコとして残すべきですか?あるいは、if行の最後に中括弧を置くことができますか?式を否定ないのはなぜこれを処理する「正しい」方法でC#が、私はプログラミングに、私の最も最近の仕事にかなり新しいです
任意の提案は
エルス適切な「何もしない」構文は、私は他の文は、forループ内にネストされた場合は、次を作成した場合あなたが見ることができます、私は文が真であれば、何もしない方法を望みます。この場合、 "// Do Nothing"をコメントアウトするのは正しい構文ですか、それとも開いて閉じたカッコとして残すべきですか?あるいは、if行の最後に中括弧を置くことができますか?式を否定ないのはなぜこれを処理する「正しい」方法でC#が、私はプログラミングに、私の最も最近の仕事にかなり新しいです
任意の提案は
をいただければ幸いですか!
for (int i = 0; i < originalColumnCells.Count; i++)
{
if (originalColumnCells[i] != sortedColumnCells[i])
{
return false;
}
}
良い...良い... upvoted – Cloud
変更にif文=と一緒に他のすべてを削除します!
if (originalColumnCells[i] != sortedColumnCells[i])
{
return false;
}
// No else
をあなたは文を逆することができます
if (originalColumnCells[i] != sortedColumnCells[i])
しかし、その後、他の何か。場合によっては、else
ブランチの必要性について考えていることを示したいが、必要がない。そして、それはこれらの線に沿って何かを使用することが私の意見で結構です(私は時々私は、複雑な文を使用して、より大きなコードブロックを持っている場合は、1つは欲しい):一般的に
else
{
// no action, already handled later on in bla bla
}
私はあなたがそのようなことを述べる必要があるならば、あなたのコードは複雑すぎて保守が難しいと主張します。 – Bauss
まあ、実際には時には、小さなコードで分けたコードでは状況が悪化することがあります(特にかなりの変数を 'ref'する必要がある場合)。私はそれも好きではありませんし、それを使用してアドバイスをしませんが、あなたが必要な場合は、大丈夫です。 –
を、あなたは珍しい事を説明したいです。
{
}
これは空白になっていますか?何かが削除されましたか?もっと自分の意思を明示的にするために:
{
// do nothing
}
しかし - また、一般的には - あなたはおそらく、あなたが空の文で終わるしないように、あなたのif文を変更したいです。私は==
から!=
に比較を変更すると言っているいくつかの答えを参照してください。しかし、
if (a > b && c == d || Object.ReferenceEquals(null, e)) ...
のようないくつかの文について何これを反転させるために、あなたは、<=
に>
を変更!=
に==
を変更、&&
と||
事業者を変更する、など私はそれが最も簡単なだけで、文全体を反転させるために見つけるだろう代わりに、各比較演算子に変更しようとしているの!
で:(!originalColumnCells [i]は= sortedColumnCells [i]は) `{/ *あなたが* /やりたい}場合
if (!(a > b && c == d || Object.ReferenceEquals(null, e))) ...
を' '' ''それを逆に – tym32167
。 'originalColumnCells [i]!= sortedColumnCells [i]'です。今、あなたは他を必要としません – FCin
文を逆転させます。 do not nothingブロックについて心配する必要はありません。 –