Как инициализировать изменяемый 2D массив char в struct C++?

Question

[Ihor Bratukh]

Здравствуйте, мучаюсь с 2D массивом char'ов в struct C++ и прошу Вашей помощи.

Что нужно сделать? Инициализировать структуру с пустым 2D массивом char. Позже нужно записать строки в этот массив и вывести в консоль отдельные элементы массива с помощью for цикла.

У меня получилось инициализировать массив сразу, а не потом, таким образом:

#include <iostream>
using namespace std;

struct Wifi {
    char *access[2][2] = {
        {
            "TEST",
            "123456"
        },
        {
            "DIR",
            "654321"
        }
    };
} wifi;

int main() {

    for (int i = 0; i < sizeof(wifi.access) / sizeof(wifi.access[0]); i++) {
        cout << wifi.access[i][0] << ": " << wifi.access[i][1] << '\n';
        // TEST: 123456
        // DIR: 654321
    }

    return 0;
}

Привожу список своих проб и ошибок:

struct Wifi {
    char *access[1][2];
} wifi;

char ssid[32] = "TEST";
char password[32] = "123456";

wifi.access[0][0] = ssid;
wifi.access[0][1] = password;

// error: 'wifi' does not name a type

struct Wifi {
    char *access[1][2];
} wifi;

char ssid[32] = "TEST";
char password[32] = "123456";

strcpy(wifi.access[0][0], ssid);
strcpy(wifi.access[0][1], password);

// error: expected constructor, destructor, or type conversion before '(' token
//strcpy(wifi.access[0][0], ssid);
//      ^

struct Wifi {
    char access[1][2][32];
} wifi;

char ssid[32] = "TEST";
char password[32] = "123456";

wifi.access[0] = { ssid, password };

// error: 'wifi' does not name a type
 // wifi.access[0] = { ssid, password };
 // ^~~~

Comments

[Евгений Шатунов]

BRAGA96 , из всего этого так и не понятно, что же ты пытаешься сделать.
В каком контексте существует твоя wifi?
В каком контексте находится строчка wifi.access[0] = { ssid, password };?

Возможно тебе лучше показать больше кода чтобы можно было определить контекст твоих выражений.
В общем смысле видно что ты хочешь выразить Wifi как агрегатную структуру. Почему в полях должен быть именно массив? Почему тип элементов должен быть именно char*, а не const char*?
Ты себе жизнь не усложняешь, случайно? Почему именно char*, а не std::string?

[Ihor Bratukh]

В каком контексте существует твоя wifi?
В каком контексте находится строчка wifi.access[0] = { ssid, password };?

В теле программы, над функцией main

Почему тип элементов должен быть именно char*, а не const char*?

Ну если я правильно понимаю то не const, потому что после инциализации структуры я хочу изменять её элементы

Ты себе жизнь не усложняешь, случайно? Почему именно char*, а не std::string?

Эти данные нужны для функций библиотеки, которая испльзует в качестве аргументов тип char. Создания string, а потом постоянное преобразование в char не является плохой практикой?

[Vitaly]

Ну и где тут C++, кроме iostream?
Ну чисто же Сишный подход и стиль программирования.

[Евгений Шатунов]

BRAGA96

В теле программы, над функцией main

Значит контекст глобальный. Ожидаемо, в глобальном контексте то, что ты хочешь сделать, не сработает.
Для кода, который ты хочешь написать, тебе нужен контекст исполнения - это контекст любой функции.

Ну если я правильно понимаю то не const, потому что после инциализации структуры я хочу изменять её элементы

Ты неправильно понимаешь значение слова const в этом месте.
char - значение можно менять.
const char - значение нельзя менять.
char* - указатель, который можно менять, на значение, которое можно менять.
const char* - указатель, который можно менять, на значение, которое нельзя менять.
char* const - указатель, который нельзя менять, на значение, которое можно менять.
const char* const - указатель, который нельзя менять, на значение, которое нельзя менять.

Константный указатель на константный указатель на константное значение будет выглядеть так: const char* const* const.

Строковые литералы в C++ являются константными, тип указателя для строкового литерала - const char*. Да и уже просто по смутному смыслу твоих действий в коде, ты не хочешь изменять символы в строке, ты хочешь только указатели на сроки менять.

[Ihor Bratukh]

Vitaly, извиняюсь если не так написал, слабо понимаю грань между С и С++

[Ihor Bratukh]

Евгений Шатунов, большое Вам спасибо за ответ. Уже начинаю разбираться, есть результаты, не далеко и решение найду. Спасибо!

[Евгений Шатунов]

BRAGA96, старайся не забывать отмечать решениями те ответы, которые тебе помогли.

Answer 1

[res2001]

Вот это работать не будет:
wifi.access[0] = { ssid, password };
Вам нужно явно делать пересылку памяти с помощью memcpy или strcpy для каждого элемента массива.

Но лучше используйте вместо charов std::string.

Comments

[Ihor Bratukh]

Спасибо за Ваш ответ!

1. Я делал через strcpy, но получаю ошибку

char ssid[32] = "TEST";
char password[32] = "123456";

strcpy(wifi.access[0][0], ssid);
strcpy(wifi.access[0][1], password);

и так, все равно получаю ошибку:

struct Wifi {
    char access[1][2];
} wifi;

char access[2][32] = { "TEST", "123456" };

wifi.access[0] = access;

2. Эти данные нужны для использования в функциях библиотеки, которые принимают тип char. Создания string для дальнейшего преобразования в char показалось мне плохой практикой (зачем string, если мне всегда будут нужны char) или я не прав?

[res2001]

BRAGA96, Определите struct WiFi для первого случая так:

struct Wifi {
    char access[1][2][32];
};

Но т.к. вы плохо понимаете работу с Си строками, лучше перейти на string. string позволяет легко получать доступ к своему содержимому как к Си строке: string.c_str(), но при этом он позволяет выполнять "интуитивные" операции со своим содержимым (то что вы сейчас пытаетесь делать с Си строками).

Answer 2

[Евгений Шатунов]

Вот так сработает.

Пример

#include <iostream>
using namespace std;

const char ssid[] = "TEST";
const char password[] = "123456";

struct Wifi {
    const char *access[2][2];
} wifi{
    {
        { ssid, password },
        { "DIR", "654321" }
    }
};

int main() {

    for (int i = 0; i < sizeof(wifi.access) / sizeof(wifi.access[0]); i++) {
        cout << wifi.access[i][0] << ": " << wifi.access[i][1] << '\n';
        // TEST: 123456
        // DIR: 654321
    }

    return 0;
}

Смотри на агрегатную инициализацию.

Comments

[Ihor Bratukh]

Спасибо! А как инициализировать Wifi struct без значений, а уже после инициализации присвоить новые элементы?

#include <iostream>
using namespace std;

struct Wifi {
    char *access[1][2];
} wifi;

const char ssid[] = "TEST";
const char password[] = "123456";

// wifi.access[0] = ...

int main() {

    for (int i = 0; i < sizeof(wifi.access) / sizeof(wifi.access[0]); i++) {
        cout << wifi.access[i][0] << ": " << wifi.access[i][1] << '\n';
        // TEST: 123456
    }

    return 0;
}

[Евгений Шатунов]

BRAGA96, ты можешь это прямо в main сделать.

int main() {
	wifi.access[0][0] = ssid;
	wifi.access[0][1] = password;
	
	// ...
}

Остальное я допишу в комментариях к вопросу.