Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Реализовать сервисный класс для выгрузки конфигурации хранилища #82

Open
ava57r opened this issue Oct 11, 2017 · 5 comments
Assignees

Comments

@ava57r
Copy link
Member

ava57r commented Oct 11, 2017

реализовать класс для сохранения конфигурации хранилища
метод save_depot_config
метод save_part_depot_config
Эти два метода в сумме 1460 строк

@dmpas
Copy link
Collaborator

dmpas commented Oct 11, 2017

1460 строк

жесть...

@ava57r
Copy link
Member Author

ava57r commented Oct 11, 2017

да вот и я листал, листал туда сюда.
Два метода дублируют функциональность и проверки.
Начал пробовать бить на части прямо в 1CD, но выглядет это страшно read_record_table_depot и тому подобное.
Буду делать отдельного "Сапожника" для выгрузки конфы.

Тут вот еще ищу альтернативу new char[]
в коде куча адресной арифметики с rec+field->offset
чем ее заменить?

@dmpas
Copy link
Collaborator

dmpas commented Oct 11, 2017

чем ее заменить?

пока ничем. ибо тут именно работа с двоичными данными.

@ava57r ava57r self-assigned this Nov 16, 2017
@dmpas dmpas added this to the pre-release milestone Jan 12, 2018
@ava57r
Copy link
Member Author

ava57r commented Feb 10, 2018

@dmpas класс не реализовали, но выгрузку основательно перетрясли.
Стоит пересмотреть или закрыть и сформулировать новую задачу?

@dmpas
Copy link
Collaborator

dmpas commented Feb 10, 2018

Оставь. Суть в том, чтобы общий алгоритм обхода выделить

@dmpas dmpas removed this from the pre-release milestone Apr 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants