Thread: extracting a specific part out of a string (errors in my code?)

Yarr ugly!

Code:

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

#define MAX_PATH 256

char * extract_name(char * buf)
{
   char * path = NULL;

   path = malloc(MAX_PATH);
   if(!path)
      return NULL;

   sscanf(buf, "%*s %256s", path);

   return path; 
}

int main(void)
{
   char * a;

   a = extract_name("putfile /foo/bar");
   if(a)
   {
      printf("%s\n", a);
      free(a);
   }
   
   return 0;
}

I haven't done the work for you either. It's an ugly solution, you can do it your way and write a far nicer one. Also don't cast calloc().

1. You don't need to traverse backwards. You know the length of the filename is the length of the buffer - the start bit.
2. You can use strncpy() you know .

In reality I'd probably strip all the trailing and leading spaces from the buffer, then again from the path after I parsed it. Probably with the above method if I was feeling lazy

You should be aware of strrchr:

Code:

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

char *extract_name (char *fpathname) {
	char *ptr=strrchr(fpathname,'/');
	ptr++;
	return ptr;
}


int main() {
	char example[]="/home/user/somedir/afile.o",
		*shortname=extract_name(example);
	printf("%s\n",shortname);
}

BE AWARE THAT THIS RETURNS ONLY A POINTER INTO THE ORIGINAL STRING. However, it does work and demonstrate strrchr, which you need to look up.

Yes, but "./foo/bar" isn't the same as "/foo/bar" nor is "../foo/bar":-)

I really don't think you should do this:

Originally Posted by snpnx

if((c == '/') || (c == '\\') || (c == '$')){ //either of these chars show me I have

If portability is a factor use an ifdef statement:

Code:

#ifdef WIN32
#define SC 92     /* a "\"   */
#else 
#define SC 47     /* a  "/" */
#endif
[...]
char *ptr=strrchr(fpathname,SC);

ps. an honest question -- what OS allows "$" in filenames or paths?

Originally Posted by zacs7

Yes, but "./foo/bar" isn't the same as "/foo/bar" nor is "../foo/bar":-)

Well, that's true. Also, blue is not the same as green. Is this relevent?

> ps. an honest question -- what OS allows "$" in filenames or paths?
The ext2/3 filesystems. So Linux :-)

I said don't cast calloc(). See the FAQ.

Originally Posted by snpnx

Not a question of the OS, it was just needed since I did go backwards through the String and thought if the user puts in the filename without any leading './' I know it's right after the '$'. But I'll use strrchr, which I didn't know and let the user know that he needs to give me a '/' with the filename.

So you mean the end of the command prompt. That WILL NOT be in your string. Also, if the string does not contain any slashes, it is not a path and sticking one on the front won't change that. This is like saying I can make an apple out of an orange by putting a hat on it.

Pick, pick pick...anyway good luck

Code:

	for(i = count; i>=0; i--){						//go backwards through string
		fname[i] = buf[length];						//and set the characters accordingly
		i--;
		length--;
	}

Code in red does what??

Code:

	fname = (char*)calloc(count+1, sizeof(char));	//allocate free space
	fname[count+1] = '\0';							//put string termination symbol

Probably should not access outside of the member you have allocated - it tends to cause bad things to happen. Remember, an n-element array has valid a valid index range of 0 .. (n-1).

--
Mats

Originally Posted by zacs7

> ps. an honest question -- what OS allows "$" in filenames or paths?
The ext2/3 filesystems. So Linux :-)

Okay, $ is allowed in a filename as long as it's not the first character (eg. the$.file) but in this:

Code:

prompt$ /home/user/file.txt

$ is not part of the filename or path.