Your code is not playable, but to work, the image needs to be in the package img in the same class level within your project in the IDE. If there is no package img, the image will not be located and will burst nullpointer. Check your project if the package exists and is at the same level of the class you are trying to access it.To exemplify the hierarchy in a package nomination project (not recommended): Being like this, class and package img within the same package, the code runs without errors.As mentioned in the comments and as you can see in the figure, you are using a hierarchy of packages proper in the eclipse, and the ideal is always to inform the complete path in your case would be:getClass().getResource("/br/com/williamcasa/servicodeordenacao/icons/iconOrganize.png");

This line here is the cause of error:add(board,buttons); You didn't put a complete code, so I'm assuming that class Board be some other component you have created.If you look at the class documentation https://docs.oracle.com/javase/8/docs/api/javax/swing/JFrame.html , will see that none of the methods add that she inherits receives 2 components at the same time, that line is wrong. If you want to add board and buttons on JFrame, need to be one at a time, you need to break this dictation:add(board); add(buttons); Only that would only still make a mistake because the class Buttons is a JFrame and you cannot add a JFrame within another. You will have to transform these classes Buttons and Board in https://docs.oracle.com/javase/8/docs/api/javax/swing/JPanel.html if you want to add them that way.It is worth mentioning the recommendation below:Avoid using absolute layout unless it is of extreme need and know the consequences of this, because absolute layout makes it difficult to maintain the screen and makes its application look different depending on the monitor and resolution being performed.There are several https://docs.oracle.com/javase/tutorial/uiswing/layout/visual.html so you don't have to worry about positioning or manual component organization. Not counting that the use of layouts makes your code easier to give maintenance than being inserting a lot of setbounds, and if you need to change the position of some component, in the absolute layout, you will have to reposition all manually.And it is also worth noting that all graphical applications involving the swing API should be https://pt.stackoverflow.com/q/2017/28595 .I recommend for study: https://docs.oracle.com/javase/tutorial/uiswing/index.html

