Ce script ne veut pas fonctionner

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Ce script ne veut pas fonctionner

_Faust_

Burn-out général

Bon, c'est le dernier....

Qu'est-ce qui marche pas dans ce foutu script.... :pfff:

Tu rentre la valeur et le dossier et il affiche tous les fichiers qui dépasse la valeur dans le dossier...

Code :

#!/bin/bash
echo "Taille min des fichiers a afficher : " $1
echo "dans le repertoire : " $2
echo
rm -rf temp
ls -lS $2 | cut -b39-50,57- >> temp
while read ligne
do
taille=`echo $ligne | cut -f1 -d' '`
fichier=`echo $ligne | cut -f4 -d' '`
date=`echo $ligne | cut -f2 -d' '`
mois=`echo $ligne | cut -f3 -d' '`
for i in $2; do
if [ -f $i ]; then
if [ ${taille} -gt ${1} ];then
echo -e "$fichier $date $mois $taille"
fi
fi
done
done <temp

Publicité

mirtouf

Light is right !

Messages d'erreurs ?

---------------
-~- Libérez Datoune ! -~- Camarade, toi aussi rejoins le FLD pour que la flamme de la Révolution ne s'éteigne pas ! -~- A VENDRE

_Faust_

Burn-out général

Si je laisse le "if" qui teste si c'est un fichier, j'ai aucun message d'erreur mais il liste rien

Si j'enlève ce "if", il liste ce que je veux, mais AUSSI les dossiers, et ca il ne faudrait pas. En plus il me faire un "unary error expected"...Pourtant je compare bien 2 valeures qui se compare...

Je suis tellement concentré dedans que je n'y vois plus rien...

aigles

Execute ton script avec affichage des commandes exécutées et analyse se qui se passe.

bash -x the_script taille rep 2>&1 | more

---------------
Jean Pierre.

_Faust_

Burn-out général

Peut être que mon code ne fait pas ce que je pense qu'il doit faire...

Code :

[---------------------]$ bash -x t.sh 40 coco
+ echo 'Taille min des fichiers a afficher : ' 40
Taille min des fichiers a afficher : 40
+ echo 'dans le repertoire : ' coco
dans le repertoire : coco
+ echo
+ rm -rf temp
+ ls -lS coco
+ cut -b39-50,57-
+ read ligne
++ echo
++ cut -f1 '-d '
+ taille=
++ echo
++ cut -f4 '-d '
+ fichier=
++ echo
++ cut -f2 '-d '
+ date=
++ echo
++ cut -f3 '-d '
+ mois=
+ '[' -f coco ']'
+ read ligne
++ echo 4096 Feb 14 sub
++ cut -f1 '-d '
+ taille=4096
++ echo 4096 Feb 14 sub
++ cut -f4 '-d '
+ fichier=sub
++ echo 4096 Feb 14 sub
++ cut -f2 '-d '
+ date=Feb
++ echo 4096 Feb 14 sub
++ cut -f3 '-d '
+ mois=14
+ '[' -f coco ']'
+ read ligne
++ echo 12 Feb 14 goe
++ cut -f1 '-d '
+ taille=12
++ echo 12 Feb 14 goe
++ cut -f4 '-d '
+ fichier=goe
++ echo 12 Feb 14 goe
++ cut -f2 '-d '
+ date=Feb
++ echo 12 Feb 14 goe
++ cut -f3 '-d '
+ mois=14
+ '[' -f coco ']'
+ read ligne
[------------------]$ bash t.sh 40 coco
Taille min des fichiers a afficher : 40
dans le repertoire : coco
[------------------]$

_Faust_

Burn-out général

Je l'ai modifié un peu mais ca donne pas encore le résultat escompté....

-Il ne doit pas lister les répertoires, juste les fichiers
-Il me fait cette erreur
-J'arrive pas à le formater

Code :

