[wytrzzeszcz]

#programowanie
Mamy funkcje która jest gigantycznym ifem
Jak to się minimalizuje?
I return dawać w środku czy zawsze na końcu?
#matematyka

[farmaceut]

@wytrzzeszcz: poka kod

[koostosh]

@wytrzzeszcz: najlepiej usunąć wszsystko i return false.( ͡° ͜ʖ ͡°)

Poka kod

[Kizio]

@wytrzzeszcz: Na przykład tak: https://pl.wikipedia.org/wiki/Strategia_(wzorzec_projektowy), ale

[wytrzzeszcz]

@Kizio: @koostosh: @farmaceut: chciałbym ale nie mogę. Już czytam Wiki

[PanPapcio]

@wytrzzeszcz: I w taki sposób powstają aplikacje ze spółek skarbu państwa :)

[koostosh]

@wytrzzeszcz: to przynajmniej opisz jak ta funkcja wyglada np

funkcja() { if (costam) {#!$%@?} else {#!$%@?; return false} return true}

to przynajmniej będzie wiadomo co rozumiesz przez gigantyczny if

[wytrzzeszcz]

If return
If return
If return
If {
If return
Return
}
Return

Coś takiego więcej komentarzy nie kodu

[T.....y]

@wytrzzeszcz: Nie wiem czy dobrze rozumiem ale po conditionie jest return od razu?

[mateusza]

@TenAnonToKlopoty: raczej chodzi o to, że każda gałąź ma osobny return; zamiast przypisania wyniku do jakiejś zmiennej i jednego returna na końcu.

[T.....y]

@mateusza: Nie wiedzac w sumie co tam sie dzieje jedyne co mozna chyba zalecic to kazda galaz dac do osobnej metody i ja jakos sensownie nazwac to bedzie przejrzyscie

[Sztuczny_Snieg]

@wytrzzeszcz: Minimalizacja sugeruje raczej że chcesz niektóre ify połączyć bo sprawdzają to samo. Tobie raczej chodzi o jakiś sensowny refaktor kodu. Sam fakt że znalazło się tam tyle ifów może źle świadczyć o kodzie, ale bez dużej ilości refaktoringu można wprowadzić np. wzorzec strategii czy Chain of Responsibility.

[wytrzzeszcz]

@mateusza: @TenAnonToKlopoty: tak ale nic tam się nie dzieje

[ponton]

@wytrzzeszcz: pokaż kod, ale wszystkie nazwy zastąp np. na 'foo1', 'foo2', itd, żebyśmy zobaczyli samą strukturę tylko

[rtoip7]

@wytrzzeszcz: rozbija się na mniejsze funkcje, a dalej to zależy od kodu

[wytrzzeszcz]

@rtoip7: @ulke: @ponton: @Sztuczny_Snieg: @TenAnonToKlopoty: @koostosh: @PanPapcio: @Kizio: @koostosh: @farmaceut: dzięki za rady przeorane i naprawione ;) zmiana kolejności w jednym miejscu i połowa kodu do kosza