C / C++ / MFC

Prakash Nadar3-Jan-06 19:49

3-Jan-06 19:49

Aljechin wrote:
szValue = chpValue;

Is not the right thing to do,

The CString copies only the first string till first zero is encontered,
You need to run a while loop(for example) to extract each key,value pair and assign it to appropriate variables.

-Prakash

<color>Aljechin 3-Jan-06 19:53

3-Jan-06 19:53

Can you show a small code snippet?

Thanks

<color>Aljechin 3-Jan-06 20:09

3-Jan-06 20:09

Heartfelt Thanks sir, the problem is solved.

Aljechin Alexander.

kakan3-Jan-06 19:54

3-Jan-06 19:54

There are other ways to do this, but this is one way. Use this func:

<br />
CString getNextItem(char * wholeThing)<br />
{<br />
    CString ret;<br />
    static char * current;<br />
<br />
    if(wholeThing) {<br />
        // Just set at first time<br />
        current = wholeThing;<br />
    }<br />
<br />
    if(*current == '\0') ret = ""; // End<br />
    else {<br />
        ret = current;<br />
        while(*current++);<br />
    }<br />
    return ret;<br />
}<br />

like this:

<br />
CString szValue;<br />
char chpValue[32767];<br />
<br />
GetPrivateProfileSection("SETTINGS",chpValue,sizeof(chpValue),"MySettings.ini");<br />
<br />
szValue = getNextItem(chpValue);<br />
<br />
while(szValue.GetLength()) {<br />
   //Do something with szValue<br />
   szValue = getNextItem(NULL);<br />
};<br />

<color>Aljechin 3-Jan-06 20:08

3-Jan-06 20:08

Heart felt thanks, sir.

One more thing: Your's is just that damn good logic.

I must accept I have almost completely screwed up my basic c++ programming logic by staying with and relying heavily on the luxuries of MFC. I need to brush those stuff up back.

Aljechin Alexander

If space and time are related in some way, at what time and which dimension of the space did this relation occur? If the universe was created at some point of time, then does the time starts right from that point? Before that there was no time?

kakan3-Jan-06 20:22

3-Jan-06 20:22

Thanks! It's mainly good old C. But it's still useful.

Kakan

Prakash Nadar3-Jan-06 20:26

3-Jan-06 20:26

It is not actually a best of doing it, it works but better way is possible.

I think you can actually optimise it.

-Prakash

-- modified at 2:26 Wednesday 4th January, 2006

kakan3-Jan-06 20:38

3-Jan-06 20:38

Such as...?
As I said: "It's one way of doing it".

Of course, I know there are limitations, such as lack of Unicode support, no ATL, etc. etc. Sleepy | :zzz:

But at least a gave the guy a solution to his problem. I haven't find another solution here, just a bunch of questions and vague suggestions how to solve the problem in question.
Anyone, feel totally free to give the guy a complete solution.

<color>Aljechin 3-Jan-06 20:57

3-Jan-06 20:57

kakan wrote:
I haven't find another solution here, just a bunch of questions and vague suggestions how to solve the problem in question.

True. Kakan actually gave me a solution that worked. I think i cannot expect him to give me the best optimised code at the begining itself, that too immediately.

Thanks to all

Aljechin Alexander

kakan3-Jan-06 21:03

3-Jan-06 21:03

IMO, that's what this site is all about. Helping people, since noone has all the answers. Big Grin | :-D

Prakash Nadar3-Jan-06 21:33

3-Jan-06 21:33

Aljechin wrote:
True. Kakan actually gave me a solution that worked. I think i cannot expect him to give me the best optimised code at the begining itself, that too immediately.

Well I assumed you knew how to tokanise a zero seperated strings, quite basic thing to do.

-Prakash

Prakash Nadar3-Jan-06 21:32

3-Jan-06 21:32

I didnt say the solution is bad, All i wanted to point out that if you want to use the code for production code, then you have to optimise that code before using it.

-Prakash

<color>Aljechin 3-Jan-06 20:55

3-Jan-06 20:55

Dude, can you optimise it?

kakan3-Jan-06 21:13

3-Jan-06 21:13

Aljechin wrote:
Dude, can you optimise it?

Prakash Nadar3-Jan-06 21:31

3-Jan-06 21:31

I can well optimise it, dont have the time. Roll eyes | :rolleyes:

Since you are asking for me to optimise it , i can understand that you dont want to spend too much effort on ur work.

-Prakash

-- modified at 3:39 Wednesday 4th January, 2006

<color>Aljechin 3-Jan-06 22:01

3-Jan-06 22:01

I have optimised and modified the code to suit my need. I never meant to say that you don't know it, but I was trying to tell you gave me some idea, but Kakan gave me the same with a proper sample.

Many thanks for your time.

Aljechin

Prakash Nadar3-Jan-06 22:03

3-Jan-06 22:03

Aljechin wrote:
I have optimised and modified the code to suit my need. I never meant to say that you don't know it, but I was trying to tell you gave me some idea, but Kakan gave me the same with a proper sample.

yes, kakan gave you the solution after we had couple of converstation to pinpoint the exact problem.

-Prakash

<color>Aljechin 3-Jan-06 22:11

3-Jan-06 22:11

Ah, yes of course. If there was a start you gave it and you were the first to reply too. I thanked you there i think. OMG | :OMG:

full of gurus here. I'm a bit scared. I am good at chess, not at MFC. LOL.

Aljechin

Owner drawn3-Jan-06 23:30

Owner drawn

3-Jan-06 23:30

See how looooooooooooong you all have been fighting.;P

Love Forgives--Love Gives--Jesus is Love Smile | :)

<marquee direction="up" height="50" scrolldelay="1" step="1" scrollamount="1" style="background-color:'#44ccff'">
--Owner Drawn
--Nothing special
--Defeat is temporary but surrender is permanent
--Never say quits
--Jesus is Lord

Prakash Nadar3-Jan-06 23:41

3-Jan-06 23:41

Owner drawn wrote:
See how looooooooooooong you all have been fighting.

I was not fighting Wink | ;)

-Prakash

<color>Aljechin 4-Jan-06 0:35

4-Jan-06 0:35

Did i hear someone saying the word fight? Who was fighting here? Big Grin | :-D

Aljechin

Owner drawn3-Jan-06 20:05

Owner drawn

3-Jan-06 20:05

Did you take note of the return value...

The return value is the number of characters copied to the buffer, not including the terminating null character.

So why don't you loop through the string instead of checking for NULL. This way you will get the full settings.

Here each string is seperated with a NULL char. So while looping you can replace these NULL chars with a semicolon or some special char.

Love Forgives--Love Gives--Jesus is Love Smile | :)

<color>Aljechin 3-Jan-06 20:10

3-Jan-06 20:10

Thanks sir, the problem is solved.

Aljechin Alexander