Of course.

Make sure the code you use is correct; in the script you posted, it was incomplete.

Just saying you made a change is not enough; I need to see the code you are trying to execute.

The case statment does not carry out the action; it makes a decision about which action is to be carried out. You then need the code to carry out the action.

You might find it helpful to put sections of your code into functions so that they can be tested idependently.

For example, you might want these functions: build_list, show_list, ask_user and move_files.

After you have written and tested the functions, the rest of your script could look like this:

Code:

build_list <file pattern>
show_list
if ask_user
then
   move_files
else
   echo "No changes made ..."
fi

I didn't suggest anything. I posted an example of what might be a better way of doing it. I don't know whether it is correct or not because you haven't explained the file selection criteria.

Does it do what you want? Does the wildcard pattern match the files you want?

What values are $1 and $2 likely to contain?

Does the following code display the files you want to work with?

Code:

printf "%s\n" *$1*$2*

It is explained in the "parameter expansion" section of your shell's man page. To see what the expansion does, try printing the individual parts:

Code:

printf "%s\n" "$i" "${i%%.Sent*}" "${i#*.Sent}"