Increment element at index









up vote
3
down vote

favorite
1












I am working on the guide.elm-lang website on the Random example.
I am trying to add a feature that shows you the total number of times you threw the dice, and some stats on how many times you got each face.



To do this I've changed the model to look like this:



type alias Model =

die_face : Int,
total_throws : Int,
stats: Array.Array Int



and this is what I do to update the model:



 
die_face = face,
total_throws = model.total_throws + 1,
stats = Array.set face ((Array.get face model.stats) + 1) model.stats



this throws an error that tells me:




This get call produces:



#Maybe# Int


But (+) only works with #Int# and #Float# values.




Which refers to Array.get not returning a Int but a Maybe and therefore I can't add it to the number 1.



I have tried using lists to achieve the same purpose but since they are not indexed I am not sure what to increment when I map over it. I am thinking of using records to do this and figure out a way to map the record keys to the face Int.



In general my question is. What is a good method to increment an element at index [x] of an Array in elm ? Or if I am just thinking about this wrong, what would be the elm way ?










share|improve this question



























    up vote
    3
    down vote

    favorite
    1












    I am working on the guide.elm-lang website on the Random example.
    I am trying to add a feature that shows you the total number of times you threw the dice, and some stats on how many times you got each face.



    To do this I've changed the model to look like this:



    type alias Model =

    die_face : Int,
    total_throws : Int,
    stats: Array.Array Int



    and this is what I do to update the model:



     
    die_face = face,
    total_throws = model.total_throws + 1,
    stats = Array.set face ((Array.get face model.stats) + 1) model.stats



    this throws an error that tells me:




    This get call produces:



    #Maybe# Int


    But (+) only works with #Int# and #Float# values.




    Which refers to Array.get not returning a Int but a Maybe and therefore I can't add it to the number 1.



    I have tried using lists to achieve the same purpose but since they are not indexed I am not sure what to increment when I map over it. I am thinking of using records to do this and figure out a way to map the record keys to the face Int.



    In general my question is. What is a good method to increment an element at index [x] of an Array in elm ? Or if I am just thinking about this wrong, what would be the elm way ?










    share|improve this question

























      up vote
      3
      down vote

      favorite
      1









      up vote
      3
      down vote

      favorite
      1






      1





      I am working on the guide.elm-lang website on the Random example.
      I am trying to add a feature that shows you the total number of times you threw the dice, and some stats on how many times you got each face.



      To do this I've changed the model to look like this:



      type alias Model =

      die_face : Int,
      total_throws : Int,
      stats: Array.Array Int



      and this is what I do to update the model:



       
      die_face = face,
      total_throws = model.total_throws + 1,
      stats = Array.set face ((Array.get face model.stats) + 1) model.stats



      this throws an error that tells me:




      This get call produces:



      #Maybe# Int


      But (+) only works with #Int# and #Float# values.




      Which refers to Array.get not returning a Int but a Maybe and therefore I can't add it to the number 1.



      I have tried using lists to achieve the same purpose but since they are not indexed I am not sure what to increment when I map over it. I am thinking of using records to do this and figure out a way to map the record keys to the face Int.



      In general my question is. What is a good method to increment an element at index [x] of an Array in elm ? Or if I am just thinking about this wrong, what would be the elm way ?










      share|improve this question















      I am working on the guide.elm-lang website on the Random example.
      I am trying to add a feature that shows you the total number of times you threw the dice, and some stats on how many times you got each face.



      To do this I've changed the model to look like this:



      type alias Model =

      die_face : Int,
      total_throws : Int,
      stats: Array.Array Int



      and this is what I do to update the model:



       
      die_face = face,
      total_throws = model.total_throws + 1,
      stats = Array.set face ((Array.get face model.stats) + 1) model.stats



      this throws an error that tells me:




      This get call produces:



      #Maybe# Int


      But (+) only works with #Int# and #Float# values.




      Which refers to Array.get not returning a Int but a Maybe and therefore I can't add it to the number 1.



      I have tried using lists to achieve the same purpose but since they are not indexed I am not sure what to increment when I map over it. I am thinking of using records to do this and figure out a way to map the record keys to the face Int.



      In general my question is. What is a good method to increment an element at index [x] of an Array in elm ? Or if I am just thinking about this wrong, what would be the elm way ?







      arrays elm maybe






      share|improve this question















      share|improve this question













      share|improve this question




      share|improve this question








      edited Nov 10 at 22:03









      glennsl

      1




      1










      asked Nov 10 at 21:51









      Nicola Pedretti

      1,84911523




      1,84911523






















          1 Answer
          1






          active

          oldest

          votes

















          up vote
          5
          down vote



          accepted










          Array.get returns a Maybe because it has to account for the case where the index is outside the array. The simplest way of getting around that is using Maybe.withDefault with a reasonable default value:



          Array.set face (((Array.get face model.stats) |> Maybe.withDefault 0) + 1) model.stats


          It might be a good idea to write a helper function for this though, to clean up the code a bit. Something like this:



          incrementAt : Int -> Array Int -> Array Int
          incrementAt index array =
          case Array.get index array of
          Just value ->
          Array.set index (value + 1) array

          Nothing ->
          array


          You can also use Array.Extra.update if you don't mind the extra dependency.






          share|improve this answer




















            Your Answer






            StackExchange.ifUsing("editor", function ()
            StackExchange.using("externalEditor", function ()
            StackExchange.using("snippets", function ()
            StackExchange.snippets.init();
            );
            );
            , "code-snippets");

            StackExchange.ready(function()
            var channelOptions =
            tags: "".split(" "),
            id: "1"
            ;
            initTagRenderer("".split(" "), "".split(" "), channelOptions);

            StackExchange.using("externalEditor", function()
            // Have to fire editor after snippets, if snippets enabled
            if (StackExchange.settings.snippets.snippetsEnabled)
            StackExchange.using("snippets", function()
            createEditor();
            );

            else
            createEditor();

            );

            function createEditor()
            StackExchange.prepareEditor(
            heartbeatType: 'answer',
            convertImagesToLinks: true,
            noModals: true,
            showLowRepImageUploadWarning: true,
            reputationToPostImages: 10,
            bindNavPrevention: true,
            postfix: "",
            imageUploader:
            brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
            contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
            allowUrls: true
            ,
            onDemand: true,
            discardSelector: ".discard-answer"
            ,immediatelyShowMarkdownHelp:true
            );



            );













            draft saved

            draft discarded


















            StackExchange.ready(
            function ()
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53243760%2fincrement-element-at-index%23new-answer', 'question_page');

            );

            Post as a guest















            Required, but never shown

























            1 Answer
            1






            active

            oldest

            votes








            1 Answer
            1






            active

            oldest

            votes









            active

            oldest

            votes






            active

            oldest

            votes








            up vote
            5
            down vote



            accepted










            Array.get returns a Maybe because it has to account for the case where the index is outside the array. The simplest way of getting around that is using Maybe.withDefault with a reasonable default value:



            Array.set face (((Array.get face model.stats) |> Maybe.withDefault 0) + 1) model.stats


            It might be a good idea to write a helper function for this though, to clean up the code a bit. Something like this:



            incrementAt : Int -> Array Int -> Array Int
            incrementAt index array =
            case Array.get index array of
            Just value ->
            Array.set index (value + 1) array

            Nothing ->
            array


            You can also use Array.Extra.update if you don't mind the extra dependency.






            share|improve this answer
























              up vote
              5
              down vote



              accepted










              Array.get returns a Maybe because it has to account for the case where the index is outside the array. The simplest way of getting around that is using Maybe.withDefault with a reasonable default value:



              Array.set face (((Array.get face model.stats) |> Maybe.withDefault 0) + 1) model.stats


              It might be a good idea to write a helper function for this though, to clean up the code a bit. Something like this:



              incrementAt : Int -> Array Int -> Array Int
              incrementAt index array =
              case Array.get index array of
              Just value ->
              Array.set index (value + 1) array

              Nothing ->
              array


              You can also use Array.Extra.update if you don't mind the extra dependency.






              share|improve this answer






















                up vote
                5
                down vote



                accepted







                up vote
                5
                down vote



                accepted






                Array.get returns a Maybe because it has to account for the case where the index is outside the array. The simplest way of getting around that is using Maybe.withDefault with a reasonable default value:



                Array.set face (((Array.get face model.stats) |> Maybe.withDefault 0) + 1) model.stats


                It might be a good idea to write a helper function for this though, to clean up the code a bit. Something like this:



                incrementAt : Int -> Array Int -> Array Int
                incrementAt index array =
                case Array.get index array of
                Just value ->
                Array.set index (value + 1) array

                Nothing ->
                array


                You can also use Array.Extra.update if you don't mind the extra dependency.






                share|improve this answer












                Array.get returns a Maybe because it has to account for the case where the index is outside the array. The simplest way of getting around that is using Maybe.withDefault with a reasonable default value:



                Array.set face (((Array.get face model.stats) |> Maybe.withDefault 0) + 1) model.stats


                It might be a good idea to write a helper function for this though, to clean up the code a bit. Something like this:



                incrementAt : Int -> Array Int -> Array Int
                incrementAt index array =
                case Array.get index array of
                Just value ->
                Array.set index (value + 1) array

                Nothing ->
                array


                You can also use Array.Extra.update if you don't mind the extra dependency.







                share|improve this answer












                share|improve this answer



                share|improve this answer










                answered Nov 10 at 22:01









                glennsl

                1




                1



























                    draft saved

                    draft discarded
















































                    Thanks for contributing an answer to Stack Overflow!


                    • Please be sure to answer the question. Provide details and share your research!

                    But avoid


                    • Asking for help, clarification, or responding to other answers.

                    • Making statements based on opinion; back them up with references or personal experience.

                    To learn more, see our tips on writing great answers.





                    Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


                    Please pay close attention to the following guidance:


                    • Please be sure to answer the question. Provide details and share your research!

                    But avoid


                    • Asking for help, clarification, or responding to other answers.

                    • Making statements based on opinion; back them up with references or personal experience.

                    To learn more, see our tips on writing great answers.




                    draft saved


                    draft discarded














                    StackExchange.ready(
                    function ()
                    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53243760%2fincrement-element-at-index%23new-answer', 'question_page');

                    );

                    Post as a guest















                    Required, but never shown





















































                    Required, but never shown














                    Required, but never shown












                    Required, but never shown







                    Required, but never shown

































                    Required, but never shown














                    Required, but never shown












                    Required, but never shown







                    Required, but never shown







                    Popular posts from this blog

                    How to how show current date and time by default on contact form 7 in WordPress without taking input from user in datetimepicker

                    Darth Vader #20

                    Ondo