CODEWITHSUNDEEP
cstrtok
baskin
baskin

Reputation: 1663

strtok program crashing

the program for strtok given on http://www.opengroup.org/onlinepubs/000095399/functions/strtok.html crashes everytime..

#include <string.h>
...
char *token;
char *line = "LINE TO BE SEPARATED";
char *search = " ";


/* Token will point to "LINE". */
token = strtok(line, search);


/* Token will point to "TO". */
token = strtok(NULL, search);

If I use a char array for variable 'line', it works. i.e. char line[] = "LINE TO BE SEPARATED" works.

Kindly explain.

Upvotes: 3

Views: 5097

Answers (4)

Arkku
Arkku

Reputation: 42149

char *line is a pointer and you are pointing it to a constant string ("LINE TO BE SEPARATED"). This fails when strtok attempts to modify that string. It would be better to qualify this variable as const char *line—still wouldn't work, but might lead to a helpful warning when you try to pass it to strtok.

Meanwhile the array char line[] can be modified (it's not const) and is only initialised to contain the string.

Upvotes: 2

Viet
Viet

Reputation: 18404

aJ said what is needed. My advice is avoid that ugly & unsafe strtok. You are using C++ so go ahead with std::string. You also can use Boost http://www.boost.org/doc/libs/1_43_0/libs/libraries.htm#String & http://www.boost.org/doc/libs/1_43_0/doc/html/string_algo.html . If you want a new string class, you may look at http://bstring.sourceforge.net/ .

Upvotes: 1

sbi
sbi

Reputation: 224119

Since this has a C++ tag: 

// Beware, brain-compiled code ahead!
#include <string>
#include <sstream>
#include <iostream>

int main()
{
  std::istringstream iss("LINE TO BE SEPARATED");
  while( iss.good() ) {
    std::string token;
    iss >> token;
    std::cout << token '\n';
  }

  return 0;
}

Edit: As Konrad said in his comment, the above loop could be replaced by std::copy working on stream iterators: 

// Beware, brain-compiled code ahead!
#include <string>
#include <sstream>
#include <iostream>
#include <algorithm>

int main()
{
  std::istringstream iss("LINE TO BE SEPARATED");
  std::copy( std::istream_iterator<string>(std::iss)
           , std::istream_iterator<string>()
           , std::ostream_iterator<string>(std::cout, "\n") );
  return 0;
}

I have to (grudgingly) admit that there's something to be said for it.

Upvotes: 2

aJ.
aJ.

Reputation: 35460

strtok modifies the input string line. 

char *line = "LINE TO BE SEPARATED";

In this case line points to the read-only memory. Hence, cannot be modified. You need to pass char array for strtok.

Upvotes: 9

Related Questions