simplify Topic is solved

[ik4u]

Code: Select all

PixelGetColor, color, 160, 160
if color=0xE0E0E0
Click 335 380
PixelGetColor, color, 195, 160
if color=0xE0E0E0
Click 380 380
PixelGetColor, color, 230, 160
if color=0xE0E0E0
Click 420, 380
PixelGetColor, color, 265, 160
if color=0xE0E0E0
Click 465 380
PixelGetColor, color, 300, 160
if color=0xE0E0E0
Click 510 380
PixelGetColor, color, 335, 160
if color=0xE0E0E0
Click 550 380     

Can this be simplified, color is same in all

[boiler]

Code: Select all

loop, 6 {
	PixelGetColor, color, 160 + (A_Index - 1) * 35, 160
	if (color = 0xE0E0E0)
		Click, % [335, 380, 420, 465, 510, 550][A_Index] "," 380
}

[ik4u]

Code: Select all

loop, 6 {
	PixelGetColor, color, 160 + (A_Index - 1) * 35, 160
	if (color = 0xE0E0E0)
		Click, % [335, 380, 420, 465, 510, 550][A_Index] "," 380
}

The code works, however she skips some clicks, can you please add a small delay
between clicks to avoid a miss

[boiler]

Code: Select all

 loop, 6 {
	PixelGetColor, color, 160 + (A_Index - 1) * 35, 160
	if (color = 0xE0E0E0) {
		Click, % [335, 380, 420, 465, 510, 550][A_Index] "," 380
		Sleep, 100
	}
}

[boiler]

If it's still missing clicks, I would try this instead:

Code: Select all

 loop, 6 {
	PixelGetColor, color, 160 + (A_Index - 1) * 35, 160
	if (color = 0xE0E0E0) {
		MouseMove, [335, 380, 420, 465, 510, 550][A_Index], 380, 0
		Sleep, 100
		Click
	}
}

Or this to make sure you don't move the mouse and cause it to click in a different location:

Code: Select all

 loop, 6 {
	PixelGetColor, color, 160 + (A_Index - 1) * 35, 160
	if (color = 0xE0E0E0) {
		BlockInput, On
		MouseMove, [335, 380, 420, 465, 510, 550][A_Index], 380, 0
		Sleep, 100
		Click, % [335, 380, 420, 465, 510, 550][A_Index] "," 380
		BlockInput, Off
	}
}