Loop not working

Code java:

---

<
{
	Rectangle rectangle = new Rectangle(0, 0, 1365, 770);
	{
		while(true)
		{
        BufferedImage image = robot.createScreenCapture(rectangle);
        search: for(int x = 0; x < rectangle.getWidth(); x++)
			{
            for(int y = 0; y < rectangle.getHeight(); y++)
				{
					if(image.getRGB(x, y) == color1.getRGB())
						{
							return true;
						}
					return false;
				}
			}
		}
	}
}
>

---

what im having a problem with is that the loop while(true) is not working and i need it to work

this method is supposed to search through a screenshot and find a pixel and if it dose not take another shot and look again but it is not doing that when i run the program

Quote:

---

the loop while(true) is not working

---

Please explain what the code does and why you think it is not working.

When the code executes the return false; statement, that will end the loop. Are you sure that you want to end the loop the first time the colors don't match?


The formatting of your code makes it very hard to read. The {}s should be aligned and not left and right as yours are.

Saying "it's not working" is as useful to us as us saying "then fix it" is useful to you. Please provide an SSCCE that demonstrates the actual problem, and ask a specific technical question. What doesn't work about your loop? What does it do that you don't expect? Have you stepped through this with a debugger, or simply added some print statements, to figure out the flow of the program? What are all of those variables whose declarations are missing? Is that really how you indent your code?

That loop only ever looks at a single pixel in a single image. To see that try:

Code :

---

Rectangle rectangle = new Rectangle(0, 0, 1365, 770);
{
    while(true)
    {
        System.out.println("About to take screen shot");
        BufferedImage image = robot.createScreenCapture(rectangle);
        search: for(int x = 0; x < rectangle.getWidth(); x++)
        {
            for(int y = 0; y < rectangle.getHeight(); y++)
            {
                System.out.println("Looking at pixel " + x + "/" + y);
                if(image.getRGB(x, y) == color1.getRGB())
                {
                    System.out.println("About to finish and return true");
                    return true;
                }
                System.out.println("About to finish and return false");
                return false;
            }
        }
    }
}

---

You might to replace the "true" while(true) with something that better expresses your intent that the loop should continue until a pixel of a specific colour has been found. Also remove the label "search:" - it isn't being used.

ok explanation, no i want that chunk of code to repeat over and over again until it returns with a true (an i found the pixel mate) and im hoping the while loop is the best one but im open to suggestions.

Remove the return false from inside of the loop.

ok so where would i put it? i took it out completely and its still not looping, do i need to put it in at the end of the method?

Quote:

---

its still not looping,

---

Add a printlns as shown in post#4 in the loop to show if it is looping.
If the method does not execute return true, where is it executing?
The return appears to be the ONLY way to exit the loop. The loop should loop forever.

Code java:

---

<
import java.awt.*;
import java.awt.event.*;
import java.awt.Robot;
import java.awt.AWTException;
import java.awt.Rectangle;
import java.awt.Color;
import java.awt.Toolkit;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
import javax.imageio.ImageIO;
 
public class Gun{
	static void Main() throws Exception 
	{
	Robot robot = new Robot();
	Color color1 = new Color(196, 195, 181);
	Color color2 = new Color(95, 118, 127);
	Color color3 = new Color(108, 25, 85); 
	Rectangle rectangle = new Rectangle(0, 0, 1365, 770);
	{
	BufferedImage image = robot.createScreenCapture(rectangle);
	search: for(int x = 0; x < rectangle.getWidth(); x++)
		{
		for(int y = 0; y < rectangle.getHeight(); y++)
			{
			if(image.getRGB(x, y) == color1.getRGB())
				{
				robot.mouseMove(x, y);
				robot.mousePress(InputEvent.BUTTON1_MASK);
				robot.delay(10);
				robot.mouseRelease(InputEvent.BUTTON1_MASK);
				break search;
				}
			}
		}
    }
    {
    BufferedImage image = robot.createScreenCapture(rectangle);
    search: for(int x = 0; x < rectangle.getWidth(); x++)
        {
            for(int y = 0; y < rectangle.getHeight(); y++)
            {
                if(image.getRGB(x, y) == color2.getRGB())
                {
                    robot.mouseMove(x, y);
					    robot.mousePress(InputEvent.BUTTON1_MASK);
						robot.delay(10);
						robot.mouseRelease(InputEvent.BUTTON1_MASK);
                    break search;
                }
            }
        }
    }
	}
}
static boolean openingboard() throws Exception
{	
	Robot robot = new Robot();
	Color color3 = new Color(108, 25, 85);
	Rectangle rectangle = new Rectangle(0, 0, 1365, 770);
		while(true)
		{
        BufferedImage image = robot.createScreenCapture(rectangle);
        search: for(int x = 0; x < rectangle.getWidth(); x++)
			{
            for(int y = 0; y < rectangle.getHeight(); y++)
				{
					if(image.getRGB(x, y) == color3.getRGB())
					{
							System.out.println("About to finish and return true");
							return true;
					}
					System.out.println("About to finish and return false");
				}
			}
		}
	}
}
 
 
 
 
 
>

---

ok now im getting the exception in thread "main" nosuchmethod error i have a Main class but do i have to have a main method? i don't know why im getting the error for the above code (yes its messy sorry ill fix and repost)

Quote:

---

exception in thread "main" nosuchmethod error i have a Main class

---

When you get an error, please copy and paste here the full text of the error message.

Quote:

---

do i have to have a main method?

---

If you want to have the java program execute the class it must have a proper main method.

Are you conscious that with the while(true) statement you are creating a infinite loop ?

Oscar Gómez
Engineer at PSL
PSL S.A. - CMMi 5 nearshore software development and outsourcing from Latin America - Home

Are you conscious that with the while(true) statement you are creating a infinite loop ¡

Oscar Gómez
Engineer at PSL
PSL S.A. - CMMi 5 nearshore software development and outsourcing from Latin America - Home

replace return false statement from inside the for loop. It should be outside the while(true) loop.