Почему при попытке вывести суммы выдаёт не то что нужно?

Question

[een0t]

Попытался написать программу, а там при попытке вывести сумму строки матрицы, занесённой в массив выдаёт вот это:
Нужно, чтобы оно считало суммы строк, а затем строчку с самой большой суммой элементов ставило на место первой строчки, но ошибка находится уже на моменте поиска суммы элементов в строке
Вот кусок кода на поиск суммы строк и записи их в отдельный массив

for (i = 1; i <= m; i++) 
	{                              
		for (j = 1; j <= n; j++)
			summ[i] += ar[i][j];

Answer 1

[Денис Загаевский]

Кода недостаточно, но думаю, что ты не инициализируешь summ нулями

Comments

[een0t]

Да, заполнение массива нулями помогло, правда теперь появилась немного другая проблема, оно криво изменяет матрицу, не как должно. Должно быть так, что первой строкой будет стоять строка с максимальной суммой элементов в ней, далее идти остальные строки по убыванию. Можешь пожалуйста сказать что не так с кодом в этом моменте? Вот сам код полный

spoiler

int ar[15][15];
	int summ[15] = {};
    int i, j, k;
    int m, n;
    int temp;
	srand(time(NULL));
	cout << "Введите количество строк: ";               //Размер масива
	cin >> n;
	cout << "Введите количество столбцов: ";
	cin >> m; 
	cout << "\n";

	for (i = 1; i <= m; i++) {                              //Записываем в основной массив случайные числа
		cout << "Введите элементы " << i << " строки: ";
		for (j = 1; j <= n; j++)
			cin >> ar[i][j];
	}
	for (i = 1; i <= m; i++) {                              //Распечатываем их
		for (j = 1; j <= n; j++)
			cout << "  " << ar[i][j];
			cout << "\n";
	}
	for (i = 1; i <= m; i++) 
	{                              //Находим сумму строк
		for (j = 1; j <= n; j++)
			summ[i] += ar[i][j];
	}
	cout << "\n";
	for (i = 1; i <= m; i++)                                //Распечатываем суммы для проверки что все идет правильно
		cout << "Сумма " << i << " строки = " << summ[i] << "\n";
	cout << "\n";
	temp = 0;
	for (i = 1; i <= m; i++) {
		if (i + 1 <= m) {
			for (k = 1; k <= m; k++) {
				if (summ[k] < summ[k + 1]) {                 //Сортируем массив S по не возростанию
					temp = summ[k];
					summ[k] = summ[k + 1];
					summ[k + 1] = temp;
					for (j = 1; j <= n; j++) {         //Сортируем строки в основном масиве
						temp = ar[i][j];
						ar[i][j] = ar[i + 1][j];
						ar[i + 1][j] = temp;
					}                        
				}
			}
		}
	}
	cout << "\n";
	for (i = 1; i <= m; i++) {                         //Распечатываем получившийся основной массив
		for (j = 1; j <= n; j++)
			cout << "  " << ar[i][j];
		cout << "\n";
	}
	cout << "\n";                                    

	system("PAUSE");
	return 0;
}

[Денис Загаевский]

een0t, по-моему, очевидно, сортировка неправильная. Сравниваешь ты одни суммы, а местами меняешь другие строки.

[een0t]

Денис Загаевский, дуб дубом в программировании и тем более на с++, не очень понимаю как там отредактировать чтобы работало как надо, но всё равно спасибо

[Денис Загаевский]

een0t, ты обмениваешь местами i и i+1 строки, а надо k и k+1.
Сорри, я не буду писать код за тебя.
Вместо "спасибо" есть "отметить решение".