Co jest z tym nodem, mam prosta metode:
getRealSizeOfTransaction(trade, correctTradePair) {
this.contractList.forEach(el => {
if(el.name == correctTradePair)
{
let quantoMultiplier = Number(el.quantoMultiplier);
let quantityToDoMultiplierByQuant = trade.quantityToDo / quantoMultiplier;
return Math.floor(quantityToDoMultiplierByQuant);
}
})
return 0;
}
I mi zwraca zawsze 0 o co tu #%%^ chodzi?
goku21 napisał(a):
Co jest z tym nodem, mam prosta metode:
getRealSizeOfTransaction(trade, correctTradePair) { return this.contractList.forEach(el => { if(el.name == correctTradePair) { let quantoMultiplier = Number(el.quantoMultiplier); let quantityToDoMultiplierByQuant = trade.quantityToDo / quantoMultiplier; return Math.floor(quantityToDoMultiplierByQuant); } }) }I mi zwraca zawsze 0 o co tu #%%^ chodzi?
goku21 napisał(a):
aaaaa no tak strzalka gubi thisa ja pier#$^$% dzieki;]
What? Nie. Żadnego związku z this to nie ma, ale za to ma z tym że nie rozumiesz jak działa programowanie funkcyjnie.
Jeśli chciałeś zwrócić jakiś element z listy i przeprowadzić na nim operacje to powinieneś użyć .find(). Jeśli wiele to .filter().map().
Strzelam że chodziło Ci o to drugie, więc to powinno wyglądać jakoś tak
getRealSizeOfTransaction(trade, correctTradePair) {
return this.contractList
.filter(el => el.name === correctTradePair)
.map(el => {
const quantoMultiplier = Number(el.quantoMultiplier);
const quantityToDoMultiplierByQuant = trade.quantityToDo / quantoMultiplier;
return Math.floor(quantityToDoMultiplierByQuant);
});
}
@TomRiddle: a jesli nic nie zanjdzie to nie zwroci 0 tylko pewnie undefined:> A z ifem wygladaloby to paskudnie :/
Noo mysle po staremu dlatego uzywam zwykle fora.
goku21 napisał(a):
@TomRiddle: a jesli nic nie zanjdzie to nie zwroci 0 tylko pewnie undefined:> A z ifem wygladaloby to paskudnie :/
Ty poważny jesteś?
this.contractList to jest array, jak rozumiem. I operacje filter() oraz .map() również działają na arrayu, i array też zwracają. Jeśli nie znajdziesz żadnego przypadku dla którego el.name === correctTradePair to dostaniesz pusty, array czyli []. Co w tym dziwnego?
@TomRiddle: Jezeli uzyje find to mi zwroci undefined.
goku21 napisał(a):
@TomRiddle: Jezeli uzyje find to mi zwroci undefined.
Tak, to wtedy musisz po prostu zaprogramować, co Twoja funkcja getRealSizeOfTransaction() powinna zrobić kiedy przekażesz jej listę elementów, w którym żaden z nich nie spełnia kryterium.
Ja bym rzucił wyjątek; ale możesz zrobić bardziej głupie rzeczy jeśli chcesz, typu zwrócić null, 0, -1 albo jakąś inną głupią wartość.
@TomRiddle: Nie ma byc wyjatku poniewaz nie ma zadnego bledu. Jezeli jakims cudem nie znajdzie w tablicy szukanego elementu, tzn ze owy element nie istnieje a to oznacza ze ilosc danego elementu wynosi 0 i taka liczba ma sie pojawic.
Napisalem tak:
getRealSizeOfTransaction(trade, correctTradePair)
{
let el = this.contractList.find(el => el.name == correctTradePair);
if(el == undefined)
return 0;
let quantoMultiplier = Number(el.quantoMultiplier);
let quantityToDoMultiplierByQuant = trade.quantityToDo / quantoMultiplier;
return Math.floor(quantityToDoMultiplierByQuant);
}
Dziala ok ale ten if jest brzydki...
goku21 napisał(a):
@TomRiddle: Nie ma byc wyjatku poniewaz nie ma zadnego bledu.
No ja bym się zdziwił gdybym zawołał getRealSizeOfTransaction(trade, "absadashdbhajsbdhasbd") albo getRealSizeOfTransaction(trade, "") i dostał 0. Moim zdaniem ta funkcja w ogóle nie powinna być zawołana z takim correctTradePair którego nie ma w Twojej liście.
No ale jak mówiłem, chcesz zwrócić 0 to zwróć 0.
Jezeli jakims cudem nie znajdzie w tablicy szukanego elementu, tzn ze owy element nie istnieje a to oznacza ze ilosc danego elementu wynosi 0 i taka liczba ma sie pojawic.
Dziwna logika, bo skoro tego elementu nie musi być 1, bo by znaczyło też że może być ich więcej niż 1? I wtedy ogarniesz tylko pierwszy?
W ogóle dziwny jest design tej funkcji, nie chcesz pokazać całej klasy?
@TomRiddle: Nie chce:> Moze byc tylko 1. Z reszta na poczatku programu pobiera liste z mozliwymi obiektami, później odwołuje sie tylko do tej listy zeby zaokraglic precyzje. Jezeli jednak ilosci jest mniej niz najmniejsza mozliwa jednostka, metoda powinna zwrocic 0 co jest logiczne moim zdaniem:> Byc moze niedokladnie nazwalem ta metode.