Thread: Problem retrieving information from a file line by line.

Code:

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


void retrieveSubdomains(void);


int main(int argc, char *argv[])
{
    
	retrieveSubdomains();
    system("pause");
    return 0;
}




void retrieveSubdomains()  //return subdomain
{
	TCHAR *filename = "subdomains.txt";
	FILE *fp = fopen(filename, "r"); /* _tfopen_s */
    TCHAR subs;
    TCHAR *fullSub = malloc(150*sizeof(TCHAR)); /* Couldn't be bothered with using realloc to update the size, so a set size will have to do. */
    short i = 0;


    if (fp != NULL)
    {
			while((subs = fgetc(fp)) != (TCHAR)feof && subs != ' ' && subs != '\n')
			{
				printf("%c", fullSub[i] = subs);
				printf("%d", strlen(fullSub));
				i++;
			}


			fullSub[i] = '\0';
			printf("%d\n", sizeof *fullSub);
			//free(fullSub);
    }else{
        fprintf(stderr, "Failed to open file '%s'!\n", filename);
    }
	for(i=0;i<strlen(fullSub);i++)
		printf("%c", fullSub[i]);
		
	free(fullSub);
}

Thant's my current code, but I am having some problems, still. Strlen returns 6..? And I need to know how to jump to the next line? Would I just increment 'i' when I reach the newline character and it'll jump to the next line?

Originally Posted by inu11byte

Code:

			while((subs = fgetc(fp)) != (TCHAR)feof && subs != ' ' && subs != '\n')

You shouldn't use casts unless you understand why they're needed. Using them to hide compiler warnings is not a good reason (and yes, you should turn on and pay attention to all compiler warnings).

Originally Posted by inu11byte

Any ideas on why I'm get unexpected results? Or more improvements I could make?

What are your unexpected results?

Right now, you're only reading the first line because of your while condition which stops when fgetc() reads in the newline character from the first line.

Bye, Andreas

You should look into using fgets(...) since it reads one line at a time.