[JAVA] non riesco a capire perchè!!

[luxorl]

Ho appena scritto un programmino facile facile in java, che basandosi su un oggetto "moneta" simuli il giochi di un lancio del testa o croce.

In un primo momento il codice era questo:

package esamepoo.giochi;
import poo.inout.*;

public class Monetina{
   private boolean faccia;
   public Monetina(){
   this.faccia=faccia;
}

public boolean lancia(Monetina m){
   double d=Math.random();
   if(d<0.5)
         m.faccia=true;
   else
         m.faccia=false;
}

public static void main(String args[]){
   Monetina m=new Monetina();
   boolean con;
   System.out.println("TESTA o CROCE?")
   int i=Console.readInt("per TESTA premi 1, per CROCE premi 0");

   if(i==1)
   con=true;
   else
   con=false;

   boolean f=lancia(m);
   
   if (f==true)
   System.out.println("TESTA");
   else
   System.out.println("CROCE");

   if(con==f)
   System.out.println("BRAVO, HAI VINTO!");
   else
   System.out.println("Peccato, Ritenta");
   }
}

compilando così (main nello stesso file) mi da l'errore:
non-static method lancia
cannot be referenced from a static context

ok, fin qua ho capito dove ho sbagliato.. anche se non ho ben capito perchè devo rendere statico il metodo lancia!! Chi me lo spiega? please!!

vado cmq ad aggiungere "static" al metodo lancia, e funziona tutto perfettamente!!

Allora decido di tagliare il main dallo stesso file, crearmi una cartella a parte, creare un altro file: "GiocaMoneta.java"

e dentro incollo lo stesso main!

import esamepoo.giochi.*;
import poo.inout;

public class GiocoMoneta{
   public static void main(String args[]){
   Monetina m=new Monetina();
   boolean con;
   System.out.println("TESTA o CROCE?")
   int i=Console.readInt("per TESTA premi 1, per CROCE premi 0");

   if(i==1)
   con=true;
   else
   con=false;

   boolean f=lancia(m);
   
   if (f==true)
   System.out.println("TESTA");
   else
   System.out.println("CROCE");

   if(con==f)
   System.out.println("BRAVO, HAI VINTO!");
   else
   System.out.println("Peccato, Ritenta");
   }
}

Compilo e mi da l'errore:
cannot resolve symbol
symbol: method lancia (esamepoo.giochi.Monetina)
location: class GiocoMoneta
boolean f=lancia(m);
1 error

Perchè lo stesso main in un altro file non va??

Datemi una mano please!
Grazie

[end.is.forever]

Il motivo dell'errore è che un metodo statico non può chiamare un metodo di istanza senza identificare l'istanza su cui chiamarlo.

I membri statici di una classe sono membri riferiti sempre e solo alla classe stessa, e non ad una istanza di quella classe; i membri non statici invece di default sono riferiti all'istanza della classe.
Per questo motivo se sei in un metodo statico, lui non ha idea di quale sia l'oggetto su cui deve invocare il tuo metodo lancia, dato che non hai anticipato alla chiamata l'oggetto su cui invocarlo.

Tu hai dichiarato il metodo lancia all'interno di Moneta come metodo non statico, questo può essere giusto, ma a che ti serve l'identificatore di un' altra moneta se già "this" è una moneta? E' molto più sensato eseguire le operazioni sull'istanza stessa invece che su un' altra istanza m.
E quindi chiamare m.lancia() invece che lancia.m().

Altrimenti se vuoi realizzare lancia come metodo statico a cui viene passata l'istanza su cui eseguire le operazioni ti basta aggiungere static prima di "boolean lancia(..." e a questo punto invocarla come "Moneta.lancia( m )" oppure se sei nella classe Moneta semplicemente "lancia( m )".

La seconda versione che hai fatto (GiocoMoneta) non cambia niente, sei sempre in un contesto "di classe" e vorresti invocare un metodo "di istanza".

[end.is.forever]

Poi noto alcuni errori concettuali, per sicurezza ti avverto:

public Monetina()
{
   this.faccia = faccia;
}

Qui quello che stai facendo è settare this.faccia al valore di this.faccia, ovvero al suo stesso valore dato che non ci sono altri identificatori "faccia" nello scope del metodo; se all'inizio non è definito continuerà a non essere definito.

public boolean lancia( Monetina m )
{
   double d = Math.random();
   if( d < 0.5 )
         m.faccia = true;
   else
         m.faccia = false;
}

Di questo te ne ho già parlato sopra, è sbagliata l'idea di "una moneta che lancia un' altra moneta": hai due possibilità:
a) una moneta espone "lancia"
b) il metodo statico "lancia" svolge le operazioni su una determinata moneta
Poi nello stesso metodo avrai anche un errore di compilazione, dato che dichiari di restituire un boolean ma non lo restituisci, per cui ti basta aggiungere prima della chiusa graffa "return faccia;".

[luxorl]

Grazie Mille
sei stato gentilissimo

cmq riepilogo così vediamo se ho capito bene,
ho reso dinuovo di default il metodo lancia, e ho cancellato dentro le sue parentesi "Monetina m", così invece di richiedere che gli venga passato l'oggetto monetina, lo invoco direttamente sull'oggetto:

m.lancia();

..infatti effetuando questa modifica, il programma gira anche mettendo il main in un'altra classe. (ho detto giusto? non si parla di altri file, ma di altre classi?!?)

cmq quando invocavo il metodo lancia dal main in un'altra classe questo mi dava l'errore perchè non capiva su che "this" invocarlo cioè su che istanza (giusto?)

e perchè invocandolo dal main nella stessa classe invece tutto andava bene? (sempre però rendendo statico il metodo lancia)

[luxorl]

Originariamente inviato da end.is.forever
Poi noto alcuni errori concettuali, per sicurezza ti avverto:

public Monetina()
{
   this.faccia = faccia;
}

Qui quello che stai facendo è settare this.faccia al valore di this.faccia, ovvero al suo stesso valore dato che non ci sono altri identificatori "faccia" nello scope del metodo; se all'inizio non è definito continuerà a non essere definito.

e come avrei dovuto fare invece?

[luxorl]

Originariamente inviato da end.is.forever

public boolean lancia( Monetina m )
{
   double d = Math.random();
   if( d < 0.5 )
         m.faccia = true;
   else
         m.faccia = false;
}

Di questo te ne ho già parlato sopra, è sbagliata l'idea di "una moneta che lancia un' altra moneta": hai due possibilità:
a) una moneta espone "lancia"
b) il metodo statico "lancia" svolge le operazioni su una determinata moneta
Poi nello stesso metodo avrai anche un errore di compilazione, dato che dichiari di restituire un boolean ma non lo restituisci, per cui ti basta aggiungere prima della chiusa graffa "return faccia;".

sisi.. ho dimenticato di copiarlo il return
non ho copiato ed incollato.. ho riscritto tutto il programmino a mano, perchè lo stavo facendo sul portatile.. che però non ha connesisone ad internet ..e sono venuto a postare invece dal desktop!