Я хочу перебирать строку и сравнивать каждый из символов в этой строке с определенным символом, например «M». std::string::find не будет работать для меня, так как порядок, в котором символы появляются в строке, имеет значение (например, в римских цифрах MC отличается от CM).
Код, который у меня есть (компилирую на C++ 11):
#include <iostream>
#include <cstring>
using namespace std;
int main ()
{
string str = ("Test Mstring");
for (auto it = str.begin(); it < str.end(); it++) {
if (strcmp(*it, "M") == 0) cout << "M!1!!1!" << endl;
}
}
Отображение ошибки консоли:
test.cc: In function ‘int main()’:
test.cc:10:16: error: invalid conversion from ‘char’ to ‘const char*’ [-fpermissive]
if (strcmp(*it, "M") == 0) cout << "M!1!!1!" << endl;
^~~
In file included from /usr/include/c++/7/cstring:42:0,
from test.cc:2:
/usr/include/string.h:136:12: note: initializing argument 1 of ‘int strcmp(const char*, const char*)’
extern int strcmp (const char *__s1, const char *__s2)
Я не понимаю, почему поиск строки у вас не работает





Разыменование итератора, полученного от std::string, возвращает char. Ваш код просто должен быть:
if (*it == 'M') cout << "M!1!!1!" << endl;
Также:
Обратите внимание на "M"! = "M". В C++ двойные кавычки определяют строковый литерал, который заканчивается нулевым байтом, тогда как одинарные кавычки определяют одиночный символ.
Не используйте endl, если вы не собираетесь очищать буфер стандартного вывода. \n намного быстрее.
strcmp в C++ - это обычно запах кода.
Элементы строки - это символы, такие как 'M', а не строки.
string str = "Test Mstring";
for (auto it = str.begin(); it < str.end(); it++) {
if (*it == 'M') cout << "M!1!!1!" << endl;
}
или
string str = "Test Mstring";
for (auto ch: str) {
if (ch == 'M') cout << "M!1!!1!" << endl;
}
strcmp сравнивает целые строки, поэтому, если вы сравниваете "mex" с "m", они не равны, и вы не можете сравнивать char с char из string в этой функции, потому что для сравнения символов вы можете использовать строку как массив, например
string c = "asd";
string d = "dss";
if (c[0]==d[0] /* ... */
if (c[0]=='a') /*... */
Помните, что it - это указатель на char в строке, поэтому при разыменовании вы должны сравнивать с char
if (*it=='c')
Кстати, а почему вы смешиваете строки C и C++? Вы используете string, как в C++, но функция strcmp берется из библиотеки C.
Вы можете сделать что-то вроде ниже:
std::for_each(str.begin(), str.end(), [](char &c){ if (c == 'M') cout<< "M!1!!1!"<<endl; });
Итератор указывает на символ в строковой переменной, и вам не нужно выполнять сравнение строк
Не ответ, а потенциально полезная ссылка для вашей метапроблемы: Пример Boost.Spirit для разбора римских цифр