Discussion :

[seanbean]

Bonsoir,

il fallait écrire un script qui prend deux noms de répertoires et qui compte le nombre de fichiers ayant le même nom dans les deux répertoires.

La stratégie que j'ai écrite, et qui ne fonctionne pas, est de mettre deux boucles imbriquées pour comparer chaque nom de fichier avec chaque fichier de l'autre répertoire.

L'appel du script est comme ça : ./script /home/sean/rep1 /home/sean/rep2

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#!/bin/bash
compteur=0
	for file1 in $1/*
	do
		for file2 in $2/*
		do
			if [ "$file1" = "$file2" ]; then
				compteur=$((compteur+1))
			fi
		done
	done
echo "le nombre de fichiers ayant le meme nom est $compteur"

Merci de votre aide !

[fransoo]

il fallait écrire un script qui prend deux noms de répertoires et qui compte le nombre de fichiers ayant le même nom dans les deux répertoires.

La stratégie que j'ai écrite, et qui ne fonctionne pas, est de mettre deux boucles imbriquées pour comparer chaque nom de fichier avec chaque fichier de l'autre répertoire.

L'appel du script est comme ça : ./script /home/sean/rep1 /home/sean/rep2

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#!/bin/bash
compteur=0
	for file1 in $1/*
	do
		for file2 in $2/*
		do
                        echo "file1='$file1' - file2='$file2'"
			if [ "$file1" = "$file2" ]; then
				compteur=$((compteur+1))
			fi
		done
	done
echo "le nombre de fichiers ayant le meme nom est $compteur"

La stratégie est bonne mais je vais te mettre sur une piste si tu ajoutes la ligne en gras dans le script.
Autre chose : si les noms de fichiers contiennent des espaces, ça ne marchera pas non plus.

[matafan]

Les espaces dans les noms de fichier ne devraient pas poser de problème.

Avec ton algo ça va vite devenir lent si tu as beaucoup de fichiers dans tes répertoires (complexité en O(n²)). Il serait plus efficace de créer une hash table des fichers du premier répertoire, puis de regarder si les fichiers du deuxième répertoire sont dans la hash table. Tu peux faire ça en awk ou en perl par exemple.

[N_BaH]

bah !?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
for file in $1/*
do [ -f "$2/${file##*/}" ] && ((nb++))
done
echo "$nb fichiers portent le même nom"

et pi sc'est tout.

[seanbean]

bah !?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
for file in $1/*
do [ -f "$2/${file##*/}" ] && ((nb++))
done
echo "$nb fichiers portent le même nom"

et pi sc'est tout.

J'admire ton intelligence, c'est une trop bonne méthode que j'ai bien compris
Etant donné que je cherche exactement le meme nom de fichier, j'ai modifié :
il ne marche toujours pas!

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
#!/bin/bash
compteur=0
	for file in $1/*
	do
		if [ -f "$2/$file" ]; then
			compteur=$((compteur+1))
		fi
	done
echo "le nombre de fichiers ayant le meme nom est $compteur"

Le problème vient de "$file" qui contient l'ensemble du chemin "/home/sean/Bureau/rep1/lenomdufichier" !!, car j'ai executé ce script pour tester et ça donne le chemin entier dans la variable "$file" :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
#!/bin/bash
for fichier in $1/*
do
	if [ -f "$fichier" ]; then
		echo $fichier
	fi
done

Résultat :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
/home/sean/Bureau/rep1/lettre
/home/sean/Bureau/rep1/photo
/home/sean/Bureau/rep1/image
/home/sean/Bureau/rep1/crayon

[seanbean]

Les espaces dans les noms de fichier ne devraient pas poser de problème.

Avec ton algo ça va vite devenir lent si tu as beaucoup de fichiers dans tes répertoires (complexité en O(n²)). Il serait plus efficace de créer une hash table des fichers du premier répertoire, puis de regarder si les fichiers du deuxième répertoire sont dans la hash table. Tu peux faire ça en awk ou en perl par exemple.

Merci pour ta réponse,
Hash table?! j'ai cherché sur internet mais je n'ai trouvé aucune réponse
Peux-tu m'expliquer plus stp ?

[fransoo]

Hash table?!

C'est un tableau qui au lieu d'avoir des entiers en index peut prendre des strings ou autre en guise d'index.

[seanbean]

C'est un tableau qui au lieu d'avoir des entiers en index peut prendre des strings ou autre en guise d'index.

Ah d'accord c'est comme en php j'imagine
Est-ce que vous avez un lien où apprendre cela svp ?

[seanbean]

La stratégie est bonne mais je vais te mettre sur une piste si tu ajoutes la ligne en gras dans le script.
Autre chose : si les noms de fichiers contiennent des espaces, ça ne marchera pas non plus.

Au fait je cherche exactement le meme fichier sans espace, donc les deux fichiers doivent etre exactement les memes.
Etant débutant, je ne comprends pas la ligne que t'as ajouté en fait.