solved  Storing a String of Data into Vectors

garrett (12)   Link to this post
I have a string str="L1 L2 C1 P2 P1 S1 S2" and I would like to store each value into vectors. The following code is what I have attempted to do, when i run the file it crashes.

I am almost sure my error is located in the line " obstypes[i]=p;"
I attempted to use ".push_back" but i faced the same problem.

Any suggestions would be gladly appreciated.

Thanks,
G


1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
 

#include <iostream>
#include <string>
#include <vector>
#include <fstream>
#include <sstream>

using namespace std;

int main ()
{
  char * cstr, *p;
  vector<string>obstypes(7);
  int  i =0;

  string str ("L1    L2    C1    P2    P1    S1    S2");

  cstr = new char [str.size()+1];
  strcpy (cstr, str.c_str());


p=strtok (cstr," ");

//obstypes.push_back(p);
for( int i = 0; i < 9; i++ )

{


    cout << i;

    cout << p << endl;
    p=strtok(NULL," ");
    obstypes[i]=p;
  }

  delete[] cstr;
return 0;
}




.exe crashes



R0mai (371)   Link to this post
I strongly recomend using std::string's functions (find() and substr()) but
Let's say you have to use c-style functions and c-style strings. then...
Here are the problems in our code :
1. You don't include <cstring>, this header defines strtok and strcpy.
2. You make a varibale 'i' on line 14, altough you never use it.
3. You don't need a for loop here, you don't need to know how many tokens will be there.
You have to test whether strtok() returns 0.
4. Before the loop you use strtok (line 22), and before you get the results out of p, in the first iteration you overwrite it again (line 33).

Here is a corrected code :
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
#include <iostream>
#include <string>
#include <vector>
#include <cstring>

using namespace std;

int main() {
    char *cstr, *p;
    vector<string> obstypes;

    string str("L1    L2   C1    P2    P1    S1    S2");

    cstr = new char[str.size() + 1];
    strcpy(cstr, str.c_str());

    p = strtok(cstr, " ");

    while (p) { // while (p != 0)
        obstypes.push_back(p);
        p = strtok(NULL, " "); //We use strtok again AFTER we took out the result.
    }


    for(unsigned i = 0; i < obstypes.size(); ++i) {
        cout << obstypes[i] << endl;
    }

    delete[] cstr;
    return 0;
}


EDIT : Always check out the reference for a function
http://www.cplusplus.com/reference/clibrary/cstring/strtok/
There is an example very similar to yours.
Last edited on
Bazzy (4096)   Link to this post
The easiest way is using stringstreams
garrett (12)   Link to this post
Thanks alot for the help.

I was actually using the "strtok" example and trying to implement vectors within it. I am new to C++ so i just experimenting with the functions.
Duoas (3449)   Link to this post
You should avoid strtok() -- it modifies the argument string.

If you use a stringstream, it would look something like:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
#include <iostream>
#include <sstream>
#include <string>
#include <vector>
using namespace std;

int main()
{
    vector<string> v;

    string src("L1    L2   C1    P2    P1    S1    S2");

    istringstream ss(src);
    string s;
    while (ss >> s)
    {
        v.push_back(s);
    }

    cout << "The vector of strings is:\n";
    for (unsigned n = 0; n < v.size(); n++)
    {
        cout << "  " << n << ": " << v[n] << endl;
    }

    return 0;
}

Hope this helps.

This topic is archived - New replies not allowed.