Форум программистов
 

Восстановите пароль или Зарегистрируйтесь на форуме, о проблемах и с заказом рекламы пишите сюда - alarforum@yandex.ru, проверяйте папку спам!

Вернуться   Форум программистов > Java программирование > Общие вопросы по Java, Java SE, Kotlin
Регистрация

Восстановить пароль
Повторная активизация e-mail

Купить рекламу на форуме - 42 тыс руб за месяц

Ответ
 
Опции темы Поиск в этой теме
Старый 02.08.2018, 06:51   #1
Nastya2018
Форумчанин
 
Регистрация: 24.07.2018
Сообщений: 133
По умолчанию Подскажите что у меня не правильно в коде

Нужно создать метод Keyboard чтобы могла с клавиатуры заполнять количество строк и столбцов, а так же каждую ячейку заданной матрицы. Вроде написала может что то не правильно делаю)

Код:
/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package matrix;

import java.util.Random;
import java.util.Scanner;


public class MatrixUtils {

    public static void fillByRandom(Matrix m) {
        int rowCount = m.getRowCount();
        int colCount = m.getColCount();
        Random random = new Random();
        
        for (int i = 1; i <= rowCount; i++) {
            

            for (int j = 1; j <= colCount; j++) {
                try {
                    m.put(i, j, random.nextInt(50));
                } catch (MatrixIndexException e) {
                     System.err.println(e.getMessage());

                }

            }
        }
    }

    public static void fillByNumber(Matrix m, int number) {
        int rowCount = m.getRowCount();
        int colCount = m.getColCount();

        for (int i = 1; i <= colCount; i++) {

            for (int j = 1; j <= rowCount; j++) {
                try {
                    m.put(i,j, number);
                    

                } catch (MatrixIndexException e){
                     System.err.println(e.getMessage());
                     
                    }

                }
            }

        }
    
    public static void fillByKeyboard(Matrix m) {
        int row, col, i, j;
        int arr[][] = new int[10][10]
        
        Scanner scan = new Scanner(System.in);

        System.out.print("Enter row for the array (max 10) : ");
        row = scan.nextInt();
        System.out.print("Enter column for the array (max 10) : ");
        col = scan.nextInt();

        System.out.println("Enter " + (row * col) + " Array Elements : ");
        for (i = 0; i < row; i++) {
            for (j = 0; j < col; j++) {
                arr[i][j] = scan.nextInt();
            }
        }

        System.out.print("The Array is :\n");
        for (i = 0; i < row; i++) {
            for (j = 0; j < col; j++) {
                System.out.print(arr[i][j] + "  ");
            }
            System.out.println();
        }
    }
}
Код:
/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package matrix;

public class Matrix {
    
 
    private int rowCount;
    private int colCount;
    private int[][] data;

    

    Matrix(int row, int col) throws MatrixIndexException {
        if (row <= 0 || col <= 0) {
            throw new MatrixIndexException("Недопустимый размер матрицы.");
        }
        this.rowCount = row;
        this.colCount = col;
        data = new int[row][col];
    }

    Matrix(Matrix matrix) {

        this.rowCount = matrix.getRowCount();
        this.colCount = matrix.getColCount();
        data = new int[rowCount][colCount];

        for (int i = 1; i < rowCount; i++) { 
            for (int j = 1; j < colCount; j++) {
                data[i][j] = matrix.data[i][j];
            }
        }
    }

    public int get(int i, int j) throws MatrixIndexException {
        if (i < 1 || i > rowCount) {  
            throw new MatrixIndexException("Недопустимое число строк: " + i);
        }
        if (j < 1 || j > colCount) {
            throw new MatrixIndexException("Недопустимое число столбцов: " + j);
        }
        
        return data [i-1][j-1];
    }

    void put(int i, int j, int value) throws MatrixIndexException {
        if (i < 1 || i > rowCount) {
            throw new MatrixIndexException("Недопустимое число строк: " + i);
        }
        if (j < 1 || j > colCount) {
            throw new MatrixIndexException("Недопустимое число столбцов: " + j);
        }
        data[i-1][j-1] = value;


    }

    public int getRowCount() {
        return rowCount;
    }

    public int getColCount() {
        return colCount;
    }

    @Override
    public boolean equals(Object obj) {
        Matrix m = (Matrix) obj;

        if (m.getRowCount() != rowCount || m.getColCount() != colCount) {
            return false;
        }

        for (int i = 0; i < rowCount; i++) {
            for (int j = 0; j < colCount; j++) {
                if (data[i][j] != m.data[i][j]) {
                    return false;
                }
            }
        }

        return true;
    }

    @Override
    public String toString() {
        StringBuilder out = new StringBuilder();
        out.append("Matrix:\n[ ");
        for (int i = 0; i < rowCount; i++) {
            if (i != 0) {
                out.append("\n");
                out.append("  ");
            }
            for (int j = 0; j < colCount; j++) {
                out.append(data[i][j]);
                if (j == colCount - 1) {
                    continue;
                }
                for (int k = 0; k < getMaxLength() - getIntLength(data[i][j]) + 2; k++) {
                    out.append(" ");
                }
            }
        }
        out.append("  ]");
        return out.toString();
    }

