Hi
One thing that I think could be a problem in this code is that Timer1 is being initailized everytime you try to receive a byte. Initialization should be done once throughout the communication process. Try refering this project on EG: https://www.engineersgarage.com/microcontroller/8051projects/interface-serialport-RS232-AT89C51-circuit