Bash script help... menu to pick file from folder

bfellenr · 02-22-2008, 09:22 AM

Hello,

I'm trying to figure out how to make a bash script that will basically give you a menu of files in a folder then let you pick one. I would like to have it desplay a number, then the tgz filename, then read text from a file (only contains a single line) with the same name but ending in txt. There would be no more than 10 files. Example folder and screen mockup below.

Folder structure...
-------------------
file-2008-02-20.tgz
file-2008-02-20.txt
file-2008-02-21.tgz
file-2008-02-21.txt
file-2008-02-22.tgz
file-2008-02-22.txt

Menu mock up...
---------------

Extraction process
==================

1. file-2008-02-20.tgz : Here is the single line read from file-2008-02-20.txt

2. file-2008-02-21.tgz : Here is the single line read from file-2008-02-21.txt

3. file-2008-02-22.tgz : Here is the single line read from file-2008-02-22.txt

Pick the file to extract:

Thanks for the help

slakmagik · 02-22-2008, 09:41 AM

select file in *; do tar xf $file; done

-- maybe not - I'm not sure I follow what you want to do with the text files but this should at least point you in the right direction.

Disillusionist · 02-22-2008, 01:52 PM

Code:

l_count=0
ls *.tgz|while read l_file
do
   l_count=`expr $l_count + 1`
   echo "${l_count}. ${l_file} `cat $(l_file%.tgz).txt`"
done
echo
echo "Pick the file to extract:"

Not tested as I am waiting to go home...

bfellenr · 02-24-2008, 10:09 AM

That worked for the most part. The problem I have that when I try to set the filename to the FILE# variable in the do loop it doesn't stay set when the loop is exited even though I don't declare it as local. I need to basically equate the number they type to the filename so I can perform my operation after their selection.

Code:

L_COUNT=0
FILE0=0 FILE2=0 FILE3=0 FILE4=0 FILE5=0 FILE6=0 FILE07= FILE8=0 FILE9=0

ls *.tgz|while read L_FILE

do
	echo $L_COUNT. $L_FILE : `cat ${L_FILE%.tgz}.txt`

	case "$L_COUNT" in
		0) FILE0=$L_FILE ;;
		1) FILE1=$L_FILE ;;
		2) FILE2=$L_FILE ;;
		3) FILE3=$L_FILE ;;
		4) FILE4=$L_FILE ;;
		5) FILE5=$L_FILE ;;
		6) FILE6=$L_FILE ;;
		7) FILE7=$L_FILE ;;
		8) FILE8=$L_FILE ;;
		9) FILE9=$L_FILE ;;
	esac

	L_COUNT=`expr $L_COUNT + 1`
done

echo
echo -n "Pick the file to extract: "
read -e CHOICE
echo

case "$CHOICE" in
	0) CHOICE=$FILE0 ;;
	1) CHOICE=$FILE1 ;;
	2) CHOICE=$FILE2 ;;
	3) CHOICE=$FILE3 ;;
	4) CHOICE=$FILE4 ;;
	5) CHOICE=$FILE5 ;;
	6) CHOICE=$FILE6 ;;
	7) CHOICE=$FILE7 ;;
	8) CHOICE=$FILE8 ;;
	9) CHOICE=$FILE9 ;;
esac

echo You picked $CHOICE

Disillusionist · 02-25-2008, 02:50 AM

export the variables:

Code:

case "$L_COUNT" in
		0) export FILE0=$L_FILE ;;

bfellenr · 02-26-2008, 06:48 AM

Tried that. While in the do loop it works just fine. I can get the value that was set in the case with/without doing the export. The minute the script exits the for loop the value is back to what it was first initialized as.

slakmagik · 02-26-2008, 08:00 AM

Code:

