Jak mógłbym zrefaktoryzować te ify?

Jak mógłbym zrefaktoryzować te ify?
WeiXiao
  • Rejestracja:około 9 lat
  • Ostatnio:około 9 godzin
  • Postów:5108
0

Witam, potrzebuje pomysłu jak mogę zmniejszyć ilość tego kodu

Kopiuj
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ść.

Kopiuj
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;
	}
}
edytowany 2x, ostatnio: WeiXiao
caer
pomijając inne rozwiązania w tym wątku, jeśli masz return to nie potrzebujesz else
krzysiek050
  • Rejestracja:ponad 12 lat
  • Ostatnio:około 4 lata
  • Postów:1272
2

Każde:

Kopiuj
if(x) return true; 
else return false;

skraca się do:

Kopiuj
return x;

Dodatkowo niektóre pary sprawdzań możesz zastąpić jednym.

Kopiuj
public static bool zwroc_wynik( int wynik_gospodarz, int wynik_gosc, string typ)
{
return 
        (wynik_gospodarz > wynik_gosc && typ == "1") ||
        (wynik_gospodarz == wynik_gosc && typ == "0") ||
        (wynik_gosc > wynik_gospodarz && typ == "2") ||
        ((wynik_gospodarz >= wynik_gosc) && typ == "10") ||
        ((wynik_gosc >= wynik_gospodarz) && typ == "02") ||
        ((wynik_gosc != wynik_gospodarz) && typ == "12");
}
Zobacz pozostałe 2 komentarze
krzysiek050
No ja z kolei nie wiem czemu dla czegoś takiego robić zmienną, którą jedynym celem jest przekazanie do returna. Debuguje w Intelij i mogę szybko sprawdzić co wychodzi z całości i poszczególnych linijek. Działa to nawet gdyby któraś z lini sięgała do bazy, czy zewnętrznego serwisu.
katelx
zgadzam sie z @fasadin, tutaj oczywiscie jest to kod ktory wklepiesz w sprawdzarke i zapomnisz ale w normalnym projekcie taki kod jest duzo gorszy do przeczytania i zrozumienia (zwlaszcza jak masz pare and/or). to ze intellij czy vs takie rzeczy ladnie wspiera w debugu to cool ale to nie jest najszybsza i najefektowniejsza metoda czytania randomowych kawalkow kodu
robertwadowski
robertwadowski
"typ" to tutaj bardziej kontekst dla relacji między pozostałymi zmiennymi tak więc wersja @katelx wydaje mi się bardziej intuicyjna
fasadin
@Juhas widac ze nie masz pojecia o czym piszesz. Debugowanie to nie sprawdzanie calego wyniku. Dodatkowo to nie jest zaden narzut bo zostanie to zoptymalizowane. Nie mam nic sie do czepiania jezeli jest poparte argumentami ktore istenija ;) @krzysiek050 rozumiem ze to przyklad, dlatego moja uwaga jest wlasnie na temat jakiegos korpo projektu.
JU
@fasadin: w tym przypadku bez problemu możesz podejrzeć w debugerze każdą zmienną, a w okienku auto cały wynik metody. Nie potrzebujesz do tego dodatkowej zmiennej, którą będziesz mógł sobie podejrzeć. Widać, że chyba nie wiesz, jak działa okienko auto. Co do optymalizacji to nie wiem. Nauczony jestem, że wynik zwracany bezpośrednio za pomocą return jest szybszy niż zwracany z uwzględnieniem dodatkowej zmiennej. Rzecz jasna różnica w standardowym kodzie nie będzie widoczna.
ML
  • Rejestracja:około 11 lat
  • Ostatnio:około 5 lat
  • Postów:9
1

Zdefiniuj sobie funkcje result

Kopiuj
int result(wynik.gospodarz, wynik.gosc){
  if (wynik.gospodarz == wynik.gosc){
    return 0
  }
  else {
    wynik.gospodarz > wynik.gosc ? (return 1) : (return 2)  //nie piszę w javie nie wiem czy to legitne
  }
}

i sprawdź za pomocą jakiejś funkcji czy w stringu typ zawiera się wynik funkcji (contains albo coś w tym stylu - powinno takie coś być w Javie(?))

edytowany 2x, ostatnio: flowCRANE
WeiXiao
Też ciekawe rozwiązanie, żeby te wyniki też rozbić :P
ML
KISS - masz to prostsze, a skoro robisz coś bukmacherskiego, to możesz potem przy danym meczu/spotkaniu/czymkolwiek dodatkowo przedstawić jaki typ wygrał
WeiXiao
tzn. to było zadanie ze spoja które polegało na sprawdzaniu wyników (rozstrzygnięcie meczu) i porównywanie ich z obstawieniem na kuponie ;p
katelx
  • Rejestracja:prawie 10 lat
  • Ostatnio:4 miesiące
  • Lokalizacja:Hong Kong
13

najprosciej chyba

Kopiuj
public static bool zwroc_wynik2(int wynik_gospodarz, int wynik_gosc, string typ)
{
	switch(typ)
	{
		case "1": return wynik_gospodarz > wynik_gosc;
		case "0": return wynik_gospodarz == wynik_gosc;
		case "2": return wynik_gospodarz < wynik_gosc;
		case "10": return wynik_gospodarz >= wynik_gosc;
		case "02": return wynik_gospodarz <= wynik_gosc;
		case "12": return wynik_gospodarz != wynik_gosc;
		default: return false;
	}
}
Zobacz pozostały 1 komentarz
katelx
@grzesiek51114: mmm komplement od jimmiego page'a, dzieki ;)
grzesiek51114
grzesiek51114
@katelx: wiesz... byłem dazed and confused ale Twoje poprawki kodu poprawiły mi humor ;)
katelx
@grzesiek51114: :) az sobie poslucham pierwszego albumu
grzesiek51114
grzesiek51114
@katelx: uwielbiam go, jest świetny :)
KA
i to jest ten przypadek gdzie switch case jest super
vpiotr
  • Rejestracja:ponad 13 lat
  • Ostatnio:prawie 3 lata
1

Kolejna mutacja: (nie znam C#)

Kopiuj
 public static bool zwroc_wynik2(int wynik_gospodarz, int wynik_gosc, string typ)
 {
    int sign = Math.Sign(wynik_gospodarz.CompareTo(wynik_gosc)); 
    switch(typ)
    {
        case "1": return sign == 1;
        case "0": return sign == 0;
        case "2": return sign == -1;
        case "10": return sign >= 0;
        case "02": return sign <= 0;
        case "12": return sign != 0;
        default: return false;
    }
 }
edytowany 1x, ostatnio: vpiotr
john_klamka
  • Rejestracja:prawie 9 lat
  • Ostatnio:prawie 5 lat
  • Postów:177
1
Kopiuj
public static bool zwroc_wynik(int wynik_gospodarz, int wynik_gosc, string typ)
        {
            var fn = new Dictionary<string, Func<int, int, bool>>
            {
                { "1", (a, b) => a > b },
                { "0", (a, b) => a == b },
                { "2", (a, b) => a < b },
                { "10", (a, b) => a >= b },
                { "02", (a, b) => a <= b },
                { "12", (a, b) => a != b }
            };

            return fn.ContainsKey(typ) && fn[typ](wynik_gospodarz, wynik_gosc);
        }
edytowany 1x, ostatnio: john_klamka
Zobacz pozostałe 11 komentarzy
V-2
@caer ale gdy pattern matching nie ma (brak wsparcia na poziomie języka; zdaje się że ma nadejść w C# 8), a my uprzemy się, by je sobie "zasymulować"... to w niektórych przypadkach łatwiej będzie to osiągnąć taką mapą asocjacyjną z predykatami, bo switch-case nie wszędzie jest możliwy. Tu - zgoda; jak sam napisałem - switch-case rzeczywiście dobrze się sprawdza. @john_klamka ośmielił się pokazać alternatywny kierunek myślenia, za co naturalnie zostanie spalony przez surową inkwizycję ;) Prometeusz, Giordano Bruno, John Klamka. Walka trwa
katelx
ej no nikt tu nie hejtuje przeciez, raczej konstruktywnie krytykuje
V-2
Inkwizycja broniła się tak samo @katelx ;) Jaja sobie robię
somekind
@V-2, przecież pattern matching w C# jest.
V-2
@somekind: faktycznie jest to w C# 7.0. Zatrzymałem się w rozwoju jak chodzi o tę ścieżkę :) Dzięki za sprostowanie
Maciej Cąderek
Maciej Cąderek
  • Rejestracja:ponad 9 lat
  • Ostatnio:około 3 lata
  • Lokalizacja:Warszawa
  • Postów:1264
3

Nie chce mi się za bardzo szukać metod C#, więc wrzucę JS - może ktoś przepisze na C#:

Kopiuj
function isWinningTicket(host, guest, bet) {
  const resultCode = host > guest ? '1' : (host < guest ? '2' : '0')

  return bet.includes(resultCode)
}

Ot cała filozofia ;)


Edit:
chyba poprawna wersja C#

Kopiuj
public static bool IsWinningTicket(int host, int guest, string bet)
{
  var resultCode = host > guest ? "1" : (host < guest ? "2" : "0");
 
  return bet.Contains(resultCode);
}
edytowany 8x, ostatnio: Maciej Cąderek
Zobacz pozostałe 3 komentarze
Maciej Cąderek
Maciej Cąderek
@vpiotr Dobry kolega na CR podpowiedziałby lepszą nazwę :P
Maciej Cąderek
Maciej Cąderek
@vpiotr Jak to nie jest kod pisany o 3 w nocy dla zabawy na forum to się raczej stosuję ;)
vpiotr
@Maciej Cąderek: no dobra, jesteś usprawiedliwiony :)
Maciej Cąderek
Maciej Cąderek
Ogólnie to bym się tych kodów pozbył i zrobił pewnie trzy stałe, DRAW, HOST_WON, GUEST_WON itd
katelx
  • Rejestracja:prawie 10 lat
  • Ostatnio:4 miesiące
  • Lokalizacja:Hong Kong
2

w takim razie dla chcacych oszczedzic klawiature

Kopiuj
static bool W(int h,int g,string b)=>b.Contains(h>g?"1":h<g?"2":"0");
V-2
code golf :)
katelx
tak wlasnie czekam kto skroci...
Maciej Cąderek
Maciej Cąderek
Zminifikować czyjś pomysł to każdy potrafi :P
katelx
myslalam ze nikt sie nie zorientuje ;)
Kliknij, aby dodać treść...

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj <u>underline</u>.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, <b>, <i>, <kbd>, <del>, <strong>, <dfn>, <pre>, <blockquote>, <hr/>, <sub>, <sup> oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H<sub>2</sub>O i m<sup>2</sup> otrzymasz: H2O i m2.

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript
document.write('Hello World');
```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy  by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

  • Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
  • Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
  • Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
  • Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

  • Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

  • Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

  • Alt+L - zaznaczenie całej linii
  • Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
  • Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
  • Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

  • Ctrl+Enter - dodaj post
  • ⌘+Enter - dodaj post (MacOS)