C++ bot do klikania w czarne pixele WinAPI GetPixel

Question

Cześć,

Chciałbym napisać bota który po wykryciu czarnego pixela w podanych współrzędnych ekranu (x, y) po prostu tam kliknie,
Podaje to co napisałem, ale mi to w ogóle nie działa - proszę o jakies wskazówki 
 

#include <windows.h>
#include <iostream>
#include <winuser.h>
#include <conio.h>
#include <unistd.h>

using namespace std;


int main(){
     while(true){


    HDC pulpit = GetDC(NULL);                       
    COLORREF kolor = GetPixel(pulpit, 580, 470);     
    COLORREF kolor1 = GetPixel(pulpit, 680, 470);
    COLORREF kolor2 = GetPixel(pulpit, 770, 470);    
    COLORREF kolor3 = GetPixel(pulpit, 880, 470);   
    //cout << kolor << '\n';                         
    ReleaseDC(NULL, pulpit);            

    
    while(true){

    if (kolor = 0){
    SetCursorPos(580,470);
    cout<<"BLACK    0"<<endl;
    mouse_event (MOUSEEVENTF_LEFTDOWN| MOUSEEVENTF_LEFTUP, 0, 0, 0, 0);
    }

    else if (kolor1 = 0){
    SetCursorPos(680,470);
    cout<<"BLACK    1"<<endl;
    mouse_event (MOUSEEVENTF_LEFTDOWN| MOUSEEVENTF_LEFTUP, 0, 0, 0, 0);
    }

    else if (kolor2 = 0){
    SetCursorPos(770,470);
    cout<<"BLACK    2"<<endl;
    mouse_event (MOUSEEVENTF_LEFTDOWN| MOUSEEVENTF_LEFTUP, 0, 0, 0, 0);
    }

    else (kolor3 = 0);{
    SetCursorPos(880,470);
    cout<<"BLACK    3"<<endl;
    mouse_event (MOUSEEVENTF_LEFTDOWN| MOUSEEVENTF_LEFTUP, 0, 0, 0, 0);
    
    }

}
return 0;
}

 

Answer 1

kolor = 0
użyłeś jednego operator porównania =, to oznacza przypisanie. Jest to wyrażenie, które zwróci 0, które automaycznie zostanie uznane, za false, czyli żaden z tych branchy ifa się nie wykona. Wszędzie musisz mieć kolor == 0, kolor1 == 0, etc.

Comments

Słuszna uwaga  Jednak problem nadal ten sam, po kompilacji dostaje na ekran "BLACK 3" i dalej nic sie nie dzieje.

---

generalnie, to to Ci się nie powinno skompilować..
else (kolor3 = 0);
tutaj nie dość, że masz błąd (do else'a nie ma nawiasów, nie ma warunków), to jeszcze średnik, który powoduje, ze to co jest w nawiasach zawsze się wykona

---

Cóż, faktycznie za dużo tych błędów...
Spacer, spojrzenie na problem 'świeżym okiem' i proszę.
Wstawiam kod, może się komuś na coś przyda. 
Program po wykryciu czarnego pixela na podanych współrzędnych (x, y) ustawia tam kursor i klika. 
 

#include <windows.h>
#include <iostream>
#include <winuser.h>
#include <conio.h>
#include <unistd.h>

using namespace std ;

int main(){
HDC dc = GetDC(NULL);


 while(true){

COLORREF color = GetPixel(dc, 600, 460);
if (color==0){
    cout<<"BLACK    0"<<endl;
  SetCursorPos(600,460);
  mouse_event(MOUSEEVENTF_LEFTUP,0,0,0,0);
  mouse_event(MOUSEEVENTF_LEFTDOWN,0,0,0,0);

}

COLORREF color1 = GetPixel(dc, 700, 460);
if (color1==0){
    cout<<"BLACK    1"<<endl;
    SetCursorPos(700,460);
  mouse_event(MOUSEEVENTF_LEFTUP,0,0,0,0);
  mouse_event(MOUSEEVENTF_LEFTDOWN,0,0,0,0);

}

COLORREF color2 = GetPixel(dc, 800, 460);
if (color2==0){
    cout<<"BLACK    2"<<endl;
      SetCursorPos(800,460);
  mouse_event(MOUSEEVENTF_LEFTUP,0,0,0,0);
  mouse_event(MOUSEEVENTF_LEFTDOWN,0,0,0,0);

}

COLORREF color3 = GetPixel(dc, 900, 460);
if (color3==0){
    cout<<"BLACK    3"<<endl;
      SetCursorPos(900,460);
  mouse_event(MOUSEEVENTF_LEFTUP,0,0,0,0);
  mouse_event(MOUSEEVENTF_LEFTDOWN,0,0,0,0);
}
}
}