getting help with 89v51rd2 serial port data receiving using C

[dongdurex]

I'm trying to receive data from PC via serial port.
But so far I can't get the data in array of chars (char[]).
Here is the program I've got from my department.

#include <p89v51rd2.h> // Header file register for P89V51RD2FN
unsigned char Get_Serial(void)
{
    bit Timeout=0;
    unsigned char count=252, Data = 0;
    while((Timeout == 0))
    {
        if(RI == 0)
        count++; //waiting
        else
        {
            Data = SBUF;
            RI=0;
            return(Data);
        }
        if(count <= 2)
        {
            Timeout = 1; //Timeout!!
        }
    }
    return(Data);
}
 
void Send_Serial(unsigned char x)
{
    TI = 0;
    SBUF = x;
    while(TI==0);
}
 
void Serial_initial()
{
    TMOD = 0x21;
    SCON = 0x50;
    TH1 =0xFD;
    TL1 =0xFD;
    TF1 = 0;
    TR1 = 1;
    TI = 0;
}
void main(void) // Main loop
{
    Serial_initial();
    P1 = 0x80;
    while(1) // Infinite loop
    {
        Data = Get_Serial(); // ******** I'm thinking that the data gets in from this line
         //My main loop which I need to apply a string like data.
        }
}
 

If I send a data from PC as "10"
The program will only read '1'
which is not correct.

please give me some advice.
ps. sorry for my English, it's not my primary language.

[weaknessforcats]

This code:

Data = SBUF;
 13.             RI=0;
 14.             return(Data);
 

causes Get_Serial to return after receiving one char. That must be your 1.

I do not see any arrays in this code.

main() needs to receive chars until a deliiter is reached that signals the end of the data. Maybe a newline char like \n.

I would expect code in main like:

char buffer[100]; //array of 100 chars
etc...
char delimiter= '\n';  //signals end of the data
char data;  //the data received
int i =0;
while ((data =Get_Serial()!= delimiter )
{
  buffer[i] =data; //store data in correct array position
  ++i; //et ready for next char to be received
}
buffer[i] = '\0';  //makes a C-string in the buffer array.
etc...
 
}

The idea is to receive chars one after the other until you reach the end of the data. So sending 123 should result in 123 in the buffer.

You can append a \0 char to the array so the buffer will work with all C string functions and can be printed using printf with a %s formatter.