Trying to obtain int from inside a string.

[TonyG]

Having trouble reading an int from a string.

categoryID is always of the format of <char>'000++' i.e. C0001 C0002...

 strncpy(temp, current->categoryID, ID_LEN);
        sprintf(temp, "%04i", atoi(intTemp) + 1);
        printf("%d\n", intTemp); // this outputs a large number not the '000++' value.

[laserlight]

It sounds like you actually want:

intTemp = atoi(current->categoryID + 1);

[cph]

how about a combination of strcspn & strtol?

[TonyG]

worked it out thanks.

[CommonTater]

Having trouble reading an int from a string.

categoryID is always of the format of <char>'000++' i.e. C0001 C0002...

sscanf ("C%d ",number);

[ssharish2005]

>sscanf ("C%d ",number);
Did you mean
sscanf ("C%d ", &number);

And to be more versatile

if( sscanf(  buffer, "%*c%d", &d ) == 1 ) { ... }
else { print error }

ssharish

[TonyG]

Actually I have change the plan for this process. And now i am trying to generate a string that will increment as above C0001, C0002, C0003... every time it does a loop.
This is the code i have at the moment but for some reason the output is not correct.

void addCategory(GJCType* menu) {

    char newCatID[ID_LEN + 1] = {0};
    char catID[ID_LEN + 1] = {"C"};
    char* catIDPtr = catID;

    char temp[menu->numCategories][ID_LEN + 1];
    int index = 0;
    char* newCatIDPtr = newCatID;

    CategoryType* current = menu->headCategory;

    while (current != NULL) {
        index++;
        current = current->nextCategory;
        sprintf(newCatID, "%04i", atoi(newCatID) + index);
        strncat(catIDPtr, newCatID, ID_LEN);
        printf("%s\n", catIDPtr);
        strncpy (catID, "C\0", ID_LEN + 1);
    }

}

OP
C0001
C0003
C0006
C0010
C0015
C0021
C0028

[quzah]

Get rid of the atoi chunk, and just stick the index there. If you took a moment to look at your output you should have been able to see what you were doing wrong.


Quzah.

[TonyG]

Just one of those things.. Kept banging my head for hrs working this out.

[CommonTater]

Just one of those things.. Kept banging my head for hrs working this out.

Hi Tony... There's a little trap I kept falling into, took me a while to get out of it too... When things start going wrong the problem appears nebulous in it's complexity, so the natural tendency is to keep adding or changing a lot of stuff... In truth, it makes more sense to stop, take a break, and try to think about how to *simplify* the problem to a level where you can *remove* code to fix it. Life got a lot easier when I finally got my head around that idea.