Созданы именованные константы в файле APIcfBase.cpp

[ava57r]

#22

[ava57r]

вот думаю может сделать по короче, вместо DATETIME_1600_YEARS_FORMAT_1C - DATETIME_1600_YEARS...

[dmpas]

_1S_EPOCH_START, 1600 - лишнее уточнение, которое должно стоять в документации к константе.

[dmpas]

И да! "1С", "1C" и "1S" лучше не использовать в названиях.

[ava57r]

И да! "1С", "1C" и "1S" лучше не использовать в названиях.

Вот так же подумал.
Так как назвать? =)

[dmpas]

EPOCH_START. можно положить внутрь класса _1CD, можно поиграться с пространствами имён. Ещё можно под дату сделать отдельный класс, внутри которого будет преобразование из/в и эта константа. Если класс будет невиртуальный и с одним полем типа int64_t, то можно все поля даты-времени безболезненно заменить на этот класс.

[artbear]

vendor вместо 1С ? вт, 13 июн. 2017 г. в 11:46, Alexander Andreev <notifications@github.com>:

…

И да! "1С", "1C" и "1S" лучше не использовать в названиях. Вот так же подумал. Так как назвать? =) — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <https://github.com/dmpas/tool1cd/pull/35#issuecomment-308049574>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACyRcWyVFxZnaTljBFqvRTQXM-0Oc4Ooks5sDkx1gaJpZM4N2d8d> .

[dmpas]

vendor вместо 1С ?

я бы всё же в определениях оперировал форматом файла, а на разработчиком.

[ava57r]

Про класс даты тоже думал.
Я думал так

1. Заменить магию
2. Пересмотреть функциональность и создать конкретные классы.
3. Замутить пространства имен.

Давайте создам задачу про пространства имен?

[dmpas]

@andreevlex
вынеси работу с датой в отдельный класс, в нём же заведи константу. Переименуй константу 7fff, пространства имён пока оставь.

[ava57r]

Тогда этот PR вливать не надо.
Создам по новой