Sfrgttk

tikz convert color string to hex value

How can I make my BBEG immortal short of making them a Lich or Vampire?

Can a Cauchy sequence converge for one metric while not converging for another?

What defenses are there against being summoned by the Gate spell?

Perform and show arithmetic with LuaLaTeX

How does one intimidate enemies without having the capacity for violence?

How much of data wrangling is a data scientist's job?

How do I deal with an unproductive colleague in a small company?

LWC SFDX source push error TypeError: LWC1009: decl.moveTo is not a function

How can bays and straits be determined in a procedurally generated map?

Replacing matching entries in one column of a file by another column from a different file

What is a clear way to write a bar that has an extra beat?

Are astronomers waiting to see something in an image from a gravitational lens that they've already seen in an adjacent image?

A newer friend of my brother's gave him a load of baseball cards that are supposedly extremely valuable. Is this a scam?

Intersection point of 2 lines defined by 2 points each

Could an aircraft fly or hover using only jets of compressed air?

Which country benefited the most from UN Security Council vetoes?

Do I have a twin with permutated remainders?

Is it inappropriate for a student to attend their mentor's dissertation defense?

Alternative to sending password over mail?

Fully-Firstable Anagram Sets

How is it possible to have an ability score that is less than 3?

Why is consensus so controversial in Britain?

Languages that we cannot (dis)prove to be Context-Free