It worked. The changes were like this:public class NewClass { public void genericMethod() { JFrame frame = new JFrame("JFrame Example"); JPanel panel = new JPanel(); panel.setLayout(new FlowLayout()); JButton button = button = new JButton(new AbstractAction("Botao") { @Override public void actionPerformed(ActionEvent e) { ativarMetodos(); } }); panel.add(button); frame.add(panel); frame.setSize(300, 300); frame.setVisible(true); } public void metodoQualquer() { try { Thread.sleep(10000); } catch (Exception e) { } } public static void main(String s[]) { EventQueue.invokeLater(new Runnable() { @Override public void run() { new NewClass1().genericMethod(); } }); // new NewClass().genericMethod(); } public void ativarMetodos() { SwingWorker worker = new SwingWorker() { @Override protected Object doInBackground() throws Exception { WaitSplash wait; wait = new WaitSplash(); wait.setVisible(true); wait.requestFocus(); metodoQualquer(); wait.dispose(); return null; } @Override protected void done() { super.done(); try { try { get(); } catch (InterruptedException ex) { Logger.getLogger(NewClass1.class.getName()).log(Level.SEVERE, null, ex); } catch (ExecutionException ex) { Logger.getLogger(NewClass1.class.getName()).log(Level.SEVERE, null, ex); } } catch (Exception e) { } } }; worker.execute(); }

Try the following:jButton.setComponentOrientation(ComponentOrientation.RIGHT_TO_LEFT); jButton.setHorizontalAlignment(SwingConstants.RIGHT); jButton.setHorizontalTextPosition(SwingConstants.LEFT); Source: https://stackoverflow.com/a/36821976/5524514 I did some tests here and it took only the line below to get the arrow to the right of the text. Take the test yourself, replacing the path of the two buttons of the code below and pointing to your icon:import java.awt.FlowLayout; import java.io.IOException; import javax.imageio.ImageIO; import javax.swing.*; public class JButtonAlinhaTextIconTest { public void genericMethod() { JFrame frame = new JFrame("JFrame Example"); JPanel panel = new JPanel(); panel.setLayout(new FlowLayout()); JButton button1 = new JButton(); JButton button2 = new JButton(); button1.setText("button1"); button2.setText("button2"); try { button1.setIcon(new ImageIcon(ImageIO.read(getClass().getResource("/res/arrow-right-black.png")))); button2.setIcon(new ImageIcon(ImageIO.read(getClass().getResource("/res/arrow-right-black.png")))); } catch (IOException e) { e.printStackTrace(); } button1.setHorizontalTextPosition(SwingConstants.LEFT); panel.add(button1); panel.add(button2); frame.add(panel); frame.setSize(300, 300); frame.setVisible(true); } public static void main(String s[]) { SwingUtilities.invokeLater(() -> new JButtonAlinhaTextIconTest().genericMethod()); } } Only the button1 was applied the configuration and only he aligned the icon to the left. See the result I got:

To change the color, just use https://docs.oracle.com/javase/9/docs/api/javax/swing/JComponent.html#setBackground-java.awt.Color- passing the desired color through the class https://docs.oracle.com/javase/9/docs/api/java/awt/Color.html , using your constants or creating custom colors:botao.setBackground(Color.red); But most https://docs.oracle.com/javase/tutorial/uiswing/lookandfeel/plaf.html already has default color settings and even changing the above method, it does not allow to change correctly in some cases. To solve this, you need to "remove" this component stylization through the methods below: botao.setContentAreaFilled(false); botao.setOpaque(true); botao.setBackground(Color.RED); Result: Note: By doing this, you are giving up all the stylings made by the LAF, and not only of the background and foreground colors.

Try: button.setText("<html>1<br>Ex</html>"); Swing components allow you https://docs.oracle.com/javase/tutorial/uiswing/components/html.html inside them. It is a solution, although in particular, to find that this form, if used with a lot of volume in a code, leaves it difficult to read and maintain.

First that you are adding a button practically on top of the other, this is the problem of using absolute layout without being absolutely sure how to use and its consequences. The method setBounds works with coordinates and dimension, where the first two arguments represent, respectively, the position x and y ( https://pt.wikipedia.org/wiki/Sistema_de_coordenadas_cartesiano ) the component on the screen, and the remaining two its width and height. Only by adjusting the position of the second as the height of the first, it is possible to display both. But I insist on recommending:Avoid using absolute layout unless it is of extreme need and know the consequences of this, because absolute layout makes it difficult to maintain the screen and makes its application look different depending on the monitor and resolution being performed.There are several https://docs.oracle.com/javase/tutorial/uiswing/layout/visual.html so you don't have to worry about positioning or manual component organization. Not counting that the use of layouts makes your code easier to give maintenance than being inserting a lot of setbounds, and if you need to change the position of some component, you will have to position all manually.Then you overwrite the method paint, and the documentation says that http://www.oracle.com/technetwork/java/painting-140037.html#swing in swing programs. If you want to make drawings on the screen, overwrite the method paintComponent from some JPanel and add it to the screen. The cause of a button not to be displayed is precisely the fact that it overwritten this method.Another problem I noticed is the fact that you do not dispatch the screen to the https://docs.oracle.com/javase/tutorial/uiswing/concurrency/dispatch.html , recommend that https://pt.stackoverflow.com/questions/2017/aplica%C3%A7%C3%A3o-swing-por-que-o-m%C3%A9todo-main-deve-despachar-a-cria%C3%A7%C3%A3o-da-gui-para-a-ed/2095#2095 the importance of this and the consequences that may occur when it is not done and https://pt.stackoverflow.com/a/193456/28595 shows some ways to start the application within this Thread.You have more problems in that code, but you run away from doubt by getting everything done, so I just adapted what was pointed out in the answer.import java.awt.Color; import java.awt.EventQueue; import java.awt.Graphics; import javax.swing.*; public class LDEGraphic extends JFrame { JButton jb = new JButton("Adicionar"); JButton jb2 = new JButton("Remover"); JPanel painel; public LDEGraphic() { setTitle("Lista Duplamente encadeada"); setSize(900, 600); setLocationRelativeTo(null); setDefaultCloseOperation(EXIT_ON_CLOSE); setVisible(true); painel = new JPanel(){ @Override protected void paintComponent(Graphics g) { super.paintComponent(g); g.setColor(Color.BLACK); g.drawRect(100, 300, 60, 25); g.fillRect(100, 300, 60, 25); } }; painel.setLayout(null); jb.setBounds(10, 10, 100, 60); painel.add(jb); jb2.setBounds(10, 70, 120, 60); painel.add(jb2); getContentPane().add(painel); } public static void main(String[] args) { EventQueue.invokeLater(() -&gt; new LDEGraphic()); } }

Its approach is not very good, as well as creating JFrames multiplos(which in most cases is not necessary, only https://docs.oracle.com/javase/9/docs/api/javax/swing/JDialog.html would already serve for this case), you are still creating a completely new instance of the main screen to validate something defined in another instance. The variable salvar is instance, creating a new object in the Register class, you will not be able to recover its value.A better approach is to create constant publicas, and from them, validate the botoes, passing as argument for the class Cadastro:public class TelaPrincipal extends javax.swing.JFrame { public TelaPrincipal() { initComponents(); this.setResizable(false); this.setLocationRelativeTo(null); } private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) { new Cadastro(Cadastro.CADASTRAR).setVisible(true); dispose(); } private void jButton2ActionPerformed(java.awt.event.ActionEvent evt) { new Cadastro(Cadastro.EDITAR).setVisible(true); dispose(); } } And in the Register class: public class Cadastro extends javax.swing.JFrame { public static final int EDITAR = 0; public static final int CADASTRAR = 1; public Cadastro(int tipo) { this.setResizable(false); initComponents(); this.setLocationRelativeTo(null); jButton1.setEnabled(tipo == CADASTRAR); jButton3.setEnabled(tipo == EDITAR);} } //no caso o botão 1 sempre está desligado e o 3 ligado não importa o valor de salvar. }

Edit.: I optimized the class so that it is not limited to only 3 colors, but the amount you find necessary.Just create a method in class MultiColorButton that receives the color array(or the 3 individual colors), assign them to a color array and force the repaint() from the button after assigning the colors in the array:class MultiColorButton extends JButton { private static final long serialVersionUID = 1L; Color[] colors; public MultiColorButton() { this(new Color[] { new Color(63, 72, 204), new Color(181, 230, 29), new Color(237, 28, 36) }); } public MultiColorButton(Color[] colors) { setContentAreaFilled(false); setFocusPainted(false); this.setColors(colors); } public void setColors(Color[] colors) { this.colors = colors; repaint(); } @Override protected void paintComponent(Graphics g) { int colorsCount = colors.length; if (colorsCount &gt; 0) { int fractionWitdh = getWidth() / colorsCount; for (int i = 0; i &lt; colorsCount; i++) { g.setColor(colors[i]); g.fillRect(fractionWitdh * i, 0, fractionWitdh, getHeight()); } } super.paintComponent(g); } } To illustrate the form of use, I made this example that generates random colors for the button, each time it is clicked:import java.awt.BorderLayout; import java.awt.Color; import java.awt.EventQueue; import java.awt.Graphics; import java.awt.Graphics2D; import java.awt.LinearGradientPaint; import java.util.Random; import javax.swing.JFrame; import javax.swing.JPanel; import javax.swing.border.EmptyBorder; import javax.swing.plaf.multi.MultiRootPaneUI; import javax.swing.JButton; import java.awt.Dimension; public class JButtonMultipleColorsTest extends JFrame { private JPanel contentPane; private JPanel panel; private MultiColorButton multiColorButton; private GradientButton gradientButton; public static void main(String[] args) { EventQueue.invokeLater(new Runnable() { public void run() { try { JButtonMultipleColorsTest frame = new JButtonMultipleColorsTest(); frame.setVisible(true); } catch (Exception e) { e.printStackTrace(); } } }); } public JButtonMultipleColorsTest() { initComponents(); pack(); setLocationRelativeTo(null); } private void initComponents() { setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); setPreferredSize(new Dimension(300, 200)); this.contentPane = new JPanel(); this.contentPane.setBorder(new EmptyBorder(5, 5, 5, 5)); this.contentPane.setLayout(new BorderLayout(0, 0)); setContentPane(this.contentPane); this.panel = new JPanel(); this.contentPane.add(this.panel, BorderLayout.CENTER); this.multiColorButton = new MultiColorButton(); this.multiColorButton.addActionListener(e -&gt; { Random rand = new Random(); Color[] colors = new Color[3]; for(int i = 0; i &lt; colors.length; i++) colors[i] = new Color(rand.nextInt(256), rand.nextInt(256), rand.nextInt(256)); multiColorButton.setColors(colors); }); this.multiColorButton.setText("multicolor btn"); this.panel.add(this.multiColorButton); } class MultiColorButton extends JButton implements iColorsButton { private static final long serialVersionUID = 1L; Color[] colors; public MultiColorButton() { this(new Color[] { new Color(63, 72, 204), new Color(181, 230, 29), new Color(237, 28, 36) }); } public MultiColorButton(Color[] colors) { setContentAreaFilled(false); setFocusPainted(false); this.setColors(colors); } public void setColors(Color[] colors) { this.colors = colors; repaint(); } @Override protected void paintComponent(Graphics g) { if (colors != null &amp;&amp; colors.length &gt; 0) { int colorsCount = colors.length; int fractionWitdh = getWidth() / colorsCount; for (int i = 0; i &lt; colorsCount; i++) { g.setColor(colors[i]); g.fillRect(fractionWitdh * i, 0, fractionWitdh, getHeight()); } } super.paintComponent(g); } } } See working:

Edit.: I have optimized the two classes so that they are not limited to only 3 colors, but the amount they find necessary.Without using gradient, applying the color division exactly as you explained:class MultiColorButton extends JButton { private static final long serialVersionUID = 1L; Color[] colors; public MultiColorButton() { this(new Color[] { new Color(63, 72, 204), new Color(181, 230, 29), new Color(237, 28, 36) }); } public MultiColorButton(Color[] colors) { setContentAreaFilled(false); setFocusPainted(false); this.setColors(colors); } public void setColors(Color[] colors) { this.colors = colors; repaint(); } @Override protected void paintComponent(Graphics g) { if (colors != null &amp;&amp; colors.length &gt; 0) { int colorsCount = colors.length; int fractionWitdh = getWidth() / colorsCount; for (int i = 0; i &lt; colorsCount; i++) { g.setColor(colors[i]); g.fillRect(fractionWitdh * i, 0, fractionWitdh, getHeight()); } } super.paintComponent(g); } } Result: Although not mentioned in the question, here I leave an alternative with linear gradient, which leaves the transition of colors more beautiful:class GradientButton extends JButton { private static final long serialVersionUID = 1L; Color[] colors; public GradientButton() { this(new Color[] { new Color(63, 72, 204), new Color(181, 230, 29), new Color(237, 28, 36) }); } public GradientButton(Color[] colors) { setContentAreaFilled(false); setFocusPainted(false); this.colors = colors; } @Override public void setColors(Color[] colors) { this.colors = colors; repaint(); } @Override protected void paintComponent(Graphics g) { final Graphics2D g2d = (Graphics2D) g; float[] fractions = new float[colors.length]; for (int i = 1; i &lt;= fractions.length; i++) { float fraction = 1.0f / fractions.length; fractions[i - 1] = i * fraction; } g2d.setPaint(new LinearGradientPaint(0, 0, getWidth(), getHeight(), fractions, colors)); g2d.fillRect(0, 0, getWidth(), getHeight()); super.paintComponent(g); } } Result: The use is like that of a common JButton.Your choice your best option

You can do this:private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) { boolean isVisible; if(jButton1.getText().equals.("Créditos")) { isVisible = false; jButton1.setText(" Voltar "); } else { isVisible = true; jButton1.setText("Créditos") } pnBusca.setVisible(isVisible); pnOpcao.setVisible(isVisible); btnGerar.setVisible(isVisible); } Remembering that the ideal is always to rescue the button from the ActionEvent. Simplifying the code a little, it could look like this:private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) { JButton btn = (JButton)evt.getSource(); boolean isVisible = btn.getText().equals.("Créditos"); btn.setText(isVisible ? " Voltar " : "Créditos"); pnBusca.setVisible(!isVisible); pnOpcao.setVisible(!isVisible); btnGerar.setVisible(!isVisible); }

Normally, it works by having the image in the same folder as the compiled file (.class). I tested your code with any image and worked perfectly (compiling "in the hand"). But, doing this way, the image will not be part of . jar generated, it only serves for testing.The recommended is to place your images in the folder Resources/Images and modify the code for something like:Icon imageIcon = new ImageIcon(Main.class.getResource("/Images/flag.png"));

A simple way is to define Action Commands using the method https://docs.oracle.com/javase/9/docs/api/java/awt/Button.html#setActionCommand-java.lang.String- :JButton b1 = new JButton("B1"); JButton b2 = new JButton("B2"); ActionListener evento = new Eventos(); b1.setActionCommand("botao1"); b1.setActionCommand("botao2"); b1.addActionListener(evento); b2.addActionListener(evento); (...) class Eventos implements ActionListener{ public void actionPerformed(ActionEvent e) { String action = e.getActionCommand(); switch(action) { case "botao1": //Ação do botao 1 break; case "botao2": //Ação do botao 2 break; } } } The advantage of using action command rather than button.getText() is that you can freely change the text of the buttons without having to update the listener. Remembering that this option is not restricted to only 2 buttons, others can be added, you just have to add case No switch, but in case of many buttons, there are better solutions to identify botoes than using switch, https://pt.stackoverflow.com/a/134554/28595 has an example using HashMap.

You have to understand the concept of thread. AWT and Swing run on a thread called EDT (see more https://pt.stackoverflow.com/a/2095/132 and also https://pt.stackoverflow.com/q/115168/132 ). EDT is responsible for receiving operating system events (mouse, keyboard, drag windows, maximize, minimize, redesign the screen, etc). EDT is also the thread that responds to events such as a click on a button.It is important for the application to be responsive, never overload EDT with heavy tasks. You should always make the best effort to get her free to do what her purpose is and nothing else. That is, tasks like reading or writing files, doing complicated operations in loop, running batch operations, uploading or downloading data on the internet, etc. from within EDT is a terrible idea.What happens is that you are locking EDT in an infinite loop. Therefore, it will be stuck in the loop and will not be able to process events from redesigning the screen and even receive the click on the other button. The solution in your case is to use another thread to release EDT:private volatile Thread runner; private void loop() { while (runner == Thread.currentThread()) { System.out.print("Olá"); } } private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) { if (runner != null) return; runner = new Thread(this::loop); runner.start(); } private void jButton2ActionPerformed(java.awt.event.ActionEvent evt) { runner = null; }

For what I understand setEnabled(false) changes the appearance of the same button, it has no way. But try this:private void Button1_1ActionPerformed(java.awt.event.ActionEvent evt) { if (Button1_1.getBackground() != Color.CYAN && Button1_1.getBackground() != Color.GREEN) { if(TextoJogador.getText().equals("Jogador 1")){ Button1_1.setBackground(Color.CYAN); TextoJogador.setText("Jogador 2"); play(sound); }else{ Button1_1.setBackground(Color.GREEN); TextoJogador.setText("Jogador 1"); play(sound); } } }

To add actions to buttons (assuming they are JButtons by your code), just add one http://docs.oracle.com/javase/7/docs/api/java/awt/event/ActionListener.html as example below:jButton1.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { //aqui você adiciona o código com o a ação //que será executada quando o botão for clicado } }); For cases where the excerpt of the action is a little more complex, you can create a method the part and just call it inside https://pt.stackoverflow.com/questions/121163/porque-nova-instancia-da-actionlistener-recebe-um-parenteses-e-um-ponto-e-v%C3%ADrg/121165#121165 (passing) ActionEvent as a parameter is optional in case you are to make use of some information from the clicked component):jButton1.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { //chama o método com a ação do botão ButtonHandler(evt); } }); //... private void ButtonHandler(java.awt.event.ActionEvent evt) { //ação do botão em um método desmembrado } References: https://docs.oracle.com/javase/tutorial/uiswing/events/actionlistener.html http://www.devmedia.com.br/javalisteners-trabalhando-com-actionlistener-e-keylistener-em-java/31850

To perform an action when the button is clicked, simply add a listener as below:seuJButton.addActionListener(new ActionListener() { public void actionPerformed(ActionEvent e) { // sua ação aqui } }); UPDATEIf you want to control if the button has been previously clicked at least once, you can create a Boolean variable as a "signal" that has already entered the event:boolean buttonHasClicked = false; Then, check in the method was clicked, and change the status of the variable:seuJButton.addActionListener(new ActionListener() { public void actionPerformed(ActionEvent e) { if(!buttonHasClicked){ //ação se já não foi clicado buttonHasclicked = true; }else{ //ação se já foi clicado antes } } }); It would be good for a read on http://docs.oracle.com/javase/tutorial/uiswing/components/button.html , has many examples there demonstrating more about components.

One of the ways to do this would be by using an exclusive variable to represent the "increment":private int xi = 1; private int increment = 1; Then, on the first button, you change to the following:private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) { xi += increment; jLabel2.setText("Your money " + xi + "$"); } And the second button:private void jButton2ActionPerformed(java.awt.event.ActionEvent evt) { increment = 2;//ou increment++; //desativa o botao apos o primeiro uso ((JButton) evt.getSource()).setEnable(false); } As it is, you don't even need to disable the button, because as much as you click it, the change will always be for 2. If you use increment++, you need to use the code that disables the button to stop the increment.

And change your listener method to:class but implements ActionListener{ private int xi; @Override public void actionPerformed(ActionEvent but){ xi++; label.setText("Your money: " + xi); } } Also note the convention of the nomenclature of the java, where classes must start with capital letter.

From what I understand, it would be enough for you to create a class that implements https://docs.oracle.com/javase/9/docs/api/java/awt/event/ActionListener.html , and assign to all buttons. The class could be like this:class AcaoBotaoColoridoListener implements ActionListener { @Override public void actionPerformed(ActionEvent e) { JButton button = (JButton) e.getSource(); if ("lapis".equals(tipo)) { switch (cor) { case "cinza": button.setBackground(new Color(101, 101, 101)); break; case "branco": button.setBackground(new Color(255, 255, 255)); break; case "preto": button.setBackground(new Color(0, 0, 0)); break; case "azulE": button.setBackground(new Color(0, 0, 255)); break; case "vermelho": button.setBackground(new Color(255, 0, 0)); break; case "verde": button.setBackground(new Color(0, 204, 0)); break; case "amarelo": button.setBackground(new Color(255, 255, 0)); break; case "laranja": button.setBackground(new Color(255, 204, 0)); break; case "rosa": button.setBackground(new Color(255, 153, 153)); break; case "azulC": button.setBackground(new Color(0, 204, 204)); break; default: break; } } } } Then just apply to each button, using the same loop you already use to create the buttons:JButton[] button = new JButton[100]; for (int i = 0; i < 100; i++){ button[i] = new JButton(); pnlPixels.add(button[i]); button[i].addActionListener(new AcaoBotaoColoridoListener()); } Note: inverted your if comparison to "lapis".equals(tipo), therefore, case tipo get naked, avoid problems with https://pt.stackoverflow.com/q/119426/28595 .UpdateThere is a more "eligant" way of doing this, which is using Map to keep the colors, so you don't need to make a swith...case with such a choice.First create a type variable Map where the key will be the string with the color names, and the value will be the proper colors:private Map<String, Color> colorMap; (...) colorMap = new HashMap<String, Color>(); colorMap.put("cinza", new Color(101, 101, 101)); colorMap.put("branco", new Color(255, 255, 255)); colorMap.put("preto", new Color(0, 0, 0)); colorMap.put("azulE", new Color(0, 0, 255)); colorMap.put("vermelho", new Color(255, 0, 0)); colorMap.put("verde", new Color(0, 204, 0)); colorMap.put("amarelho", new Color(255, 255, 0)); colorMap.put("laranja", new Color(255, 204, 0)); colorMap.put("rosa", new Color(255, 153, 153)); colorMap.put("azulC", new Color(0, 204, 204)); Then just check if the color exists in Map and set the color to the button. See how the class got:class AcaoBotaoColoridoListener implements ActionListener { private Map&lt;String, Color&gt; colorMap; public AcaoBotaoColoridoListener() { colorMap = new HashMap&lt;String, Color&gt;(); colorMap.put("cinza", new Color(101, 101, 101)); colorMap.put("branco", new Color(255, 255, 255)); colorMap.put("preto", new Color(0, 0, 0)); colorMap.put("azulE", new Color(0, 0, 255)); colorMap.put("vermelho", new Color(255, 0, 0)); colorMap.put("verde", new Color(0, 204, 0)); colorMap.put("amarelho", new Color(255, 255, 0)); colorMap.put("laranja", new Color(255, 204, 0)); colorMap.put("rosa", new Color(255, 153, 153)); colorMap.put("azulC", new Color(0, 204, 204)); } @Override public void actionPerformed(ActionEvent e) { JButton button = (JButton) e.getSource(); if ("lapis".equals(tipo) &amp;&amp; colorMap.containsKey(cor)) { button.setBackground(colorMap.get(cor)); } } } Notice that the code has now become more readable and easier to maintain.