Groovy (Fichier IO): recherche de tous les fichiers et renvoyer tous les fichiers - le Groovy façon

Ok, ça devrait être facile...

Je suis nouveau sur groovy et je suis à la recherche pour mettre en œuvre la logique suivante:

def testFiles = findAllTestFiles();

Jusqu'à présent, je suis venu avec le code ci-dessous qui ont réussi à s'imprime tous les noms de fichiers. Cependant, au lieu de l'impression, j'ai juste besoin de les placer dans une collection. Bien sûr, je pouvais faire ce que le vieux java façon: il vous suffit d'instancier une collection, ajouter tous les éléments et à les renvoyer. Cependant, ce n'est pas de m'enseigner quoi que ce soit.

Alors, comment faites-vous cela à la cool, "Groovy"?

static File[] findAllTestFiles() {
    def directory = new File("src/test/java");
    def closure = {File f -> if(f.name =~ /Test\.java$/) println f }
    directory.eachFileRecurse FileType.FILES, closure
    return null;
}

Je suis à la recherche pour mettre en œuvre findAlltestFiles() en Groovy, en utilisant aussi peu de code que possible, tout en restant lisible.

OriginalL'auteur gMale | 2010-10-29

  1. 12

    Je voudrais essayer d'éviter la construction de la collection. À l'aide de fermetures, vous pouvez séparer la logique de sélectionner les fichiers à partir de ce que vous voulez vraiment faire, comme suit:

    import groovy.io.FileType

def withEachTestFile(Closure closure) {
    new File("src/test/java").eachFileRecurse(FileType.FILES) {
        if (it.name =~ /Test\.java$/) {
            closure.call(it)
        }
    }
}

    Alors si vous voulez faire quelque chose sur les fichiers de test, vous pouvez le faire directement sans la constitution d'une liste en mémoire:

    withEachTestFile() { println it }

    ou si vraiment vous voulez la liste, vous pouvez facilement générer, à l'aide de la collection de sens:

    def files = []
withEachTestFile() { files << it }
    Je sens que j'ai vraiment besoin de la liste des fichiers, mais je suis probablement mal (comme c'est mon premier Groovy de la classe, jamais). Je vais tenir compte de votre première suggestion de manière plus approfondie--il semble comme un grand aperçu. Je vais pouvoir jouer avec elle pour comprendre pleinement ce que vous dites et comment l'appliquer, c'est une manière complètement différente de penser à des choses! Je l'aime... groovy est croissante sur moi, vite.
    J'ai ré-écrit la totalité de mon script, en essayant de concevoir avec fermetures à l'esprit. Parfois, ça fait mal à ma tête un peu pour penser à elle, profondément. Mais je suis vendu sur Groovy, j'aurai donc un peu de livres, et commencer à maîtriser cette nouvelle approche. Je suis ravi d'apprendre comment utiliser Groovy de la Console... merci pour votre aide.

    OriginalL'auteur ataylor

  2. 10

    Une nouvelle, et plus générique de chemin à parcourir un répertoire qui prend en charge de multiples fermeture rappels est parcourir.

    import static groovy.io.FileType.FILES
...
def files = [] 
new File("src/test/java").traverse(type: FILES, nameFilter: ~/Test\.java$/) {
    files << it    
}

    OriginalL'auteur btiernay

  3. 2

    quelque chose comme cela devrait fonctionner

    def files = []
new File("src/test/java").eachFileRecurse(FILES) {
    if(it.name =~ /Test\.java$/)) {
        println f
        files << f
    }
}

    ou je pense que la modification de votre code comme ceci

    static File[] findAllTestFiles() {
    def files = []
    def directory = new File("src/test/java");
    def closure = {File f -> if(f.name =~ /Test\.java$/) {println f; files << f} }
    directory.eachFileRecurse FileType.FILES, closure
    return files;
}
    merci! votre réponse était au point et très utile. Je souhaite que je pourrais également vous attribuer des points pour la bonne réponse! Merci de prendre le temps de l'aider.

    OriginalL'auteur Aaron Saunders

  4. 2

    La suite n'a pas été testé, mais la méthode findAll devrait aider à rendre votre code très concis:

    List<File> files = new File("src/test/java").listFiles().findAll { it.name =~ /Test\.java$/ }

    OriginalL'auteur Yevgeniy Brikman