    private int getMaxLength() {
        int max = Integer.MIN_VALUE;
        for (int i = 0; i < rowCount; i++) {
            for (int j = 0; j < colCount; j++) {
                int k = data[i][j];
                if (k > max) {
                    max = k;
                }
            }
        }
        return getIntLength(max);
    }

    private int getIntLength(int i) {
        return String.valueOf(i).length();
    }

}

Код:
/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package matrix;

import java.util.LinkedList;
import java.util.List;


public class Main extends Exception {

        
        
        Matrix m1 = new Matrix(4, 4);        
        MatrixUtils.fillByRandom(m1);
        System.out.println(m1.toString());
        
        Matrix m3 = new Matrix(5, 5);
        MatrixUtils.fillByNumber(m3, 1);
        System.out.println(m3.toString());
        
        
       
       
        
        /*Matrix m1 = new Matrix(3, 3);
        Matrix m4 = new Matrix(100, 10);
        
        try {
            m1.get(1, 1);

        } catch (MatrixIndexException e) {
            System.out.println(e.getMessage());
        }

        System.out.println();

        System.out.println(m1.toString());

        Matrix m2 = new Matrix(m1);

        System.out.println(m2.toString());

        System.out.println(m1.equals(m2));

        Matrix m3 = new Matrix(2, 2);
        try {
            m3.put(1, 1, 10);
            m3.put(1, 2, 53);
            m3.put(2, 1, 20);
            m3.put(2, 2, 106);
        } catch (MatrixIndexException e) {
            System.out.println("Out of index");
        }
        System.out.println(m3.toString());
        System.out.println(m1.equals(m3));*/
    }
}
Nastya2018 вне форума Ответить с цитированием
Старый 03.08.2018, 09:36   #2
Serge_Bliznykov
Старожил
 
Регистрация: 09.01.2008
Сообщений: 26,238
По умолчанию

во-первых, чем Вас не устроило заполнение в коде #1 ?
кроме неэффективного выделения памяти под массив, собственно ошибок в заполнении матрицы я не вижу.

во-вторых, определитесь, Вы делаете статический метод, где двухмерный массив задаётся в самом методе, или Вы описываете класс Matrix и создаёте методы у этого класса.
Serge_Bliznykov вне форума Ответить с цитированием
Старый 03.08.2018, 09:57   #3
Serge_Bliznykov
Старожил
 
Регистрация: 09.01.2008
Сообщений: 26,238
По умолчанию

посмотрел ваш код, понял, что Вы творите смесь бульдога с носорогом!
у Вас и класс и статические методы, куда Вы класс передаёте.

не знаю, "скушает" ли Java, если передать в метод неиницилизированный класс:

Код:
    Matrix m1;
    fillByKeyboard(m1);
если скушает, тогда вот Вам статический метод:
Код:
    public static void fillByKeyboard(Matrix m) {


        int row, col;

        Scanner scan = new Scanner(System.in);
        System.out.print("Enter row count for the array: ");
        row = scan.nextInt();
                
        System.out.print("Enter column count for the array: ");
        col = scan.nextInt();
       
        
        // int[][] arr = new int[row][col];
        m = new Matrix(row, col);
        
        for (int i = 0; i < row; i++) {
            for (int j = 0; j < col; j++) {
                System.out.print("Enter element arr[" + i + "][" + j + "]:");
                int next = scan.nextInt();
                m.put(i, j, next);
            
            }   
        }
        scan.close();

    }

если так нельзя, то можно попробовать сделать так, чтобы статический метод возвращал экземпляр класса:
Код:
    public static Matrix fillByKeyboard() {

        int row, col;

        Scanner scan = new Scanner(System.in);
        System.out.print("Enter row count for the array: ");
        row = scan.nextInt();
                
        System.out.print("Enter column count for the array: ");
        col = scan.nextInt();
        
        Matrix m = new Matrix(row, col);
        
        for (int i = 0; i < row; i++) {
            for (int j = 0; j < col; j++) {
                System.out.print("Enter element arr[" + i + "][" + j + "]:");
                int next = scan.nextInt();
                m.put(i, j, next);
            
            }   
        }
        scan.close();

        return m;

    }
тогда вызывать надо так:
Код:
Matrix m1 = fillByKeyboard();

Последний раз редактировалось Serge_Bliznykov; 03.08.2018 в 10:11.
Serge_Bliznykov вне форума Ответить с цитированием
Старый 03.08.2018, 14:21   #4
Nastya2018
Форумчанин
 
Регистрация: 24.07.2018
Сообщений: 133
По умолчанию

Cпасибо вам) но я после обеда сделала сама) но все же еще раз спасибо)
Nastya2018 вне форума Ответить с цитированием
Ответ


Купить рекламу на форуме - 42 тыс руб за месяц

Опции темы Поиск в этой теме
Поиск в этой теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Что не правильно в коде? michael.lion Помощь студентам 2 09.08.2017 14:18
Не знаю что у меня не правильно вроде все написал но что-то не хочет выполнятся помогите исправить Денис123321 Паскаль, Turbo Pascal, PascalABC.NET 5 16.03.2017 09:23
Подскажите, что у меня не так _Freak_ Общие вопросы Delphi 6 13.10.2013 02:39
Подскажите, что не правильно? Татьяна_16 Помощь студентам 1 10.06.2011 15:09
Динамические массивы в Борланд Си... (Что у меня не правильно в программе?) Алекс...))) Помощь студентам 2 09.02.2009 21:47