33

I am using std::ptr_fun as follows:

static inline std::string &ltrim(std::string &s) {
    s.erase(s.begin(), std::find_if(s.begin(), s.end(), std::not1(std::ptr_fun<int, int>(std::isspace))));
    return s;
}

as presented in this answer.

However this does not compile with C++17 (using Microsoft Visual Studio 2017), with the error:

error C2039: 'ptr_fun': is not a member of 'std'

How can this be fixed?

4
  • 2
    And the error is....? Jul 7, 2017 at 14:42
  • 1
    The error is: error C2039: 'ptr_fun': is not a member of 'std' Jul 7, 2017 at 14:56
  • 1
    edit your full error message in the question
    – Passer By
    Jul 7, 2017 at 15:06
  • 1
    If MVS2017 supports it, I think you could simply do std::not_fn(::isspace).
    – Holt
    Jul 7, 2017 at 15:35

7 Answers 7

66

You use a lambda:

static inline std::string &ltrim(std::string &s) {
    s.erase(s.begin(), std::find_if(s.begin(), s.end(), [](int c) {return !std::isspace(c);}));
    return s;
}

The answer you cited is from 2008, well before C++11 and lambdas existed.

11
  • 1
    Compiles perfectly and it's easier to understand than the 2008 solution. Thanks. Jul 7, 2017 at 14:51
  • I took the liberty of editing the 2008 answer with link to this one.
    – SergeyA
    Jul 7, 2017 at 15:11
  • 2
    @T.C. I don't understand - std::isspace() takes an int. What difference does it make if the conversion to int happens into isspace() or into another function that just calls isspace()?
    – Barry
    Jul 7, 2017 at 18:29
  • 5
    @Barry The point is that you aren't supposed to call any of the C character classification functions with a char or with a char converted (directly) to int. You are supposed to call them with a char converted to unsigned char and then to int. The intermediary unsigned char conversion is necessary to correctly handle negative-valued chars.
    – T.C.
    Jul 7, 2017 at 18:34
  • 5
    @T.C. That's just a horrible interface then. Yay C.
    – Barry
    Jul 7, 2017 at 18:43
20

Just use a lambda:

[](unsigned char c){ return !std::isspace(c); }

Note that I changed the argument type to unsigned char, see the notes for std::isspace for why.

std::ptr_fun was deprecated in C++11, and will be removed completely in C++17.

0
5

According to cppreference, std::ptr_fun is deprecated since C++11 and discontinued since C++17.

Similarly, std::not1 is deprecated since C++17.

So best don't use either, but a lambda (as explained in other answers).

4

Alternatively, you might use std::not_fn:

static inline std::string &ltrim(std::string &s) {
    s.erase(s.begin(), std::find_if(s.begin(), s.end(),
        std::not_fn(static_cast<int(*)(int)>(std::isspace))));
    return s;
}
0
4

My answer is similar to the already given answers in this thread. But instead of

int isspace(int c);

function from the standard C library, I am suggesting to use

bool isspace(char c, const locale& loc);

function instantiation from the standard C++ library (http://en.cppreference.com/w/cpp/locale/isspace), which is more type-correct. In this case you don't need to think about char -> unsigned char -> int conversions and about the current user's locale.

The lambda which searches for non-space will looks like this then:

[](char c) { return !std::isspace(c, std::locale::classic()); }

And the full code of ltrim function will look like this:

static inline std::string& ltrim(std::string& s) {
    auto is_not_space = [](char c) { return !std::isspace(c, std::locale::classic()); };
    auto first_non_space = std::find_if(s.begin(), s.end(), is_not_space);
    s.erase(s.begin(), first_non_space);
    return s;
}
1

You use Lambda as suggested by Nicol Bolas but you can use auto and type will be deduced there, as follow:-

    static inline std::string &ltrim(std::string &s) {
    s.erase(s.begin(), std::find_if(s.begin(), s.end(), [](auto c) {return 
       !std::isspace(c);}));
    return s;
  }
0

use std::isspace with lambda instead of std::ptr_fun,

static inline std::string &ltrim(std::string &s) {
    str.erase(str.begin(), std::find_if(str.begin(), str.end(),
    [](unsigned char c) { return !std::isspace(c);}));
    return s;
}

you can find more details about std::isspace

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.