Récuperer un identifiant dans un nom de fichier avec une REGEXP

Bonjour,

merci cela ne correspondait pas exactement a ce que je voulais mais je l'ai adapté :
([sS]\d{1,2}[eE]\d{1,2})

, je souhaite que seul le numéro de l'épisode match en fait, j'écrase le reste du nom de fichier

en tous cas merci pour le site il est génial !

Bonjour,

merci pour tes conseils, j'avais déjà commencer a refactorer mon code avec certain de tes conseil avant :) Mais je prend note de certain oubli en effet :)

voici la version avant ta réponse :

	public static void main(String[] args) {

		long start = System.currentTimeMillis();

		if (args.length == 0) {
			System.out.println(
					"USAGE : RenameSeriesFiles racineDuNomDeFichier [repertoire des fichier][-auto] \n -auto : repertoire courrant");
			return;
		}

		String racine = args[0];
		String root = args[1];

		if (root == "-auto") {
			root = ".\\";
		}
		System.out.println("root : " + root);
		System.out.println("serie : " + racine);

		renameProcess(root, racine);
		
		long end = System.currentTimeMillis();
		System.out.println("temps de traitement : " + (end - start) + "ms");
	}
	
	private static void renameProcess(String root, String prefixeNomFichier) {
		File dossier = new File(root);

		if (dossier.isDirectory()) {

			// Pattern pattern =
			// Pattern.compile("#^(.*)-S?([0-9]*)E([0-9]*)#i");
			Pattern pattern = Pattern.compile("S\\d\\dE\\d\\d");

			File[] files = dossier.listFiles();

			System.out.println("nombre de fichiers trouvés : " + files.length + "\n");
			int i = 0;
			int j = 0;
			for (File file : files) {
				if (!file.isDirectory()) {
					i++;
					String name = file.getName().toUpperCase();
					String extension = name.substring(name.lastIndexOf(".")).toLowerCase();
					//file.renameTo(new File(dossier.getPath() + "\\" + name.replaceAll(".S4E", ".S04E")));
					Matcher mattcher = pattern.matcher(name);
					if (mattcher.find()) {
						j++;
						String identifiant = mattcher.group(0);
						String newName = "\\" + prefixeNomFichier + " " + identifiant + extension;
						System.out.println(file.getName());
						System.out.println("rename >> " + newName + "\n");
						File newFile = new File(dossier.getPath() + newName);
						if (!fileExistInDirectory(dossier, newFile.getName())) {
							file.renameTo(new File(dossier.getPath() + newName));
						} else {
							System.err.println("un fichier porte déjà ce nom dans le dossier : " + newFile.getName());
						}
					} else {
						System.err.println(
								"l'identifiant de l'episode n'a pas été trouvé pour le fichier : " + file.getName());
					}
				}
			}

			System.out.println("nombre de fichiers : " + i);
			System.out.println("nombre de repertoires : " + (files.length - i));
			System.out.println("nombre de fichiers renomés : " + j);
		}
	}

	private static boolean fileExistInDirectory(File directory, String fileName) {
		boolean result = false;

		if (directory.isDirectory() && fileName != null) {
			for (File file : directory.listFiles()) {
				if (fileName.equals(file.getName())) {
					return true;
				}
			}
		} else {
			result = true;
		}
		return result;
	}

Remarque :

Généralement on mets

x.equals(y)

lorsque c'est la variable

x

qui nous intéresse et que l'on regarde la valeur

y

. Dans ton cas il serait plus logique de mettre

root.equals(AUTO)

De toute façon args[1] ne peut pas être null... donc root non plus, du coup

AUTO.equals(root)

ou

root.equals(AUTO)

les deux ne posent aucun risque.

Et s'il y avait un risque d'avoir null, il serait mieux de tester explicitement la valeur avec un

if (x==null)

plutôt que déléguer ce contrôle à une méthode qui même si elle gère le cas peut faire plusieurs opérations avant de contrôler la valeur null.

Exemple, dans ton cas c'est la méthode equals de la classe String, voici son code :

    public boolean equals(Object anObject) {
        if (this == anObject) {
            return true;
        }
        if (anObject instanceof String) {
            ...
        }
        return false;
    }

Le cas où anObject est null est géré par instanceof qui renvoie false, mais la méthode a d'abord testé

this == anObject

au préalable, ce qui est inutile... on aurait tout simplement pu ne jamais appeler la méthode equals.