Problema passaggio di variabile in classe dello stesso package

[roquentin]

Ciao a tutti,
ho dichiarato una variabile in una classe di un package:

 public String elemento;

e sto cercando di utilizzarla all'interno di un'altra classe dello stesso package

import java.io.File;

public class CheckFile {

    private static boolean foundFolder = false;

    public static void main(String[] args) {
        File dir = new File("dirIniziale");
        findDirectory(dir);
    }

    private static void findDirectory(File parentDirectory) {
        if(foundFolder) {
            return;
        }
        File[] files = parentDirectory.listFiles();
        for (File file : files) {
            if (file.isFile()) {
                continue;
            }
            if (file.getName().equals(String (elemento))) {
                foundFolder = true;    
                break;
                
            }
            if(file.isDirectory()) {
               findDirectory(file);
            }
        }
    }

}

in modo tale da poterla utilizzare come chiave di ricerca di una cartella all'interno della directory "dirIniziale" ma non va!
Dove sbaglio???

[Alex'87]

1. Per usare elemento devi avere un'istanza della classe che la contiene (a meno che elemento non sia static)
2. elemento è public: pessima cosa
3. Se elemento è String, String(elemento) cosa sarebbe? Un cast? (che comunque è (String) elemento))

[roquentin]

1. Per usare elemento devi avere un'istanza della classe che la contiene (a meno che elemento non sia static)
2. elemento è public: pessima cosa
3. Se elemento è String, String(elemento) cosa sarebbe? Un cast? (che comunque è (String) elemento))

Perchè è una pessima cosa dichiarare elemento public ?
Non è preferibile se voglio utilizzare elemento al di fuori della classe dove è stato creato ?

[roquentin]

Ho fatto in questo modo:

import java.io.File;

public class CheckFile {
    

    private static boolean foundFolder = false;

    public static void main(String[] args) {
        File dir = new File("dirIniziale");
        findDirectory(dir);
    }

    private static void findDirectory(File parentDirectory) {
        
        PV p = new PV();
        
       
        
        if(foundFolder) {
            return;
        }
        File[] files = parentDirectory.listFiles();
        for (File file : files) {
            if (file.isFile()) {
                continue;
            }
            if (file.getName().equals(p.elemento)) {
                foundFolder = true;  
                break;
            }
            if(file.isDirectory()) {
               findDirectory(file);
            }
        }
    }

ma non riesco a capire se va bene!

[roquentin]

import java.io.File;

public class CheckFile {
    
    private static String s;
    private static boolean foundFolder = false;

    public static void main(String[] args) {
        File dir = new File("dirIniziale");
        findDirectory(dir);
        System.out.println(s);
    }

    private static void findDirectory(File parentDirectory) {
        
        PVEstimation p = new PVEstimation();
        
       
        
        if(foundFolder) {
            return;
        }
        File[] files = parentDirectory.listFiles();
        
        for (File file : files) {
            if (file.isFile()) {
                continue;
            }
            if (file.getName().equals(p.elemento)) {
                foundFolder = true;  
                s = file.getName();
                break;
            }
            if(file.isDirectory()) {
               findDirectory(file);
            }
        }
        
    }

}

Non capisco perchè mi restituisce null !!!

Potete aiutarmi?

[Alex'87]

Beh, tu stai istanziando un PVEstimation senza inizializzare nulla. Pertanto tutti i suoi campi saranno null.
Da nessuna parte fai p.elemento = qualcosa...

[roquentin]

import java.io.File;

public class CheckFile {
    
    private static String s;
    private static boolean foundFolder = false;

    public static void main(String[] args) {
        File dir = new File("dirIniziale");
        findDirectory(dir);
        System.out.println(s);
    }

    private static void findDirectory(File parentDirectory) {
        
        PVEstimation p = new PVEstimation();
        
       
        
        if(foundFolder) {
            return;
        }
        File[] files = parentDirectory.listFiles();
        
        for (File file : files) {
            if (file.isFile()) {
                continue;
            }
            if (file.getName().equals(p.elemento)) {
                foundFolder = true;  
                s = file.getName();
                break;
            }
            if(file.isDirectory()) {
               findDirectory(file);
            }
        }
        
    }

}

Non capisco perchè mi restituisce null !!!

Potete aiutarmi?

il metodo listFiles ho notato che mi permette di avere i pathname di tutti gli elementi contenuti in una cartella.
Come posso fare per avere invece del percorso, solo i nomi degli elementi contenuti nella directory ????

[Alex'87]

Perchè è una pessima cosa dichiarare elemento public ?
Non è preferibile se voglio utilizzare elemento al di fuori della classe dove è stato creato ?

No. L'accessi ai campi va sempre fatto attraverso gli opportuni metodi (getter e setter). Questo perché in questo modo è possibile controllare l'accesso ai campi ed evitare stati inconsistenti.

Ti faccio un esempio.
Abbiamo una classe "Person" che modella una persona. Avrà nome, cognome, età ecc.
L'età ha un vincolo: deve essere non negativa (ovviamente!)

Approccio "public":

public class Person {
    public int age;
    ... // altri campi, non ci interessano...
}

Potrei ora fare questo:

Person p = new Person();
p.age = -100; // non ha senso ma non c'è nulla che me lo impedisce!

Un ipotetico metodo calcolaPensione(p) mi darebbe risultati sbagliati! BOOM!

Approccio serio:

public class Person {
    private int age;
    
    public void setAge(int age) {
        if (age < 0) {
            throw new IllegalArgumentException("age must be >= 0!");
        }
        
        this.age = age;
    }

    public int getAge() { 
        return age;
    }

    ... // altra roba...
}

La situazione ora cambia:

Person p = new Person();
p.age = -100; // ERRORE DI COMPILAZIONE! age è privato, non posso accederci!

p.setAge(-100); // ERRORE A RUNTIME: mi stampa sulla console "age must be >= 0!"
p.setAge(40); // OK!

int age = p.getAge();

[roquentin]

Si capisco benissimo quello che dici, però nel caso una variabile sia "interna" nel senso che la uso per metterci solo determinati valori definiti da me in un menù a tendina, potrebbe anche andare, no?

[roquentin]

Beh, tu stai istanziando un PVEstimation senza inizializzare nulla. Pertanto tutti i suoi campi saranno null.
Da nessuna parte fai p.elemento = qualcosa...

elemento l'ho dichiarato public all'interno di PVEstimation, poi l'ho utilizzato all'interno del metodo privato (jButton), sempre della classe PVEstimation. Ora, all'interno della classse di cui sopra (dello stesso package) io vorrei utilizzare il valore corrente di elemento, che si trova nel metodo privato, pensando che istanziando la classe mi avrebbe dato direttamente il valore di elemento in p.elemento.