Skip to content

Homework #6#6

Open
mashfeii wants to merge 8 commits intomasterfrom
hm-6
Open

Homework #6#6
mashfeii wants to merge 8 commits intomasterfrom
hm-6

Conversation

@mashfeii
Copy link
Owner

@mashfeii mashfeii commented Jul 17, 2024

Признаюсь честно, я не думал, что потрачу на это столько времени...
Я пытался написать адекватный Makefile, но ничего не получилось, поэтому разработал план:

  • python3 -m venv venv
  • source venv/bin/activate или source venv/bin/activate.fish в моём случае
  • ./setup.py install + chmod +x setup.py до этого если не сработает
  • python3 -m pip install -r requirements.txt
  • А дальше вроде уже можно жить спокойно и делать make/make speed

Я написал две функции loads, наверное, стоило вынести в отдельный метод модуля, а не файл... После первой рекурсивной реализации хотел ускорить через работу с boost библиотекой, а в итоге написал через std::regex, тем более давно хотел их попробовать
loads не понимает float, не баг, а разработчик запитонился 🙂

Copy link
Collaborator

@tarasovxx tarasovxx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В целом я ничего критического не подметил, так что прекрасная работа, реально круто сделал:)

И круто, что преисполнился, значит ты всё сделал правильно)

return nullptr;
}

std::regex expression("\"([A-Za-z]*)\"\\s*:\\s*(\"?\\w*\"?)\\s*[},]?\\s*");
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Всегда помни фразу, может и не в этом случае, но просто помни.

"Хочешь решить проблему регуляркой, теперь у тебя 2 проблемы"
Сказал твой ментор:)

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

Successfully merging this pull request may close these issues.

2 participants