Вернуть все четные числа массива в виде нового массива

@luigration · Регистрация: 04.01.2013

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

Функция должна вернуть все четные числа массива в виде нового массива.

C++ (Qt)

int *f2 (int *mas,int N)
{
    int size=0;
 
    for(int i=0; i<N; i++)
    {
        if(mas[i]%2==0)
        {
            size++;
        }
    }
 
    int *even = new int [size];
 
    
    int j=0;
    for(int i(0); i<N; i++)
    {
        if(mas[i]%2==0)
        {
            even[j] = mas[i];
            j++;
        }
    }
    
    return even;
    delete []even;
    
}

Подскажите, в чем ошибка.

@zss · 28.04.2014, 20:40

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

delete []even;

Писать не надо, хотя все равно эта строчка не выполняется,
т.к. стоит после return

@alekopoko · 28.04.2014, 20:44

delete не выполняется,выход происходит раньше,return even;

@luigration · 28.04.2014, 21:00 **[ТС]**

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

delete не выполняется,выход происходит раньше,return even;

Это понятно. Эта строчка случайно сюда попала. Что с delete, что без - все равно не работает.

@alekopoko · 28.04.2014, 21:03

luigration, зачем тебе указатель на функцию?
может так?

Код

int f2 (int *mas,int N)

@luigration · 28.04.2014, 21:12 **[ТС]**

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

luigration, зачем тебе указатель на функцию?
может так?

Код

int f2 (int *mas,int N)

Так то же самое будет..
Мне казалось, что если функция должна возвращать указатель или массив, то нужно писать со свездой, разве нет? Если убрать звезду, то функция не сможет вернуть массив.
Указатели на функцию объявляются по-другому..

C++ (Qt)

1	int(*p)(int)=&f2;

@cooller · 28.04.2014, 21:15

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

зачем тебе указатель на функцию?

а где ты увидел там указатель на функцию?

Добавлено через 2 минуты
luigration, мне интересно, а как ты выводишь потом этот массив?

@luigration · 28.04.2014, 21:19 **[ТС]**

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

а где ты увидел там указатель на функцию?

Добавлено через 2 минуты
luigration, мне интересно, а как ты выводишь потом этот массив?

Я в главной функции пишу

C++ (Qt)

1	cout << f2(mas,n);

И мне кажется, делаю совершенно неправильно.

@alekopoko · 28.04.2014, 21:19

cooller,

а где ты увидел там указатель на функцию?

забыл прототипизацию указателя на функцию) int (*f2) (int *mas,int N);

luigration,

C++

#include <iostream>
 int j=0;
using namespace std;
 int *f2 (int *mas,int N)
{
    int size=0;
 
    for(int i=0; i<N; i++)
    {
        if(mas[i]%2==0)
        {
            size++;
        }
    }
 
    int *even = new int [size];
    for(int i(0); i<N; i++)
    {
        if(mas[i]%2==0)
        {
            even[j] = mas[i];
            j++;
        }
    }
    
    return even;
    
}
 void main()
 {
    int a[]={1,2,3,4,5,6,7,8,9,10};
    int *b;
    b=f2(a,10);
    for(int i=0;i<j;i++)
    printf("%d\n",b[i]);
 }

@luigration · 28.04.2014, 21:24 **[ТС]**

То есть в главной функции мне нужно было создать еще один массив, так?
И почему j - глобальная переменная?

@cooller · 28.04.2014, 21:25

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

И мне кажется, делаю совершенно неправильно.

массив нужно выводить в цикле, а для этого нужно знать его размер
так что ты должен вернуть из цикла и размер тоже, либо использовать std::vector

@alekopoko · 28.04.2014, 21:25

luigration,

Так то же самое будет..
Мне казалось, что если функция должна возвращать указатель или массив, то нужно писать со свездой, разве нет? Если убрать звезду, то функция не сможет вернуть массив.
Указатели на функцию объявляются по-другому..
Код C++ (QT)
1
int(*p)(int)=&f2;

все правильно говоришь да) это я накосячил)

@luigration · 28.04.2014, 21:26 **[ТС]**

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

массив нужно выводить в цикле, а для этого нужно знать его размер
так что ты должен вернуть из цикла и размер тоже, либо использовать std::vector

Вектором пользоваться не умею. Пока что.

@alekopoko · 28.04.2014, 21:27

luigration,

И почему j - глобальная переменная?

это переменная определяет сколько элементов выводить в Main
ну можно по-разному это все реализовать

@luigration · 28.04.2014, 21:29 **[ТС]**

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

luigration,
это переменная определяет сколько элементов выводить в Main
ну можно по-разному это все реализовать

Не работает все равно..

@alekopoko · 28.04.2014, 21:33

luigration,

Не работает все равно..

почему это?печатается же 2 4 6 8 10
и это массив b
которому ты выделил нужный размер в функции f2
какая ошибка?

@luigration · 28.04.2014, 21:34 **[ТС]**

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

luigration,
почему это?печатается же 2 4 6 8 10
и это массив b
которому ты выделил нужный размер в функции f2
какая ошибка?

Все, понятно теперь. Все работает, пардон. Спасибо огромное!

@cooller · 28.04.2014, 21:36

alekopoko, лучше не глобальной переменную делать, а передавать по ссылке

@alekopoko · 28.04.2014, 21:43

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

@luigration · 28.04.2014, 22:22 **[ТС]**

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

alekopoko, лучше не глобальной переменную делать, а передавать по ссылке

Что именно передавать по ссылке? Вернее где ее передавать?

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

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

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

