Замена элементов в массиве

@JanCover · Регистрация: 12.11.2015

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

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

Java

import java.util.Arrays;
import java.util.List;
import java.util.Random;
 
public class massiv {
    public static void main(String[] args) {
 
        int a = 6, b = 7, c = 0, d = 0, max = 0, k = 0;
        int m[][] = new int[a][b];
        int n[] = new int[b];
 
        for (int i=0;i < a;i++){
            for (int j=0;j < b;j++){
                m[i][j]=(int)(Math.random()*10);
            }
        }
        for (int i=0;i < a;i++){
            for (int j=0;j < b;j++){
                System.out.print(m[i][j]+" ");
            }
            System.out.println();
        }
        System.out.println();
 
        for (int i = 0; i<a; i++) {
            max = m[i][c]; // заносим в макс первый элемент, что бы с ним сравнивать
            for (int j = 0; j < b; j++) {
                if (m[i][j] > max){
                    max = m[i][j]; //заносим в макс новый элемент
                    k = j; //заносим индекс этого элемента
                }
                if (c != k){
                    int p = m[i][c];  // заносим в новую переменную элемент стоящий на первой позиции
                    m[i][c] = max; // заносим на место первого элемент из макс
                    m[i][k] = p; // заносим первый элемент на позицию нибольшего элемента
                }
            }
        }
 
        for (int i=0;i < a;i++){
            for (int j=0;j < b;j++){
                System.out.print(m[i][j]+" ");
            }
            System.out.println();
        }
    }
}

ninjacut · 16.06.2016, 16:08

У тебя в цикле замена максимум должна происходить когда ты всю строку прошел, а у тебя оно происходит на каждом шаге итерации.
И переменным бы давать осмысленные имена и читалось бы хоть как-то и убрал бы лишнее, вот я не понял зачем переменная c если она всегда 0.

@HOBATOP · 16.06.2016, 17:13

JanCover, даже не знаю чего бы прокомментировать... Одним словом: смотрите, сравнивайте, ищите ошибки...

Java

class massiv {
    static int[][] m;
 
    public static void main(String[] args) {
 
        int a = 6, b = 7, c = 0, d = 0;
        m = new int[a][b];
        int n[] = new int[b];
 
        for (int i = 0; i < a; i++) {
            for (int j = 0; j < b; j++) {
                m[i][j] = (int) (Math.random() * 10);
            }
        }
        for (int i = 0; i < a; i++) {
            for (int j = 0; j < b; j++) {
                System.out.print(m[i][j] + " ");
            }
            System.out.println();
        }
        System.out.println();
 
        for (int i = 0; i < a; i++) {
            int max = Integer.MIN_VALUE;
            int maxI = 0;
            int maxJ = 0; // заносим в макс первый элемент, что бы с ним сравнивать
            for (int j = 0; j < b; j++) {
                if (m[i][j] > max) {
                    max = m[i][j]; //заносим в макс новый элемент
                    maxI = i;
                    maxJ = j; //заносим индекс этого элемента
                }
            }
            int p = m[i][0];  // заносим в новую переменную элемент стоящий на первой позиции
            m[i][0] = max; // заносим на место первого элемент из макс
            m[maxI][maxJ] = p; // заносим первый элемент на позицию нибольшего элемента
        }
 
        for (int i = 0; i < a; i++) {
            for (int j = 0; j < b; j++) {
                System.out.print(m[i][j] + " ");
            }
            System.out.println();
        }
    }
}

Хотя я бы написал как-нибудь вот этак:

Кликните здесь для просмотра всего текста

Java

class HelloArray {
    public static int[][] createMatrix(int n, int m) {
        int[][] array = new int[n][m];
        for (int[] temp : array) {
            for (int i = 0; i < temp.length; i++) {
                temp[i] = (int) (Math.random() * 30 - 10);
            }
        }
        return array;
    }
 
    public static int[][] replaceElements(int[][] array) {
        for (int[] temp : array) {
            int max = Integer.MIN_VALUE, indexMax = 0;
            for (int i = 0; i < temp.length; i++) {
                if (temp[i] > max) {
                    max = temp[i];
                    indexMax = i;
                }
            }
            int tmp = temp[0];
            temp[0] = temp[indexMax];
            temp[indexMax] = tmp;
        }
        return array;
    }
 
    public static void showMatrix(int[][] array) {
        for (int[] temp : array) {
            for (int tmp : temp) {
                System.out.printf("%4d", tmp);
            }
            System.out.println();
        }
    }
 
    public static void main(String[] args) {
        int[][] array = createMatrix(6, 8);
        showMatrix(array);
        System.out.println("===================================");
        showMatrix(replaceElements(array));
    }
}

ninjacut 155 / 154 / 53 Регистрация: 30.04.2016 Сообщений: 321
	16.06.2016, 16:08	2
	У тебя в цикле замена максимум должна происходить когда ты всю строку прошел, а у тебя оно происходит на каждом шаге итерации. И переменным бы давать осмысленные имена и читалось бы хоть как-то и убрал бы лишнее, вот я не понял зачем переменная c если она всегда 0. 0

	16.06.2016, 17:13