Change:
in step 13 of the dice game (js beta), the instructions say: Your detectFullHouse function should check if the user has rolled three of one number and two of another number. If so, it should update the third radio button to display a score of 25, with the correct attributes. If not, it should update the last radio button to display a score of 0, with the correct attributes. However when we look at the code in the following step 14 for this function we see this: Notice how the last radio button is updated regardless of the full house check? But the instructions said: "If not, it should update the last radio button...". People are trying to follow this instruction and failing and not understanding why (they use an if-else statement for example as in the code below) The instructions should be changed to match the requested code: Your detectFullHouse function should check if the user has rolled three of one number and two of another number. If so, it should update the third radio button to display a score of 25, with the correct attributes. It should also update the last radio button to display a score of 0, with the correct attributes. However, I noticed that the functions getHighestDuplicates and detectFullHouse are both updating the last radio button. So it seems like one of these need not do that? (so we could just not ask for this in this step since it is gonna get updated elsewhere anyway?) No response N/A I noticed this issue when I was attempting to help a camper on the forum:
I don't know why we are updating the (5,0) radio button when there is a non zero value i.e. full house, three of a kind, four of a kind ? Anything positive integer will result in more high score. I think we should only implement the last radio button the zero one when nothing above works. There is two scenario whether the None of the above is a choice which should be present every time OR we must give it as a last option to the user or maybe make it default when rolls are over. I just removed the updateRadioOption(5, 0); from the getHighestDuplicates && detectFullHouse locally in my machine and initiated it when all the 3 roll options are over, then it became the obvious choice. ! ! If you like my implementation I would love to raise a PR or if you have any other implementation logic, I would like to work on this issue :)
I propose updating the text to:
the text works but should we look at why so many of the functions in this project need to update the last radio button? I feel like updating the last radio should just happen independently of the various functions like getHighestDuplicates or detectFullHouse
In a production app, sure. Here, I think the reinforcement of how conditions work and how code after an block executes and such is good.
Okay fair enough, thanks.

Discussion:
If the user rolls three of one number, and two of another number, this is called a full house. Declare a `detectFullHouse` function that accepts a single argument. The function will be passed the `diceValuesArr` array when called. <del> Your `detectFullHouse` function should check if the user has rolled three of one number and two of another number. If so, it should update the third radio button to display a score of `25`, with the correct attributes. If not, it should update the last radio button to display a score of 0, with the correct attributes. </del> <ins> Your `detectFullHouse` function should check if the user has rolled three of one number and two of another number. If so, it should update the third radio button to display a score of `25`, with the correct attributes. Regardless, it should always update the last radio button to display a score of 0, with the correct attributes. </ins> Don't forget to call your new function when the dice are rolled.