Почему не выводит средние значения строк и столбцов?

Question

[Amigo2019]

Добрый вечер!
Метод принт не выводит средние значения колонок и столбцов.Кажется чтото не так в методе принт или меан,чего то не хватает..

import java.util.Scanner;

    public class TwoDimensionalArray {
        //создание и заполнение массива
        static double[][] createRandomArray(int size) {
            double[][] da = new double[size][size];
            for(int i=0; i<da.length; i++) {
                for(int j=0; j<da[i].length; j++) {
                    da[i][j] = (Math.random()*100);
                }
            }
            return da;
        }
        //задание найти среднее арифметическое - сумма всех чисел деленная на их количество
        static double mean(double[] onearray) {
            double total=0;
            int totallength = 0;
            for(int i=0;i<onearray.length;i++) {
                 {
                    total += onearray[i];
                    totallength++;
                }
            }
            return total/(totallength);
        }
        //print
        static double[][] print(double[][] da){
            for(int i=0; i<da.length; i++) {
                for(int j=0; j<da[i].length; j++) {
                    System.out.print(da[i][j] + "\t");
                    System.out.println("Среднее значение "+"["+i+"]"+" строки: "+"["+mean(double[] (da)[i]+"]");
                    System.out.println("Среднее значение "+"["+j+"]"+" cтолбца "+"["+mean(double[] (da)[j]+"]");
                }
                System.out.println();
            }
            return da;
        }
        //майн
        public static void main(String args[]) {
            Scanner num = new Scanner(System.in);
            System.out.println ("Введите размер матрицы ");
            int size= num.nextInt();//считывает число и присваивает значение в size
            System.out.println("Вы ввели  размер "    +size);
            System.out.println("Матрица :  " );
            double[][] array=new double[size][size];
            array = createRandomArray(size);
            print(array);
        }
    }

Буду рад любой помощи! Спасибо!

Comments

[Ксения]

Это что за фраза:
mean(double[] (da)[i]+"]") ?
Вы создаете новый массив и потом хотите на нем посчитать среднее значение?

Answer 1

[Ксения]

Это мое видение ситуации. Язык C#, если что.

// MeanLineValue вернет массив, состоящий из чисел, каждое из которых – среднее значение по строке
    public float[] MeanLineValue(float[,] array)
    {
        // Создаем массив нужной длины (т.е. такой длины, сколько строк нам нужно обсчитывать)
        float[] values = new float[array.GetLength(0)];
        // Складываем значения
        for (int i = 0; i < array.GetLength(0); i++)
        {
            for (int j = 0; j < array.GetLength(1); j++)
            {
                values[i] += array[i, j];
            }
            // Ищем среднее для каждой строки
            values[i] /= array.GetLength(1);
        }
        // Возвращаем массив со средними значениями по строкам
        return values;
    }

// Создаем массив
int size = 3;
float[,] array = new float[size, size * 2];
// Заполняем рандомными значениями
for (int i = 0; i < array.GetLength(0); i++)
    {
        for (int j = 0; j < array.GetLength(1); j++)
        {
            array[i, j] = Random.value * 10f;
         }
    }
// Выводим средние по строкам
for (int i = 0; i < size; i++)
{
    Debug.Log(MeanLineValue(array)[i]);
}

Answer 2

[Денис Кузнецов]

1)

static double mean(double[] onearray) {
            double total=0;
            int totallength = 0;
            for(int i=0;i<onearray.length;i++) {
                 {
                    total += onearray[i];
                    totallength++;
                }
            }
            return total/(totallength);
        }

Если я не ошибаюсь totallength у вас будет равен onearray.length по итогу , так что наверное можно просто делить на onearray.length
2) в качестве параметра передаваемого в mean не надо передавать тип ( тип указываем только в сигнатуре самого метода) + скобочка пропущена из-за чего он не распознает число
Так не надо:

