Надо понимать что {}
- является константой. Теоретически константа не должна менятся, но на практике, проигнорировав сообщение компилятора, константу получится поменять, это может при особых случаях прокатить, но в будущем станет причиной багов или падений программы. Если программа не слетит - то баг будет проявляться в том, что присвоив значение в две разные переменные - значения в двух переменных окажется одинаковым (ведь это константа).
Вместо {}
нужно использовать new int[размер_массива_цифрой]
в вашем случае my_var = new int[2]
P.S. после взятия памяти, память желательно освобождать используя delete
. Иначе можно рано ли позно столкнуться с тем что память закончилась. Для вашего случая можно сделать как ниже (в вашем случае такая утечка памяти не повлияет на работу, но в будущем пригодится):
char * a = string_to_time(timing);
...
delete a;
Как вариант, что бы не использовать delete
, если вы хотите использовать "временную" переменную, но не хотите её делать глобальной, можете обьявить её с ключевым словом static
- тогда компилятор зарезервирует область памяти под эту переменную из глобально пространства, а не из стека. Если вас устраивает то что переменная будет после вызова ф-ции может быть использована "один раз" (вызвали функцию - вывели или скопировали и не храним, как у вас сейчас) то тогда можно вашу программу реализовать так
(так можно для случая single-thread как у вас сейчас, multi-thread может дать тихий баг как описано вначале - там сложнее: надо бы или thread-var или буфер передавать):
char* f() {
static char x_out[2];
return x_out;
}
Что будет эквивалентно, но x_out не будет мешать в глобальной области.
char x_out[2];
char* f() {
return x_out;
}
Но в вашем примере есть ещё ошибки.
Одну ошибку можно решить таким образом (если без delete)
char & a = *f();
cout << ((char*)&a)[0] << ' ' << ((char*)&a)[1]; //(вторая ошибка)
Но что бы достать значения - прийдётся мучатся опять с указателями, поэтому вторую ошибку... Ошибка заключается в том, что в char хранится одно число, что бы испоьзовать больше чем одно - обязательно тип ссылка или указатель. Можно ошибку исправить так
out * a = f();
cout << a[0] << ' ' << a[1];
Или использовать ASCII-z.
Я ещё подумал, есть третий вариант реализации, без использования памяти вообще. К сожелению на с++ этот вариант чуть сложнее (хотя стандарт позволяет обьявить тип "массив размером 2 елемента", использовать его не получится, и приходится использовать структуру). Хотя на практике стараются делать так что бы функция возвращала либо одну сущность, либо массив.
typedef struct { char item[2]; } ARRAYC2;
ARRAYC2 string_to_time(string time){
ARRAYC2 out;
out.item[0] = atoi(time.substr(time.length() - 2, 2).c_str());
out.item[1] = atoi(time.substr(0, time.length() - 2).c_str());
return out;
}
int main(){
string timing;
cin >> timing;
ARRAYC2 a = string_to_time(timing);
cout << a.item[0] << ' ' << a.item[1];
}
А если принять что item[0] и item[1] числа от 0 до 255 - то можно применить "упаковку" числа в байты, упростив выражение выше:
int string_to_time(string time){
int out;
out = atoi(time.substr(time.length() - 2, 2).c_str());
out += atoi(time.substr(0, time.length() - 2).c_str())*256;
return out;
}
int main(){
string timing;
cin >> timing;
int a = string_to_time(timing);
cout << a % 256 << ' ' << a / 256; // упрощенный вариант
cout << a & 255 << ' ' << ( a >> 8 ); // вариант для гурманов
}
Ну и на последок, правильный ответ. На практике, если вам нужно вернуть две величины, обычно "общепринятым" считается передавать назад всё в параметрах (или на крайний случай первую величину через return, остальные через параметры которые "в хвосте" функции):
void string_to_time(string time, char * ret1 , char * ret2){
*ret1 = atoi(time.substr(time.length() - 2, 2).c_str());
*ret2 = atoi(time.substr(0, time.length() - 2).c_str())*256;
}
void main(){
string timing;
cin >> timing;
char a[2];
string_to_time(timing, &a[0], &a[1]);
cout << a[0] << ' ' << a[1];
}
Ещё добавлю...
Есть умные указатели для С++ (Мне подсказывают), тогда не нужно delete. Они являются частью новой стандартной библиотеки (если не ошибаюсь)
Есть GC для c++, тогда тоже не нужно delete (не стандартная библиотека)
Ещё есть пара malloc
free
и подобные, есть пулы и много ещё чего.
можно вернуть int32 и оспользовать его отдельные байты
c_str и string - дополнительные удобства, для економии памяти так же можно использовать ascii-z функции strlen.