为什么在函数块中定义枚举可能是一个坏习惯？

我已经编写了一个状态流处理程序，为减少在状态流处理程序之外使用状态流枚举的风险，我在函数块中定义了状态流枚举。

我的代码如下：

static void RequestHandler(bool isThisRequestANewRequest)
{
    typedef enum
    {
        STATE_NEW_REQUEST,
        STATE_1,
        STATE_2,
        STATE_ERROR,
    } States;

    static States  state = STATE_ERROR;

    if(isThisRequestANewRequest == true)
    {
        state = STATE_NEW_REQUEST;
    }

    switch(state)
    {    
        case STATE_NEW_REQUEST:
            //init request flags
            state = STATE_1;
            //lint -fallthrough
        case STATE_1:
            //do something
            break;
        case STATE_2:
            //do something else
            break;
        case STATE_ERROR:
        default:
            //do something in case of error
            break;
     }
}         

这可以算是一种好习惯吗？有没有风险？有什么缺点吗？（维护，阅读，...）

我的一位同事告诉我这不是，但我正在等待基于事实的答案，而不仅仅是原始意见。

注意：我的问题适用于单线程和多线程任务。