I think you meant

    int value = c - '0';

Also, remember that sizeof("string") includes the null terminator while strlen() does not.