#!/bin/bash
echo "Taille min des fichiers a afficher : " $1
echo "dans le repertoire : " $2
echo
rm -rf temp
#ls -lS $2 | cut -b39-50,57- >> temp
ls -lS $2 >> temp
while read ligne
do
taille=`echo $ligne | cut -f 5 -d' '`
fichier=`echo $ligne | cut -f 9 -d' '`
date=`echo $ligne | cut -f 7 -d' '`
mois=`echo $ligne | cut -f 6 -d' '`
#for i in $2; do
#if [ -f "$i" ]; then
if [ ${taille} -gt ${1} ];then
echo -e "\t$fichier\t$date\t$mois$taille"
fi
#fi
#done
done <temp
#fin
[--------@--------]$ bash t.sh 60 coco
Taille min des fichiers a afficher : 60
dans le repertoire : coco
t.sh: line 19: [: -gt: unary operator expected
sub 14 Feb4096
tp1no5.sh 14 Feb996
t.sh 14 Feb492
coloce 14 Feb166
[--------@--------]$

BMOTheKiller

tu veux du récursif aussi, c'est à dire lister les fichiers des sous-répertoires aussi ?

BMOTheKiller

bon, avec du récursif :

#!/bin/sh

taille=$1
rep="$2"

[ "$1" = "" ] && read -p "Entrer la taille maximale en octet : " taille || echo "Taille maximale en octet : $taille"
[ "$2" = "" ] && read -p "Entrer le répertoire racine à lister : " rep || echo "Répertoire racine à lister : $rep"

count=0
echo
for i in `find "$rep" -type f`
do
tf=`du -b $i | cut -f1 -d'/'`
[ $tf -gt $taille ] && count=`expr $count + 1` && echo "$i : $tf"
done

[ $taille -gt 1 ] && s="s "
[ $count -gt 1 ] && ss="s"

echo
echo "Nombre de fichier$ss dans '$rep' dont la taille est supérieure à $taille octet$s: $count"
echo

résultat :

<-=[ ~/test-scripts ]=-> ./test
Entrer la taille maximale en octet : 500000
Entrer le répertoire racine à lister : /tmp

/tmp/kde-bmo/ksycoca : 1052672
/tmp/coreutils_5.0.91-2_i386.deb : 2449408
/tmp/coreutils-5.1.2-1mdk.i586.rpm : 2244608
/tmp/coreutils-5.0.91.tgz : 2461696
/tmp/coreutils-5.1.2.tgz : 2252800

Nombre de fichiers dans /tmp dont la taille est supérieure à 500000 octets : 5

Message édité par BMOTheKiller le 15-02-2004 à 08:46:38

aigles

_Faust_, je pense que ton problème vient des 'cut'.Dasn le résultat de la commande 'ls' les colonnes sont séparées par plusieurs espaces et pour cut chaque espace délimite un champ (ce n'est pas comme pour le shell).

-rwxr-xr-x 1 jp mkgroup 36 Feb 6 19:43 a.aw
^f1 ^f5

Tu te retrouves avec taille="", ce qui te donne pour le test
if [ -gt val$1 ]; then
d'où le pb de -gt: unary operator expected

Il vaut mieux mettre les opérandes entre guillemets dans le test
if [ "${taille}" -gt "${1}" ]; then

Il vaut mieux utiliser 'read' pour décomposer la ligne.
De plus comme tu liste les fichiers par ordre décroissant (-S),
tu peux arrêter de tester dés que tu tu rencontres un fichier plus petit.
[/fixed]
ls -lS $2 | \
while read perms hlinks owner group taille date mois fichier
do
if [ "${taille}" -gt "${1}" ]
then
echo -e "\t$fichier\t$date\t$mois$taille"
else
break
fi
done
[/fixed]

---------------
Jean Pierre.

aigles

aigles a écrit :

-rwxr-xr-x 1 jp mkgroup 36 Feb 6 19:43 a.aw
^f1 ^f5

ls -lS $2 | \
while read perms hlinks owner group taille date mois date3 fichier
do
if [ "${taille}" -gt "${1}" ]
then
echo -e "\t$fichier\t$date\t$mois$taille"
else
break
fi
done

---------------
Jean Pierre.

Publicité

_Faust_

Burn-out général

aigles, ca marche ceci chez toi ou il y a qq chose que je n'ai pas compris?

Code :

#!/bin/bash
echo "Taille min des fichiers a afficher : " $1
echo "dans le repertoire : " $2
echo
ls -lS $2 | \
while read perms hlinks owner group taille date mois date3 fichier
do
if [ "${taille}" -gt "${1}" ]
then
echo -e "\t$fichier\t$date\t$mois$taille"
else
break
fi
done
#fin

aigles

Désolé, ça ne marche effectivement pas, 'read' ne semble pas capable de lire des données depuis un pipe (avec ksh pas de pb).

Voila donc une nouvelle version du script:

Code :

#!/usr/bin/bash
echo "Taille min des fichiers a afficher : " $1
echo "dans le repertoire : " $2
echo
ls -lS $2 | tail +2 > $$.tmp
while read perms hlinks owner group taille date mois date3 fichier
do
if [ "${taille}" -gt "${1}" ]
then
echo -e "\t$fichier\t$date\t$mois$taille"
else
break
fi
done < $$.tmp
rm -f $$.tmp

---------------
Jean Pierre.

_Faust_

Burn-out général

bon, grâce à vous ca très bien. J'ai changé quelques trucs pour le peaufiner, et j'ai fais une boucle pour ne pas lister les dossiers.

D'ailleurs, c'est la seule facon que j'ai trouvé pour ne pas lister les dossiers, mais je me demandais si c'était "de la programmation conforme" ou si c'est inhabituel.

Code :

#!/usr/bin/bash
min=${1:?"Vous devez specifiez la taille minimum"}
folder=${2:?"Specifiez un dossier"}
echo -e "\nTaille min des fichiers a afficher : " $min
echo "dans le repertoire : " $folder
echo
if [ ! -d $folder ]
then
echo "$folder n'est pas un dossier"
else
ls -lS $folder | tail +2 > $$.tmp
while read perms hlinks owner group taille date mois date3 fichier
do
if [ "${taille}" -gt "${min}" ]
then
if [ "$perms" != "drwx------" ]
then
echo -e "\t$fichier \t\t$date $mois\t\t$taille"
fi
else
break
fi
done < $$.tmp
fi
rm -f $$.tmp

Message édité par _Faust_ le 15-02-2004 à 22:11:58

aigles

Tu ne devrais pas tester de cette façon si le fichier est un répertoire.
Les permissions sur le répertoires ne sont pas obligatoirement "drwx------", par contre cela commence toujours par "d". Il suffit fonc d'éliminer les fichiers pour lesquels perms commence par "d" :

Code :

ls -lS $folder | tail +2 > $$.tmp
while read perms hlinks owner group taille date mois date3 fichier
do
[[ "$perms" = d* ]] && continue
[[ "${taille}" -gt "${min}" ]] && break
echo -e "\t$fichier \t\t$date $mois\t\t$taille"
done < $$.tmp

---------------
Jean Pierre.

FORUM HardWare.fr

Linux et OS Alternatifs

Codes et scripts

Ce script ne veut pas fonctionner

Sujets relatifs
Pg_DUmp : script de sauvegarde, mais demande automatique du password ?	Script qui fonctionne, mais
[2.6.2] je n'arrive plus à faire fonctionner ma carte son !	Utiliser Lynx - links - W3m dans un script shell
Script à lancé au démarage sous l'user "xyz"	Créer un petit menu dans un script bash
serveur, script inscription automatique	"depmod : ...Makefile is not an ELF file" ca veut dire koi ? pliz help
[script sh] tester si une partition est montée?	Tomcat ne veut pas se lancer sur le port 80
Plus de sujets relatifs à : Ce script ne veut pas fonctionner

Page générée en 0.111 secondes