System.out.println("Среднее значение "+"["+i+"]"+" строки: "+"["+mean(double[] (da)[i]+"]");
                    System.out.println("Среднее значение "+"["+j+"]"+" cтолбца "+"["+mean(double[] (da)[j]+"]");

Так надо:

System.out.println("Среднее значение " + "[" + i + "]" + " строки: " + "["+ mean(da[i])+ "]");
                System.out.println("Среднее значение " + "[" + j + "]" + " cтолбца " + "["+ mean(da[j]) +"]");

3) в майне вы выделяете память под массив с длиной и шириной(типо используете конструктор), а потом применяете функцию createRandomArray(size) (которая типо и есть конструктор для вашей матрицы уже определенной вами) , так что вместо этого

double[][] array=new double[size][size];
            array = createRandomArray(size);

можно написать просто
double[][] array = createRandomArray(size);
4)в методе mean у вас среднее значение строки считается нормально 1 потом 2 потом 3 , а вот значение среднее столбца считается 1 при подсчете первой строки, 1 и 2 столбца отдельно при подсчете второй строки 1, 2 ,3 столбца при подсчете 3 строки, подумайте как это исправить, это не сложно
5) ну и ставьте пожалуйста пробелы между знаками и прочим)) (и разбивайте строки)

Comments

[Amigo2019]

