Codice java con errore

[luca93abc]

Buongiorno a tutti, premetto di essere alle prime armi con la programmazione java. Poco fa ho provato a scrivere un semplice programma che dato il raggio restituisce l'area.

import java.util.Scanner;

public class cerchio {
    private int raggio;

    @SuppressWarnings("unused")
    private void raggio(int raggio){
        this.raggio = raggio;
    }
    public int getRaggio() {
        return raggio;
    }

    public void setRaggio(int raggio) {
        Scanner input = new Scanner(System.in);
        raggio = input.nextInt();
        this.raggio = raggio;
    }
    
    public static void main(String[] args){
        cerchio area = new cerchio();
        double area1 = area1.getRaggio(raggio)*area1.getRaggio(raggio)*3.1415;
    }
}

Siccome intendo programmare ad oggetti non ho seguito il suggerimento di Eclipse di rendere static la variabile raggio, che ora figura come errore. Come posso risolverlo?
Grazie

[andbin]

    public void setRaggio(int raggio) {
        Scanner input = new Scanner(System.in);
        raggio = input.nextInt();
        this.raggio = raggio;
    }

Questo non ha assolutamente senso.

    @SuppressWarnings("unused")
    private void raggio(int raggio){
        this.raggio = raggio;
    }

Questo ha il nome del metodo inappropriato, oltre che essere private quindi poco utile.

Siccome intendo programmare ad oggetti

Bene, allora definisci una classe Cerchio che ha la proprietà "raggio" (quindi il campo private e relativi metodi getter/setter) e un metodo che calcola e restituisce l'area. Stop, non deve fare input, nessun uso di Scanner o cose del genere.

Poi fai una classe es. TestCerchio dove nel main usi Scanner, chiedi raggio, crei un cerchio con il raggio e determini l'area che poi stampi.

[LeleFT]

PS: oltre a tutto quello che già ha detto andbin, stai anche facendo un altro errore (che sicuramente Eclipse segnala... probabilmente è un errore di copia-incolla qui sul forum):

    public static void main(String[] args){
        cerchio area = new cerchio();
        double area1 = area1.getRaggio(raggio)*area1.getRaggio(raggio)*3.1415;
    }

Stai cercando di richiamare il metodo "getRaggio()" su una variabile di tipo double, che, oltretutto, stai definendo sulla stessa riga...


Ciao.

[luca93abc]

Grazie mille! Penso ora di aver capito: creo le due classi in questo modo

package cerchio;

public class cerchio {
    private int raggio;

    public cerchio(int raggio) {
        this.raggio = raggio;
    }

    public int getRaggio() {
        return raggio;
    }

    public void setRaggio(int raggio) {
        this.raggio = raggio;
    }
    public double area(int raggio){
        double area = raggio * raggio * 3.1415;
        return area;
    }
    public double lunghezza(int raggio){
        double lunghezza = raggio * 2 * Math.PI;
        return lunghezza;
    }
}

e la seconda

package cerchio;

import java.text.DecimalFormat;
import java.util.Scanner;

public class area {
    public static void main(String[] args) {
        Scanner input = new Scanner(System.in);
        System.out.println("Raggio: ");
        int raggio = input.nextInt();
        cerchio cer1 = new cerchio(raggio);
        System.out.println("L'area del tuo cerchio è: " + cer1.area(raggio) + " e la lunghezza della circonferenza è: " +cer1.lunghezza(raggio));
    }

}

La compilazione va a buon fine e il codice funziona (fa quello che deve). Formalmente è corretto come programmazione ad oggetti?
Grazie

[andbin]

    public double area(int raggio){
        double area = raggio * raggio * 3.1415;
        return area;
    }
    public double lunghezza(int raggio){
        double lunghezza = raggio * 2 * Math.PI;
        return lunghezza;
    }

Così no. Non devi ricevere un ulteriore raggio ... devi usare lo "stato" dell'oggetto su cui invochi area o lunghezza.

Formalmente è corretto come programmazione ad oggetti?

Salvo quanto ti ho indicato sì, è appropriato. Le classi andrebbero con le parole in maiuscolo, es. DecimalFormat, Scanner, ecc...