Thread: problems with string

Code:

//* String to Int *//

#include <iostream>
#include <string>


using namespace std;


int sum_string_numbers(string x);


int main
{
    string my_string = "1,2,5,4,3,7";
    sum_string_numbers(my_string);
    
    return 0;
}


int sum_string_numbers(string x)
{
    int sum=0;
    for(i=0; x!='/0';i++)
    {
        if (isdigit(x[i]) == 1)
        {
            sum += stoi(x);
            i++;
        }
        i++;
    }
    return sum;
}

well, I try to use string var, but I get this error:

expected primary expression before 'my_string'.

please helpppp

This really has nothing to do with the string, that "before" in your error message is very important. So start looking at things "before" that line. For example how do you implement a function, remember main() is a function.

Jim

Code:

//* String to Int *//


#include <iostream>
#include <string>


using namespace std;


int sum_string_numbers(string x);


int main()
{
    string my_string = "1,2,5,4,3,7";
    sum_string_numbers(my_string);


    return 0;
}


int sum_string_numbers(string x)
{
    int sum=0;
    int i;
    for(i=0; x[i] != '\0';i++)
    {
        if (isdigit(x[i]) == 1)
        {
            sum += stoi(x[i]);
            i++;
        }
        i++;
    }
    return sum;
}

THANK YOU!!!

now the last problem is with stoi function
'stoi' was not declared in this scope <----------------

What is the exact version of the compiler are you using? The stoX() series of functions are C++11 features and may not be available with your compiler.

Jim

GNU GCC Compiler (from CodeBlocks)

And the version number of the compiler is? The version of your IDE (Code::Blocks) is? Also did you modify the project settings to enable C++11 mode?

Jim

code block 13.12, no ver on compiler, and I didnt modify... how can I do that ?

Not that you need to use stoi() anyways.

Code:

string digits = "0123456789";
for (string::size_type i = 0; i < digits.size(); i++) {
  int iNum = digits[i] - '0';
  cout << iNum << '\n';
}

Control your loops with string::size(). std::string does not need to be zero terminated, so comparisons that rely on it being terminated may fail to work.

how can I do that ?

From the command line type "gcc --version".


Jim

can you explain please "string::size_type" ..?

Originally Posted by noririco

can you explain please "string::size_type" ..?

It's just a typedef, usually for std::size_t, which in turn is usually just a typedef for unsigned int.

It's just a name for an unsigned integer type. You can use unsigned int I think, and there will be no problem, unless the string is extra long.

You can use unsigned int I think, and there will be no problem

A size_t or string::size_type are not necessarily an unsigned int, it could be any unsigned type, an unsigned long for example. You are better off using the correct type, size_t or std::string size_type then you will be correct no matter what operating system or compiler you happen to be using.

Jim

Thanks Jim. The rest of my post was good too.