А как сделать без глобализации j?

@cooller 571 / 539 / 280 Регистрация: 25.12.2013 Сообщений: 1,456
	28.04.2014, 21:25	11
	Сообщение от luigration И мне кажется, делаю совершенно неправильно. массив нужно выводить в цикле, а для этого нужно знать его размер так что ты должен вернуть из цикла и размер тоже, либо использовать std::vector 0

@alekopoko 1 / 1 / 1 Регистрация: 28.04.2014 Сообщений: 33
	28.04.2014, 21:33	16
	luigration, Не работает все равно.. почему это?печатается же 2 4 6 8 10 и это массив b которому ты выделил нужный размер в функции f2 какая ошибка? 0

@luigration 12 / 9 / 5 Регистрация: 04.01.2013 Сообщений: 205
	28.04.2014, 21:34 [ТС]	17
	Сообщение от alekopoko luigration, почему это?печатается же 2 4 6 8 10 и это массив b которому ты выделил нужный размер в функции f2 какая ошибка? Все, понятно теперь. Все работает, пардон. Спасибо огромное! 0

@cooller 571 / 539 / 280 Регистрация: 25.12.2013 Сообщений: 1,456
	28.04.2014, 21:36	18
	alekopoko, лучше не глобальной переменную делать, а передавать по ссылке 0

@zss Модератор 13507 / 10757 / 6412 Регистрация: 18.12.2011 Сообщений: 28,712
	28.04.2014, 20:40	2
	Сообщение от luigration delete []even; Писать не надо, хотя все равно эта строчка не выполняется, т.к. стоит после return 0

@alekopoko 1 / 1 / 1 Регистрация: 28.04.2014 Сообщений: 33
	28.04.2014, 20:44	3
	delete не выполняется,выход происходит раньше,return even; 0

@luigration 12 / 9 / 5 Регистрация: 04.01.2013 Сообщений: 205
	28.04.2014, 21:00 [ТС]	4
	Сообщение от alekopoko delete не выполняется,выход происходит раньше,return even; Это понятно. Эта строчка случайно сюда попала. Что с delete, что без - все равно не работает. 0

@alekopoko 1 / 1 / 1 Регистрация: 28.04.2014 Сообщений: 33
	28.04.2014, 21:03	5
	luigration, зачем тебе указатель на функцию? может так? Код int f2 (int *mas,int N) 0

@cooller 571 / 539 / 280 Регистрация: 25.12.2013 Сообщений: 1,456
	28.04.2014, 21:15	7
	Сообщение от alekopoko зачем тебе указатель на функцию? а где ты увидел там указатель на функцию? Добавлено через 2 минуты luigration, мне интересно, а как ты выводишь потом этот массив? 0

@luigration 12 / 9 / 5 Регистрация: 04.01.2013 Сообщений: 205
	28.04.2014, 21:24 [ТС]	10
	То есть в главной функции мне нужно было создать еще один массив, так? И почему j - глобальная переменная? 0

@alekopoko 1 / 1 / 1 Регистрация: 28.04.2014 Сообщений: 33
	28.04.2014, 21:25	12
	luigration, Так то же самое будет.. Мне казалось, что если функция должна возвращать указатель или массив, то нужно писать со свездой, разве нет? Если убрать звезду, то функция не сможет вернуть массив. Указатели на функцию объявляются по-другому.. Код C++ (QT) 1 int(*p)(int)=&f2; все правильно говоришь да) это я накосячил) 0

@luigration 12 / 9 / 5 Регистрация: 04.01.2013 Сообщений: 205
	28.04.2014, 21:26 [ТС]	13
	Сообщение от cooller массив нужно выводить в цикле, а для этого нужно знать его размер так что ты должен вернуть из цикла и размер тоже, либо использовать std::vector Вектором пользоваться не умею. Пока что. 0

@alekopoko 1 / 1 / 1 Регистрация: 28.04.2014 Сообщений: 33
	28.04.2014, 21:27	14
	luigration, И почему j - глобальная переменная? это переменная определяет сколько элементов выводить в Main ну можно по-разному это все реализовать 0

@luigration 12 / 9 / 5 Регистрация: 04.01.2013 Сообщений: 205
	28.04.2014, 21:29 [ТС]	15
	Сообщение от alekopoko luigration, это переменная определяет сколько элементов выводить в Main ну можно по-разному это все реализовать Не работает все равно.. 0

@alekopoko 1 / 1 / 1 Регистрация: 28.04.2014 Сообщений: 33
	28.04.2014, 21:43	19
	cooller, ну с точки зрения затрат памяти или с точки зрения - что где-то там случайно совпадут индентификаторы и я не замечу, то да. Но программа мизерная,так что разницы нет. Ну если я правильно понимаю, то ты намекаешь что это плохой стиль программирования) да) не спорю) 0

@luigration 12 / 9 / 5 Регистрация: 04.01.2013 Сообщений: 205
	28.04.2014, 22:22 [ТС]	20
	Сообщение от cooller alekopoko, лучше не глобальной переменную делать, а передавать по ссылке Что именно передавать по ссылке? Вернее где ее передавать? Добавлено через 15 минут Сообщение от alekopoko cooller, ну с точки зрения затрат памяти или с точки зрения - что где-то там случайно совпадут индентификаторы и я не замечу, то да. Но программа мизерная,так что разницы нет. Ну если я правильно понимаю, то ты намекаешь что это плохой стиль программирования) да) не спорю) А как сделать без глобализации j? 0

Вернуть все четные числа массива в виде нового массива

Решение