Преобразование цветов

Question

Я пытаюсь написать метод для своей игры, который примет изображение, шестнадцатеричное значение для старого цвета и шестнадцатеричное значение для нового цвета, а затем преобразует все пиксели старого цвета в новый цвет. В настоящий момент метод окрашивает весь образ в новый цвет, как если бы оператор if вообще не работал. Этот метод:

private void convertColors(BufferedImage img, int oldColor, int newColor) 
{ 
    Graphics g = img.getGraphics(); 
    g.setColor(new Color(newColor)); 
    Color old = new Color(oldColor); 

    for(int x = 0; x < img.getWidth(); x++) 
    { 
     for(int y = 0; y < img.getHeight(); y++) 
     { 
      Color tmp = new Color(img.getRGB(x, y)); 
      if(tmp.equals(old)); 
      { 
       System.out.println("Temp=" + tmp.toString() + "Old=" + old.toString() + "New=" + g.getColor().toString()); 
       g.fillRect(x, y, 1, 1); 
      } 
     } 
    } 
    g.dispose(); 
} 

* Шестнадцатеричное для oldColor является 0xFFFFFF (белый) и для newColor является 0xFF0000 (красный).

Используя метод Println я получаю такого рода результаты:

Temp=java.awt.Color[r=0,g=0,b=0]Old=java.awt.Color[r=255,g=255,b=255]New=java.awt.Color[r=255,g=0,b=0] 
Temp=java.awt.Color[r=255,g=255,b=255]Old=java.awt.Color[r=255,g=255,b=255]New=java.awt.Color[r=255,g=0,b=0] 

scond линия выглядит правильно, цвет температура и старый одинаковы, но это явно не тот случай с первым. Я также попытался создать новый BufferedImage и скопировать пиксели, но это оставляет тот же результат ... Не работает ли метод equals, как я думаю, он делает или весь этот метод просто не работает, и есть лучший способ сделать это? Спасибо за помощь.

Answer 1

Я получил его на работу; это рабочий метод convertColors:

private BufferedImage convertColors(BufferedImage img, int oldColor, int newColor) 
{ 
    int [] pixels = new int [img.getWidth() * img.getHeight()]; 
    img.getRGB(0, 0, img.getWidth(), img.getHeight(), pixels, 0, img.getWidth()); 
    Color old = new Color(oldColor); 
    Color newC = new Color(newColor); 

    for(int x = 0; x < img.getWidth(); x++) 
    { 
     for(int y = 0; y < img.getHeight(); y++) 
     { 
      Color tmp = new Color(pixels[x + y * img.getWidth()]); 
      if(tmp.equals(old)) 
      { 
       pixels[x + y * img.getWidth()] = newC.getRGB(); 
      } 
     } 
    } 
    img.setRGB(0, 0, img.getWidth(), img.getHeight(), pixels, 0, img.getWidth()); 

    return newImg; 
} 

Answer 2

Просто удалите ; после if(tmp.equals(old)).
В противном случае вы сравниваете цвета и ничего не делаете после сравнения и всегда выбираете новый цвет.

Кроме того, что вам нужно реорганизовать ваш код немного, чтобы сделать его немного более эффективным:

Graphics g = img.getGraphics(); 
g.setColor(new Color(newColor)); 

for(int x = 0; x < img.getWidth(); x++) { 
    for(int y = 0; y < img.getHeight(); y++) { 
     if(img.getRGB(x, y) == oldColor) {//check if pixel color matches old color 
      g.fillRect(x, y, 1, 1);//fill the pixel with the right color 
     } 
    } 
} 
g.dispose(); 

Просто потому, что я был заинтересован в этой теме: полагаться на фильтры изображения вы можете сделать все, что с помощью:

class ColorSwapFilter extends RGBImageFilter { 
    int newColor, oldColor; 
    public ColorSwapFilter(int newColor, int oldColor) { 
    canFilterIndexColorModel = true; 
    this.newColor = newColor; 
    this.oldColor = oldColor; 
    } 

    @Override 
    public int filterRGB(int x, int y, int rgb) { 
    return rgb == oldColor ? newColor : oldColor; 
    } 
} 

, который должен быть вызван с помощью

BufferedImage img;//your image 
ColorSwapFilter filter = new ColorSwapFilter(...,...);//your colors to be swapped. 
ImageProducer producer = img.getSource(); 
producer = new FilteredImageSource(producer, filter); 
Image im = Toolkit.getDefaultToolkit().createImage(producer); 

Answer 3

У вас есть точка с запятой непосредственно после вашей, если заявление if(tmp.equals(old));

Это esentially говорит Java, что ваш, если оператор имеет только одну команду, связанную с ним, и эта команда является одной точкой с запятой, фактически означает «ничего не делать». Если вы удалите его, он восстановит условие на блоке кода под ним, которое прямо сейчас выполняется каждый раз независимо от состояния.