I'm mocking up a small version now that uses 4 momentary switches, 2 digital inputs that I'm using is as momentary switch inputs
4 switches on two pins? Or 6 switches on 6 pins?
Why don't they all use the internal pullup resistors? Do the other 4 have external pullup or pulldown resistors?
int loopState1 = 0;
int loopState2 = 0;
int loopState3 = 0;
int loopState4 = 0;
Crappy names. You are not saving the state of a loop. Are you sure these shouldn't be longs? After all, you do need to store a value as high as 15 (to be 255) in them.
It's hard to understand what your code is supposed to be doing, given meaningless names like presetN and loopStateN.
You should be using arrays, not numbered variable names. Then, you could use a for loop to iterate over all of them, instead of cut and paste.
Where are you going to save these? If you are going to save them in EEPROM, bytes will be easier.
while (digitalRead (pb2) == HIGH);
While the switch state is HIGH, do nothing. When it goes LOW, delay for a millisecond. Why? Even removing the semicolon from the end, to make the delay(1) call the body of while loop doesn't make sense. Why do you care how many times the while loop iterates? Once or a million times makes no difference.