Спасибо большое.
Но есть вопрос/просьба,вы не могли бы мне чуть помочь(сорри по джаве 1 лекция была :-()
Воспользовался почти всеми рекомендациями что смог,ещё раз спасибо,но ключевое ..что смог..:-(

import java.util.Scanner;

    public class TwoDimensionalArray {
        //создание и заполнение массива
        static double[][] createRandomArray(int size) {
            double[][] da = new double[size][size];
            for(int i=0; i<da.length; i++) {
                for(int j=0; j<da[i].length; j++) {
                    da[i][j] = (Math.random()*100);
                }
            }
            return da;
        }
        //задание найти среднее арифметическое - сумма всех чисел деленная на их количество
        static double mean(double[] onearray) {
            double total=0;
            int totallength = 0;
            for(int i=0;i<onearray.length;i++) {
                 {
                    total += onearray[i];
                    totallength++;
                }
            }
            return total/(totallength);
        }
        //print
        static double[][] print(double[][] da){
            for(int i=0; i<da.length; i++) {
                for(int j=0; j<da[i].length; j++) {
                    System.out.print(da[i][j] + "\t");
                    System.out.println("["+mean(da[i])+"]");
                    System.out.println("["+mean(da[j])+"]");
                }
                System.out.println();
            }
            return da;
        }
        //майн
        public static void main(String args[]) {
            Scanner num = new Scanner(System.in);
            System.out.println ("Введите размер матрицы ");
            int size= num.nextInt();//считывает число и присваивает значение в size
            System.out.println("Вы ввели  размер "    +size);
            System.out.println("Матрица :  " );
            double [][] array = createRandomArray(size);
            print(array);
        }
    }

Счас выходит как то так
Если вас не затруднит ещё чуть подмочь..Заранее громадное спасибо

[Ксения]

Я, может, чего-то не понимаю с массивом массива, но вот тут вы, как по мне, делаете неверно:

for(int i=0; i<da.length; i++) {
    for(int j=0; j<da[i].length; j++) {
        System.out.print(da[i][j] + "\t");
        System.out.println("["+mean(da[i])+"]");
        System.out.println("["+mean(da[j])+"]");
    }
    System.out.println();
}

Вот сюда заходите, в оба цикла и выполняете операции для каждого (!) элемента:

for(int i=0; i<da.length; i++) {
    for(int j=0; j<da[i].length; j++) {

Вот это – вывод элементов матрицы:

for(int i=0; i<da.length; i++) {
    for(int j=0; j<da[i].length; j++) {
        System.out.print(da[i][j] + "\t");
    }
}

Вот типа такого должен выглядеть – вывод среднего для строк:

for(int i=0; i<da.length; i++) {
    System.out.println("["+mean(da[i])+"]");
}

Т.е. вы методу mean должны передать полную строку. Строк всего da.length. Или сколько там?

А вот как передать столбец, если вы используете массив массивов, а не двумерный массив? Чота я не знаю.

(Кстати, почему вы используете массив массивов, а не двумерный массив? Или в Java - [][] - двумерный?)

[Ксения]

Забавно, что c# не дает такого делать:
double[][] da = new double[size][size];
Поэтому я и не знаю, зачем такая форма записи. Это массив массивов все же? Или двумерный массив?

[Ксения]

Не знаю, как это сделать лучше.
Я бы, короче, создала бы новый массив (транспонированная матрица) - поменять строки и столбцы местами.
newArray[i][j] = array[j][i];
Потом применила бы уже подсчет среднего снова на строках) Получилось бы среднее на столбцах изначального массива.

[Денис Кузнецов]

Ксения, все верно в Java - [][] - двумерный , если ваш вопрос по реализации был адресован ко мне, то я скажу сразу, ничего не менял в коде, просто сделал чтоб работало и тот цикл

for(int i=0; i<da.length; i++) {
    for(int j=0; j<da[i].length; j++) {
        System.out.print(da[i][j] + "\t");
        System.out.println("["+mean(da[i])+"]");
        System.out.println("["+mean(da[j])+"]");
    }
    System.out.println();
}

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

[Amigo2019]

Ксения, интересно,тоже вариант и довольно интересный сам подумывал..

[Amigo2019]

Денис Кузнецов,
К сожелению это надо реализвать в одном методе и у меня связаны руки
Счас вот так выдает..

[Денис Кузнецов]

Amigo2019, Ксения, Интересная реализация, весьма креативно , я бы не стал париться с транспонированной матрицей , у меня получилось так

//В строке
static void middleNumByLine(double[][] array)
    {
        double sum = 0;

        for(int i = 0; i < array.length; ++i)
        {
            for(int j = 0; j < array.length; ++j)
            {
                sum += array[i][j];
            }
            System.out.println(sum / array.length);
            sum = 0;
        }
    }

//В столбце 
    static void middleNumByTomb(double[][] array)
    {
        double sum = 0;

        for(int i = 0; i < array.length; ++i)
        {
            for(int j = 0; j < array.length; ++j)
            {
                sum += array[j][i];
            }
            System.out.println(sum / array.length);
            sum = 0;
        }
    }

как видите я просто сначала проходился по всем эл-там столбца 0 , далее столбца 1 ...

[Денис Кузнецов]

Amigo2019,
Вот пока реализация с разделением методов :

public class MT
{
    static double[][] createRandomArray(int size) {
        double[][] array = new double[size][size];
        for(int i = 0; i < array.length; i++) {
            for(int j = 0; j < array[i].length; j++) {
                array[i][j] = (Math.random() * 10);
            }
        }
        return array;
    }

    static void middleNumByLine(double[][] array)
    {
        double sum = 0;

        for(int i = 0; i < array.length; ++i)
        {
            for(int j = 0; j < array.length; ++j)
            {
                sum += array[i][j];
            }
            System.out.println(sum/array.length);
            sum = 0;
        }
    }

    static void middleNumByTomb(double[][] array)
    {
        double sum = 0;

        for(int i = 0; i < array.length; ++i)
        {
            for(int j = 0; j < array.length; ++j)
            {
                sum += array[j][i];
            }
            System.out.println(sum/array.length);
            sum = 0;
        }
    }


    static void print(double[][] array)
    {
        for (int i = 0; i < array.length; ++i)
        {
            for(int j = 0; j < array.length; ++j)
            {
                System.out.print(array[i][j] + " ");
            }
            System.out.println();
        }
    }

    public static void main(String[] args) {
        Scanner num = new Scanner(System.in);
        System.out.println ("Введите размер матрицы ");
        int size = num.nextInt();
        double[][] array = createRandomArray(size);

        print(array);

        System.out.println("________________");
        middleNumByLine(array);

        System.out.println("________________");
        middleNumByTomb(array);
    }
}

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

[Ксения]

Денис Кузнецов, я на C# писала ответ, сделала так же, как вы предложили. Отдельный метод для строк, отдельный для столбцов. Это как-то проще для моей головы :)

[Amigo2019]

Ксения,
Мне тоже но это всё требуют в одном методе и это не шутка
и Вывод и средние
Поэтому и проблема

[Денис Кузнецов]

Ксения,
в Java не совсем понятно, как задается массив , ибо все максимально лаконично
int[][] array = new int[size][size];
гораздо понятнее все становится если взглянуть на это в С++

int **arr = new int*[n];
	for (int i = 0; i < n; ++i)
	{
		arr[i] = new int[n];
	}

то есть фактически мы создаем линию чисел(массив), а потом под каждым числом создаем столбец

[Ксения]

К сожелению это надо реализвать в одном методе и у меня связаны руки
Счас вот так выдает..

А как именно задание звучит? Мб вы себя ограничиваете. В одном методе можно все это сделать, но разделить же никто не запрещает подсчет и вывод в рамках одного метода?

[Ксения]

Денис Кузнецов, в c# как раз либо двумерный так, например: float[,] array = new float[size, size]. И это называется двумерным массивом.
Либо float[][] array = new float[size, ];
И тогда необходимо сделать такой же финт ушами :)
for (int i = 0; i < size; i++) {
array[i] = new float[size];
}
И это называется массив массивов уже :) Потому что каждую строку можно делать разной длины, по идее.

[Денис Кузнецов]

Ксения, а кстати вот для реализации всего в одном методе транспонированная матрица наверное будет удобным решением

[Ксения]

Денис Кузнецов, ну да. Иначе я не знаю, как это сделать? Не совсем понятно, какое там ограничение в задаче. Один метод, который считает среднее и по строкам и по столбцам?

[Денис Кузнецов]

Ксения, Amigo2019 ,это канеш можно сделать, но придется в пустую прогонять циклы
В общем вот решение с выводом в одном методе

static double[][] transpose(double[][] array)
    {
        double[][] tmatrix = new double[array.length][array.length];

        for(int i = 0; i < array.length; ++i)
        {
            for(int j = 0; j < array.length; ++j)
            {
                tmatrix[j][i] = array[i][j];
            }
        }

        return tmatrix;
    }

    static void print(double[][] array)
    {
        double sumLine = 0;
        double sumTomb = 0;

        double[][] tmatrix = transpose(array);

        for (int i = 0; i < array.length; ++i)
        {
            for(int j = 0; j < array.length; ++j)
            {
                sumTomb += tmatrix[i][j];
                sumLine += array[i][j];
                System.out.print(array[i][j] + " ");
            }
            System.out.println("MiddleNumLine " + sumLine / array.length + " MiddleNumTomb " + sumTomb / array.length);
            sumLine = 0;
            sumTomb = 0;
        }
    }

    public static void main(String[] args) {
        Scanner num = new Scanner(System.in);
        System.out.println ("Введите размер матрицы ");
        int size = num.nextInt();
        double[][] array = createRandomArray(size);

        print(array);
    }

[Ксения]

Денис Кузнецов, решение-то рабочее, но мне до сих пор не понятно, почему в учебной задаче такое странное ограничение)))

