Помогите найти ошибку в коде с++

[Dezzez]

не могу найти ошибку в коде
при повторном нахождении минимума выводит просто ноль

# include <conio.h>
# include <iostream>
# include <ctime>
# include <cstdlib>
using namespace std;
int main() {
	setlocale(0, "rus");
	int N, M, A, B, Q, C, u, y, o, p;
	float min = 10000000, max = 0, sr;
	for ( ; ; ) {
		cout << "Введите размеры массива" << endl;
		cin >> N >> M;
		cout << "Введите диапазон рандома" << endl;
		cin >> A >> B;
		A = A * 100000; B = B * 100000; C = B - A;
		float **arr;
		arr = new float *[N];
		for (int i = 0; i < N; ++i) arr[i] = new float[M];
		for (int i = 0; i < N; ++i) {
			for (int k = 0; k < M; ++k) {
				arr[i][k] = (A + rand() % C)*1.00001; cout << arr[i][k] << "\t";
			} cout << endl;}
		do {
			for (int i = 0; i < N; ++i) {
				for (int k = 0; k < M; ++k) {
					if (max < arr[i][k]) { max = arr[i][k]; y = i; u = k; }
					if (min > arr[i][k]) { min = arr[i][k]; o = i; p = k; }}}
			cout << "макс=" << max << endl << "мин=" << min << endl;
			sr = (min + max) / 2; min = 10000000, max = 0;
			cout << "Среднее мин и макс " << sr << endl;
			arr[y][u] = 0; arr[o][p] = 0;
			cout << "Выберите следующее действие:" << endl;
			cout << "1.Повторить с изменением размера массива" << "   " << "2.Обнулить минимальное и максимальное и повторить вычисления" << "  " << "0.Выход" << endl;
			cin >> Q;
			if (Q == 0) { return 0; }
		} while (Q == 2);}
		_getch();}

______________________
Используйте тег [CODE] (кнопка [CODE] в форме сообщения) при вставке кода на форум. Подробнее в FAQ

[Serge_Bliznykov]

не вижу принципиальных косяков (ну, кроме крайне сомнительной иницилизации переменных, да и max, min не нужны, достаточно y u o p )
https://ideone.com/8z787t

что не так?

[digitalis]

Попробовать перед началом цикла за минимум и максимум принять arr[0][0] (конечно, запомнив индексы) и поиск вести уже от arr[0][1] ... до конца массива.

Serge_Bliznykov

да и max, min не нужны

Вообще не нужны ??

[Serge_Bliznykov]

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

Вообще не нужны ??

ага.

			y=0; u=0; o=0; p=0;
			for (int i = 0; i < N; ++i) {
				for (int k = 0; k < M; ++k) {
					if (arr[y][u] < arr[i][k]) {y = i; u = k; }
					if (arr[o][p] > arr[i][k]) {o = i; p = k; }}}
			cout << "макс=" << arr[y][u] << endl << "мин=" << arr[o][p] << endl;

[digitalis]

Когда память была маленькой, а дискеты - большими , нам объясняли, почему так не надо делать: при выполении цикла два обращения к элементу массива, что медленнее (на несколько нс ), чем к скалярной переменной и элементу массива.

[Serge_Bliznykov]

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

Когда память была маленькой, а дискеты - большими , нам объясняли, почему так не надо делать

Это было давно.
Сейчас уже можно

[p51x]

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

[Serge_Bliznykov]

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

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

ну что Вы занудствуете. ну да, конечно, есть разница!

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

Это всё разговоры про "пустой мешок"!

Лучше помогите автору топика найти ошибку в коде.

[p51x]

Для этой учебной задачи вообще достаточно https://en.cppreference.com/w/cpp/al...minmax_element . А забивавшие на такие простые правила, потому что "задача учебная", дофига на работу приходит, хорошо если потом переучиваются.

Там вообще нужно обратить внимание на утечку памяти - память выделяется, указатель теряется и не освобождается.

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

Лучше помогите автору топика найти ошибку в коде.

Так авор пропал и что за проблема у него не доконца понятно. Если проблема в том, что после выбора 2 у него минимум становится 0... то чего он ожидал обнуляя элемент массива.

[Serge_Bliznykov]

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

Для этой учебной задачи вообще достаточно https://en.cppreference.com/w/cpp/al...minmax_element .

ну, не уверен. разве minmax возвращает индексы (он же занулять min/max собирается).

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

Так авор пропал и что за проблема у него не доконца понятно. Если проблема в том, что после выбора 2 у него минимум становится 0... то чего он ожидал обнуляя элемент массива.

согласен.