0

機能が期待どおりに動作しません。目標は、埋め込みデバイス (LCD 付き) 上の人間が読める物理単位の文字列をバッファーに出力することです。たとえば、 where as の1234uVように表示されます。オプションで適切な調整を行う正しい (高速で安全な) 実装はどのようになっていますか (で)?+1.234mV-100023uV-1.000,230 VuV

uint8 voltage_string(char* buf, int32 uVolt)
{
    static const int32 VOLT = 1000000;
    static const int32 MILLIVOLT = 1000;

    const int32  V =  uVolt / VOLT;
    const int32 mV = (uVolt - V*VOLT) / MILLIVOLT;
    const int32 uV = (uVolt - V*VOLT - mV*MILLIVOLT);

    uint8 n = 0;

    if(abs(V) > 0) {
        n  = sprintf(buf,     "%+d",    V);
        n += sprintf(buf + n, ",%3d",   abs(mV));
        n += sprintf(buf + n, ".%3d V", abs(uV));

        return n;
    }

    if(abs(mV) > 0) {
        n  = sprintf(buf,     "%+d",     mV);
        n += sprintf(buf + n, ",%3d mV", abs(mV));

        return n;
    }

    if(abs(uV) > 0) {
        n  = sprintf(buf,     "%+3d uV", uV);

        return n;
    }

    return n;
}
4

2 に答える 2

0

コードは必要以上に複雑です。次のように単純化できます。

uint8_t voltage_string(char* buf, int32_t uVolt)
{
    const int32_t VOLT = 1000000;
    const int32_t MILLIVOLT = 1000;

    uint8_t n = 0;

    if (abs(uVolt) >= VOLT)
    {
        n = sprintf(buf, "%+.6f V", uVolt / (double)VOLT);
    }
    else if (abs(uVolt) >= MILLIVOLT)
    {
        n = sprintf(buf, "%+.3f mV", uVolt / (double)MILLIVOLT);
    }
    else
    {
        n = sprintf(buf, "%+d µV", uVolt);
    }

    return n;
}
于 2012-09-06T10:44:35.363 に答える
0 
uint8_t voltage_string(char* buf, int32_t uVolt) 
{     
    const int32_t VOLT = 1000000;     
    const int32_t MILLIVOLT = 1000;      
    int v=0,mv=0,uv=0;
    uint8_t n = 0;      
    v = uVolt/VOLT;
    mv = (uVolt % VOLT) / MILLIVOLT;
    uv = (uVolt % VOLT) % MILLIVOLT;
    n = sprintf(buf, "%d.%03d%03d  V", v,mv,uv); 
    return n; 
}
于 2012-09-06T12:38:53.717 に答える