[Денис Кузнецов]

Ксения, ахахаха, у меня вообще препод по функциональному программированию и языку Haskell дал задание, допилить в проге определенный функционал, и я как ни пытался найти решение сам, как ни спрашивал на форумах ничего не получалось. Причем перейти к следующим задачам он мне не давал, и говорил мол время есть, ищи, пробуй. Под конец семестра ничего не изменилось (а эта задача была пороговой на допуск к экзу) , и тогда он на последнем занятии сам сел, и под конец пары заключил, что такое в принципе выполнить невозможно))))))

[Amigo2019]

Это дали после первой лекции и первой практической
https://yadi.sk/i/RQIsdSFMVndK0Q
Счас код такой

import java.util.Scanner;

    public class TwoDimensionalArray {
        //создание и заполнение массива
        static int[][] createRandomArray(int size) {
            int[][] da = new int[size][size];
            for(int i=0; i<da.length; i++) {
                for(int j=0; j<da[i].length; j++) {
                    da[i][j] = (int) (Math.random()*10);
                }
            }
            return da;
        }
        //среднее
        static int mean(int[] onearray) {
            int total=0;
            int totallength = 0;
            for(int i=0;i<onearray.length;i++) {
                 {
                    total += onearray[i];
                    totallength++;
                }
            }
            return total/(totallength);
        }
        //print
        static int[][] print(int[][] da){
            for(int i=0; i<da.length; i++) {
                for (int j = 0; j < da[i].length; j++) {
                    System.out.print(da[i][j] + "\t");
                }
                System.out.println("["+mean(da[i])+"]");
            }

            return da;
        }
        //майн
        public static void main(String args[]) {
            Scanner num = new Scanner(System.in);
            System.out.println ("Введите размер матрицы ");
            int size= num.nextInt();//считывает число и присваивает значение в size
            System.out.println("Вы ввели  размер "    +size);
            System.out.println("Матрица :  " );
            int [][] array = createRandomArray(size);
            print(array);
        }
    }

И выглядит так,попробую завтра транспонирование в принт засунуть

[Amigo2019]

Почти

[Amigo2019]

ПОЛУЧИЛОСЬ!!!!!!!!!!!!
ВСЕМ СПАСИБО!!!!!!!!!!!!!!!!

[Денис Кузнецов]

Amigo2019, поздравляю))