Po pierwsze: Umieść kod w specjalnym bloczku kodu!
Skasuj linijkę
string login, haslo
Nie zakończyłeś jej średnikiem. A potem w kodzie tworzysz nowe zmienne o takich samych nazwach. Ta linijka jest zupełnie zbędna.
Mam takie pytanie: dlaczego postanowiłeś utworzyć zmienne globalne? Bardzo często ludzie umieszczają tu kody źródłowe zawierające pełno zmiennych globalnych. To jest coś, czego uczą w szkole?