СИ, матрица, поменять max и min элементы во всех строках...

[Immoralist]

Дана матрица размера M x N. Преобразовать матрицу, поменяв местами минимальный и максимальный элемент в каждой строке.

PHP код:

#include "stdafx.h"
#include <stdlib.h>
#include <time.h>
#include <stdio.h>

void main()
{
    const long m=7, n=5;
    long a[m][n], i, j, min, max, x, l, r;
    srand(time(0));
    //формируем и выводим матрицу
    for (i=0; i<m; i++)
    {
        printf("\n\n");
        for (j=0; j<n; j++) 
           {
            a[i][j]=25+rand()%56;
            printf("%d\t",a[i][j]);
           }
    }
    //ищем min и max, меняем местами
    for (i=0; i<m; i++)
    {
       min=999999;
       max=0;
       for (j=0; j<n; j++) 
       {
        if (j==n) {x=a[i][l]; a[i][l]=a[i][r]; a[i][r]=x;}
        if (min>a[i][j]) {min=a[i][j]; l=j;}
        if (max<a[i][j]) {max=a[i][j]; r=j;}
       }
    }
    printf("\n\n");
    //выводим то что получилось
    for (i=0; i<m; i++)
    {
        printf("\n\n");
        for (j=0; j<n; j++) 
           {
            printf("%d\t",a[i][j]);
           }
    }
    printf("\n\n");
} 


Выводятся две абсолютно одинаковые матрицы. В чем проблема? Ошибок в алгоритме не вижу...помогите...

[Helloween]

А если так?

//ищем min и max, меняем местами 
    for (i=0; i<m; i++) 
    { 
       min=999999; 
       max=0; 
       for (j=0; j<n; j++)  
	   {
		if (a[i][j]<min) {min=a[i][j]; l=j;}
		if (a[i][j]>max) {max=a[i][j]; r=j;}
	   }
	   a[i][l] = max;
	   a[i][r] = min;
	}

[Immoralist]

Спасибо, работает...

[Hacker19_90]

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

А если так?

//ищем min и max, меняем местами 
    for (i=0; i<m; i++) 
    { 
       min=999999; 
       max=0;  // и что это?
       for (j=0; j<n; j++)  
	   {
		if (a[i][j]<min) {min=a[i][j]; l=j;}
		if (a[i][j]>max) {max=a[i][j]; r=j;}
	   }
	   a[i][l] = max;
	   a[i][r] = min;
	}

так не проще?

for (i = 0; i < n; i++)
{
  min = 0;
  max = 0;
  for (j = 1; j < m; j++)
  {
     if (a[i][j] < a[i][min]) min = j;
     if (a[i][j] > a[i][max]) max = j;
  }
  if (max != min)
  {
    a[i][max] = a[i][max]^a[i][min];
    a[i][min] = a[i][max]^a[i][min];
    a[i][max] = a[i][max]^a[i][min];
  }
}

[Helloween]

Я исправил текущий вариант как бы =) Вообще по-нормальному нужно брать max и min - первым элементом массива. Понятно что я сам так не стал бы писать, но так хоть человек понял в чем была его ошибка.

[Эля856]

Я сделала вот так но он почему то выводит матрицу в одну строчку... Помогите :conf used:


{ // Дана матрица размером М x N. Преобразовать матрицу, поменяв местами минимальные и максимальные элемент в каждой строке.

class Program
{
static void Main(string[] args)
{
int r=(0), min, max, n=(4), m=(4), l=(0);
int M = 4; // Число строк
int N = 4; // Число столбцов
int[,] Z = {{1,3,8,0},
{4,6,2,5},
{3,7,0,6},
{3,9,11,2}};
int i, j;
Console.WriteLine("Исходная матрица");
for (i = 0; i < M; i++)
{
for (j = 0; j < N; j++)
Console.Write(Z[i, j] + "\t");
Console.WriteLine();
}
//ищем min и max, меняем местами
for (i = 0; i < m; i++)
{
min = 999999;
max = 0; // и что это?
for (j = 0; j < n; j++)
{
if (Z[i, j] < min) { min = Z[i, j]; l = j; }
if (Z[i, j] > max) { max = Z[i, j]; r = j; }
}
Z[i, l] = max;
Z[i, r] = min;
// Вывод преобразованной матрицы
Console.WriteLine("Преобразованная матрица");
for (i = 0; i < M; i++)
{
for (j = 0; j < N; j++)
Console.Write(Z[i, j] + "\t");
}

}
Console.ReadKey();
}
}
}

[Son Of Pain]

Нужно вставить Console.WriteLine(); так же, как и в первом цикле