[trzeciodkonca]

Dlaczego jak wywołuje funkcje z nowym argumentem array, to zwraca mi starego array?

W tempArr usunąłem powiązanie przez klonowanie.

https://paste.ofcode.org/nAuLZb48bbfr7ZP85VurLv

Dzięki

#javascript

[S.....8]

@trzeciodkonca: z ciekawosci: co ta funkcja ma robic?

[m4kb0l]

@trzeciodkonca: ale o co chodzi? Że tempArr to samo co arr?

[trzeciodkonca]

@Sheppard28: Robie zadania na codewars i mam usunąć kierunki przeciwne, ale leżące obok siebie z array.

[S.....8]

@trzeciodkonca: a to nie lepiej zrobic jakas mape, w ktorej liczbom parzystym odpowiadalyby np kierunki NS a nieparzystym EW i wtedy tylko sprawdzasz czy w jednej tablicy sa liczby parzyste lub niepatrzyste? Czy czegos nie rozumiem

[trzeciodkonca]

@m4kb0l: Chodzi o to że jak wrzucę już raz przefiltrowany arr do funkcji checkPairs, to chciałbym, aby została znów przefiltrowana.
Pierwsze wrzucam arr do funkcji z której powstaje nowy arr. Klonuje go za pomocą spread operator i jeszcze raz wrzucam do funkcji. Niestety wyniki są takie same, czyli tak jakbym wykonał to z arr.
checkPairs(tempArr, arrOposite).

[trzeciodkonca]

@Sheppard28: hmmm... Co mi daje sprawdzenie parzystości? Chcę usunąć z array takie kierunki jak EW, WE, NS i SN. I np. Zostanie w tablicy samo W bo nie ma pary lub ESWN bo nie są obok siebie. Problem jest że po pierwszym filtrowaniu niektóre kierunki przeciwstawne, mogą znałeś się obok siebie i znów trzeba przefiltrować

[S.....8]

@trzeciodkonca:

wiem, ze to nie jest odpowiedz na Twoje pytanie ale wydaje mi sie ze masz strasznie zagmatwany ten kod. Tutaj masz przyklad dziallajacego algorytmu, ktory wymyslillem:

Mozesz go sobie teraz ulepszyc (bardzo) latwo tak, zeby obslugiwal kierunki NE, NW, SW, SE.

https://jsfiddle.net/unxat4dk/

[trzeciodkonca]

@Sheppard28:
Nie jestem jeszcze przyzwyczajony do tych nowych metod i twój kod wydaje mi się bardziej skomplikowany. ( ͡° ͜ʖ ͡°)

Może łatwiej sprawdzić o co chodzi jak podam dane do testów:

Test.assertSimilar(dirReduc(["NORTH", "SOUTH", "SOUTH", "EAST", "WEST", "NORTH", "WEST"]), ["WEST"])
Test.assertSimilar(dirReduc(["NORTH", "WEST", "SOUTH", "EAST"]), ["NORTH", "WEST", "SOUTH", "EAST"])
Test.assertSimilar(dirReduc(["NORTH", "SOUTH", "EAST", "WEST", "EAST", "WEST"]), [])

[S.....8]

@trzeciodkonca: To jaki ma byc wynik funkcji dirReduc w tych 3 przypadkach? I wynik testow?

[trzeciodkonca]

@Sheppard28: za nawiasem ostatni array.
1 ["WEST"]
2 ["NORTH", ...
3 []

[trzeciodkonca]

@Sheppard28: Część testów wykonano się dobrze i to uśpiło moją czujność. Błąd był w warunku.

Powinien być:

if ((arrPair.toString() === arrOposite[0].toString()) || (arrPair.toString() === arrOposite[1].toString()))

Błąd był też w zerowaniu licznika i. Powinno być i = -1;

Teraz nie trzeba sprawdzać kolejny raz, bo z każdą usuniętą parą, sprawca array od początku :)

A tutaj rozwiązanie z użyciem reduce:

function dirReduc(plan) {
var opposite = {
'NORTH': 'SOUTH', 'EAST': 'WEST', 'SOUTH':

[trzeciodkonca]

@trzeciodkonca: W każdym razie dzięki za pomoc.