Hi,
I have taken your code and put ALL { and } brackets on thier own line.
Then executed an "Auto-Format" which is CTRL-T, or under the TOOLS tab
This indents you code so you can see you code loops.
I'm not sure if this is code you have written or editing from an example.
void setup()
{
lc.shutdown(0, false);
lc.setIntensity(0, 0);
lc.clearDisplay(0);
pinMode(rockoffenseSwitch, INPUT);
pinMode (paperoffenseSwitch, INPUT);
pinMode (scissorsoffenseSwitch, INPUT);
pinMode(rockdefenseSwitch, INPUT);
pinMode (paperdefenseSwitch, INPUT);
pinMode (scissorsdefenseSwitch, INPUT);
// randomSeed(analogRead(A0)); //A: I dont think i need this?
}
void loop()
{
uor = 0;
uop = 0;
uos = 0;
udr = 0;
udp = 0;
uds = 0;
challengeDisplay();
rockoState = digitalRead(rockoffenseSwitch);
paperoState = digitalRead(paperoffenseSwitch);
scissorsoState = digitalRead(scissorsoffenseSwitch);
rockdState = digitalRead(rockdefenseSwitch);
paperdState = digitalRead(paperdefenseSwitch);
scissorsdState = digitalRead(scissorsdefenseSwitch);
void challengeDisplay()
{
// ?
for (int row = 0; row < 8; row++)
{
lc.setRow(0, row, RPS[7][row]);
}
delay (200);
lc.clearDisplay(0);
}
if (rockoState == HIGH)
// Rock, A: if the rock button is pressed, set rock value to one (now uor/s/p), then make the comptuer choose a number, whichwas removed
{
for (int row = 0; row < 8; row++)
{
}
lc.setRow(0, row, RPS[0][row]);
}
delay (500);
uor = 1;
lc.clearDisplay(0);
}
if (paperoState == HIGH)
// Paper
{
for (int row = 0; row < 8; row++)
{
}
//lc.setRow(0, row, RPS[1][row]);}
delay (500);
uop = 1;
lc.clearDisplay(0);
}
if (scissorsoState == HIGH)
// Scissors
{
for (int row = 0; row < 8; row++)
{
}
//lc.setRow(0, row, RPS[2][row]);}
delay (500);
uos = 1;
lc.clearDisplay(0);
}
if (rockdState == HIGH)
// Rock, A: if the rock button is pressed, set rock value to one (now udr/s/p), then make the comptuer choose a number. which isnt gonna happen anymore
{
for (int row = 0; row < 8; row++)
{
}
//lc.setRow(0, row, RPS[0][row]);}
delay (500);
udr = 1;
lc.clearDisplay(0);
}
if (paperdState == HIGH)
// Paper
{
for (int row = 0; row < 8; row++)
{
}
//lc.setRow(0, row, RPS[1][row]);}
delay (500);
udp = 1;
lc.clearDisplay(0);
}
if (scissorsdState == HIGH)
// Scissors
{
for (int row = 0; row < 8; row++)
{
//lc.setRow(0, row, RPS[2][row]);}
delay (500);
uds = 1;
lc.clearDisplay(0);
}
}
I have only edited part of your code, but as you can you have many { } pairs that do nothing as well as missing brackets.
Tom... 