Hi
I'm new to Perl language and I have this assignment which is killing me, CAN SOMEONE HELP ME PLEASE. Below is the question:

You are to write a Perl program that analyses text files to obtain statistics on their content. The program should operate as follows:

1) When run, the program should check if an argument has been provided. If not, the program should prompt for, and accept input of, a filename from the keyboard.

2) The filename, either passed as an argument or input from the keyboard, should be checked to ensure it is in MS-DOS format. The filename part should be no longer than 8 characters and must begin with a letter or underscore character followed by up to 7 letters, digits or underscore characters. The file extension should be optional, but if given is should be ".TXT" (upper- or lowercase).

If no extension if given, ".TXT" should be added to the end of the filename. So, for example, if "testfile" is input as the filename, this should become "testfile.TXT". If "input.txt" is entered, this should remain unchanged.

3) If the filename provided is not of the correct format, the program should display a suitable error message and end at this point.

4) The program should then check to see if the file exists using the filename provided. If the file does not exist, a suitable error message should be displayed and the program should end at this point.

5) Next, if the file exists but the file is empty, again a suitable error message should be displayed and the program should end.

6) The file should be read and checked to display crude statistics on the number of characters, words, lines, sentences and paragraphs that are within the file.


I HAVE MANAGE TO WRITE THE FOLLOWING BUT ANY TIME I EXECUTE IT I GET THE ff: Global symbol "filename" requires explicit package name form C: directory.

Below is my code, can anyone check and find out what is wrong, please:

#!/usr/bin/perl 

use strict; 
use warnings; 

if ($#ARGV == -1) #no filename provided as a command line argument. 
{ 
print("Please enter a filename: "); 
$filename = <STDIN>; 
chomp($filename); 
} 
else  
{ 
$filename = $ARGV[0]; 
} 
 
if ($filename !~ m/^[a-z]{1,7}\.TXT$/i) 
{ 
die("File format not valid\n"); 
} 

if ($filename !~ m/\.TXT$/i) 
{ 
$filename .= ".TXT"; 
} 
 
if (-e $filename) 
{ 
die("File does not exist\n"); 
} 
 
if (-s $filename) 
{ 
die("File is empty\n"); 
} 

my $i = 0; 
my $p = 1; 
my $words = 0; 
my $chars = 0; 

open(READFILE, "<$data.txt") or die "Can't open file '$filename: $!"; 

while (<READFILE>) { 
chomp;  
$i = $.; #"$". is the input record line numbers, $i++ will also work 
$p++ if (m/^$/); #count paragraphs 
$my @t = split (/\s+/); #split sentences into "words" 
$words ++;= @t; #add count to $words 
$chars += tr/ //c; #tr/ //c count all characters except spaces and add to $chars 
} 
 
print "There are $i lines in $data\n"; 
print "There are $p Paragraphs in $data\n"; 
print "There are $words in $data\n"; 
print "There are $chars in $data \n"; 

close(READFILE);

I am surprised that you got this program to run at all.

$my @t = split (/\s+/); #split sentences into "words"
$words ++;= @t; #add count to $words

These two lines contain syntax errors.

The reason your program will not run (and you're getting the error that you're getting) is because you are using strict, but not programming in a strict fashion. You're not declaring the scope of the variable filename, which is why it is bombing. If you take strict out, you will see that you have some fairly straight forward syntax errors in your code. Take out strict and fix the basic errors. That will probably fix the code, although I'd probably take a completely different approach to this problem. I just don't have time to rewrite the entire program.

I am surprised that you got this program to run at all.

Considering they did not write it, and probably never even ran it, your surprise should be alleviated.

I am surprised that you got this program to run at all.

$my @t = split (/\s+/); #split sentences into "words"
$words ++;= @t; #add count to $words

These two lines contain syntax errors.

The reason your program will not run (and you're getting the error that you're getting) is because you are using strict, but not programming in a strict fashion. You're not declaring the scope of the variable filename, which is why it is bombing. If you take strict out, you will see that you have some fairly straight forward syntax errors in your code. Take out strict and fix the basic errors. That will probably fix the code, although I'd probably take a completely different approach to this problem. I just don't have time to rewrite the entire program.

Hi!,
As I said earlier on I am new to Perl, I wrote the code by reading thru other peoples work and my course book. In fact I do not understand most of the syntax myself that is why I have put it online for help. I'll would be grateful if people could comment and make corrections line by line so that I can follow the steps. But it seems all the comments is so far had not helped me to add any new thin to my code. I really need help please!

But it seems all the comments is so far had not helped me to add any new thin to my code.

Your code? You didn't write one single thing in that code, you just copied and pasted it from the other forum. You really have no shame.

CAN ANYBODY HELP ME IM DOING A TMA 10 SECTION B FOR SKILLSTRAIN analyse a text file to obtain statistics on its contents ANYBODY START ME OFF THANX

CAN ANYBODY HELP ME IM DOING A TMA 10 SECTION B FOR SKILLSTRAIN analyse a text file to obtain statistics on its contents ANYBODY START ME OFF THANX

hi there
i need help with same TMA
can anybody help

hi there
i need help with same TMA
can anybody help

Hello man, Have you completed for Perl Fundamental Write Perl Program TMA? Do you need help?

Regards

try adding


if (($ch eq "\t" || $ch eq " " || $ch eq "\n") &&

($lastchar ne "\t" && $lastchar ne " " && $lastchar ne "\n"))

{
$words++;
}

after the while

This article has been dead for over six months. Start a new discussion instead.