Заменить элементы массива кратные трём на сумму нечётных по значению элементов

@romcheg · Регистрация: 01.11.2020

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

Изучаем функции, была задача создать динамический массив, и заменить элементы, кратные трём, на сумму нечётных по значению элементов. Всё заставляют делать через функции, я около часа возился над кодом, но в конце начали уже ошибки вылетать, а вывод был неправильным(сумма нечётных увеличивалась постепенно, и были вписаны разные значения). Может кто-то подсказать, в чём беда?

C++

#include<iostream>
#include<ctime>
#include<iomanip>
using namespace std;
void Make_array(int*, int);
int Suma_nechotnih(int*, int);
void Zamina_kratnih_3(int*, int);
int main() {
    srand(time(0));
    int i, n;
    cout << "Enter n:";
    cin >> n;
    int* array = new int;
    Make_array(array, n);
    cout << "\n\n";
    Zamina_kratnih_3(array, n);
    delete[] &array;
    cout << endl;
    system("pause");
    return 0;
}
void Make_array(int* array, int n) {
    int i;
    for (i = 0; i < n; i++) {
        array[i] = -50 + rand() % 201;
        cout << setw(4) << array[i] << "|";
    }
}
 
int Suma_nechotnih(int* array, int n) {
    int i, s=0;
    for (i = 0; i < n; i++) {
        if (array[i] % 2 != 0) { s += array[i]; }
    }
    return s;
}
 
void Zamina_kratnih_3(int* array, int n) {
    int i;
    for (i = 0; i < n; i++) {
        if (array[i] % 3 == 0) {
            array[i] = Suma_nechotnih(array, n);
        }
    }
    for (i = 0; i < n; i++) {
        cout << setw(4) << array[i] << "|";
    }
}

@Yetty · 05.11.2020, 17:55

массив квадратный ? напишите оригинал условия задачи

@zss · 05.11.2020, 17:58

Строки с ошибками я отметил //!!!!!!!!!!!!!!

C++

#include<iostream>
#include<ctime>
#include<iomanip>
using namespace std;
void Make_array(int*, int);
int Suma_nechotnih(int*, int);
void Zamina_kratnih_3(int* arr, int n,int value); //!!!!!!!!!!!!!!
int main() {
    srand(time(0));
    int i, n;
    cout << "Enter n:";
    cin >> n;
    int* array = new int[n]; //!!!!!!!!!!!!!!
    Make_array(array, n);
    cout << "\n\n";
    Zamina_kratnih_3(array, n, Suma_nechotnih(array, n));//!!!!!!!!!!!!!!
    delete[] array;  //!!!!!!!!!!!!!!
    cout << endl;
    system("pause");
    return 0;
}
void Make_array(int* array, int n) {
    int i;
    for (i = 0; i < n; i++) {
        array[i] = -50 + rand() % 201;
        cout << setw(4) << array[i] << "|";
    }
}
 
int Suma_nechotnih(int* array, int n) {
    int i, s = 0;
    for (i = 0; i < n; i++) {
        if (array[i] % 2 != 0) { s += array[i]; }
    }
    return s;
}
 
void Zamina_kratnih_3(int* array, int n,int value) {  //!!!!!!!!!!!!!!
    int i;
    for (i = 0; i < n; i++) {
        if (array[i] % 3 == 0) {
            array[i] = value;  //!!!!!!!!!!!!!!
        }
    }
    for (i = 0; i < n; i++) {
        cout << setw(4) << array[i] << "|";
    }
}

@romcheg · 05.11.2020, 18:12 **[ТС]**

Создать динамический массив(одномерный), длина вводится с клавы. Заменить элементы, кратные трём, на сумму нечётных по значению элементов. Всё заставляют делать через функции(заполнение, обработка, вывод и т.п.), в main только вызывать эти функции. При чём функции объявлять(или как там) нужно выше main, а настраивать ниже. Ну, как в коде сделано в общем. Чёткого условия нет, т.к. дали задачу и добавили все эти пожелания

Добавлено через 5 минут
127| 107| 76| 22| 42| 49| -25| -33| 131| 127| 37| 7| -42| 20| 93| - стартовый массив
127| 107| 76| 22| 620| 49| -25| 620| 131| 127| 37| 7| 653| 20|1306| - после обработки, + ошибки(похоже утечка памяти)

Добавлено через 6 минут
Спасибо, помогло!!! пойду разбираться с этим всем

@Yetty · 05.11.2020, 18:25

C++

#include <iostream>
#include <cstdlib>
#include <ctime>
 
using namespace std;
 
int* Create(int n) // создание 
{
    int* a = new int [n];    
    return a;
}
 
void Random(int* a, int n) // заполнение 
{    
    for (int i = 0; i < n; i++)        
        a[i]=rand()%9+1;
}
 
void Print(int *a, int n) // вывод на экран
{
    for (int i = 0; i < n; i++)    
        cout << a[i] << " ";
    cout << "\n";    
}
 
int SumOdd(int* a, int n) 
{
    int sum_odd=0;
    for (int i = 0; i < n; i++) 
        if (a[i]%2) sum_odd+=a[i];    
return sum_odd;
}
 
void Replace(int* a, int n) 
{
    int s=SumOdd(a,n);
    for (int i = 0; i < n; i++) 
        if (a[i]%3==0) a[i]=s;
}
 
int main() 
{
    srand((int)time(0));
    int n, *a;
    cout << "n="; cin >> n;
    
    a = Create(n);
    
    Random(a,n);
    cout << "Array:\n";
    Print(a,n);
    cout << "sum odd = " << SumOdd(a,n) << "\n";
    Replace(a,n);
    cout << "Modified array:\n";
    Print(a,n);       
    delete[]a;
system("pause");
return 0;
}

функции вниз сами перенесите если это требуется

@Yetty 7437 / 5029 / 2892 Регистрация: 18.12.2017 Сообщений: 15,692
	05.11.2020, 17:55	2
	массив квадратный ? напишите оригинал условия задачи 0

@romcheg 0 / 0 / 0 Регистрация: 01.11.2020 Сообщений: 40
	05.11.2020, 18:12 [ТС]	4
	Создать динамический массив(одномерный), длина вводится с клавы. Заменить элементы, кратные трём, на сумму нечётных по значению элементов. Всё заставляют делать через функции(заполнение, обработка, вывод и т.п.), в main только вызывать эти функции. При чём функции объявлять(или как там) нужно выше main, а настраивать ниже. Ну, как в коде сделано в общем. Чёткого условия нет, т.к. дали задачу и добавили все эти пожелания Добавлено через 5 минут 127\| 107\| 76\| 22\| 42\| 49\| -25\| -33\| 131\| 127\| 37\| 7\| -42\| 20\| 93\| - стартовый массив 127\| 107\| 76\| 22\| 620\| 49\| -25\| 620\| 131\| 127\| 37\| 7\| 653\| 20\|1306\| - после обработки, + ошибки(похоже утечка памяти) Добавлено через 6 минут Спасибо, помогло!!! пойду разбираться с этим всем 0

	05.11.2020, 18:25

Заменить элементы массива кратные трём на сумму нечётных по значению элементов

Решение