Инициализация 'const char **' из несовместимого типа указателя 'char *'

Примечание: Как я уже упоминал в верхних комментариях, я был ограничен тем, что мог делать на мобильном телефоне. Я снова за своим столом.

Проблемы были:

1. X находится в области действия функции, поэтому выходит за ее пределы, когда функция возвращается. Нам нужно использовать malloc [поэтому нам нужно stdlib.h]
2. Поскольку elements — это const char **, нам нужно const char **X

Нам нужно использовать [следует использовать] strdup, чтобы дублировать элементы.

В противном случае оба списка будут Поделиться с одними и теми же данными. Это мая желаемое намерение. Но из описания проблемы не ясно, каким должен быть правильный код.

Большинство списков должны быть полностью независимы друг от друга. В исходном коде это делает нет. Итак, списки делятся данными. Но если это правда, зачем создавать отдельный список? Мы могли бы просто «обратить» его, проиндексировав в обратном направлении без, создав новый список:

for (int i = list.count - 1;  i >= 0;  --i)

Итак, я решил дублировать строки. Исходное поведение (общие строки) при компиляции с -DNODUP.

Вот рефакторинг кода:

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

#if 0
#include "api.h"
#else
typedef struct {
    int count;
    const char **elements;
} string_list;
#endif

void
init()
{
    // You can initiate and calculate things here
}

/**
 * Return the given list in reversed order. Each element in the given list contains a single character.
 *
 * typedef struct {
 *   int count;
 *   const char * * elements;
 * } string_list;
 *
 * list_of_chars: string_list
 *
 * returns: string_list
 */
string_list
reverse(string_list list)
{
    const char **X = malloc(sizeof(*X) * list.count);

    // share the strings
#if NODUP
    for (int i = 0; i < list.count; i++)
        X[list.count - 1 - i] = list.elements[i];

    // make lists independent (more usual/expected)
#else
    for (int i = 0; i < list.count; i++)
        X[list.count - 1 - i] = strdup(list.elements[i]);
#endif

    string_list ret = {
        .count = list.count,
        .elements = X,
    };

    return ret;
}

Обновлено:

In the link above was the exercise page where we can choose GCC for C. I pasted your code exactly to the website bit still see same errors.

Ну наконец-то... ;-)

1. Веб-сайт использует версию стандарта C древний (например, C99). У нет есть strdup.
2. Веб-сайт [принудительно] выполнит #include "api.h" перед включением кода, поэтому возникает конфликт с определением, которое я использовал.

Вот исправленный код (который успешно работает на веб-сайте):

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

#if 1
#include "api.h"
#else
typedef struct {
    int count;
    const char **elements;
} string_list;
#endif

void
init()
{
    // You can initiate and calculate things here
}

/**
 * Return the given list in reversed order. Each element in the given list contains a single character.
 *
 * typedef struct {
 *   int count;
 *   const char * * elements;
 * } string_list;
 *
 * list_of_chars: string_list
 *
 * returns: string_list
 */
string_list
reverse(string_list list)
{
    const char **X = malloc(sizeof(*X) * list.count);

    // share the strings
#if 1
    for (int i = 0; i < list.count; i++)
        X[list.count - 1 - i] = list.elements[i];

    // make lists independent (more usual/expected)
#else
    for (int i = 0; i < list.count; i++)
        X[list.count - 1 - i] = strdup(list.elements[i]);
#endif

    string_list ret = {
        .count = list.count,
        .elements = X,
    };

    return ret;
}

Примечание: Я бы взял хорошую книгу по C и изучил ее, чтобы получить более базовые знания о C. Попытка использовать «конкурентные» веб-сайты — не лучший способ, IMO.

Ряд проблем, которые вы видели (но не понимали), были бы очевидны при лучшем базовом понимании синтаксиса и семантики C.

Вот список: Полное руководство и список книг C