Thread: How do I make this work with lines of any length

So rather than actually try the suggestion made, you decided you would make a new thread in hopes that someone would do everything for you?

Quzah.

OKAY I'm now doing something like

Code:

	int i = fgetc(fileOne);
		        int j = fgetc(fileTwo);
			while(i != '\n' && i != EOF)
			{
				puthcar(i);
				i = fgetc(fileOne);	
			}
			printf(%s, sep)
			while( j != '\n' && != EOF)
			{
				putchar(j);
				j = fgetc(fileTwo);
			}

needs finishing, on the right track?

Originally Posted by Adak

So immediately AFTER I recommend treating strings as strings, you come back with something like code, that deals with chars individually?

I SO don't understand that. Buffers are just a temporary storage area. Many times, used to hold strings.

I know I'm sorry but I tried it couldn't do it and I've latched onto my chars idea now...

If I get this working, then you can maybe show me how you'd code it with a buffer? I can't expect you to give me the answer to my problem but I also cannot do what you're suggesting because I can't comprehend it.. which is my problem but.. yeah. Silly I know but just... it is how it is.

I'm getting segmentation fault when using a really long line in file 1

Code:

#include <stdio.h>
#include <string.h>

int main(int argc, char *argv[])
{
	// check num of args
	if(argc < 4)
	{
		fprintf(stderr,"error: not enough arguments\n");	
	}
	else
	{
		FILE *fileOne =fopen( argv[2], "r");
		FILE *fileTwo =fopen( argv[3], "r");
		char *sep = argv[1];
		if ( fileOne == 0 || fileTwo == 0)
		{
			/* error*/
 			fprintf(stderr, "Error: Can't open file");
			
		}
		else	
		{
			int i = fgetc(fileOne);
		        int j = fgetc(fileTwo);
			while(i!=EOF || j!=EOF)
			{
				while((i != '\n') && (i != EOF))
				{
					putchar(i);
					i = fgetc(fileOne);	
				}
				printf("%s", sep);
				while((j != '\n') && (j != EOF))
				{
					putchar(j);
					j = fgetc(fileTwo);
				}
			}
								


		}	
	}
		return 0;
}

Is this a thumbnail version of the job?

You have lines of text of varying length. You want to append a space, and a second line, onto the first line of text, but only if line1 is not the same as line2.

Is that right?

There's a story of a guy out hunting for rabbits. He's got a hunting dog, and a shotgun, and he knows there are rabbits in his fields.

Crossing a small creek, he walks up the bank and there they are - scores of rabbits, who get scared, and start running all around, including right by him and his startled dog.

He couldn't aim at any ONE rabbit, because there were so many, all hopping around. He and his dog did not get a rabbit that day.

You need to stick with someone's advice that you believe makes sense, once you know that they understand the job you want to do. There's always more than one way to do a job.

works fine for normal files, but when I use a big file with 700+ chars in one line it freaks out

Code:

#include <stdio.h>
#include <string.h>

int main(int argc, char *argv[])
{
// check num of args
	if(argc != 4)
	{
		fprintf(stderr,"Usage is %s <number>\n", argv[0]);
	}
	else
	{
		FILE *fileOne =fopen( argv[2], "r");
		FILE *fileTwo =fopen( argv[3], "r");
		char *sep = argv[1];
		if ( fileOne == 0 || fileTwo == 0)
		{
			/* error*/
 			fprintf(stderr, "Error: Can't open file");
			
		}
		else	
		{
			int i = fgetc(fileOne);
		        int j = fgetc(fileTwo);
			while(i!=EOF || j!=EOF)
			{
				while((i != '\n') && (i != EOF))
				{
					putchar(i);
					i = fgetc(fileOne);	
				}
				printf("%s", sep);
				while((j != '\n') && (j != EOF))
				{
					putchar(j);
					j = fgetc(fileTwo);
				}
			}
								


		}	
	}
		return 0;
}

Originally Posted by Adak

Is this a thumbnail version of the job?

You have lines of text of varying length. You want to append a space, and a second line, onto the first line of text, but only if line1 is not the same as line2.

Is that right?

I want to just concatinate the line from file 1 with a separator string and the line from file two for all the lines in the file, and if there is no line for one of the files but for another just continue to print the sep and the line that exists.

Atm, it is working for a file with 116 chars, but not with about 700 chars in 1 line. I get a segmentation fault.. no idea what the problem is.

Originally Posted by Adak

Crossing a small creek, he walks up the bank and there they are - scores of rabbits, who get scared, and start running all around, including right by him and his startled dog.

He couldn't aim at any ONE rabbit, because there were so many, all hopping around. He and his dog did not get a rabbit that day.

You need to stick with someone's advice that you believe makes sense, once you know that they understand the job you want to do. There's always more than one way to do a job.

The reason I did not follow your advice is because I could not make sense of it, not because it didn't make sense. I hope you are not offended. In programming I know that there are lots of different ways of doing things each with their own advantages...

Originally Posted by Adak

Check the value of argv[1] and sep.

Are i and j being declared anew every time through the loop?

Depends on your compiler, but I would move variable declarations to the very top of the function, to be sure.

You're getting no error message at all, and no warnings either?

you mean just move the int i; int j; to the top? then assign later? - DONE and still throws error

no nothing just segmentation fault error msg

FIXED IT

Code:

#include <stdio.h>
#include <string.h>

int main(int argc, char *argv[])
{
// check num of args
	if(argc != 4)
	{
		fprintf(stderr,"Usage is %s <number>\n", argv[0]);
	}
	else
	{
		FILE *fileOne =fopen( argv[2], "r");
		FILE *fileTwo =fopen( argv[3], "r");
		char *sep = argv[1];
		int i, j;
		if ( fileOne == 0 || fileTwo == 0)
		{
			/* error*/
 			fprintf(stderr, "Error: Can't open file");
			
		}
		else	
		{
			
			while(i!=EOF || j!=EOF)
			{
				i = fgetc(fileOne);
		        	j = fgetc(fileTwo);
				while((i != '\n') && (i != EOF))
				{
					putchar(i);
					i = fgetc(fileOne);	
				}
				if(i == '\n') i = fgetc(fileOne);
				printf("%s", sep);
				while((j != '\n') && (j != EOF))
				{
					putchar(j);
					j = fgetc(fileTwo);
				}
				if(j == '\n') j = fgetc(fileTwo);
				putchar('\n');
			}
								


		}	
	}
		return 0;
}