Как оптимизировать логическую операцию?

Question

У меня есть два RadioButton s: r1 и r2.

я должен включить как на Enabled = true и отключить только бесконтрольно на Enabled = false

public bool Enabled 
{ 
    set 
    { 
    if(value) 
    { 
     r1.Enabled = value; // true 
     r2.Enabled = value; 
    } 
    else 
    { 
     if(!r1.Checked) 
     { 
     r1.Enabled = value; // false if not checked 
     } 
     if(!r2.Checked) 
     { 
     r2.Enabled = value; 
     } 
    } 
    } 
} 

Какой оператор я использовать, чтобы написать каждое условие в одной строке?

Answer 1

Ключа, что включено свойство остается как есть, когда значение является ложным и соответствующим флажок is проверено. Так что попробуйте это:

public bool Enabled 
{ 
    set 
    { 
    r1.Enabled = !r1.Checked? value: value || r1.Enabled; 
    r2.Enabled = !r2.Checked? value: value || r2.Enabled; 
    } 
} 

или

public bool Enabled 
{ 
    set 
    { 
     r1.Enabled = r1.Checked? value || r1.Enabled: value; 
     r2.Enabled = r2.Checked? value || r2.Enabled: value; 
    } 
} 

Answer 2

Я думаю, что это достаточно ясно:

public bool Enabled 
{ 
    set 
    { 
    if (value || !r1.checked) { 
     r1.Enabled = value; 
    } 
    if (value || !r2.checked) { 
     r2.Enabled = value; 
    } 
    } 
} 

Answer 3

Следующая будет похоже на то, что вы делаете, но у вас есть необработанное случай в исходном коде: если ((значение) & & (r1.Checked)), r1.Enabled никогда не устанавливается (такое же условие для r2). Если вы установили r1.Enabled и r2.Enabled в true, по умолчанию где-то может быть достаточно кода.

 
r1.Enabled = value || r1.Checked; 
r2.Enabled = value || r2.checked; 

Если у вас есть странная зависимость, я ничего особо чистое, не видя ...