C Низове с различна дължина, малки към главни букви

Вие обърквате „настоящото съдържание на изходния параметър“ с „налично пространство“. Първото е без значение, а единствената информация, която имате за второто, е „има поне толкова свободно място“.

Сега ще зависи от това как на първо място е зададено пространството за output. Ако сте направили нещо подобно

char output[100];
strcpy(output, "hello");

Ще получите място за 100 знака, но реално използвани само 6 (5+1). Следователно можете да вземете низа "THIS IS a STRING" и да го обработите с вашата функция без проблем.

Но това не е безопасно, защото не знаете колко място има. Следният подход би бил по-добър:

char *output;
output = malloc(100);

Сега променете своя прототип на функция на

void makeUpper( const unsigned char* input, unsigned char** output ) 

и във вашата функция, вие го правите

inputLength = strlen(input);
*output = realloc(*output, inputLength + 1);

Това ще гарантира, че е отделено достатъчно място за изхода. Или можете да върнете стойността във входния вектор - вече знаете, че там има достатъчно място...

РЕДАКТИРАНЕ В примера, който давате, има достатъчно място в output; въпросът просто се превръща в "безопасно копиране" на (с главни букви) вход към изхода. В който случай вашата функция може да изглежда така:

void makeUpper( const unsigned char* input, unsigned char* output ) 
{

    int inputLength = strlen(input);
    int outputLength = strlen(output);
    int ii;
    for (ii = 0; ii < inputLength; ii++)
    {
        output[ii] = toupper(input[ii]);
    }
    if(outputLength < inputLength) output[ii] = '\0';
}

Последният ред е там, за да се уверите, че ако сте увеличили дължината на output (отново, ако приемем, че това е памет, до която имате валиден достъп), тогава все пак трябва да се уверите, че има завършващ знак nul в края на низа. Във вашия пример искате „останалата част от изходния низ“ все още да е там, когато input е по-къс от output, така че имате нужда от условието if.

Като цяло - ако не знаете със сигурност, че output е достатъчно голям, няма начин да го направите по-голям, без да имате достъп до адреса на указателя - понякога наричан "дръжка".

Ако може да се приеме, че изходният буфер е създаден във функцията:

// C++ version
void makeUpper(const unsigned char* input, unsigned char*& output)
{
    // assume output = null
    int inputLength = strlen((const char*)input);
    output = new unsigned char[inputLength + 1];
    memset(output, 0, inputLength + 1); // initialize the array to 0's

    for (int i = 0; i < inputLength; i++)
    {
        output[i] = ::toupper(input[i]); // why reinvent the wheel
    }
}

// C version
void makeUpper(const unsigned char* input, unsigned char** output) 
{
    // assume output = null
    int inputLength = strlen((const char*)input);
    *output = (unsigned char*)malloc((inputLength + 1) * sizeof(unsigned char));
    memset(*output, 0, (inputLength + 1) * sizeof(unsigned char)); // initialize the array to 0's

    for (int i = 0; i < inputLength; i++)
    {
        (*output)[i] = ::toupper(input[i]);
    }
}

output ще трябва да бъде изтрит/освободен от всеки, който поеме контрола над него.

Ако не искате правилно да оразмерите output във функцията:

void makeUpper(const unsigned char* input, unsigned char* output, unsigned int output_size) 
{
    // assume output != null, the current contents of output are irrelevant - you need it's size
    int inputLength = strlen((const char*)input);
    int maxLength = (inputLength < output_size - 1 ? inputLength : output_size - 1);
    memset(output, 0, output_size); // clear output

    for (int i = 0; i < maxLength; i++)
    {
        output[i] = ::toupper(input[i]); // why reinvent the wheel
    }
}

ЗАБЕЛЕЖКА: Тагът C++ беше премахнат, докато пишех отговора си. Това наистина засяга само първото решение (тъй като бихте използвали malloc вместо new). Бих се отклонил от realloc или calloc, освен ако вашите изисквания не ги налагат абсолютно.