Дан одномерный массив. Найти последние чётное, удалить отрицательные, найти среднеарифметическое. с++

[zevs 12]

Lardes пробывал

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

// удаление отрицательных элементов из массива
    for (int i = 0; i < length; i++) {
        if (array[i] <= 0) {

ноль - это отрицательный элемент?

нет ноль это не отрицательный элемент он и не - и не +

короче должно считать количество элементов а не число(((

[rrrFer]

короче должно считать количество элементов а не число(((

Чем "количество элементов" отличается от "числа элементов"?

[igoreshka3333]

Я же тебе решение давал уже? Что в нем не Ясно????????

#include <iostream>
using namespace std;
int lasteval (int[], int);
void negnumb (int[], int&);
void average (int[], int);

int main ()
{
    int n;
    int* X = new int;
    cout << "\nInsert the size of the array: "; cin >> n;
    X = new int[n];
    cout << "\nFill an array: \n";
    for (int i=0; i<n; i++) { cout << "X[" << i+1 << "]= "; cin >> X[i]; }
    int temp = lasteval(X, n);
    cout << "The last element with an even value: " << temp;
    cout << "\nArray without negative values: \n";
    negnumb (X, n);
    cout << "\nArray with the average of the number: \n";
    average (X, n);
    delete[] X;
    cout << endl;
    system ("pause");
}

//-------------------------------------------------------------
int lasteval (int X[], int n)
{
    int last;
    for (int i=0; i<n; i++)
    {
        if (X[i]%2==0 && X[i]>=0) last = X[i];
    }
    return last;
}
//--------------------------------------------------------------
void negnumb (int X[], int& n )
{
    for (int i=0; i<n; i++)
        if (X[i]<0)
        {
            for (int j=i; j<n; j++)
                X[j] = X[j+1];
            i--;
            n--;
        }
    for (int i=0; i<n; i++)
    {
        cout << "X[" << i+1 << "]= " << X[i];
        cout << endl;
    }
}
//--------------------------------------------------------
void average (int X[], int n)
{
    int temp = 0;
    for (int i=0; i<n; i++)
    {
        temp += X[i];
    }
    int aver = temp/n;
    int* X1 = new int[n+1];
    for (int i=0; i<n; i++)
    {
        X1[i] = X[i];
    }
    X1[n] = aver;
    for (int i=0; i<(n+1); i++)
    {
        cout << "X[" << i+1 << "]= " << X1[i];
        cout << endl;
    }
}

Смотри последнюю функцию - сопоставляй!

[rrrFer]

int* X = new int;
    cout << "\nInsert the size of the array: "; cin >> n;
    X = new int[n];

утечка

int last;
    for (int i=0; i<n; i++)
    {
        if (X[i]%2==0 && X[i]>=0) last = X[i];
    }
    return last;

UB если все элементы нечетные или отрицательные. По заданию вроде как проверять тут надо было только четность (вторая проверка лишняя 100%)

int aver = temp/n;

если все элементы были отрицательными - ты получишь деление на ноль.

int* X1 = new int[n+1];

память из под этого массива ты освободить забыл, при выходе из функции указатель теряется - утечка.

Код смотрел по диагонали, наверняка еще баги естью

[igoreshka3333]

Спасибо за замечания)

int* X = new int;

- да, эт не красиво!

void average (int X[], int n)

- это функция, если не обратил внимания, просто выводит на экран значения все элементы массива со сред. арифм. в конце, так что утечка не страшна при выходе, то педантичности ради можно и освободить)))

если все элементы были отрицательными - ты получишь деление на ноль

Будем надеятся, что хоть одно будет положительным!

[8Observer8]

    int* X = new int;
    cout << "\nInsert the size of the array: "; cin >> n;
    X = new int[n];

Нужно было так написать:

    int* X;
    cout << "\nInsert the size of the array: "; cin >> n;
    X = new int[n];

Утечка будет только если, к примеру, average() вызвать много раз. Если приложение, в котором есть утечка проработает какое-то время, то закончится вся оперативная память. Потом виртуальная. А дальше не знаю, что будет. Может копьютер перегрузится

А в таком примере можно забить, так как после завершения приложения кусок оперативки вернётся операционке

На практике нужно использовать std::vector. Привыкать на таких вот примерах, как эта задачка. Об утечках можно не беспокоиться. Сравните функции. Почти всё тоже самое:

void average (int X[], int n)
{
    int temp = 0;
    for ( int i=0; i < n; i++ )
    {
        temp += X[i];
    }
    int aver = temp/n;
    int* X1 = new int[n+1];
    for (int i=0; i<n; i++)
    {
        X1[i] = X[i];
    }
    X1[n] = aver;
    for (int i=0; i<(n+1); i++)
    {
        cout << "X[" << i+1 << "]= " << X1[i];
        cout << endl;
    }
}

void average (const std::vector<int> &X)
{
    int n = X.size();

    int temp = 0;
    for ( int i=0; i < n; i++ )
    {
        temp += X[i];
    }
    int aver = temp/n;
    std::vector<int> X1(n+1);   // int* X1 = new int[n+1];
    for (int i=0; i<n; i++)
    {
        X1[i] = X[i];
    }
    X1[n] = aver;
    for (int i=0; i<(n+1); i++)
    {
        cout << "X[" << i+1 << "]= " << X1[i];
        cout << endl;
    }
}

[igoreshka3333]

Спасибо! Я бы тоже использовал вектор - так безопасней и возни меньше, но студенту нужен был массив...

[zevs 12]

спасибо всем !!!

[rrrFer]

Утечка будет только если, к примеру, average() вызвать много раз. Если приложение, в котором есть утечка проработает какое-то время, то закончится вся оперативная память. Потом виртуальная. А дальше не знаю, что будет. Может копьютер перегрузится

А в таком примере можно забить, так как после завершения приложения кусок оперативки вернётся операционке

Утечка будет в любом случае - нет разницы сколько раз ты вызвал функцию.
Если вызовешь много раз или для больших массивов - утечет больше памяти.

А в таком примере можно забить, так как после завершения приложения кусок оперативки вернётся операционке

Будем надеятся, что хоть одно будет положительным!

Лучше сразу забить на ВУЗ и забрать уже документы из деканата

[8Observer8]

Утечка будет в любом случае - нет разницы сколько раз ты вызвал функцию.
Если вызовешь много раз или для больших массивов - утечет больше памяти.

Я имел ввиду ситуацию, когда память будет реально утекать. Есты вызвать average() один раз, то ничего не утечёт