Communication between Raspberry pi and Arduino via GPIO

Thankyou for your advice

PiState = digitalRead(piPin1);
 if (PiState == HIGH) {
  digitalWrite(piPin1, HIGH);
  }
 else {
  digitalWrite(piPin1 , LOW);
 }

 PiState = digitalRead(piPin2);
 if (PiState == HIGH) {
  digitalWrite(piPin2, HIGH);
  }
 else {
  digitalWrite(piPin2 , LOW);
 }

I've update my code, tell me if i still missing something