Home Stretch! :)

I'm nearly finished working out all the bugs out of my WordLinker. There is just one problem left.
[php]
void addDirLink(char* str)
{
char temp[50] = "";
strcpy(temp, str);
for(int i = 0; temp[i]; i++)
{
linkcount++;
//here is the problem
//Somehow its passing a pointer to temp[i]
//rather then the value....
//I already have memory allocated all I need is the value... not a pointer
links[linkcount] = temp[i];
}
linkcount++;
links[linkcount] = ' ';
}

char* getLinks()
{
cout<<"What is links? " << links[0] <<endl;
return links;
}
[/php]

Some how link[linkcount] is being passed a pointer to temp[i] when what I want is the value to be stored in memory that won't disappear...

I might be able to figure it out myself.... =/

You need to learn how to make hacks bish.

Damn that was quick... and that is what I'm doing... =/

You can't rush perfection.

Maybe three more months if I'm good... :l

Originally Posted by why06

Damn that was quick... and that is what I'm doing... =/

You can't rush Obama.

Maybe three more months if I'm good... :l

Yeah I feel you

What was your declare for links? And how did you assign it its values? Did you pass by reference or value? We need to see some more code. Also if links[Linkcount] is being passed pointers to the values you need you can get those values using the pointer.

Sorry... that's the whole class:

[php]
class Word
{
int maxlinks;
char* wordname;
char* links;
int linkcount;

public:
Word(char str[]){strcpy(wordname, str);}
Word(){
maxlinks = 1000;
links = new char[maxlinks];
strcpy(links, "");
wordname = "";
linkcount = 0;
}

void newWord(char* str)
{
wordname =(char*)str;
}

char* getWord(){return wordname;}

void addDirLink(char* str)
{
char temp[50] = "";
strcpy(temp, str);
for(int i = 0; temp[i]; i++)
{
linkcount++;
//here is the problem
//Somehow its passing a pointer to temp[i]
//rather then the value....
//I already have memory allocated all I need is the value... not a pointer
links[linkcount] = temp[i];
}
linkcount++;
links[linkcount] = ' ';
}

char* getLinks()
{
cout<<"What is links? " << links[0] <<endl;
return links; //here getLinks returns nothing! D;
}
};
[/php]

Not sure exactly what you're trying to do, but shouldn't you replace:

Code:

for(int i = 0; temp[i];i++)

with:

Code:

for(int i = 0; i < strlen(str); i++)

I don't get how what you wrote works.

You declared Links as a pointer. So try *Links[linkcount] = temp[i]. And that should pass the value pointed to by the pointer links[linkcount].

Originally Posted by wtfiwantthatname

You declared Links as a pointer. So try *Links[linkcount] = temp[i]. And that should pass the value pointed to by the pointer links[linkcount].

Thanks, but I tried that before. I get this error:

Code:

52 C:\Dev-Cpp\My Projects\WordLinker.cpp invalid type argument of `unary *'

I've also tried char links[5000]; =/

theres another way to dereference it but im not good at it so ill just demonstrate it.

Code:

but before this you need to put the Word in pointer(I FORGOT HOW).

Word->links

Haha finally figured it out... it seems so obvious now I look like an idiot. The problem was my incrementing of linkcount... I never overwrote the first char which was a NULL ... silly me :P Well now writing to the file is essentially out of the way. Now all I have to worry about is properly reading from it.

EDIT: btw... I was meaning to tell you this earlier David, but I was a little preocupied. my for loop simply checks to see whether temp has encountered a NULL char or that we reached the end of the string. It's shorthand for what you wrote.

2 more monthes

Home Stretch! :)

Similar Threads

Tags for this Thread