Thread: Tricky request and response structure.

More appropriately code must be like this

Code:

case REQUEST4:
        request_response(&request[4], &response);
        if (prev_request != REQUEST_NONE)
            handle_prev_response(prev_request, &response);
        prev_request = REQUEST3;
        break;
case REQUEST4: ////send same REQUEST4 for 3 different modes and repeat again
        request_response(&request[4], &response);  //for mode1,2 and 3
        if (prev_request != REQUEST_NONE)
            handle_prev_response(prev_request, &response);
        prev_request = REQUEST4;
        break;


}

void handle_prev_response(RequestType rt, data_st *response) {
    switch (rt) {
    case REQUEST3:
        // process response for request-3 type
        break;
    case REQUEST4:   
        // process response for request-4 type mode1
    break;

    case REQUEST4:
        // process response for request-4 type mode2
    break;
    case REQUEST4:
        // process response for request-4 type mode3


        break;
 case REQUEST4:   
        // process response for request-4 type mode1

    // etc.
    }
}

Please guide.
--david.

Maybe something like this:

Code:

typdef struct{
    uint16_t data0;
    uint16_t data1;
    uint16_t data2;
    uint16_t data3;
} data_st;
 
enum RequestType { REQUEST_NONE, REQUEST0, REQUEST1, REQUEST2, REQUEST3,
                   REQUEST4 };
 
void handle_request(RequestType request_type) {
 
    static RequestType prev_request = REQUEST_NONE;
    static int prev_mode = 0;
 
    static data_st request[5] = {
        { 0x0102, 0x0304, 0x0506, 0x0708 },   // whatever the data is
        // ...
    };
 
    data_st response;
 
 
    switch (request_type) {
 
    case REQUEST0:
        request_response(&request[0], &response);
        if (prev_request != REQUEST_NONE)
            handle_response(prev_request, prev_mode, &response);
        prev_request = REQUEST0;
        prev_mode = 0;
        break;
 
    case REQUEST1:
        request_response(&request[1], &response);
        if (prev_request != REQUEST_NONE)
            handle_response(prev_request, prev_mode, &response);
        prev_request = REQUEST1;
        prev_mode = 0;
        break;
 
    // ... case REQUEST2 and case REQUEST3 ...
    
    case REQUEST4:
        request_response(&request[4], &response);
        if (prev_request != REQUEST_NONE)
            handle_response(prev_request, prev_mode, &response);
        if (prev_request == REQUEST4)
            prev_mode = (prev_mode + 1) % 3;
        else
            prev_mode = 0;
        prev_request = REQUEST4;
        break;
    
    }
}
 
void handle_response(RequestType type, int mode, data_st *response) {
    switch (type) {
    case REQUEST0:
        // process response for request-0 type
        break;
    case REQUEST1:
        // process response for request-1 type
        break;
    case REQUEST2:
        // process response for request-2 type
        break;
    case REQUEST3:
        // process response for request-3 type
        break;
    case REQUEST4:
        switch (mode) {
        case 0:
            // process response for request-4, mode-0 type
            break;
        case 1:
            // process response for request-4, mode-1 type
            break;
        case 2:
            // process response for request-4, mode-2 type
            break;
        }
    }
}

Hi Algorism....looks great. I need to keep repeating REQUEST4 for all three mode again and again and process the corresponding data. . Is this will do the same? I will try and let you know. Really great help.

--David.

Originally Posted by davidmeetsall

I need to keep repeating REQUEST4 for all three mode again and again and process the corresponding data. . Is this will do the same? I will try and let you know.

I can't be sure since I'm not testing it. Give it a try.

You also need a case in handle_request to allow you to send a dummy request to handle the final response. You could use REQUEST_NONE like handle_request(REQUEST_NONE). THe case would be something like:

Code:

    case REQUEST_NONE:
        if (prev_request != REQUEST_NONE)
            handle_response(prev_request, prev_mode, &response);
        prev_request = REQUEST_NONE;
        prev_mode = 0;
        break;

Ok. I will try and let you know. Many thanks.
---David.