Thread: Characters

Hello,
I am trying to make a program that will open up as many files as the user specifies in the command line (argv) and compare each file to see which has the most characters(bytes) and which file has the most words inside of it. I am having trouble figuring out how to manipulate the argv array so that i can accompany many user inputted filenames. i was thinking argv[i] and creating a loop, but i do not think that it works. Anyway can someone help me out here?

Code:

#include <stdio.h>
#include <stdlib.h>
int i;
int main(int argc,char **argv[])
{
  char name[100],name2[100],d;
  int c,nw[w],w,numfile;
  FILE *f[n],*g;
  n=0;
  numfile=argc;
  for (i=1;argv <= numfile;i++){
    f[n]=fopen(argv[i],"r");
    if(f[n]==NULL){
      printf("Can't open %s \n",argv[i]);
      exit(0);
    }
  }
  // fgetc(f[n]);
  // while(c != EOF){
  //fputc(c,g);
  // c=getc(f);
    


    


  
 
}

I'm pretty sure that argv[i] and creating a loop is going to work. You can't create nw[w] if you don't know what w is, nor f[n] if you don't know what n is.

Put all the code to open, process, and close a file inside a loop. Loop through argv[i] (you were close) from 0 to argc. That should do it!

You're gonna run into problems if you try and open an undetermined amount of files. If that's what you'd really like to do, look into malloc() and dynamic memory allocation. But for practical purposes (depending on what you're doing to the files), I would just open one, process it, close it, move to the next one...

its not liking this line..

Code:

 for (i=1;argv[i] <= argc;i++)

im gettting this error in the terminal when compiling

Code:

2-9.c: In function ‘main’:
2-9.c:12: warning: comparison between pointer and integer

Wny not

Code:

for (i=1; i < argc; i++)

instead?

still getting the same warning... i wnat argv at 1. i compile with ./a.out then followed by the inputted file names. So i would start reading at argv[1].

here is my unfinished code so far...

Code:

//Compares upto 100 files, and decides which one has the most characters(bytes) and which has the most words.
#include <stdio.h>
#include <stdlib.h>
int i;
int main(int argc,char **argv)
{
  char name[100],name2[100],d;
  int c,w,n,numc,numw,numtotal;
  FILE *f[n],*g;
  n=0;
  numw=0;
  numtotal=0;
  for (i=1;argv[i] < argc;i++){
    f[n]=fopen(argv[i],"r");
    if(f[n]==NULL){
      printf("Can't open %s \n",argv[i]);
      exit(0);
    }
    else{
      fgetc(f[n]);
      while(c != EOF){
	if(c==' '){
	  numw++;
	}
	c=getc(f[n]);
      }
      if (c > numc){
	c=numc;
      }
      if (numw > numtotal){
	numw=numtotal;
      }

    }
    n++;
    exit(1);
  }
  }

The problem before (and now) was that you were comparing

Code:

argv[i] < argc

argv [ anything ] is a string pointer. argc is an integer - you can't compare them. But since argc is the number of elements in the array argv, it is an integer, and so you can compare them. You should be saying

Code:

i < argc

.

Originally Posted by Lucid15

still getting the same warning... i wnat argv at 1. i compile with ./a.out then followed by the inputted file names. So i would start reading at argv[1].

But you have to test if i < argc, not argv[i] < argc.

Beyond that, I'd need to see your changed code.

I also need to add in that i would like my program to print the file name that has the most characters and/or words and if there are some that are equal print ALL of those. Im confused now..

Here is my FIXED code as of NOW..

Code:

//Compares upto 100 files, and decides which one has the most characters(bytes) and which has the most words.
#include <stdio.h>
#include <stdlib.h>
int i;
int main(int argc,char **argv)
{
  char name[100],name2[100],d;
  int c,w,n,numc,numw,numtotal;
  FILE *f[n],*g;
  n=0;
  numw=0;
  numtotal=0;
  for (i=1;i < argc;i++){
    f[n]=fopen(argv[i],"r");
    if(f[n]==NULL){
      printf("Can't open %s \n",argv[i]);
      exit(0);
    }
    else{
      fgetc(f[n]);
      while(c != EOF){
	if(c==' '){
	  numw++;
	}
	c=getc(f[n]);
      }
      if (c > numc){
	c=numc;
      }
      if (numw > numtotal){
	numw=numtotal;
      }

    }
    n++;
    exit(1);
  }

  }

hmm...thank you. any more input? im still confused as all hell

when i try to run the program to make it just tell me how many characters are in a file i get a segmentation fault...

Code:

//Compares upto 100 files, and decides which one has the most characters(bytes) and which has the most words.
#include <stdio.h>
#include <stdlib.h>
int i;
int main(int argc,char **argv)
{
  int c,w,n,max_char[n],max_words[n],numtotal,numwords;
  char most_words,most_chars;
  FILE *f[n],*g;
  n=0;
  numtotal=0;
  for (i=1;i <= argc;i++){
    f[n]=fopen(argv[i],"r");
    if(f[n]==NULL){
      printf("Can't open %s \n",argv[i]);
      exit(0);
      break;
    }
    else{
      fgetc(f[n]);
      while(c != EOF){
	if(c==' '){
	  numwords++;
	}
	c=getc(f[n]);
      }
     
    }
    n++;
    exit(1);
  }
  printf("%d",c);

  }