Witam, potrzebuje pomysłu jak mogę zmniejszyć ilość tego kodu
public static bool zwroc_wynik( int wynik_gospodarz, int wynik_gosc, string typ)
{
if (wynik_gospodarz > wynik_gosc && typ == "1")
{
return true;
}
else if (wynik_gospodarz == wynik_gosc && typ == "0")
{
return true;
}
else if (wynik_gosc > wynik_gospodarz && typ == "2")
{
return true;
}
else if ((wynik_gospodarz > wynik_gosc || wynik_gospodarz == wynik_gosc) && typ == "10")
{
return true;
}
else if ((wynik_gosc > wynik_gospodarz || wynik_gospodarz == wynik_gosc) && typ == "02")
{
return true;
}
else if ((wynik_gosc > wynik_gospodarz || wynik_gospodarz < wynik_gosc) && typ == "12")
{
return true;
}
else
{
return false;
}
}
Teoretycznie chyba mógłbym zrobić to tak, ale czy jest to poprawne i nie da się tego lepiej zrobić?
Chociaż wizualnie, żeby zwiększyć przejrzystość.
public static bool zwroc_wynik( int wynik_gospodarz, int wynik_gosc, string typ)
{
if (
(wynik_gospodarz > wynik_gosc && typ == "1") ||
(wynik_gospodarz == wynik_gosc && typ == "0") ||
(wynik_gosc > wynik_gospodarz && typ == "2") ||
((wynik_gospodarz > wynik_gosc || wynik_gospodarz == wynik_gosc) && typ == "10") ||
((wynik_gosc > wynik_gospodarz || wynik_gospodarz == wynik_gosc) && typ == "02") ||
((wynik_gosc > wynik_gospodarz || wynik_gospodarz < wynik_gosc) && typ == "12")
)
{
return true;
}
else
{
return false;
}
}