Frozen game

Question

Hi!
I am programming the Frozen game (available for Android) which is pretty much like Candy Crush Saga.
The program should swap two elements of an array selected by the player clicking on them. 
My problem is that the program keeps on swapping the elements. 
And I don't know how to change the numbers of the array to pictures (i.e. number 1 is a blue stone, 2 is a red one,...).
Please somebody help me!
Thanks in advance
Attachments:
mygame.vi ‏65 KB

Answer 1

Start with a few simple LabVIEW tutorial before starting such a project. (For example you might learn that "index array" is resizable.)
Then sit down and think about the code layout and design template that would be most approriate.
After glancing at your code for a few milliseconds, here are some of the glaring mistakes I noticed (I am sure there are many more...):
Your random number generator is flawed, because it does not produce an fair and even distribution. Edge elements only occur with half the probability of the other elements.
Why are there so many stop buttons?
You have a loop that does a gigantic amount of data manipulation based on exactly two local variables as fast as the computer allows. Except for filling a few meaningless indicators, most of the code gets stripped out by the compiler because it produces no output. So why is it there?
You have another small while loop that operates on the array based on the value of four local variables, repeating the same operation over and over, again as fast as the computer allows. If this calculation is important, it only needs to be done if one ot the inputs changes, and not over and over, millions of times per second. This code belongs in the event loop to execute when a mouse-down occurs and the code is in the correct state. It seems to swap two elements of a 2D array, this is much more easily done using the "in place element structure" and "" (or simply cross the wires).
Now let's have a look at the loop with the event structure:
Delete the timeout case since you don't use it.
You don't need a stacked shift register. Get the current value after the event structure and use a feeback node for the immediately previous value.
In the "mouse down" event you do some exceedingly convoluted calcuation based on the coordinates. For some ideas on how to do it better, have a look at my old 4x4 tic tac toe game posted here long ago. Your game is much simpler and I truly believe that your game could be written with code fitting on a postcard. Try it!
LabVIEW Champion . Do more with less code and in less time .