[Java] среднее геометрическое Опции

[Shashlyk]

Добрый День!!!
помогите Пожалуйста написать метод, который будет в целочисленной матрице заменять все элементы средним геометрическим абсолютных значений их соседей.
Мой код, с добавлением "единичной" границы к матрице:

int SrGeom; 
     int Product; 
     for(int i = 1; i < n+1; i++){
            for(int j = 1;j < m+1; j++){
      Product = Math.abs(unitmatrix[i-1][j]) * Math.abs(unitmatrix[i+1][j]) *
               Math.abs(unitmatrix[i][j-1]) * Math.abs(unitmatrix[i][j+1]);
      SrGeom = (int)Math.pow(Product, 0.25);
            }
        }
     for (int i = 0; i < n; i++){ 
           for (int j = 0; j < m; j++){
        array[i][j] = SrGeom;       
           }
     }

Помогите Пожалуйста переделать код, с подсчётом среднего геометрического через логарифмы или экспоненты и с правильной заменой

[IUnknown]

Я бы не стал менять на экспоненту с логарифмом. Pow прекрасно справляется с задачей. Единственное, что я бы сделал - это убрал бы добавление

"единичной" границы к матрице:

. Вот так:

        public static int Count;

        int Counter() {
            Count -= 1;
            return 1;
        }

        void Action() {
            int [][] ca = new int [n][m];
            for(int i = 0; i < n; i++)
                for(int j = 0; j < m; j++) ca[i][j] = a[i][j];
            
            for(int i = 0; i < n; i++) {
                for(int j = 0; j < m; j++) {
                    Count = 4;
                    int Product = ((i > 0) ? Math.abs(ca[i-1][j]) : Counter()) *
                            ((i < n - 1) ? Math.abs(ca[i+1][j]) : Counter()) *
                            ((j > 0) ? Math.abs(ca[i][j-1]) : Counter()) *
                            ((j < m - 1) ? Math.abs(ca[i][j+1]) : Counter());
                    int SrGeom = (int)Math.pow(Product, 1.0 / Count); // Вычисляем правильное значение
                    a[i][j] = SrGeom;
                }
            }
        }

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

[Shashlyk]

Я бы не стал менять на экспоненту с логарифмом. Pow прекрасно справляется с задачей. Единственное, что я бы сделал - это убрал бы добавление . Вот так:

        public static int Count;

        int Counter() {
            Count -= 1;
            return 1;
        }

        void Action() {
            int [][] ca = new int [n][m];
            for(int i = 0; i < n; i++)
                for(int j = 0; j < m; j++) ca[i][j] = a[i][j];
            
            for(int i = 0; i < n; i++) {
                for(int j = 0; j < m; j++) {
                    Count = 4;
                    int Product = ((i > 0) ? Math.abs(ca[i-1][j]) : Counter()) *
                            ((i < n - 1) ? Math.abs(ca[i+1][j]) : Counter()) *
                            ((j > 0) ? Math.abs(ca[i][j-1]) : Counter()) *
                            ((j < m - 1) ? Math.abs(ca[i][j+1]) : Counter());
                    int SrGeom = (int)Math.pow(Product, 1.0 / Count); // Вычисляем правильное значение
                    a[i][j] = SrGeom;
                }
            }
        }

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

Скажите Пожалуйста, а как учесть ещё и "диагональные соседи" при подсчёте ?