It’s because fgets stores the newline character so when strstr does a comparison it fails. From the man page: fgets() reads in at most one less than size characters from stream and stores them into the buffer pointed to by s. Reading stops after an EOF or a newline. If a newline is read, it … Read more
The simplest naive solution is simply: $file = “/path/to/file”; $data = file($file); $line = $data[count($data)-1]; Though, this WILL load the whole file into memory. Possibly a problem (or not). A better solution is this: $file = escapeshellarg($file); // for the security concious (should be everyone!) $line = `tail -n 1 $file`;
The function fgets might add a newline at the end of the string read. You’ll have to check that: size_t ln = strlen(input) – 1; if (input[ln] == ‘\n’) input[ln] = ‘\0’; or even strtok(input, “\n”);
I’ll bet it’s because of the \n stuck in the input stream. See one of these questions: I am not able to flush stdin. How do I go about Flushing STDIN here? scanf() causing infinite loop or this answer. Also: Why not to use scanf(). P.S. fgets() is a function, not an instruction.
If you don’t need any special processing, this should do what you’re looking for $lines = file($filename, FILE_IGNORE_NEW_LINES);
strcmp is one of the few functions that has the reverse results of true and false…if the strings are equal, the result is 0, not 1 as you would think…. if (strcmp(a, b)) { /* Do something here as the strings are not equal */ } else { /* Strings are equal */ } Speaking … Read more
Never use gets. It offers no protections against a buffer overflow vulnerability (that is, you cannot tell it how big the buffer you pass to it is, so it cannot prevent a user from entering a line larger than the buffer and clobbering memory). Avoid using scanf. If not used carefully, it can have the … Read more
scanf() reads exactly what you asked it to, leaving the following \n from the end of that line in the buffer where fgets() will read it. Either do something to consume the newline, or (my preferred solution) fgets() and then sscanf() from that string.
Perhaps the simplest solution uses one of my favorite little-known functions, strcspn(): buffer[strcspn(buffer, “\n”)] = 0; If you want it to also handle ‘\r’ (say, if the stream is binary): buffer[strcspn(buffer, “\r\n”)] = 0; // works for LF, CR, CRLF, LFCR, … The function counts the number of characters until it hits a ‘\r’ or … Read more
In order to use gets safely, you have to know exactly how many characters you will be reading, so that you can make your buffer large enough. You will only know that if you know exactly what data you will be reading. Instead of using gets, you want to use fgets, which has the signature … Read more