Shell Programming and Scripting

Code:

cp $PATHLOGS/$DATE/*.* $TMP/logs_tmp/
cp $PATHLOGS/$DATE1/*.* $TMP/logs_tmp/

Before copying the files I have to check if the directory $DATE1 and $DATE2 exists.
If directory exists then, check if the folder contains some files.
if the file exists then, check if the file size is greater than 0.

If any of the above condition doesn't satisfy in the above directory, I have should stop the script executing.

In this case, you just check if the file size is 0 or not.

Because if file size is bigger then 0, that's mean the directory is exist, the file is exist.

---------- Post updated at 03:59 PM ---------- Previous update was at 03:31 PM ----------

Code:

[ -s FILE ]             #  True if FILE exists and has a size greater than zero.

Code:

if [ -d "$PATHLOGS/$DATE1" -a -d "$PATHLOGS/$DATE2" ]; then
  for files in `ls -A PATHLOGS/$DATE1`
  do
    if [ -s "$files" ]; then
      cp $files $TMP/logs_tmp
    fi
  done
fi

citaylor uses gives a good example, however to not incur the overhead of a seperate subshell (backticks open a new shell) in your for loop...
I would actually allow the shell to do some of that work.

Code:

#!/bin/bash

# With shell 'globbing' the '$PATHLOGS/$DATE1/* ' in the for loop below
# will be expanded to "ls $PATHLOGS/$DATE1/* " without the
# overhead of a new shell or even the "ls" command!

if [ -d "$PATHLOGS/$DATE1" -a -d "$PATHLOGS/$DATE2" ]; then
  for f in $PATHLOGS/$DATE1/*  #<------ Shell 'globbing' & expansion
  do
    if [ -s "$f" ]; then
      /bin/cp $f $TMP/logs_tmp
    fi
  done
fi

Another example that would be even more simple would be:

Code:

#!/bin/bash
if [ -d "$PATHLOGS/$DATE1" -a -d "$PATHLOGS/$DATE2" ]; then
    find $PATHLOGS/$DATE1 -type f ! -size 0 -exec cp -v {} $TMP/logs_tmp \;
fi

To explain:

The find command will look under '$PATHLOGS/$DATE1' for all files that exist and are not folders (-type f) that are not 0 bytes (! -size 0).
Next it will execute a copy command on all files found matching our requirements and copy them to $TMP/logs_tmp ('-exec cp -v {} $TMP/logs_tmp \;').

The '{}' is a place holder for each file as it finds them, and the '\;' at the end is to tell find that this is the end of the exec command.
The nature of the find command is to act on each file as it finds them, much like a for loop would. This way it can exec the command on each file one by one.

Note that the semicolon must be escaped (hence the '\;').
Also note the space between the '!' (not modifier) and the '-size' flag.