Массивы - не тот результат.

[amuleg]

Вот задания звучит так - "Дан массив А из к целых положительных чисел. Записать все четные по значению элементы массива А в массив В.


Я вроде все правильно написал(ну с моей точки зрения))) но результат, Правильно показывает число полжительных чисел, тоесть количество элементов в массиве В, а вот их значения полный брэд)) Очень надеюсь на вашу помощь) Вот сообственно мой код

# include <iostream>
# include <stdlib.h>
# include <time.h>
using namespace std;
int main ()
{ 
	int i;
	const int size=10;
	int Ar[size], B[size];
	srand(time(NULL));
	for(int i=0; i<size; i++)
	{
		Ar[i]=1+rand()%20;
		cout<<"Ar["<<i<<"] = "<<Ar[i]<<endl;
	}
	int m=0;
	for(int i=0; i<size; i++)
		
	{
		if (Ar[i]%2==0) 
		
			    m++; B[m]=Ar[i];
		
	}
			if (m==0)
			{cout<<"B[m] = 0"<<endl;
			}
			else
				for (i=1; i<m; i++)
				{
					cout<<"B["<<i<<"] = "<<B[m]<<endl;
				}
}

[_Bers]

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

Вот задания звучит так - "Дан массив А из к целых положительных чисел. Записать все четные по значению элементы массива А в массив В.


Я вроде все правильно написал(ну с моей точки зрения))) но результат, Правильно показывает число полжительных чисел, тоесть количество элементов в массиве В, а вот их значения полный брэд)) Очень надеюсь на вашу помощь) Вот сообственно мой код

# include <iostream>
# include <stdlib.h>
# include <time.h>
using namespace std;
int main ()
{ 
	int i;
	const int size=10;
	int Ar[size], B[size];
	srand(time(NULL));
	for(int i=0; i<size; i++)
	{
		Ar[i]=1+rand()%20;
		cout<<"Ar["<<i<<"] = "<<Ar[i]<<endl;
	}
	int m=0;
	for(int i=0; i<size; i++)
		
	{
		if (Ar[i]%2==0) 
		
			    //m++; B[m]=Ar[i]; //<--1 мина
		               B[m]=Ar[i]; ++m;
	}
			if (m==0)
			{cout<<"B[m] = 0"<<endl;
			}
			else
				for (i=0; i<m; ++i)  //<--- исправлены последствия 1й мины
				{
					//cout<<"B["<<i<<"] = "<<B[m]<<endl; //<--- 2 мина
                                        cout<<"B["<<i<<"] = "<<B[i]<<endl; 


				}
}

см комментарии в коде

[amuleg]

Спасибо конечно, но тперь вот что пишет

[_Bers]

Вот смотри, я ничего на самом деле глобально в коде не изменял. Я лишь переформатировал стиль кода. Ну то есть, расставил фигурные скобочки, сделал красивые отступы. И все. Больше ничего не делал. У меня получилось вот что:

#include <iostream>
#include <stdlib.h>
#include <time.h>
using namespace std;
int main ()
{ 
    int i;     const int size=10;    int Ar[size], B[size];     srand(time(NULL));  int m=0;

    for(int i=0; i<size; i++)   {   Ar[i]=1+rand()%20;   cout<<"Ar["<<i<<"] = "<<Ar[i]<<endl;  }
    
    for(int i=0; i<size; i++)
    {
        if (Ar[i]%2==0) {  B[m]=Ar[i]; ++m;} 
    }
    
    if (m==0) {cout<<"B[m] = 0"<<endl; }
    else 
    { 
         for (i=0; i<m; ++i){  cout<<"B["<<i<<"] = "<<B[i]<<endl;  } 
    }
}

Я проверил - все работает, как часики.
Отсюда вывод - в твоём коде присутствует синтаксические ошибки, которые я при форматировании кода даже и не заметил (мой собственный code-style от таких ошибок привентивно защищает).

Всем рекомендую использовать такую же модель, а именно:

туловища циклов и условий всегда заключать в фигурные скобки. Это простое правило делает код лучше читаемым, и уберегает от глупых синтаксических ошибок.