#!/bin/sh
tarballs=(*.tar.gz)
txtfiles=(*.txt)
for (( i=0; i<${#tarballs[*]}; i++ )); do
    echo $i: ${tarballs[$i]}: $(cat ${txtfiles[$i]})
done
read -ep "Which one? "
tar xf ${tarballs[$REPLY]}

The above untars the file I pick by number. A "loop runs in a subshell when it's part of a pipeline." FAQ #24

-- Oh - it might be considered more user-friendly to start the list from 1.

Code:

#!/bin/sh
tarballs=(*.tar.gz)
txtfiles=(*.txt)
for (( i=0; i<${#tarballs[*]}; i++ )); do
    echo $(($i+1)): ${tarballs[$i]}: $(cat ${txtfiles[$i]})
done
read -ep "Which one? "
tar xf ${tarballs[$(($REPLY-1))]}

520Soul · 01-13-2011, 03:21 PM

hm...let me try the last script to see if it work....thanks guy...

---------- Post added 01-13-11 at 04:21 PM ----------

Quote:

Originally Posted by slakmagik

Code:

#!/bin/sh
tarballs=(*.tar.gz)
txtfiles=(*.txt)
for (( i=0; i<${#tarballs[*]}; i++ )); do
    echo $i: ${tarballs[$i]}: $(cat ${txtfiles[$i]})
done
read -ep "Which one? "
tar xf ${tarballs[$REPLY]}

The above untars the file I pick by number. A "loop runs in a subshell when it's part of a pipeline." FAQ #24

-- Oh - it might be considered more user-friendly to start the list from 1.

Code:

#!/bin/sh
tarballs=(*.tar.gz)
txtfiles=(*.txt)
for (( i=0; i<${#tarballs[*]}; i++ )); do
    echo $(($i+1)): ${tarballs[$i]}: $(cat ${txtfiles[$i]})
done
read -ep "Which one? "
tar xf ${tarballs[$(($REPLY-1))]}

######### i still got problem using the script that you post. #############

Code:

#!/bin/sh
tarballs=(*.tar.gz)
txtfiles=(*.txt)
for (( i=0; i<${#tarballs[*]}; i++ )); do
    echo $(($i+1)): ${tarballs[$i]}: $(cat ${txtfiles[$i]})
done
read -ep "Which one? "
tar xf ${tarballs[$(($REPLY-1))]}

[/QUOTE]

################## sorry i'm a newbie on bash...please help..below is the problem that i've been having. #####################

echo This is test
1: *.tar.gz:
Which one? zzz
./test.sh: line 60: tarballs: bad array subscript <----------
tar: Option f requires an argument <---------- this two line show the problem. it need " f " for an argument? please help ..
Usage:
List: tar -tf <archive-filename>
Extract: tar -xf <archive-filename>
Create: tar -cf <archive-filename> [filenames...]
Help: tar --help

slakmagik · 01-14-2011, 03:57 PM

I can't quite follow your post. If you're trying the little snippet and getting error messages it's because it's basically a demo that does no error checking and expects the tarballs and text files to be in the current directory and expects a valid argument. If you don't have such files, you'll get '1: *.tar.gz' when glob expansion fails and the array won't be populated and tar will have nothing to work on.

Disillusionist · 01-15-2011, 03:44 AM

Additionally, if you are on a system where /bin/sh is not a symbolic link to /bin/bash you would get the following error:

Quote:

./test3.sh: 2: Syntax error: "(" unexpected

Therefore you should replace

Code:

#!/bin/sh

with

Code:

#!/bin/bash

As per slakmagik, this code was specific for the Original Poster's task with no real error checking in place.

If you are running this exact code then your error message should have said:

Quote:

1: *.tar.gz: This is a test
Which one? q
./test.sh: line 8: tarballs: bad array subscript
tar: Old option `f' requires an argument.
Try `tar --help' or `tar --usage' for more information.

Your error occurs because the response was not numeric, but you would still have had issues with a numeric value outside the range for the array.

Disillusionist · 01-15-2011, 09:43 AM

Version with basic testing added:

Code:

#!/bin/bash
tarballs=(*.tar.gz)
txtfiles=(*.txt)
for (( i=0; i<${#tarballs[*]}; i++ )); do
    if [ $i -eq 0 ]
    then
       if [ "${tarballs[$i]}" == '*.tar.gz' ]
       then
          echo "no tar files found!"
          exit 1
       fi
    fi
    if [ "${txtfiles[$i]}" == '*.txt' -o "${txtfiles[$i]}" == '' -o $i -gt ${#txtfiles[*]}]
    then
       echo $(($i+1)): ${tarballs[$i]}
    else
       echo $(($i+1)): ${tarballs[$i]}: $(cat ${txtfiles[$i]})
    fi
done
read -ep "Which one? "
if [ "${REPLY}A" == "$(echo $REPLY|sed 's/[^0-9]//g')A" -a "$REPLY" != '']
then
   ## Numeric
   if [ $REPLY -le ${#tarballs[*]} ]
   then
      echo "Extracting ${tarballs[$((REPLY-1))]}"
      tar xf ${tarballs[$(($REPLY-1))]}
   else
      echo "numeric value out of range"
      exit 2
   fi
else
   echo "$REPLY not numeric"
   exit 3
fi

schneidz · 01-15-2011, 10:23 AM

Quote:

Originally Posted by bfellenr

Hello,

I'm trying to figure out how to make a bash script that will basically give you a menu of files in a folder then let you pick one. I would like to have it desplay a number, then the tgz filename, then read text from a file (only contains a single line) with the same name but ending in txt. There would be no more than 10 files. Example folder and screen mockup below.

Folder structure...
-------------------
file-2008-02-20.tgz
file-2008-02-20.txt
file-2008-02-21.tgz
file-2008-02-21.txt
file-2008-02-22.tgz
file-2008-02-22.txt

Menu mock up...
---------------

Extraction process
==================

1. file-2008-02-20.tgz : Here is the single line read from file-2008-02-20.txt

2. file-2008-02-21.tgz : Here is the single line read from file-2008-02-21.txt

3. file-2008-02-22.tgz : Here is the single line read from file-2008-02-22.txt

Pick the file to extract:

Thanks for the help

heres my stab at it (season to taste):

Code:

[liveuser@localhost temp]$ cat bfellenr.ksh 
#!/bin/bash

echo select a file:
num=1
for item in `ls -1 *.mt`
do
 txt=`echo $item | sed s/mt/txt/`
 echo $num: $item: `cat $txt` 
 num=`expr $num + 1`
done | tee menu.lst

read num
#echo num = $num
sed -n "$num"p menu.lst | cut -d : -f 2
[liveuser@localhost temp]$ ./bfellenr.ksh 
select a file:
1: akuma.mt: gouki
2: chun-li.mt: shadow-law
3: h4x0rz.mt: hacker
4: hello.mt: hi
5: l33t.mt: elite
6: world.mt: planet
5
 l33t.mt

slakmagik · 01-15-2011, 01:56 PM

Quote:

Originally Posted by Disillusionist

Additionally, if you are on a system where /bin/sh is not a symbolic link to /bin/bash you would get the following error:

Therefore you should replace

Code:

#!/bin/sh

with

Code:

#!/bin/bash

Good point. The OP was asking for bash and that's what it is, but I must have typed sh out of habit.

Disillusionist · 01-15-2011, 03:30 PM

Quote:

Originally Posted by schneidz

heres my stab at it (season to taste):

Code:

[liveuser@localhost temp]$ cat bfellenr.ksh 
#!/bin/bash

echo select a file:
num=1
for item in `ls -1 *.mt`
do
 txt=`echo $item | sed s/mt/txt/`
 echo $num: $item: `cat $txt` 
 num=`expr $num + 1`
done | tee menu.lst

read num
#echo num = $num
sed -n "$num"p menu.lst | cut -d : -f 2
[liveuser@localhost temp]$ ./bfellenr.ksh 
select a file:
1: akuma.mt: gouki
2: chun-li.mt: shadow-law
3: h4x0rz.mt: hacker
4: hello.mt: hi
5: l33t.mt: elite
6: world.mt: planet
5
 l33t.mt

To resolve any issues with missing txt files you could modify it slightly:

Code:

 echo $num: $item: `cat $txt 2>/dev/null`