reading from a file + list of strings

[stewie1986]

Hi
Just wanted to know if anyone could help me with a reading from file problem.
When the program reads from a text file which is set out like this:

p
pbc
pcb
b

it only prints out the conversion of the last string, in this example case "b".
I'm guessing this is because of the while statement.

I just wanted to know if it possible to read in all the strings from a file and print out to another file, all the converted strings.

The only problem is that this was the given method by the teacher. I was told before that this way isnt the correct standard, but the problem is that was the structure we were asked to use.If you can help I would greatly appreciate it.

Here is my modified code, because this is a coursework assignment and I know a few people in my class use these boards.

                printf("Please enter a file input path ");
	scanf("%s", &myfile);
	 
	file_in=fopen(myfile, "r");
	
                while (!feof(file_in))
	{
		fscanf(file_in, "%s", array1);
	}
	
	fclose(file_in);
	
	length = strlen(array1);
	strupr(array1);
	
	for ( counter = 0; counter < length; counter++ )
       {
	
	if ( romanchar[counter] == 'p' )
	{
		sum = sum + 3; 
	}
	else if ( romanchar[counter] == 'b' )
	{
		sum = sum + 4; 
	}
	else if ( romanchar[counter] == 'c' )
	{
		sum = sum + 5; 
	}
   }
	
	printf("Please enter a file output path ");
	scanf("%s", &myfile);
                
                file_out = fopen(myfile, "w");
	fprintf(file_out, "%d\n", sum);
	fclose(file_out);

[MacGyver]

You have to think this through before you write code. You're reading in string after string into the same buffer and then only taking the length of the last one you read, and working on that one.

Some known issues which we probably told you at least some of them before:

• Any scanf() function that uses %s is unsafe unless it is specified to use a specific size.
• Using feof() in a control loop will likely cause your program to do weird things. Don't do it. Use the input function's return value as the loop condition.
• When you open a file with fopen(), check if the returned FILE * is NULL or not. Not doing this will probably result in crashes if something goes wrong.

[Elysia]

scanf("%s", ...) is unsafe as MacGyver mentions. I have links in my signature. Better read them.
While you're at it, read the faq entry for feof as loop control (also in signature).