А проблема была вот здесь:

for(int i=0; i<size; i++)
{    
    if (Ar[i]%2==0) 
	    m++; B[m]=Ar[i];   //поскольку туловище не заключено в фигурные скобки, условно выполнится только первая инстркция m++
                                     //а инструкция B[m]=Ar[i]; будет выполнятся всегда

                     //такую глупую ошибку очень просто по запарке допустить, если экономить на фигурных скобках
                     //но привентивно невозможно допустить, если туловища циклов и условий всегда заключать в фигурные скобки
}

Код можно сделать ещё более "красивым". В нем присутствует некоторые "недочеты".

[amuleg]

Спасибо, большое))

[amuleg]

Я понимаю может это уже нагло, но теперь мне надо отсортировать, и опять фигня) -

#include <iostream>
#include <stdlib.h>
#include <time.h>
using namespace std;
int main ()
{ 
    int i;
    const int size=10;
    int Ar[size], B[size];
    srand(time(NULL));
    for(int i=0; i<size; i++)   {   Ar[i]=1+rand()%20;   cout<<"Ar["<<i<<"] = "<<Ar[i]<<endl;  }
    int m=0;
    for(int i=0; i<size; i++)
    {
        if (Ar[i]%2==0) {  B[m]=Ar[i]; ++m;} 
    }
	cout<<endl;
	cout<<endl;
    if (m==0) {cout<<"B[m] = 0"<<endl; }
    else { for (i=0; i<m; ++i){  cout<<"B["<<i<<"] = "<<B[i]<<endl;  }
	
	}
	for ( int j=0; j<size-1; j++)
		for (m=0; m<size-1; m++)
			if (B[m]<B[m-1])
			{
				int C=B[m-1];
				B[m-1]=B[m];
				B[m]=C;
			}
			cout<<endl;
			cout<<endl;
for (m=0; m<size; m++)
{
	cout<<"B["<<m<<"] = "<<B[m]<<endl;
}
return 0;
}

[_Bers]

#include <iostream>
#include <stdlib.h>
#include <time.h>
using namespace std;

//ниже приведенные макросы служат лишь для тестирования программулички, и не рекомендованы к использованию в боевом коде
#define VIEW_ELEMENT(Arr, index) cout<<#Arr<< '['<<index<< "] = "<<Arr [index]<<endl;
#define VIEW_ARR(Arr, sizeArr) for (size_t i=0; i<sizeArr; ++i){  VIEW_ELEMENT(Arr, i);  }

//Функция ищет минимальный элемент в массиве, и возвращает его индекс
//int* Arr - массив, где ищем
//size_t sizeArr - количество элементов в массиве
//size_t startPos - позиция с которой начинается поиск
inline size_t FindMin(int* Arr, size_t sizeArr, size_t startPos)
{
    size_t find_index=startPos; 
    for(size_t i=startPos; i<sizeArr; ++i)  
    { 
        if (Arr[find_index]>Arr[i])	{ find_index=i; } 
    }
    return find_index;
}

int main ()
{ 
    const int size=10;  int Ar[size], B[size];   srand( (size_t) time(NULL)); size_t m=0;
    
    for(size_t i=0; i<size; i++) {  Ar[i]=1+rand()%20; VIEW_ELEMENT(Ar, i); }  //заполнение массива Ar
    for(size_t i=0; i<size; i++) {  if (Ar[i]%2==0) {  B[m]=Ar[i]; ++m;}    }      //заполнение массива B
    
    cout<<"\n\n";
    if (m==0) {cout<<"Массив B пуст!\n"; }  else { VIEW_ARR(B, m); }  //Покажет содержимое массива B

    for( size_t i=0; i<m; ++i) {  swap( B[i], B[FindMin(B, m, i)] ); }  //Отсортирует массив B по возврастанию
    
    cout<<"\n\n";  VIEW_ARR(B, m);    
    return 0;
}

Если ты сталкиваешься с алгоритмом, который для тебя оказывается слишком сложным - используй принцип "разделяй и властвуй".

То бишь, раздели одну большую сложную задачу на кучку более мелких, и более простых. И тогда все сразу станет ясно.