[Java] problema

[gigippa]

Devo definire una classe QuantiDispari contenente solo un main il quale legga da consolle una
sequenza di interi terminata da 0 e scriva su consolle quanti sono i numeri dispari
nella sequenza

import java.util.Scanner;
public class QuantiDispari{
public static void main(String[] args) {

Scanner tastiera = new Scanner(System.in);
int pari = 0;
int dispari = 0;
int contatore = 0;
int risposta;
System.out.println("immetti un valore opppure \"0\" per uscire");
risposta=tastiera.nextInt();
while(risposta!=0) {
risposta=tastiera.nextInt();
contatore = contatore + 1;

System.out.println("immetti un valore opppure \"0\"per uscire");
risposta=tastiera.nextInt();
//}


if(risposta %2 = 1){
dispari ++;}

else{ pari++;}
System.out.println("hai inserito numeri" +(contatore));
System.out.println("i pari sono"+pari);
System.out.println("i dispari sono"+dispari);

}
}
}
Ma non riesco a fare la divisione in modulo 2. Mi potete aiutare? grazie

[andbin]

Originariamente inviato da gigippa

Devo definire una classe QuantiDispari contenente solo un main il quale legga da consolle una
sequenza di interi terminata da 0 e scriva su consolle quanti sono i numeri dispari
nella sequenza

Ma non riesco a fare la divisione in modulo 2. Mi potete aiutare? grazie

Innanzitutto posta sempre il codice tra i tag [CODE], così è più leggibile.

Ci sono (secondo me) troppi nextInt(). Ne basta 1 solo.

do {
    System.out.println("immetti ......");
    if ((risposta = tastiera.nextInt()) == 0)
        break;
    ....
} while (true);

Il test dove fai il modulo non è giusto, usa == invece di =.

[gigippa]

Originariamente inviato da andbin

Innanzitutto posta sempre il codice tra i tag [CODE], così è più leggibile.

Ci sono (secondo me) troppi nextInt(). Ne basta 1 solo.

do {
    System.out.println("immetti ......");
    if ((risposta = tastiera.nextInt()) == 0)
        break;
    ....
} while (true);

Il test dove fai il modulo non è giusto, usa == invece di =.

Grazie con i tuoi consigli ho risolto