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

@u-geen · Регистрация: 04.03.2014

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

Добрый вечер,

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

уже и переставлял и что только не делал, не пойму в чем дело

C++

#include <iostream>
#include<time.h>
using namespace std;
 
int main()
{
    setlocale(LC_ALL, "rus");
    srand(time(NULL));
    const int m = 2;
    const int n = 4;
    int matr[m][n];
    int max=matr[0][0];
    int min=matr[0][0];
 
    for (int i = 0; i < m; i++)
    {
        int maxs = matr[i][0];
        int mins = matr[i][0];
        for (int j = 0; j < n; j++)
        {
            matr[i][j] = -5+rand() % 20;
            cout << matr[i][j] << "\t";
            
            if (matr[i][j]<min) {
                min = matr[i][j];
            }
            if (matr[i][j] > max)
            {
                max = matr[i][j];
                
            }
            if (matr[i][j]<mins)
            {
                mins = matr[i][j];
            }
            if (matr[i][j]>maxs)
            {
                maxs = matr[i][j];
            }
            
        }
        cout << "Макс-й элемент " << i << " строки= " << maxs  <<", мин-й элемент= " << mins << endl;
        cout << endl;
    }
    cout << "Максимальный элемент массива= " << max << endl;
    cout << "Минимальный элемент массива= " << min << endl;
}

Croessmah · 18.02.2015, 23:59

У Вас элементы матрицы не инициализированы, а Вы "берете" их значения

@u-geen · 19.02.2015, 00:11 **[ТС]**

я ставил
int maxs = matr[i][0];
int mins = matr[i][0];
и после
matr[i][j] = -5+rand() % 20;

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

@Krowli · 19.02.2015, 15:49

Так как

C++

1 2	int max=matr[0][0]; int min=matr[0][0];

они принимаю одно и тоже значение, оно и выводит вам такой бред. Что min = Null(-858..) и max = Null(-858..)
Да, программа выведет вам правильный максимум, но минимум останется такой же, так как меньше -858.. нету.
Лучше сделайте так

C++

int main()
{
    setlocale(LC_ALL, "rus");
    srand(time(NULL));
    const int m = 2;
    const int n = 4;
    int matr[m][n];
    for (int i = 0; i < m; i++)
    {
        for (int j = 0; j < n; j++)
        {
            matr[i][j] = -5+rand() % 20;
            cout << matr[i][j] << "\t";      
        }
    }
        int max = matr[0][0];
        int min = matr[0][0]; 
    for (int i = 0; i < m; i++)
    {
        for (int j = 0; j < n; j++)
        {
            if (matr[i][j]<min) {
                min = matr[i][j];
            }
            if (matr[i][j] > max)
            {
                max = matr[i][j];     
            }
        }
    }
    cout << "Максимальный элемент массива= " << max << endl;
    cout << "Минимальный элемент массива= " << min << endl;
system("PAUSE");
 
}

@u-geen · 19.02.2015, 16:31 **[ТС]**

Все гениальное просто!

Спасибо огромное!!! Все работает!

Croessmah Неэпический 17815 / 10586 / 2044 Регистрация: 27.09.2012 Сообщений: 26,627 Записей в блоге: 1
	18.02.2015, 23:59	2
	У Вас элементы матрицы не инициализированы, а Вы "берете" их значения 0

@u-geen 0 / 0 / 0 Регистрация: 04.03.2014 Сообщений: 32
	19.02.2015, 00:11 [ТС]	3
	я ставил int maxs = matr[i][0]; int mins = matr[i][0]; и после matr[i][j] = -5+rand() % 20; но результат тот же, если я вас правильно понял.... и тогда я проверял пошагово, и переменным присваивается значение указанных элементов, но при выходе из цикла в них появляется мусор, т.е или стирается присвоенное значение или компилятор "забывает", вообще не пойму почему 0

@u-geen 0 / 0 / 0 Регистрация: 04.03.2014 Сообщений: 32
	19.02.2015, 16:31 [ТС]	5
	Все гениальное просто!Спасибо огромное!!! Все работает! 0

Опции темы

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

Решение