Lots of switches print to two lines

Wondering if this looks OK ...
Using it for an Antenna/Transmitter switchbox display. 6 antenna AND 6 Transmitter switches on 2 banks.
I want to read the position of the top and bottom bank and display the selected position on a LED 20x4 display. Trying it first as serial.print then will be having the 1 to 6 choice print on the first line and the a to f choice print on the 3rd line. but for not the code is just printing on the serial monitor, What I want to know is...
DOES this look ok? It seems to work just fine... but...

code...

#include <Wire.h>




const int sw1Pin = 2;
const int sw2Pin = 3;
const int sw3Pin = 4;
const int sw4Pin = 5;
const int sw5Pin = 6;
const int sw6Pin = 7;
const int swaPin = 8;
const int swbPin = 9;
const int swcPin = 10;
const int swdPin =11;
const int swePin = 12;
const int swfPin = A0;




int sw1State = 0; 
int sw2State = 0;
int sw3State = 0;
int sw4State = 0;
int sw5State = 0;
int sw6State = 0;
int swaState = 0;
int swbState = 0;
int swcState = 0;
int swdState = 0;
int sweState = 0;
int swfState = 0;

void setup()   

{
    Serial.begin(9600); 
    pinMode(sw1Pin, INPUT);
    pinMode(sw2Pin, INPUT);
    pinMode(sw3Pin, INPUT);
    pinMode(sw4Pin, INPUT);
    pinMode(sw5Pin, INPUT);
    pinMode(sw6Pin, INPUT);
    pinMode(swaPin, INPUT);
    pinMode(swbPin, INPUT);
    pinMode(swcPin, INPUT);
    pinMode(swdPin, INPUT);
    pinMode(swePin, INPUT);
    pinMode(swfPin, INPUT);
}



void loop()  
  {

  sw1State = digitalRead(sw1Pin);
  sw2State = digitalRead(sw2Pin);
  sw3State = digitalRead(sw3Pin);
  sw4State = digitalRead(sw4Pin);
  sw5State = digitalRead(sw5Pin);
  sw6State = digitalRead(sw6Pin);
  swaState = digitalRead(swaPin);
  swbState = digitalRead(swbPin);
  swcState = digitalRead(swcPin);
  swdState = digitalRead(swdPin);
  sweState = digitalRead(swePin);          
  swfState = digitalRead(swfPin);
    
  if (sw1State == HIGH)
  
  {
  Serial.println("");  
  Serial.print("SW 1 Pin2 is now high");
  delay(500);  
  } 
  
  else if (sw2State == HIGH)
  
  {
  Serial.println(""); 
  Serial.print(" SW 2 Pin3 is now high");
  delay(500);  
  }

  else if (sw3State == HIGH)
  
  {
  Serial.println(""); 
  Serial.print("   SW 3 Pin4 is now high");
  delay(500);  
  }

  else if (sw4State == HIGH)
  
  {
  Serial.println(""); 
  Serial.print("    SW 4 Pin5 is now high");
  delay(500);  
  }

  else if (sw5State == HIGH)
  
  {
  Serial.println(""); 
  Serial.print("     SW 5 Pin6 is now high");
  delay(500);  
  }

  else if (sw6State == HIGH)
  
  {
  Serial.println(""); 
  Serial.print("      SW 6 Pin7 is now high");
  delay(500);  
  }
  
  else
  
  {
  Serial.println(""); 
  Serial.print("SWITCH 1-6 off-low");
  delay(500); 
  }

 if (swaState == HIGH)
  
  {
  Serial.println("");  
  Serial.print("\t\t\t\t SW A Pin8 is now high");
  delay(500);  
  } 

 else if (swbState == HIGH)
  
  {
  Serial.println("");  
  Serial.print("\t\t\t\t SW B Pin9 is now high");
  delay(500);  
  } 

 else if (swcState == HIGH)
  
  {
  Serial.println("");  
  Serial.print("\t\t\t\t SW C Pin10 is now high");
  delay(500);  
  } 

 else if (swdState == HIGH)
  
  {
  Serial.println("");  
  Serial.print("\t\t\t\t SW D Pin11 is now high");
  delay(500);  
  } 

 else if (sweState == HIGH)
  
  {
  Serial.println("");  
  Serial.print("\t\t\t\t SW E Pin12 is now high");
  delay(500);  
  } 

 else if (swfState == HIGH)
  
  {
  Serial.println("");  
  Serial.print("\t\t\t\t SW F PinA0 is now high");
  delay(500);  
  } 
  
  else
  
  {
  Serial.println(""); 
  Serial.print("\t\t\t\t SWITCH A-F off-low");
  delay(500); 
  }

  
  }

If your code seems to work fine .... nobody will check your code and help you to find errors. What do you expect?

However, your code is so simple, that I had a look at it anyway. It looks good.

DOES this look ok?

It looks a bit long to me.

I wasn't expecting anyone to find errors for me. I was trying to find out if there was a better way or is what I'm trying to do is ok as is. But thank you for your concern and assistance in this Arduino Project Guidance Forum.

I'd try avoid delay(). Why is it there?

Secondly, "Port Manipulation" would speed the code up slightly and also maybe make it neater for a human reader.

The delay is only there while I am using the serial monitor, once I get it ready and change it to print on the LED Display, I'll remove it. Thanks, I''ll have to read up on the "Port Manipulation" now...

This seems to be the perfect candidate for a saying that gets thrown around on this forum quite a lot:

When you start numbering variables, it's a good sign you should be using arrays.

saximus:
This seems to be the perfect candidate for a saying that gets thrown around on this forum quite a lot:

When you start numbering variables, it's a good sign you should be using arrays.

See reply #2

saximus:
When you start numbering variables, it's a good sign you should be using arrays.

Or to put it another way "god that is so turgid".

To be a grumpy old man for once ( stop it ) in my day every one of those statements would have to be a punched card, you very quickly learned ways of minimizing the number of statements. Copy and paste makes bad programming easier.

Have a read:- Arrays