Не могу найти ошибку (С++)

[Kristino4ka]

Здравствуйте! Пожалуйста помогите найти ошибку в коде С++

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
int main()
{
	srand (time(NULL));
	int i,j,n,m;
	int A[100][100];
	setlocale (LC_ALL, "Russia");
	cout<<"Введите количество строк:"; cin>>n;
	cout<<"Введите количество столбцов:"; cin>>m;
	{
	for (i=0;i<n;i++)
	for (j=0;j<m;j++)
	A[i][j]=rand()%50;
	}
    for (int i = 0; i < n; i++)
 {
    for(int j = 0; j < m; j++)
    cout << A[i][j] << " ";
    cout << endl;
 }
	
	int P[10];
	int k;
	int sum;
{
    
	for (j=0;j<m;j++)
	for (k=0;k<m;k++)
{
	for (i=0;i<n;i++)
	if (A[i][j]%2!=0)
	sum=sum+A[i][j];
	P[k]=sum;
	cout << P[k] << " ";
	sum=0;
}
}
	
    cout << endl;
	system("pause");
	return 0;
}

Вот само задание: Вычислить сумму нечётных элементов для каждого столбца, результаты занести в одномерный массив. Число строк и столбцов вводится с клавиатуры. Элементы массива заполняются с помощью генератора случайных чисел.
Вот что выходит:
Безымянный.JPG
Не понимаю откуда берётся первое число и почему копирует третье....

[whatever]

Цикл for (k=0;k<m;k++) определенно лишний. Ну и еще пару строчек поправил. Вроде бы правильно теперь.

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
int main()
{
	srand (time(NULL));
	int i,j,n,m;
	int A[100][100];
	setlocale (LC_ALL, "Russia");
	cout<<"Введите количество строк:"; cin>>n;
	cout<<"Введите количество столбцов:"; cin>>m;
	{
	for (i=0;i<n;i++)
	for (j=0;j<m;j++)
	A[i][j]=rand()%50;
	}
    for (int i = 0; i < n; i++)
 {
    for(int j = 0; j < m; j++)
	cout << A[i][j] << " ";
	cout << endl;
 }

	int P[10];
	int k;
	int sum=0;
{

	for (j=0;j<m;j++)
//	for (k=0;k<m;k++)
{
	for (i=0;i<n;i++)
	if (A[i][j]%2!=0)
	sum=sum+A[i][j];
	P[k]=sum;
	cout << P[k] << " ";
	k++;
	sum=0;
}
}

	cout << endl;
	system("pause");
	return 0;
}

[BDA]

/*опоздал*/

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
int main()
{
    srand (time(NULL));
    int i,j,n,m;
    setlocale (LC_ALL, "Russia");
    cout<<"Введите количество строк:"; cin>>n;
    cout<<"Введите количество столбцов:"; cin>>m;
    
    int A[n][m];
    int P[m];
    
    for (i=0;i<n;i++)
        for (j=0;j<m;j++)
            A[i][j]=rand()%50;
            
    for (int i = 0; i < n; i++) {
        for(int j = 0; j < m; j++)
            cout << A[i][j] << " ";
    cout << endl;
    }
 
    for (j=0;j<m;j++) {
        P[j] = 0;
        for (i=0;i<n;i++)
            if (A[i][j]%2!=0)
                P[j] += A[i][j];
        cout << P[j] << " ";
    }
    cout << endl;
    system("pause");
    return 0;
}

[Kristino4ka]

Продолжение этого же задания:
2) Поменять местами строки 1 и 2, 3 и 4, 5 и 6 и т.д.
вот я продолжила тот код

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
int main()
{
	srand (time(NULL));
	int i,j,n,m;
	int A[100][100];
	setlocale (LC_ALL, "Russia");
	cout<<"Введите количество строк:"; cin>>n;
	cout<<"Введите количество столбцов:"; cin>>m;
	{
	for (i=0;i<n;i++)
	for (j=0;j<m;j++)
	A[i][j]=rand()%50;
	}
    for (int i = 0; i < n; i++)
 {
    for(int j = 0; j < m; j++)
	cout << A[i][j] << " ";
	cout << endl;
 }

	int P[10];
	int k=0;
	int sum=0;
{

	for (j=0;j<m;j++)

{
	for (i=0;i<n;i++)
	if (A[i][j]%2!=0)
	sum=sum+A[i][j];
	
	P[k]=sum;
	cout << P[k] << " ";
	k++;
	sum=0;
}
}

	cout << endl;
	int K[10];
	int p;
	{
	for (i=0;i<n;i+=2)
		{
			for(j=0;j<m;j++)
				for (p=0;p<m;p++)
					K[p]=A[i][j];
			A[i][j]=A[i+1][j];
			A[i+1][j]=K[p];
		}
	}
	    for(i=0;i<n;i++)
		{
			for(j=0;j<m;j++)
				cout<<A[i][j]<<" ";
			cout<<endl;
		}
	system("pause");
	return 0;
}

в итоге выводит тот же самый массив

[BDA]

#include <iostream>
#include <cstdlib>
#include <ctime>
 
using namespace std;
 
int
main()
{
    srand (time(NULL));
    int n, m;
    setlocale (LC_ALL, "Russia");
    cout << "Введите количество строк:";
    cin >> n;
    cout << "Введите количество столбцов:";
    cin >> m;
    int A[n][m];
    int P[m];
    for (int i = 0; i < n; ++i) {
        for (int j = 0; j < m; ++j) {
            A[i][j] = rand() % 50;
        }
    }
    cout << "Массив:" << endl;       
    for (int i = 0; i < n; ++i) {
        for(int j = 0; j < m; ++j) {
            cout << A[i][j] << " ";
        }
        cout << endl;
    }
    cout << "Сумма в столбцах: " << endl;
    for (int j = 0; j < m; ++j) {
        P[j] = 0;
        for (int i = 0; i < n; ++i) {
            if (A[i][j] % 2 != 0) {
                P[j] += A[i][j];
            }
        }
        cout << P[j] << " ";
    }
    cout << endl;
    for (int i = 0; i < n - 1; i += 2) {
        for (int j = 0; j < m; ++j) {
            int tmp = A[i][j];
            A[i][j] = A[i + 1][j];
            A[i + 1][j] = tmp;
        }
    }
    cout << "Измененный массив:" << endl;       
    for (int i = 0; i < n; ++i) {
        for(int j = 0; j < m; ++j) {
            cout << A[i][j] << " ";
        }
        cout << endl;
    }
    system("pause");
    return 0;
}

int A[n][m];
int P[m];
Такое объявление массивов надежнее, чем Ваше