Просто се чудех дали това е възможно, защото започнах да използвам троични оператори, за да намаля редовете код и ми харесва.
if (x==y)
{
z += x;
} else if (x==z)
{
z += y;
} else {
z += 1;
}
мога да направя това сега, ако има само един оператор if като този:
z = x == y ? z += x : z += 1;
Би било така:
z =
x == y ? z + x :
x == z ? z + y :
z + 1;
Ако използвате z += x като операнд, в крайна сметка ще направите z = (z += x). Въпреки че работи в този специален случай, тъй като резултатът от израза z += x е крайната стойност на z, може да не работи в други случаи.
Въпреки това, тъй като всички операции имат общо z +=, можете да направите така:
z +=
x == y ? x :
x == z ? y :
1;
Но използвайте внимателно. Кодът често е по-четлив и поддържаем, колкото по-прост е, а вложените условни операции не са много четливи. Също така, използвайте това само когато имате израз като резултат от условната операция, това не е заместваща замяна на оператора if.
Можеш да използваш
z += x == y ? x : x == z ? y : 1;
Но честно казано, това не е по-четливо от това, което сте имали преди. Можете да го направите малко по-ясно, като добавите скоби:
z += x == y ? x : (x == z ? y : 1);
Но като цяло бих стоял далеч от вложени условни оператори, освен ако не играя голф.
Четири реда код и най-четливият, IMO. Тук няма нужда от троичен оператор:
if (x == y || x == z)
z += y;
else
z++;
Ако трябваше да го напиша с троичен код, бих направил:
z += (x == y || x == z) ? y : 1;
x или y, ако и двете ще бъдат еднакви, а ако не са, тогава добавяте y все пак.
- person BoltClock; 11.07.2012
трябва да направите това, като използвате скоби като тези:
(x==y)?(z+=x):((x==z)?(z+=y):(z+=1))
Това е лесно да продължите с троичен оператор, а не с if else if условие, просто трябва да продължите същото дори след ":". по-долу е пробата.
var result = a ? x : b ? y : z;
За да превърна изчислението z в един ред, бих направил нещо подобно:
public int GetZBasedOnXY(int z, int x, int y)
{
// Chose this solution, but any can go in here and work.
if (x == y || x == z)
return z + y;
else
return z + 1;
}
Тогава:
z = GetZBasedOnXY(z, x, y);
По-четливо и ако именуването е добро и методът има покритие на единичен тест, още по-добре.
