Skip to content
Advertisement

Check if a directory exists and is accessible

I want to check if a directory exists and it has access rights; if it does, then perform the tasks. This is the code I wrote, which might not have proper syntax.

Can you help me to correct it?

dir_test=/data/abc/xyz
if (test -d $dir_test & test –x $dir_test -eq 0);
 then
cd $dir_test
fi

I believe this can also be written like this.

dir_test=/data/abc/xyz
test -d $dir_test
if [ $? -eq 0 ];
then
test –x $dir_test
if [ $? -eq 0 ];
then
cd $dir_test
fi
fi

How can we write this more efficiently?

Advertisement

Answer

The best way to write the original test-based solution would be

if test -d "$dir_test" && test –x "$dir_test";
then
    cd $dir_test
fi

although what will you do if the test fails and you don’t change directories? The remainder of the script will probably not work as expected.

You can shorten this by using the [ synonym for test:

if [ -d "$dir_test" ] && [ -x "$dir_test" ]; then

or you can use the conditional command provided by bash:

if [[ -d "$dir_test" && -x "$dir_test" ]]; then

The best solution, since you are going to change directories if the tests succeed, is to simply try it, and abort if it fails:

cd "$dir_test" || {
  # Take the appropriate action; one option is to just exit with
  # an error.
  exit 1
}
User contributions licensed under: CC BY-SA
2 People found this is helpful
Advertisement