1. s=0 //Przypisujesz, a nie porównujesz (==)
2. warunek: s<=0 oznacza, że dopóki s jest mniejsze bądź równe 0 to pętla będzie się wykonywać.. jak sam widzisz.. to nie ma sensu.. warunek powinien być s>=0 lub s>0
3. nie wiem czy ta liczba nie jest czasem za duża.. czym jest s? intem? jeżeli tak to zdecydowanie jest zbyt duża.. ustaw tą liczbę mniejszą.
4. nawet jeżeli ta pętla będzie się wykonywać to zmiana koloru czcionki będzie następować tak szybko, że będzie to przypominać jeden kolor.
Moja propozycja kodu to stworzyć zmienną, która będzie się inkrementować w pętli głównej programu, a gdy nastąpi warunek if(zmienna%10000==0) to wykonać funkcję (nazwijmy ją) switch(), która zamieni kolory (z pierwszego na drugi, a przy kolejnym wywołaniu z drugiego na pierwszy).