Thanks again Peter, and don't worry! You helped me a lot! I added your
solution like this :
let
currentBox = List.concat model.cells
|> List.filter (\box -> box.id == boxId)
|> List.head
|> Maybe.withDefault (Box -1 Nothing)
updateRow row =
let
updateBox box =
if boxId == box.id then
{box | player = Just model.current}
else
box
in
List.map updateBox row
in
case currentBox.player of
Nothing ->
({model | cells = List.map updateRow model.cells, current =
changePlayer model.current, boxClicked = Just boxId}, Cmd.none)
Just _ ->
(model, Cmd.none)
and it worked! Now, I'm asking one question : are all these stuffs on
manipulating lists (concat, map and so on) and immutability more efficient
than traditionnal OOP?
Thanks again!
Le jeudi 27 octobre 2016 13:56:47 UTC+2, Peter Damoc a écrit :
>
> Maybe is used to wrap an operation that might fail.
> So, when you ask for the head of a list, you might get Just the head OR
> you might get Nothing if the list is empty.
>
> However, the issue here is again, an error on my part. Sorry about that.
>
>
> The actual solution to the currentBox is:
>
> currentBox =
> List.concat model.cells
> |> List.filter (\box -> box.id == boxId)
> |> List.head
> |> Maybe.withDefault (Box -1 Nothing) -- this value should never
> happen in your context
>
>
> You don't need to filter each row, you need to concatenate the rows into a
> single list and filter that list on the id of the box.
>
> Sorry about the misleading code from the previous message (I should have
> tested it but was too lazy).
>
>
> --
> There is NO FATE, we are the creators.
> blog: http://damoc.ro/
>
--
You received this message because you are subscribed to the Google Groups "Elm
Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
