View difference between Paste ID: 1Az7ZWZw and xbZEr8hm

 #!/bin/bash
 #ismael moreno
 #for.sh
 #peso de cada perfil,superan 200MB ¿bloquear el accesso?,bin/false
 #se que me falta comentarle
 
-for VARIABLE in `ls - /home`
+# Vale, pues lo comentaré yo. (angeldp)
-do
+
-PERSOPERFILES=`du -s "/home/$VARIABLE" 2> /dev/null | cut -d: -f1`
+
-echo " el peso de los perfiles es de $PESOPERFILES "
+# Vale, lista de carpetas en /home
-done
+# el problema es que
-sleep 2
+for VARIABLE in `ls /home`
-echo " este perfil es de $VARIABLE "
+	do
 # Mejor du -sh
-if [ "$PERSOPERFILES" -gt "200000" ]
+	PESOPERFILES=`du -s "/home/$VARIABLE" 2> /dev/null | cut -d: -f1`
-then
+# Mejor echo "El directorio $VARIABLE ocupa $PESOPERFILES"
-echo " la carpeta /home/$VARIABLE del usuario $VARIABLE pasa del limite permitido"
+	echo " el peso de los perfiles es de $PESOPERFILES "
-echo " ¿quiere bloquear la cuenta? "
+# done
-sleep 2
+# El problema del bucle for anterior es que no lo aprovechas.
-echo " pulse s para aceptar o n para denegar "
+# por eso comento el fin del bucle. Debe finalizar abajo del todo
-read -n1 OPC
+# Todo lo que pones a continuación debería ir dentro. Una
-fi
+# vez que ha finalizado $VARIABLE y $PESOPERFILES no se modifican
 # sleep 2
-case $OPC in
+ 	echo " este perfil es de $VARIABLE "
-s | S )
+
-echo " se esta procediendo a bloquear la cuenta "
+# Las comillas son solo para cadenas de caracteres, con números no
-echo  " espere "
+#	if [ "$PERSOPERFILES" -gt "200000" ]
-sleep 1
+	if [ $PESOPERFILES -gt 200000 ]
-echo "."
+		then
-sleep 1
+		echo " la carpeta /home/$VARIABLE pasa del limite permitido"
-echo "."
+		echo " ¿quiere bloquear la cuenta? "
-sleep 1
+		echo " pulse s para aceptar o n para denegar "
-echo "."
+		read -n1 OPC
-BLOQUEAR=`more etc/passwd | grep ^{$VARIABLE}: | cut -d ":" -f7 > campo bueno.txt
+# El if no acaba aquí
-# creo que eh cogido bien el campo siete del usuario que yo queria ahora no se como cogiendo ese campo como modificarle... y cambiarle por /bin/bash.
+# fi
-echo " terminaria de hacerlo" ;;
+		case $OPC in
 			s | S )
 			echo " se esta procediendo a bloquear la cuenta "
 			echo  " espere "
 			sleep 1
 			echo "."
 			sleep 1
 			echo "."
 			sleep 1
 			echo "."
 # La variable BLOQUEAR no pinta nada, ojo con las llaves,corta los campos buenos (-f1-6) y ojo con espacios en nombres de archivos
 # BLOQUEAR=`more etc/passwd | grep ^{$VARIABLE}: | cut -d":" -f7 > campo bueno.txt
 			LINEA=`more /etc/passwd | grep ^${VARIABLE}: | cut -d: -f1-6`
 # creo que eh cogido bien el campo siete del usuario que yo queria ahora no se como cogiendo ese campo como
 # modificarle... y cambiarle por /bin/bash.
 # Pues algo así como añadir /bin/false a la línea que no tenía campo 7
 			LINEA="${LINEA}:/bin/false"
 Solo queda eliminar la línea del archivo con grep -v
 			more /etc/passwd | grep -v ^${VARIABLE}: > miPasswd
 # Añadir la línea al fichero modificado
 			echo $LINEA >> miPasswd
 # Y mover el archivo a su sitio (tienes que ser root, igual deberías comprobarlo antes y redireccionar los errores
 			mv miPasswd /etc/passwd;;
 # echo " terminaria de hacerlo" ;;
 n | N )
 echo " gracias por todo "
 exit
 esac
 # cuando lo leas no seas malo y no te rias mucho de mi ajajajjaja
 # el exit le eh puesto hay pero dudaba si ponerle antes
 # no pongas el examen muy dificil que mira que nivel mas bajo tengo jajaj