Как вернуть массив из структуры?

Question

[RandomCraft]

Есть код, в котором определена структура хранящая массив из 3 ячеек. Определена функция, которая умножает значение ячеек массива переменной моей структуры. Хочу проверить, работает ли всё так, как я хочу, но не могу. Если просто вставлять в cout переменную iv3d или iv3d.array, то ничего не получается (с первым мне в целом понятно почему, а вот от второго я до сих пор в недоумении). Определил в структуре метод get, но он постоянно ругается, что я хочу вернуть int*, а в определении метода int. Если заменить и в методе на int*, то во время работы main возникает ошибка, т.к. я пытаюсь запихнуть в массив типа int тип int*... Я перестал в этот момент понимать что-либо... Что вообще происходит, почему в структуре массив int*, а в основном теле просто Int... Я не понимаю.
Подскажите пожалуйста где у меня ошибка, что нужно сделать? Я сам ответ не могу найти.

#include <iostream>
using namespace std;

struct ivector3d {

	int array[3];
  int get() {
		return array;
	}
};

void scale(ivector3d* v, int k) {
	for (int i = 0; i != 3; ++i)
		v->array[i] *= k;
}


int main() {

	ivector3d iv3d = {1,1,1};
	scale(&iv3d, 2);

	int b[3]={iv3d.get()};

	cout << b << endl;
}

Comments

[res2001]

Метод get у тебя должен возвращать int, а в return у тебя указан array. array - это массив, следовательно ты пытаешься вернуть массив там где нужно вернуть int. В С/С++ массивы на прямую не возвращаются и не передаются (как параметры в функции), вместо них возвращается (и передается) указатель. Отсюда и соответствующая ошибка.
Если ты действительно хочешь возвращать int, то надо писать что-то вроде: return array[0];
Если тебе надо возвращать массив, то тип возвращаемого значения должен быть int*, о чем и написано в ошибке.
В остальном смотри ответ Александр Ананьев

Answer 1

[Александр Ананьев]

В данном случае в вашей программе нет необходимости использовать массив, лучше завести 3 переменные в структуре ivector3d

struct ivector3d {
  int x = 0;
  int y = 0; 
  int z = 0;
};

Затем обращаться к ним из функции scale.
Если вас интересует возвращение массива из функции, то вам нужно вернуть указатель на первый элемент массива

int* get() {
    return array;
  }

Но это плохой способ, т.к. при таком подходе теряется информация о размере массива. Эту проблему можно решить возвращая размер массива в параметрах функции.

int* get(int& size) {
    size = 3;
    return array;
  }

Но, согласитесь, это не очень удобно. В таком случае предлагается изменить код на использование контейнера std::array.

struct ivector3d {

  std::array<int, 3> array;
  std::array<int, 3> get() {
    return array;
  }
};

По итогу, программа будет выглядеть следующим образом

#include <iostream>
#include <array>

using namespace std;

struct ivector3d {

  std::array<int, 3> array;
  std::array<int, 3> get() {
    return array;
  }
};

void scale(ivector3d* v, int k) {
  for (int i = 0; i != 3; ++i)
    v->array[i] *= k;
}


int main() {

  ivector3d iv3d = {1,1,1};
  scale(&iv3d, 2);

  auto b = iv3d.get();
  for(auto i : b)
  {
      cout << i << " ";
  }
}

Comments

[RandomCraft]

Cпасибо, благодаря комментарию стал лучше понимать указатели. А ещё понял почему в верном варианте кода(который ниже), выводилось что-то непонятное - выводился адрес массива, а не сам массив, я этого никак понять не мог, думал что что-то я не так написал (надо же, вроде C++ начал раньше изучать, а ошибка по итогу вылезла из-за знания python(там массив можно полностью выводить по его имени) ), написал то я всё верно до вывода, а вот с выводом запорол)). По этому мне следовало написать "cout << b[0] << b[1] << b[2] << endl;" вместо "cout << b << endl;".

Зато с вашим комментарием углубил свои знания языка. Спасибо.

#include <iostream>
using namespace std;

struct ivector3d {

    int arrray[3];
    int* get() {
        return arrray;
    }
};

void scale(ivector3d* v, int k) {
    for (int i = 0; i != 3; ++i)
        v->arrray[i] *= k;
}


int main() {

    ivector3d iv3d = { 1,1,1 };
    scale(&iv3d, 2);

    int* b = iv3d.get();
    cout << b << endl;
}