Найти максимальный и минимальный элемент каждого столбца и строки матрицы

@Maxsum19090 · Регистрация: 15.06.2014

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

Здравствуйте!Дали задание:
1.Создать матрицу размером 5х5.
2.Вывести матрицу на экран.
3.Найти макс и мин элемент каждого столбца и строки.
Вот код, который, собственно, выполняет все эти задачи,но...Иногда не правильно показывает номер столбца, чаще всего нулевого и 3-его (начало массива-ноль)...Помогите пожалуйста, где я накосячил?

C++

#include "stdafx.h"
#include "conio.h"
#include "stdlib.h"
#include "iomanip"
#include "iostream"
#include "ctime"
const int n=5;
using namespace std;
 
void zapolnatb (int A[n][n])
{
    srand(time(0));
    for (int i=0;i<n;i++)
    {
        for (int j=0;j<n;j++)
            A[i][j]=rand()%100-50;
}
}
void vivodmat (int A[n][n])
{for (int i=0;i<n;i++)
{for (int j=0;j<n;j++)
    cout<<setw(3)<<A[i][j];
cout<<endl;}
}
void naxogdenue (int A[n][n]) //max element v stroke
{
    int max;
    int str=0;
    int stl=0;
    for(int i=0;i<n;i++)
    {
        max=A[i][0];
        for (int j=0;j<n;j++)
            if (max<A[i][j]){max=A[i][j];
        str=i;
        stl=j;}
        cout<<setw(4)<<max<<setw(4)<<stl<<endl;
    }
}
void naxogdenuee (int A[n][n]) //min element v stroke
{
    int min;
    int str=0;
    int stl=0;
    for(int i=0;i<n;i++)
    {
        min=A[i][0];
        for (int j=0;j<n;j++)
            if (min>A[i][j]){min=A[i][j];
        str=i;
        stl=j;}
        cout<<setw(5)<<min<<setw(5)<<stl<<endl;
    }
}
void naxogdenueee (int A[n][n]) //min element v stolbce
{
    int min;
    int str=0;
    int stl=0;
    for(int j=0;j<n;j++)
    {
        min=A[0][j];
        for (int i=0;i<n;i++)
            if (min>A[i][j]){min=A[i][j];
        str=i;
        stl=j;}
        cout<<setw(6)<<min<<setw(6)<<str<<endl;
    }
}
void naxogdenueeee (int A[n][n]) //max element v stolbce
{
    int min;
    int str=0;
    int stl=0;
    for(int j=0;j<n;j++)
    {
        min=A[0][j];
        for (int i=0;i<n;i++)
            if (min<A[i][j]){min=A[i][j];
        str=i;
        stl=j;}
        cout<<setw(7)<<min<<setw(7)<<str<<endl;
    }
}
    int main ()
    {
        int (A[n][n]);
        zapolnatb (A);
        vivodmat (A);
        naxogdenue (A);
        naxogdenuee (A);
        naxogdenueee (A);
        naxogdenueeee (A);
        system ("pause");
        return 0;
    }

Заранее прошу прощения, за не правильно расставленные "Мин" и "макс", а так же за повторяющиеся "Нахождения", просто так быстрее было

@MayaNash · 25.06.2014, 07:00

Не по теме:

Maxsum19090, неужели так сложно нормально, читабельно ставить отступы и давать функциям нормальные названия?

Добавлено через 4 минуты
Ошибка:

C++

