Сформировать убывающий массив из чисел, которые делятся на 3

@vlad697704 · Регистрация: 09.10.2017

Author24 — интернет-сервис помощи студентам

Сформировал убывающий массив из чисел, которые делятся на 3.
Правильно ли я сделал? Можете показать свои варианты? Может лучше что то изменить.

C++

int main() {
    setlocale(LC_ALL, "Russian");
    srand(time(NULL));
    int size_massive = 0;
    size_massive = 10 + rand() % 11;
    int* massive = new int[size_massive];
    cout << size_massive << endl;
    for (int i = 1; i < size_massive; i++) {
        massive[0] = size_massive*3;
        massive[i] = massive[i-1]-3;
    }
    for (int i = 1; i <= size_massive;i++) {
        cout << i << " = " << massive[i-1] << endl;
    }
    system ("pause");
    return 0;
}

@nmcf · 02.07.2018, 18:44

9 вынести из цикла. В конце освободить память.

@k0vpack · 02.07.2018, 18:48

Сообщение от vlad697704

Может лучше что то изменить

Сообщение от vlad697704

C++

1 2	for (int i = 1; i <= size_massive;i++) { cout << i << " = " << massive[i-1] << endl;

диковина, почему бы просто не пойти с 0 и не установить строгий знак?

Новичок · 02.07.2018, 18:49

Названия переменных лучше поменять. Транслит это ужасно. Зачем massive когда можно array? Зачем size_massive когда достаточно просто size. Строку 9 нет смысла оставлять внутри цикла - лучше один раз перед циклом. Строки 4-5 объединить в одну - зачем присваивать 0 если на следующей строке той же переменной присваивается другое значение?

@vlad697704 · 02.07.2018, 18:49 **[ТС]**

Спасибо, а то иду подряд, вроде получается. А правильно ли xD

@~~tankist2018~~ · 02.07.2018, 18:49

Сообщение от nmcf

. В конце освободить память.

Зачем? Что она в конце программы не освободится разве?

Новичок · 02.07.2018, 18:49

А, ну и да, в конце delete[] надо.

@~~tankist2018~~ · 02.07.2018, 18:50

Сообщение от Новичок

А, ну и да, в конце delete[] надо.

@vlad697704 · 02.07.2018, 18:55 **[ТС]**

Сообщение от k0vpack

диковина, почему бы просто не пойти с 0 и не установить строгий знак?

просто cout сначала в первом цикле был, не правильно выводил массив и я перенес во второй цикл, забыв поменять i на 0 и знак.

Добавлено через 1 минуту

Сообщение от tankist2018

Зачем? Что она в конце программы не освободится разве?

Наверное лучше сразу привыкать освобождать память, во избежание ошибок в будущем

@nmcf 7785 / 6553 / 2983 Регистрация: 14.04.2014 Сообщений: 28,621
	02.07.2018, 18:44	2
	9 вынести из цикла. В конце освободить память. 0

Новичок 1642 / 1091 / 487 Регистрация: 17.07.2012 Сообщений: 5,345
	02.07.2018, 18:49	4
	Сообщение было отмечено vlad697704 как решение Решение Названия переменных лучше поменять. Транслит это ужасно. Зачем massive когда можно array? Зачем size_massive когда достаточно просто size. Строку 9 нет смысла оставлять внутри цикла - лучше один раз перед циклом. Строки 4-5 объединить в одну - зачем присваивать 0 если на следующей строке той же переменной присваивается другое значение? 0

@vlad697704 0 / 0 / 0 Регистрация: 09.10.2017 Сообщений: 33
	02.07.2018, 18:49 [ТС]	5
	Спасибо, а то иду подряд, вроде получается. А правильно ли xD 0

@~~tankist2018~~ Заблокирован
	02.07.2018, 18:49	6
	Сообщение от nmcf . В конце освободить память. Зачем? Что она в конце программы не освободится разве? 0

Новичок 1642 / 1091 / 487 Регистрация: 17.07.2012 Сообщений: 5,345
	02.07.2018, 18:49	7
	А, ну и да, в конце delete[] надо. 0

@~~tankist2018~~ Заблокирован
	02.07.2018, 18:50	8
	Сообщение от Новичок А, ну и да, в конце delete[] надо. 0

@vlad697704 0 / 0 / 0 Регистрация: 09.10.2017 Сообщений: 33
	02.07.2018, 18:55 [ТС]	9
	Сообщение от k0vpack диковина, почему бы просто не пойти с 0 и не установить строгий знак? просто cout сначала в первом цикле был, не правильно выводил массив и я перенес во второй цикл, забыв поменять i на 0 и знак. Добавлено через 1 минуту Сообщение от tankist2018 Зачем? Что она в конце программы не освободится разве? Наверное лучше сразу привыкать освобождать память, во избежание ошибок в будущем 0

Опции темы

Сформировать убывающий массив из чисел, которые делятся на 3

Решение