выполнения зависимостей для станд :: конкатенации

Question

std::string sAttr(""); 
sAttr = sAttr+VAL_TAG_OPEN+sVal->c_str()+VAL_TAG_CLOSE; 

еще где в коде я определил

const char VAL_TAG_OPEN[] = "<value>"; 

sVal является переменной, которая извлекается прочь массив строковых указателей. Это отлично работает в большинстве систем, окон и Linux. Однако на сайте клиента, где, как я полагаю, есть версия Linux, на которой мы провели некоторое тщательное тестирование, произведите результат, как если бы я никогда не использовал VAL_TAG_OPEN и VAL_TAG_CLOSE. Полученные результаты приведены для

sAttr = sAttr+sVal->c_str(); 

Что происходит?. Содержит ли std :: string конкатенация в зависимости от времени выполнения?

Answer 1

Почему ->c_str()? Если sVal - std::string, попробуйте удалить этот вызов. Помните, что порядок оценки не определен, поэтому вы можете добавить указатели вместо конкатенации строк, потому что VAL_TAG_OPEN, sVal->c_str() и VAL_TAG_CLOSE - все простые строки C. Я предлагаю вам использовать оператор присваивания добавления +=, например. :

sAttr += VAL_TAG_OPEN; 
sAttr += *sVal; /* sVal->c_str() ? */ 
sAttr += VAL_TAG_CLOSE; 

(что должно быть быстрее в любом случае).

Answer 2

Нет, std :: string конкатенация не должна зависеть от времени выполнения, но каким-то образом VAL_TAG_OPEN и VAL_TAG_CLOSE кажутся пустыми.

Я бы предположил, что где-то есть какой-то переполнение буфера или неправильная арифметика указателя, так что ваша программа перезаписывает память, содержащую эти «постоянные» значения. Всякий раз, когда ваша память заканчивается, это действительно время выполнения (и, следовательно, версия ОС). В прошлом я был в ловушке подобных вещей, переключая компиляторы или параметры оптимизатора.

Как вы отмечаете, что при указании исходных указателей на std :: string экземпляры в необработанных массивах такие ошибки действительно не все невероятны, но их может быть сложно обнаружить, поскольку использование сборки DEBUG не даст вам никаких проверок итератора с все это все для RAW вещей ... Удачи.

Answer 3

sAttr = sAttr+VAL_TAG_OPEN+sVal->c_str()+VAL_TAG_CLOSE; 

Как и fbonnet, это вопрос оценки.

Если эта строка оценивается строго слева направо, то результатом каждого добавления является объект std :: string, который имеет перегрузку оператора для добавления, и все работает так, как вы ожидаете.

Если он не оценивается слева направо, вы завершаете добавление указателей вместе, и кто знает, что вы получите.

Избегайте этой конструкции и просто используйте оператор + = на std :: string.

Answer 4

У меня нет оценки, которая вызывает проблему. Его из-за постоянные массивы полукокса в начале и в конце

оператора

const char VAL_TAG_OPEN[] = "<value>"; 
const char VAL_TAG_CLOSE[] = "</value>" 

конкатенации думал VAL_TAG_OPN и VAL_TAG_CLOSE как не пустая строка терминатора. Следовательно, оптимизатор просто проигнорировал их, считая это мусором.

sAttr += std::string(VAL_TAG_OPEN); 
sAttr += *sVal; 
sAttr += std::string(VAL_TAG_CLOSE); 

Это решение.