Как лучше реализовать локализацию?

Question

[Mercury13]

У нас был такой код…

namespace loc {
  const wchar_t* get(const char* key);
}

Выяснилась такая штука. Для языков, которые мы не знаем и вынуждены «забивать» английским, а правила множественного числа другие, выводит что-то типа «5 day remaining» — просто потому, что у японского нет этих правил.

Хорошо, корректируем вот так.

struct TaggedCWstr
{
    const PluralRule* rule;
    const wchar_t* text;

    operator const wchar_t* () const { return text; }
    operator std::wstring () const { return text; }
};

namespace loc {
  TaggedCWstr get(const char* key);
}

И тут возник камень преткновения на operator std::wstring. Если я его комментирую, выводит ошибку на

std::wstring xxx() {
  return loc::get(key);    // could not convert from TaggedCWstr to wstring
}

Если оставляю, выводит ошибку на

std::wstring s;
s = loc::get("key");    // ambiguous overload for op=

Можно ли как-то удовлетворить обоих?

Comments

[Армянское Радио]

Текст ошибки приведите. Ну и это, wstring и utf-16 вам все мозги вынесет при локализации, utf-8 получше будет.

[Mercury13]

Армянское Радио, Написал текст второй ошибки.

[Mercury13]

Армянское Радио, Повторю в комментах: could not convert from TaggedCWstr to wstring

[Mercury13]

Ну и это, wstring и utf-16 вам все мозги вынесет при локализации, utf-8 получше будет.

Эту операцию мы уже прошли. А один хрен в Windows используется UTF-16. А при хранении — UTF-8.

[Mercury13]

У нас очень много старого кода, понимаете… А все эти поля типа return loc::get("key").text; ухудшают понимание.

[Армянское Радио]

Mercury13, слишком мало кода, чтобы понять, где ошибка.

[Mercury13]

Если же сделать особую функцию типа
Wprintf(loc.getTagged("key"))(nDays) — код будет ошибкоопасным.

[Mercury13]

Армянское Радио, Мне в бега, но попробую сделать маленький пример.

[Mercury13]

Армянское Радио,

#include <iostream>

class PluralRule {};

PluralRule DEFAULT_RULE;

struct TaggedCWstr
{
    const PluralRule* rule;
    const wchar_t* text;

    operator const wchar_t* () const { return text; }
    operator std::wstring () const { return text; }
};

namespace loc {
    TaggedCWstr get(const char* key) { return { &DEFAULT_RULE, L"text" }; }
}

std::wstring xxx() {
  return loc::get("key");    // could not convert from TaggedCWstr to wstring
}

int main()
{
    std::wstring s;
    s = loc::get("key");   // ambiguous overload for op=
    // Я забыл — в какой-то момент в Си++ отказались от op= в string x = "y";
    return 0;
}

Answer 1

[Евгений Шатунов]

s = loc::get("key");    // ambiguous overload for op=

неоднозначность у тебя появляется потому что оба конструктора std::wstring (конструктор копирования и конструктор преобразования из const wchar_t*) являются неявными.
Операторы преобразования TaggedCWstr так же являются неявными.

Эту неоднозначность нужно исключить. Замени, например, оба оператора преобразования на оператор преобразования в std::wstring_view.

Я бы не стал делать неявный оператор преобразования в std::wstring и этим позволять бесконтрольно обращаться к динамической памяти без явного понимания этого процесса. Лучше для получения std::wstring сделать operator *, а еще лучше - вообще не ломать семантику операторов и сделать метод с говорящим именем.

Comments

[Mercury13]

Решил писать новый движок с нуля по твоим рекомендациям. Как-то работает.