[Java] Problemi di codice

[Nicolas9000]

Stavo tentando di creare un programma che ricevesse prima un numero da tastiera, poi un operatore matematico, un secondo numero ed infine restituisse un risultato. Ho tentato per giorni ma non riesco ad uscirne da solo.
Potreste controllare cosa non va nel codice, per favore?
P.S. Scusate errori logici o di sintassi troppo banali.

import java.lang.*;
public class calcolo
{
public static void main(string args[])
{
String stringa, stringa2, stringa3;
Int numero1, numero2, numero3;
Char singolo;
System.out.println("Benvenuto nella calcolatrice.");
System.out.println("Inserisci il numero con cui vuoi lavorare");
numero1 = SavitchIn.ReadLine();
System.out.println("Inserisci l'operatore matematico");
singolo = SavitchIn.ReadLine();

If (singolo == 'x'); {
System.out.println("Inserisci il secondo numero");
numero2 = SavitchIn.ReadLine();
numero3 = numero1 * numero2;
System.out.println("Il risultato è" + numero3);
}
If (singolo == '+'); {
System.out.println("Inserisci il secondo numero");
numero2 = SavitchIn.ReadLine();
numero3 = numero1 + numero2;
System.out.println("Il risultato è" + numero3);
}
If (singolo == '-'); {
System.out.println("Inserisci il secondo numero");
numero2 = SavitchIn.ReadLine();
numero3 = numero1 - numero2;
System.out.println("Il risultato è" + numero3);
}
If (singolo == '/'); {
System.out.println("Inserisci il secondo numero");
numero2 = SavitchIn.ReadLine();
numero3 = numero1 / numero2;
System.out.println("Il risultato è" + numero3);
}
else
{
Do {
System.out.println("Hai inserito un carattere errato, riprova.");
}
while (numero1 != int)(numero2 != int)(singolo != char)

}

}

[andbin]

Originariamente inviato da Nicolas9000

Stavo tentando di creare un programma che ricevesse prima un numero da tastiera, poi un operatore matematico, un secondo numero ed infine restituisse un risultato. Ho tentato per giorni ma non riesco ad uscirne da solo.

numero1 = SavitchIn.ReadLine();
System.out.println("Inserisci l'operatore matematico");
singolo = SavitchIn.ReadLine();

Non conosco questa classe SavitchIn (ma l'ho già vista usare in "esercizi" fatti da altri). Ma sei sicuro che usando solo ReadLine() tu possa prendere prima un int, poi un char ecc...? Io non credo proprio!
Direi che dovresti rivedere un po' bene l'uso dei vari tipi di dati.

[Nicolas9000]

E se invece con ReadInt, ReadChar...?

[geko84k]

si ti conviene usare readInt() e readChar(), inoltre la classe deve stare nella stessa directory del tuo programma...

[andbin]

Originariamente inviato da geko84k

la classe deve stare nella stessa directory del tuo programma...

Basta che sia in "classpath". Poi dove sia fisicamente non è molto rilevante.

[Nicolas9000]

Ciò che avevo in mente e che qualcuno potesse provare a compilare quel codice, verificare di persona gli errori e suggerirmi le correzioni. L'ho fatto perchè è un codice abbastanza facile, c'è qualcuno che può farmi questo favore?

[Don[ITA]]

Allora....gli errori sistattici sono:

public static void main(String args[])

int numero1, numero2, numero3;
char singolo;

if

la i non è maiuscola

if (singolo == 'x') {

non ci vanno i punti e virgola dopo la condizione dell'if..
Errori logici:

numero1 = SavitchIn.readLineInt();

per leggere un numero con la SavitchIn devi usare questo metodo...readLine() ritorna una stringa e a te serve un intero..

singolo = SavitchIn.readLineNonwhiteChar();

stesso di prima...

while (numero1 != int)(numero2 != int)(singolo != char)

questa sintassi è tutta errata...
suggerimento correzione:quando leggi il simbolo fai questo controllo:

do{
        System.out.println("Inserisci l'operatore matematico: ");
        singolo = SavitchIn.readLineNonwhiteChar();
    }while(singolo!='+'|singolo!='x'|singolo!='-'|singolo!='/');

altro suggerimento:

System.out.println("Inserisci il secondo numero: ");
        numero2 = SavitchIn.readLineInt();

questa parte puoi toglierla dagli if, mettila fuori.

Poi non usi ne le 3 stringhe che hai messo all'inizio ne qualcosa presente in java.lang quindi puoi toglierli.

Se vuoi vedere il tuo codice corretto guarda lo spoiler qui sotto...

Spoiler:

public class calcolo { public static void main(String args[]) { int numero1, numero2, numero3; char singolo; System.out.println("Benvenuto nella calcolatrice."); System.out.println("Inserisci il numero con cui vuoi lavorare: "); numero1 = SavitchIn.readLineInt(); do{ System.out.println("Inserisci l'operatore matematico: "); singolo = SavitchIn.readLineNonwhiteChar(); }while(singolo!='+'|singolo!='x'|singolo!='-'|singolo!='/'); System.out.println("Inserisci il secondo numero: "); numero2 = SavitchIn.readLineInt(); if (singolo == 'x') { numero3 = numero1 * numero2; System.out.println("Il risultato è: " + numero3); } else if (singolo == '+') { numero3 = numero1 + numero2; System.out.println("Il risultato è: " + numero3); } else if (singolo == '-') { numero3 = numero1 - numero2; System.out.println("Il risultato è: " + numero3); } else { numero3 = numero1 / numero2; System.out.println("Il risultato è: " + numero3); } } }

Credo sia tutto...ciauz

[Nicolas9000]

Mmm... c'è qualcosa che non va.

3 errors found:
File: C:\Documents and Settings\nicolas\Desktop\Progetti\Calcolo.java  [line: 9]
Error: C:\Documents and Settings\nicolas\Desktop\Progetti\Calcolo.java:9: cannot find symbol
symbol  : variable SavitchIn
location: class Calcolo
File: C:\Documents and Settings\nicolas\Desktop\Progetti\Calcolo.java  [line: 12]
Error: C:\Documents and Settings\nicolas\Desktop\Progetti\Calcolo.java:12: cannot find symbol
symbol  : variable SavitchIn
location: class Calcolo
File: C:\Documents and Settings\nicolas\Desktop\Progetti\Calcolo.java  [line: 15]
Error: C:\Documents and Settings\nicolas\Desktop\Progetti\Calcolo.java:15: cannot find symbol
symbol  : variable SavitchIn
location: class Calcolo

[wingman87]

Quegli errori significano che il compilatore non trova la classe SavitchIn

[Nicolas9000]

Si dev'essere così. Cosa posso fare?

[wingman87]

Allora, siccome non conoscevo questa classe ho provato a cercare con google e ho trovato questo:
http://www-cse.ucsd.edu/users/savitc.../SavitchIn.txt
copia tutto e mettilo dentro un file con nome SavitchIn.java e metti questo file nella stessa cartella del tuo programma, dovrebbe funzionare. Ciao!

[Nicolas9000]

Ho fatto come hai detto ed infatti la compilazione è stata eseguita senza errori, ma una volta lanciato il programma e inserito il primo numero, alla schermata che chiede l'operatore matematico, io lo inserisco, premo invio ma sembra non accettarlo, difatti ripete in continuazione la stringa "Inserisci un operatore matematico" non accettando alcun valore. Help

[wingman87]

Devi guardare cosa c'è che non va in questa parte di codice, in pratica non esce dal while:

do{
System.out.println("Inserisci l'operatore matematico: ");
singolo = SavitchIn.readLineNonwhiteChar();
}while(singolo!='+'|singolo!='x'|singolo!='-'|singolo!='/');

[Don[ITA]]

Strano....
eppure sintatticamente, e a mio avviso pure logicamente, è corretto...
tradotto non dovrebbe essere qualcosa tipo "finchè singolo è diverso da + o da - o da x o da / continua a richiederlo"???

[wingman87]

Sì, vuol proprio dire quello, è lì il problema

[Nicolas9000]

Boh non capisco.
Cioè dice al programma di ripetere "Inserisci operatore matematico" fin quando scrivo qualcosa diverso da + - x /. Però poi gli If e gli else If ci sono, cioè va bene che ripeta la stringa se è una cosa diversa, ma se scrivo un operatore dovrebbe innescarsi l'if relativo. No?

[Caldwell83]

Scusate se dico castronerie, il mo java e' un po' arrugginito.
Non dovrebbero esse doppi gli "|" ?

do{
System.out.println("Inserisci l'operatore matematico: ");
singolo = SavitchIn.readLineNonwhiteChar();
}while( (singolo!='+') || (singolo!='x') || (singolo!='-') || (singolo!='/') );

A parte quello

Originariamente inviato da wingman87

Sì, vuol proprio dire quello, è lì il problema

Spoiler:

Mettendo le or non uscira' mai dail ciclo perche' il carattere che tu inserisci sara' sempre diverso da uno dei quattro.

[wingman87]

Se scrivi "|" tra un'espressione e l'altra vengono calcolate tutte le espressioni, se invece metti "||" appena viene trovata un'espressione true le altre non vengono calcolate (perché il risultato sarà sempre true).

[Don[ITA]]

Originariamente inviato da wingman87

Sì, vuol proprio dire quello, è lì il problema

e quale sarebbe????

[wingman87]

Se scrivi (A or B or C) se anche solo una di queste è vera tutta l'espressione è vera.