MacOS и size_t

[gogi]

да, извиняюсь, скрин в аттаче

У тебя параметр str на стеке создаётся при вызове функции, а потом ты возвращаешь static QString. Что-то здесь не то.

[gogi]

С size_t есть проблема. Безобидное, казалось бы, изменение в коде
for(size_t i = 0; i< v.size(); i++)

for(size_t i = 0; i< v.size()-1; i++)

приводит к печальным последствиям...

Вычитание уже само по себе намекает на знаковый тип, об этом даже могут сообщать шибко настроенные на варнинги компиляторы/анализаторы. Поэтому size_t заменить на ssize_t. Ну или если не хочется с нестандартность связываться - на int.

[GreatSnake]

У тебя параметр str на стеке создаётся при вызове функции, а потом ты возвращаешь static QString. Что-то здесь не то.

Он возвращает копию str. А 'static' относится к 'видимости' функции, а не к её возвращаемому значению.
Другое дело, что аргумент по-хорошему должен быть 'const QString&'.

[Пантер]

Другое дело, что аргумент по-хорошему должен быть 'const QString&'.

Это почему же? Тогда бы пришлось вводить свою переменную внутри функции.

[Пантер]

Если уж на то пошло, то правильнее будет:

Код

C++ (Qt)
static QString invertSlash (QString str)
{
  return str.replace ("/", "\\");
}
 

[GreatSnake]

Это почему же? Тогда бы пришлось вводить свою переменную внутри функции.

Ну да, зато прототип красивый будет, как у троллей

[LisandreL]

Вычитание уже само по себе намекает на знаковый тип, об этом даже могут сообщать шибко настроенные на варнинги компиляторы/анализаторы. Поэтому size_t заменить на ssize_t.

ptrdiff_t вполне стандартный знаковый, совпадающий по разрядности с размером указателя (как и size_t).
Но вообще, как и писал Авварон, раз QString индексируется int'ом, то и цикл лучше делать по нему.

[Авварон]

Свои 2 копейки внесу.

Код:

static QString invertSlash (const QString &str)
{
  return QString(str).replace("/", "\\");
}

[Пантер]

Авварон, по сравнению с моим вариантом профита никакого.

[Авварон]

Кодестайлинг и фсе дела