strTok function (thread safe, supports empty tokens, doesn't change string)

String case reverse function in CGet line from string functionTDD: String Calculator KataC - K&R getint() variationSimple function to generate an HTML-safe stringGeneric Pairing Heap PerformancePattern for writing a generic string transformation functionChange a string into a function/def activatorC++ string tokenizing without streams, with certain conditionsRead consecutive blanks in array

.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty{ margin-bottom:0;
}

2

$begingroup$

I'm new to C language and want to explode a string like we do in PHP explode() function, I searched for a built-in function with the C standard library, and I found strtok , but It doesn't support empty tokens like 1,2,3,,5 . Inspired by the answers I found in this SO question I made this function, it is supposed to be thread safe and support empty tokens and doesn't change the original string

char* strTok(char** newString, char* delimiter)

{

    char* string = *newString;

    char* delimiterFound = (char*) 0;

    int tokLenght = 0;

    char* tok = (char*) 0;



    if(!string) return (char*) 0;



    delimiterFound = strstr(string, delimiter);



    if(delimiterFound){

        tokLenght = delimiterFound-string;

    }else{

        tokLenght = strlen(string);

    }



    tok = malloc(tokLenght + 1);

    memcpy(tok, string, tokLenght);

    tok[tokLenght] = '';



    *newString = delimiterFound ? delimiterFound + strlen(delimiter) : (char*)0;



    return tok;

}

I designed it to be used like

char* input = "1,2,3,4,5,6,7,,,10,";

char** inputP = &input;

char* tok;

while( (tok=strTok(inputP, ",")) ){

    printf("%sn", tok);

}

beginner c strings

share|improve this question

asked 4 hours ago

Accountant مAccountant م

1727

$endgroup$

• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  Better user-interface then the original strtok. You may be interested in strsep, too. code.woboq.org/userspace/glibc/string/strsep.c.html
  $endgroup$
  – Neil Edelman
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  @NeilEdelman thanks I never saw this function before, I will check it.
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  It's not in the standard C libraries, but in POSIX, (any type of gcc.) However, like strtok, it obliterates the char to replace it with , so it's not the same.
  $endgroup$
  – Neil Edelman
  1 hour ago
  

  
  
  
  

add a comment | 

2

$begingroup$

I'm new to C language and want to explode a string like we do in PHP explode() function, I searched for a built-in function with the C standard library, and I found strtok , but It doesn't support empty tokens like 1,2,3,,5 . Inspired by the answers I found in this SO question I made this function, it is supposed to be thread safe and support empty tokens and doesn't change the original string

char* strTok(char** newString, char* delimiter)

{

    char* string = *newString;

    char* delimiterFound = (char*) 0;

    int tokLenght = 0;

    char* tok = (char*) 0;



    if(!string) return (char*) 0;



    delimiterFound = strstr(string, delimiter);



    if(delimiterFound){

        tokLenght = delimiterFound-string;

    }else{

        tokLenght = strlen(string);

    }



    tok = malloc(tokLenght + 1);

    memcpy(tok, string, tokLenght);

    tok[tokLenght] = '';



    *newString = delimiterFound ? delimiterFound + strlen(delimiter) : (char*)0;



    return tok;

}

I designed it to be used like

char* input = "1,2,3,4,5,6,7,,,10,";

char** inputP = &input;

char* tok;

while( (tok=strTok(inputP, ",")) ){

    printf("%sn", tok);

}

beginner c strings

share|improve this question

asked 4 hours ago

Accountant مAccountant م

1727

$endgroup$

• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  Better user-interface then the original strtok. You may be interested in strsep, too. code.woboq.org/userspace/glibc/string/strsep.c.html
  $endgroup$
  – Neil Edelman
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  @NeilEdelman thanks I never saw this function before, I will check it.
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  It's not in the standard C libraries, but in POSIX, (any type of gcc.) However, like strtok, it obliterates the char to replace it with , so it's not the same.
  $endgroup$
  – Neil Edelman
  1 hour ago
  

  
  
  
  

add a comment | 

$begingroup$

I'm new to C language and want to explode a string like we do in PHP explode() function, I searched for a built-in function with the C standard library, and I found strtok , but It doesn't support empty tokens like 1,2,3,,5 . Inspired by the answers I found in this SO question I made this function, it is supposed to be thread safe and support empty tokens and doesn't change the original string

char* strTok(char** newString, char* delimiter)

{

    char* string = *newString;

    char* delimiterFound = (char*) 0;

    int tokLenght = 0;

    char* tok = (char*) 0;



    if(!string) return (char*) 0;



    delimiterFound = strstr(string, delimiter);



    if(delimiterFound){

        tokLenght = delimiterFound-string;

    }else{

        tokLenght = strlen(string);

    }



    tok = malloc(tokLenght + 1);

    memcpy(tok, string, tokLenght);

    tok[tokLenght] = '';



    *newString = delimiterFound ? delimiterFound + strlen(delimiter) : (char*)0;



    return tok;

}

I designed it to be used like

char* input = "1,2,3,4,5,6,7,,,10,";

char** inputP = &input;

char* tok;

while( (tok=strTok(inputP, ",")) ){

    printf("%sn", tok);

}

beginner c strings

share|improve this question

asked 4 hours ago

Accountant مAccountant م

1727

$endgroup$

char* strTok(char** newString, char* delimiter)

{

    char* string = *newString;

    char* delimiterFound = (char*) 0;

    int tokLenght = 0;

    char* tok = (char*) 0;



    if(!string) return (char*) 0;



    delimiterFound = strstr(string, delimiter);



    if(delimiterFound){

        tokLenght = delimiterFound-string;

    }else{

        tokLenght = strlen(string);

    }



    tok = malloc(tokLenght + 1);

    memcpy(tok, string, tokLenght);

    tok[tokLenght] = '';



    *newString = delimiterFound ? delimiterFound + strlen(delimiter) : (char*)0;



    return tok;

}

char* input = "1,2,3,4,5,6,7,,,10,";

char** inputP = &input;

char* tok;

while( (tok=strTok(inputP, ",")) ){

    printf("%sn", tok);

}

share|improve this question

asked 4 hours ago

Accountant مAccountant م

1727

share|improve this question

asked 4 hours ago

Accountant مAccountant م

1727

asked 4 hours ago

Accountant مAccountant م

1727

asked 4 hours ago

Accountant مAccountant م

1727

2 Answers
2

active

oldest

votes

1

$begingroup$

• delimiterFound + strlen(delimiter) sounds like a bug. If the delimiter is longer than one character, *newString will point too far into the original, maybe even beyond the end. Correct me if I am wrong, delimiterFound + 1 is what you are actually after.



• 
  

  Modern C allows, and strongly encourages, to declare variables as close to their use a possible. Consider

  
  
  

  char * delimiterFound = strstr(string, delimiter);
  
  ....
  
  char * tok = malloc(tokLenght + 1);
  
  

  
  
  

  etc.

  
  



• Always test that malloc didn't fail.



• 
  

  More spaces - around keywords, braces, etc - definitely improve readability:

  
  
  

      if (....) {
  
          ....
  
      } else {
  
          ....
  
      }
  
  

  
  

share|improve this answer

answered 2 hours ago

vnpvnp

40.6k233103

$endgroup$

• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  Thaaank you very much for these precious points, regarding the delimiter length bug, ummmm, I want to support long delimiters more than 1 characters like the boundary string in http requests that has content-type multi-part, and I don't think it's a bug because delimiterFound + strlen(delimiter) can never be after the 0 byte that terminates the original string!, right ? "fooDELIMITER" 3 + 12
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  @Accountantم Long delimiters here refer to, say ",;.", in where any character delimits the string on its own right.
  $endgroup$
  – vnp
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  I didn't get it, I'm sorry, can you please give me an example input that can break this code, exploiting this bug ?
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  @Accountantم Sorry for not being clear. I should realize that your intentions are different (and read man strstr more carefully). Consider it my blinder - since you mentioned strtok, I expected the strtok semantics.
  $endgroup$
  – vnp
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  It's my fault because I said it's strtok, I wanted strtok that can support delimiters more than 1 characters, because I need this feature a lot. So do you mean it's not a bug ??
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  

 | 
show 1 more comment

1

$begingroup$

From a readability viewpoint, you should use NULL instead of (char*) 0 as it is easier to recognize what you're trying to do. Also, the tokLenght misspells "length", and should probably be tokLength.

You leak memory, as the memory allocated to hold the returned string is never freed.

share|improve this answer

answered 3 hours ago

1201ProgramAlarm1201ProgramAlarm

3,6532925

$endgroup$

• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  Thank you very much I will use NULL from now on, and I will remember to free() memory , 'I miss PHP garbage collector :(', I didn't get the tokLength spelling note, aren't they the same ?
  $endgroup$
  – Accountant م
  2 hours ago
  
  
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  It's a matter of style, and including .h if you want to use NULL. However, you don't have to cast (char *)0, just use 0 (or NULL.) It knows from the return type.
  $endgroup$
  – Neil Edelman
  2 hours ago
  

  
  
  
  

add a comment | 

Your Answer

StackExchange.ifUsing("editor", function () {
return StackExchange.using("mathjaxEditing", function () {
StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix) {
StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
});
});
}, "mathjax-editing");

StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");

StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "196"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});

function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});


}
});

draft saved

draft discarded

Sign up or log in

StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});

Sign up using Google

Sign up using Facebook

Sign up using Email and Password

Post as a guest

Name

Email

Required, but never shown

StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f216956%2fstrtok-function-thread-safe-supports-empty-tokens-doesnt-change-string%23new-answer', 'question_page');
}
);

Post as a guest

Name

Email

Required, but never shown

1

$begingroup$

• delimiterFound + strlen(delimiter) sounds like a bug. If the delimiter is longer than one character, *newString will point too far into the original, maybe even beyond the end. Correct me if I am wrong, delimiterFound + 1 is what you are actually after.



• 
  

  Modern C allows, and strongly encourages, to declare variables as close to their use a possible. Consider

  
  
  

  char * delimiterFound = strstr(string, delimiter);
  
  ....
  
  char * tok = malloc(tokLenght + 1);
  
  

  
  
  

  etc.

  
  



• Always test that malloc didn't fail.



• 
  

  More spaces - around keywords, braces, etc - definitely improve readability:

  
  
  

      if (....) {
  
          ....
  
      } else {
  
          ....
  
      }
  
  

  
  

share|improve this answer

answered 2 hours ago

vnpvnp

40.6k233103

$endgroup$

• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  Thaaank you very much for these precious points, regarding the delimiter length bug, ummmm, I want to support long delimiters more than 1 characters like the boundary string in http requests that has content-type multi-part, and I don't think it's a bug because delimiterFound + strlen(delimiter) can never be after the 0 byte that terminates the original string!, right ? "fooDELIMITER" 3 + 12
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  @Accountantم Long delimiters here refer to, say ",;.", in where any character delimits the string on its own right.
  $endgroup$
  – vnp
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  I didn't get it, I'm sorry, can you please give me an example input that can break this code, exploiting this bug ?
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  @Accountantم Sorry for not being clear. I should realize that your intentions are different (and read man strstr more carefully). Consider it my blinder - since you mentioned strtok, I expected the strtok semantics.
  $endgroup$
  – vnp
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  It's my fault because I said it's strtok, I wanted strtok that can support delimiters more than 1 characters, because I need this feature a lot. So do you mean it's not a bug ??
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  

 | 
show 1 more comment

1

$begingroup$

• delimiterFound + strlen(delimiter) sounds like a bug. If the delimiter is longer than one character, *newString will point too far into the original, maybe even beyond the end. Correct me if I am wrong, delimiterFound + 1 is what you are actually after.



• 
  

  Modern C allows, and strongly encourages, to declare variables as close to their use a possible. Consider

  
  
  

  char * delimiterFound = strstr(string, delimiter);
  
  ....
  
  char * tok = malloc(tokLenght + 1);
  
  

  
  
  

  etc.

  
  



• Always test that malloc didn't fail.



• 
  

  More spaces - around keywords, braces, etc - definitely improve readability:

  
  
  

      if (....) {
  
          ....
  
      } else {
  
          ....
  
      }
  
  

  
  

share|improve this answer

answered 2 hours ago

vnpvnp

40.6k233103

$endgroup$

• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  Thaaank you very much for these precious points, regarding the delimiter length bug, ummmm, I want to support long delimiters more than 1 characters like the boundary string in http requests that has content-type multi-part, and I don't think it's a bug because delimiterFound + strlen(delimiter) can never be after the 0 byte that terminates the original string!, right ? "fooDELIMITER" 3 + 12
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  @Accountantم Long delimiters here refer to, say ",;.", in where any character delimits the string on its own right.
  $endgroup$
  – vnp
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  I didn't get it, I'm sorry, can you please give me an example input that can break this code, exploiting this bug ?
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  @Accountantم Sorry for not being clear. I should realize that your intentions are different (and read man strstr more carefully). Consider it my blinder - since you mentioned strtok, I expected the strtok semantics.
  $endgroup$
  – vnp
  2 hours ago
  

  
  
  
  


• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  It's my fault because I said it's strtok, I wanted strtok that can support delimiters more than 1 characters, because I need this feature a lot. So do you mean it's not a bug ??
  $endgroup$
  – Accountant م
  2 hours ago
  

  
  
  
  

 | 
show 1 more comment

$begingroup$

• delimiterFound + strlen(delimiter) sounds like a bug. If the delimiter is longer than one character, *newString will point too far into the original, maybe even beyond the end. Correct me if I am wrong, delimiterFound + 1 is what you are actually after.



• 
  

  Modern C allows, and strongly encourages, to declare variables as close to their use a possible. Consider

  
  
  

  char * delimiterFound = strstr(string, delimiter);
  
  ....
  
  char * tok = malloc(tokLenght + 1);
  
  

  
  
  

  etc.

  
  



• Always test that malloc didn't fail.



• 
  

  More spaces - around keywords, braces, etc - definitely improve readability:

  
  
  

      if (....) {
  
          ....
  
      } else {
  
          ....
  
      }
  
  

  
  

share|improve this answer

answered 2 hours ago

vnpvnp

40.6k233103

$endgroup$

share|improve this answer

answered 2 hours ago

vnpvnp

40.6k233103

answered 2 hours ago

vnpvnp

40.6k233103

answered 2 hours ago

vnpvnp

40.6k233103

1

$begingroup$

From a readability viewpoint, you should use NULL instead of (char*) 0 as it is easier to recognize what you're trying to do. Also, the tokLenght misspells "length", and should probably be tokLength.

You leak memory, as the memory allocated to hold the returned string is never freed.

share|improve this answer

answered 3 hours ago

1201ProgramAlarm1201ProgramAlarm

3,6532925

$endgroup$

• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  Thank you very much I will use NULL from now on, and I will remember to free() memory , 'I miss PHP garbage collector :(', I didn't get the tokLength spelling note, aren't they the same ?
  $endgroup$
  – Accountant م
  2 hours ago
  
  
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  It's a matter of style, and including .h if you want to use NULL. However, you don't have to cast (char *)0, just use 0 (or NULL.) It knows from the return type.
  $endgroup$
  – Neil Edelman
  2 hours ago
  

  
  
  
  

add a comment | 

1

$begingroup$

From a readability viewpoint, you should use NULL instead of (char*) 0 as it is easier to recognize what you're trying to do. Also, the tokLenght misspells "length", and should probably be tokLength.

You leak memory, as the memory allocated to hold the returned string is never freed.

share|improve this answer

answered 3 hours ago

1201ProgramAlarm1201ProgramAlarm

3,6532925

$endgroup$

• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  Thank you very much I will use NULL from now on, and I will remember to free() memory , 'I miss PHP garbage collector :(', I didn't get the tokLength spelling note, aren't they the same ?
  $endgroup$
  – Accountant م
  2 hours ago
  
  
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  It's a matter of style, and including .h if you want to use NULL. However, you don't have to cast (char *)0, just use 0 (or NULL.) It knows from the return type.
  $endgroup$
  – Neil Edelman
  2 hours ago
  

  
  
  
  

add a comment | 

$begingroup$

From a readability viewpoint, you should use NULL instead of (char*) 0 as it is easier to recognize what you're trying to do. Also, the tokLenght misspells "length", and should probably be tokLength.

You leak memory, as the memory allocated to hold the returned string is never freed.

share|improve this answer

answered 3 hours ago

1201ProgramAlarm1201ProgramAlarm

3,6532925

$endgroup$

share|improve this answer

answered 3 hours ago

1201ProgramAlarm1201ProgramAlarm

3,6532925

answered 3 hours ago

1201ProgramAlarm1201ProgramAlarm

3,6532925

answered 3 hours ago

1201ProgramAlarm1201ProgramAlarm

3,6532925