Неправильный зеленый канал в изображении

Question

Цель моей программы - сделать изображение ярче. Вот как это делается:

1. Проверяет RGB-канал пикселя
2. Находит наибольшее значение rgb ч / б все они
3. Он находит множитель для значения этого канала 255
4. Применяет множитель ко всему каналу

Код:

package Assign_4_B;

import Media.*;                  // for Picture and Sound etc.
import java.awt.*;               // for Color objects and methods
import static java.lang.Math.*;  // for math constants and functions
import static java.awt.Color.*;  // for Color constants


/** This class ...
  *
  * @author <your name>
  * @version 1.0 (<date>)                                                        */

public class Brighten {
    private PictureDisplayer display;
    private Picture pic;

    public Brighten ( ) {
      display = new PictureDisplayer();
      pic = new Picture();
      display.placePicture(pic);
      display.waitForUser();
      normIntensity(pic);
      display.close();   
    }; // constructor

    private int maxChannel(Pixel p){
      int red;
      int blue;
      int green;

      red = p.getRed();
      blue = p.getBlue();
      green = p.getGreen();

      if((red >= blue) && (red >= green)){
        return red;
      }
      else if ((blue >= red) && (blue >= green)){
        return blue;
      }
      else {return green;}
      }


    private void normIntensity (Picture img){
      Pixel a;
      int r;
      int b;
      int g;
      int hv;
      int multi;


      while(img.hasNext()){
        a = img.next();

        hv = maxChannel(a);
        multi = 255 / hv;

        r = a.getRed();
        b = a.getBlue();
        g = a.getGreen();

        a.setRed(multi * r);
        a.setBlue(multi * b);
        a.setGreen(multi * g);
      } 
    };

    public static void main ( String[] args ) { Brighten s = new Brighten(); };   
} // <className>

Проблема в моем коде в том, что он портит значение зеленого канала.

Если исходное изображение таково:

Результат, который я хочу получить, таков:

Это закончится так:

Он будет делать это только до тех пор, пока не выдаст мне этот код ошибки:

java.lang.ArithmeticException: / by zero
    at Assign_4_B.Brighten.normIntensity(Brighten.java:59)
    at Assign_4_B.Brighten.<init>(Brighten.java:23)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
    at java.lang.reflect.Method.invoke(Unknown Source)
    at edu.rice.cs.drjava.model.compiler.JavacCompiler.runCommand(JavacCompiler.java:267)

Answer 1

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

Классы, похоже, из этой библиотеки: https://www.cosc.brocku.ca/sites/all/files/documentation/Brock_packages/Media/Picture.html Но я еще не использовал эту библиотеку, и поэтому могут потребоваться некоторые корректировки в приведенном ниже коде, но он должен работать примерно так:

_{(Обновлено после некоторых комментариев:)}

import java.util.Iterator;

// for Picture and Sound etc.
import Media.Picture;
import Media.PictureDisplayer;
import Media.Pixel;

/**
 * This class ...
 *
 * @author <your name>
 * @version 1.0 (<date>)
 */

public class Brighten
{
    private PictureDisplayer display;
    private Picture pic;

    public Brighten()
    {
        display = new PictureDisplayer();
        pic = new Picture();
        display.placePicture(pic);
        display.waitForUser();
        normIntensity(pic);
        display.close();
    }; // constructor

    private int maxChannel(Picture img)
    {
        Pixel m;
        int red;
        int blue;
        int green;

        int max = 0;

        while (img.hasNext())
        {
            m = img.next();

            red = m.getRed();
            blue = m.getBlue();
            green = m.getGreen();

            if ((red >= blue) && (red >= green) && (red >= max))
            {
                max = red;
            }
            if ((blue >= red) && (blue >= green) && (blue >= max))
            {
                max = blue;
            }
            if ((green >= red) && (green >= blue) && (green >= max))
            {
                max = green;
            }
        }
        return max;
    }

    private void normIntensity(Picture img)
    {
        // Compute the maximum value of any channel over ALL pixels:
        int max = maxChannel(img);

        // Nothing to do if the image is all black...
        if (max == 0)
            return;

        // Compute the multiplier. This should probably be a double value:
        double multi = 255.0 / max;

        // Use the multiplier to scale all pixels
        Iterator<Pixel> iterator = img.iterator();
        while (iterator.hasNext())
        {
            Pixel a = iterator.next();

            int r = a.getRed();
            int g = a.getGreen();
            int b = a.getBlue();

            a.setRed((int) (multi * r));
            a.setGreen((int) (multi * g));
            a.setBlue((int) (multi * b));
        }
    };

    public static void main(String[] args)
    {
        Brighten s = new Brighten();
    };
} // <className>