Better now?
Yes, but there is still room for improvement. You look to see if there is at least one byte to read. If so, you wait for the rest of the data to arrive. Why not skip the processing until there are 4 bytes to read?
if(RFID.available() >= 4)
{
// No waiting required. Just read the 4 bytes