Как перевернуть строку в массиве в C

Это для цикла

for (int i = left; i < right; i++)
{
    temp = result[left];
    result[left] = result[right];
    //..

вызывает неопределенное поведение, поскольку динамически выделяемый массив не был инициализирован, и в результате выражения result[left] и result[right] имеют неопределенные значения. В массиве нечего менять местами. Массив должен быть заполнен символами.

Также массив ни в коем случае не содержит строку, потому что он не дополняется завершающим нулевым символом '\0';.

Если я правильно понял задание, вам нужно что-то вроде следующего. Обратите внимание, что функция может быть реализована по-разному.

Например

char * problem58( void )
{
    char *s = malloc( 2 * ( 'Z' - 'A' + 1 ) + 1 );

    if ( s != NULL )
    {
        char *p = s;
        char c = 'A';

        do
        {
            *p++ = c;
        } while ( c++ != 'Z' );

        while ( c != 'A' )
        {
            *p++ = --c;
        }

        *p = '\0';
    }

    return s;
}
   

или

char * problem58( void )
{
    size_t n = 2 * ( 'Z' - 'A' + 1 );

    char *s = malloc( n + 1 );

    if (s != NULL)
    {
        char *first = s, *last = s + n;

        *last = '\0';

        char c = 'A';

        do
        {
            *first++ = *--last = c;
        } while (c++ != 'Z');
    }

    return s;
}

     

И функция вызывается в основном как

char *s = problem58();

if ( s != NULL ) puts( s );

free( s );

Ожидаемый результат вызова функции

ABCDEFGHIJKLMNOPQRSTUVWXYZZYXWVUTSRQPONMLKJIHGFEDCBA

Вот мой ответ

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

char* problem58()
{
    int n = 0;
    int left = 0, right = 52 - 1;

    // A--->Z and Z--->A total number is 52, because string end with '\0'(0), so add 1 is 53.
    char* result = (char*)malloc(53 * sizeof(char));

    // Check if it is possible to dynamically allocate memory for 53 variables of type "char"
    // You can also write like this: if (result == NULL)
    if (NULL == result){
        printf("No space\n");
        exit(1);
    }

    // Initial result with 0,because string end with ('\0')0.
    // So result[0] will be 0, result[1] will be 0 ...... result[52] will be 0
    for(int m = 0; m < 53;m++){
        result[m] = 0;
    }

    for (char i = 'A'; i <= 'Z'; i++)
    {
        result[left] = i;
        //printf("%c", i);
        left++;
    }

    // Now left is 26, reset it to 0, because result[0] is 'A'
    left = 0;

    // n is execute times, need to let result[51] = result[0] = 'A', and result[50] = result[1] = 'B' ......
    //so right-- and left++
    for (n = 0; n < 26; n++)
    {
        result[right] = result[left];
        right--;
        left++;
    }
    return result;
}

int main()
{

    char* str = problem58();
    printf("str's length is %ld\n",strlen(str));
    printf("%s\n",str);
    free(str);
    return 0;
}

Запустите, он выведет

str's length is 52
ABCDEFGHIJKLMNOPQRSTUVWXYZZYXWVUTSRQPONMLKJIHGFEDCBA