1	if (max<A[i][j]){max=A[i][j]

Добавлено через 1 минуту

C++

1	min>A[i][j]){min=A[i][j];

Ты в курсе что такое минимум и максимум вообще?..

Добавлено через 4 минуты
Maxsum19090, это вот вообще не было быстрее. Замени нормально, может и ошибку найдешь. А то даже рыться не хочется в таком коде.

C++

#include "conio.h"
#include "stdlib.h"
#include "iomanip"
#include "iostream"
#include "ctime"
const int n=5;
using namespace std;
 
void zapolnatb (int A[n][n])
{
    srand(time(0));
    for (int i=0;i<n;i++)
    {
        for (int j=0;j<n;j++)
            A[i][j]=rand()%100-50;
    }
}
 
void vivodmat (int A[n][n])
{
    for (int i=0;i<n;i++)
    {
        for (int j=0;j<n;j++)
            cout<<setw(3)<<A[i][j];
        cout<<endl;
    }
}
 
void naxogdenue (int A[n][n]) //max element v stroke
{
    int max;
    int stl=0;
    for(int i=0;i<n;i++)
    {
        max=A[i][0];
        for (int j=0;j<n;j++)
            if (max>A[i][j])
            {
                max=A[i][j];
                stl=j;
            }
        cout<<setw(4)<<max<<setw(4)<<stl<<endl;
    }
}
 
void naxogdenuee (int A[n][n]) //min element v stroke
{
    int min;
    int stl=0;
    for(int i=0;i<n;i++)
    {
        min=A[i][0];
        for (int j=0;j<n;j++)
            if (min<A[i][j])
            {
                min=A[i][j];
                stl=j;
            }
        cout<<setw(5)<<min<<setw(5)<<stl<<endl;
    }
}
 
void naxogdenueee (int A[n][n]) //min element v stolbce
{
    int min;
    int str=0;
    for(int j=0;j<n;j++)
    {
        min=A[0][j];
        for (int i=0;i<n;i++)
            if (min>A[i][j])
            {
                min=A[i][j];
                str=i;
            }
        cout<<setw(6)<<min<<setw(6)<<str<<endl;
    }
}
 
void naxogdenueeee (int A[n][n]) //max element v stolbce
{
    int min;
    int str=0;
    int stl=0;
    for(int j=0;j<n;j++)
    {
        min=A[0][j];
        for (int i=0;i<n;i++)
            if (min<A[i][j])
            {
                min=A[i][j];
                str=i;
                stl=j;
            }
        cout<<setw(7)<<min<<setw(7)<<str<<endl;
    }
}
 
int main ()
{
    int (A[n][n]);
    zapolnatb (A);
    vivodmat (A);
    naxogdenue (A);
    naxogdenuee (A);
    naxogdenueee (A);
    naxogdenueeee (A);
    system ("pause");
    return 0;
}

С нормальными отступами, легче искать будет

@Maxsum19090 · 25.06.2014, 07:01 **[ТС]**

Я в курсе что такое минимум и максимум.А в чем конкретней ошибка?Что не так с

C++

1	min>A[i][j]){min=A[i][j];

?

alsav22 · 25.06.2014, 07:01

Сообщение от Керра

Ошибка:

В чём?

@Maxsum19090 · 25.06.2014, 07:08 **[ТС]**

C++

  int max;
    int stl=0;
    for(int i=0;i<n;i++)
    {
        max=A[i][0];
        for (int j=0;j<n;j++)
            if (max>A[i][j])
            {
                max=A[i][j];
                stl=j;
            }

Сделал так же как у вас, абсолютно та же ошибка.

@MayaNash · 25.06.2014, 08:24

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

Добавлено через 41 секунду
alsav22, ошибка в том, что когда ищут максимум, надо искать наибольший, а не наименьший элемент. И называть нормально переменные.

alsav22 · 25.06.2014, 08:27

Сообщение от Керра

alsav22, ошибка в том, что когда ищут максимум, надо искать наибольший, а не наименьший элемент. И называть нормально переменные.

Там и ищут наибольший, и называют:

Сообщение от Керра

if (max<A[i][j]){max=A[i][j]

Если A[i][j] больше max, то max присваивают значение A[i][j]. Что, по-вашему, не так?

@MayaNash · 25.06.2014, 13:29

alsav22, блин. Привыкла к собственному написанию.

@Maxsum19090 · 25.06.2014, 20:14 **[ТС]**

Я конечно безумно рад, что мой неграмотный, грязно написанный, нечитабельный код раскритиковали, но может кто то после критики оставит мааа-а-аленький код C++, решающий ошибку в дебрях написанного мной?

@zss · 25.06.2014, 22:15

C++

void naxogdenueMax (int A[n][n]) //max element v stroke
{
    int max;
    for(int i=0;i<n;i++)
    {
        int stl=0;
        max=A[i][0];
        for (int j=1;j<n;j++)
            if (A[i][j]>max)
            {
                max=A[i][j];
                stl=j;
            }
        cout<<setw(4)<<max<<setw(4)<<stl<<endl;
    }
}

@Maxsum19090 · 26.06.2014, 16:36 **[ТС]**

zss, спасибо большое

@Maxsum19090 0 / 0 / 0 Регистрация: 15.06.2014 Сообщений: 15
	26.06.2014, 16:36 [ТС]	11
	zss, спасибо большое 0

alsav22 5497 / 4892 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	25.06.2014, 07:01	4
	Сообщение от Керра Ошибка: В чём? 0

@MayaNash 1296 / 469 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	25.06.2014, 08:24	6
	Maxsum19090, я же говорила что не исправляла код. Напишите так, чтобы все было понятно, т.е. исправьте имена функций, минимумов и максимумов. Если где-то логическая ошибка, то ее сложно найти в таком коде. Добавлено через 41 секунду alsav22, ошибка в том, что когда ищут максимум, надо искать наибольший, а не наименьший элемент. И называть нормально переменные. 0

alsav22 5497 / 4892 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	25.06.2014, 08:27	7
	Сообщение от Керра alsav22, ошибка в том, что когда ищут максимум, надо искать наибольший, а не наименьший элемент. И называть нормально переменные. Там и ищут наибольший, и называют: Сообщение от Керра if (max<A[i][j]){max=A[i][j] Если A[i][j] больше max, то max присваивают значение A[i][j]. Что, по-вашему, не так? 0

@MayaNash 1296 / 469 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	25.06.2014, 13:29	8
	alsav22, блин. Привыкла к собственному написанию. 0

@Maxsum19090 0 / 0 / 0 Регистрация: 15.06.2014 Сообщений: 15
	25.06.2014, 20:14 [ТС]	9
	Я конечно безумно рад, что мой неграмотный, грязно написанный, нечитабельный код раскритиковали, но может кто то после критики оставит мааа-а-аленький код C++, решающий ошибку в дебрях написанного мной? 0

Опции темы

Найти максимальный и минимальный элемент каждого столбца и строки матрицы

Решение