Change:
va11 = digitalRead(1);
va12 = digitalRead(2);
va13 = digitalRead(3);
va14 = digitalRead(4);
va15 = digitalRead(5);
va16 = digitalRead(6);
va17 = digitalRead(7);
To:
val1 = digitalRead(1);
val2 = digitalRead(2);
val3 = digitalRead(3);
val4 = digitalRead(4);
val5 = digitalRead(5);
val6 = digitalRead(6);
val7 = digitalRead(7);
Yes, the C/C++ compiler is really that picky. It works on what you type in, not what you meant to type in.
Lefty