#!/usr/bin/perl
use strict;
use warnings;
use File::Copy;    
my $value0 = 1;                        #change this to 0 and execute a
+nd back to one and execute
my $value1 ='true';
open (INFO, "> tmp");
print INFO "this is simple\n";
close (INFO);
open (FILE, ">> RESULT_LOG") ;
    print FILE " STATUS obtained : $value1\n";
if ($value0 == 1)
    {                                # If true then copies the informa
+tion to the RESULT_LOG
    copy("tmp","RESULT_LOG");}
else {
print FILE "Check keyword supplied\n";}
close(FILE);
[download]

Replace:

    copy("tmp", "RESULT_LOG");
[download]

with:

    open my $tmpIn, '<', "tmp" or die "Failed to open tmp: $!";
    open my $log, '>>', or die "Failed to open RESULT_LOG: $!";
    print {$log} <$tmpIn>;
    close $log;
    close $tmpIn;
[download]

Copy overwrites so that is not what you wanted.

Note: you should always use three parameter open and lexical file handles.

True laziness is hard work

#!/usr/bin/perl
use strict;
use warnings;
use File::Copy;    

#change this to 0 and execute, then back to one and execute
my $value0 = 1;
my $value1 ='true';

## marmot: open for overwriting
open (INFO, "> tmp");
print INFO "this is simple\n";
close (INFO);

## marmot: open RESULT_LOG for appending
open (FILE, ">> RESULT_LOG") ;
print FILE " STATUS obtained : $value1\n";

# If true then copies the information to the RESULT_LOG

## marmot: translation --> if $value0 is true, overwrite RESULT_LOG wi
+th tmp.
##         By the way, <FILE>, which is RESULT_LOG, is still open for 
+appending, 
##         so this is probably a bad idea.
if ($value0 == 1) { 
    copy("tmp","RESULT_LOG");}
## marmot: Otherwise, append RESULT_LOG with text.
else {
    print FILE "Check keyword supplied\n";
}
close(FILE);
[download]

Apart from confusing copying with appending, and trying to overwrite a file that's already open for appending, this code can be made to work with the changes below.

if ($value0 == 1) { 
    open INFO, "<tmp";
    print FILE <INFO>; # Appends to RESULT_LOG
else {
    print FILE "Check keyword supplied\n";
}
close(FILE);
close(INFO);
[download]

I realize this is just a snippet to show what you're trying to accomplish, but I have to wonder if it makes sense to open 'tmp' and write text to it, when you're just going to append it to another file later. Why not put this information in a variable and keep it around until you need to write it to a file?

Also, for the sake of your own clarity, you might want to give your file handles names that are closer to the file names they represent, or perhaps the functions they perform. E.g.,

open TMP,     "<tmp";
open LOG,     ">>RESULT_LOG";
open RESULTS, ">>RESULT_LOG";
[download]

--marmot

Hi Marmot, Thanks for the help. I was able to get my problem solved . Using this code

if (true)        
    {open (my $fh, ">> RESULT_LOG");
    copy "Battery_Status", $fh;}
else {print "Error";}
[download]

I was not storing it in a variable because it can contain a lot of information, since its an output coming after the execution of a command on a hardware device ! So had to store in a file and also I am parsing it for some keyword and if thats true then the above code is executed to append the result from the file to the log.