PDA

View Full Version : [Java] Problemi di codice


Nicolas9000
21-01-2008, 20:10
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
21-01-2008, 20:30
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
21-01-2008, 20:31
E se invece con ReadInt, ReadChar...?

geko84k
21-01-2008, 21:47
si ti conviene usare readInt() e readChar(), inoltre la classe deve stare nella stessa directory del tuo programma...

andbin
21-01-2008, 21:55
la classe deve stare nella stessa directory del tuo programma...Basta che sia in "classpath". Poi dove sia fisicamente non è molto rilevante.

Nicolas9000
22-01-2008, 14:54
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]
22-01-2008, 19:02
Allora....gli errori sistattici sono:
public static void main(String args[])
int numero1, numero2, numero3;
char singolo;
ifla i non è maiuscola
if (singolo == 'x') {non ci vanno i punti e virgola dopo la condizione dell'if..:asd:
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...

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
22-01-2008, 20:27
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
22-01-2008, 20:41
Quegli errori significano che il compilatore non trova la classe SavitchIn

Nicolas9000
22-01-2008, 20:43
Si dev'essere così. Cosa posso fare?

wingman87
22-01-2008, 20:56
Allora, siccome non conoscevo questa classe ho provato a cercare con google e ho trovato questo:
http://www-cse.ucsd.edu/users/savitch/java/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
22-01-2008, 21:12
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 :p

wingman87
22-01-2008, 21:21
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]
22-01-2008, 21:54
Strano....:eekk:
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
22-01-2008, 22:04
Sì, vuol proprio dire quello, è lì il problema ;)

Nicolas9000
23-01-2008, 12:47
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
23-01-2008, 14:23
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
Sì, vuol proprio dire quello, è lì il problema ;)
Mettendo le or non uscira' mai dail ciclo perche' il carattere che tu inserisci sara' sempre diverso da uno dei quattro.

wingman87
23-01-2008, 18:33
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]
23-01-2008, 21:37
Sì, vuol proprio dire quello, è lì il problema ;)

e quale sarebbe????:mc:

wingman87
23-01-2008, 21:48
Se scrivi (A or B or C) se anche solo una di queste è vera tutta l'espressione è vera.

Don[ITA]
23-01-2008, 22:07
CAAAAAAAAAAAVOLOOOOOOOOOOOOOOO hai raggioneeeeeeeeeeeeeeeeee!!!!!!
Non ci avevo fatto caso!!!!!!!!! Miii che errore da niubbo che ho commessoooo:doh: :doh: :doh: :doh: :doh:

Dunque ecco il codice corretto:

public class calcolo
{
public static void main(String args[])
{
int numero1, numero2;
char singolo;
System.out.println("Benvenuto nella calcolatrice.");
System.out.println("Inserisci il numero con cui vuoi lavorare: ");
numero1 = SavitchIn.readLineInt();
System.out.println("Inserisci l'operatore: ");
singolo = SavitchIn.readLineNonwhiteChar();
boolean control = true;
while(control){
switch(singolo){
case '+':{
control = false;
System.out.println("Inserisci il secondo numero: ");
numero2 = SavitchIn.readLineInt();
System.out.println("Il risultato è: "+(numero1+numero2));
break;
}
case '-':{
control = false;
System.out.println("Inserisci il secondo numero: ");
numero2 = SavitchIn.readLineInt();
System.out.println("Il risultato è: "+(numero1-numero2));
break;
}
case 'x':{
control = false;
System.out.println("Inserisci il secondo numero: ");
numero2 = SavitchIn.readLineInt();
System.out.println("Il risultato è: "+(numero1*numero2));
break;
}
case '/':{
control = false;
System.out.println("Inserisci il secondo numero: ");
numero2 = SavitchIn.readLineInt();
System.out.println("Il risultato è: "+(numero1/numero2));
break;
}default:{
System.out.println("Errore!!!\nL'operatore può essere solo +, x, - o /.\nInserire nuovamente l'operatore: ");
singolo = SavitchIn.readLineNonwhiteChar();
break;
}
}
}
}
}

:Prrr: