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

swifty_viper префикс подставляется только в названия фаилов, но не классов/протоколов #19

Open
aknew opened this issue Jan 5, 2017 · 2 comments

Comments

@aknew
Copy link

aknew commented Jan 5, 2017

Добрый день.
Столкнулся с таким поведением - создал новый проект, запустил generamba setup, благополучно прошел этап настройки в котором указал The project prefix (пусть будет для примера SV). Создаю модуль MyModule (да, название не удачное, но с ходу лучше ничего не придумал для тестового проекта) и с удивление вижу что у файлов префикс имеется, а у сущностей внутри его нет т.е. например, есть файл SVMyModuleInteractorInput.swift внутри которого объявлен протокол MyModuleInteractorInput без префикса и так со всеми классами\протоколами, полез в сами шаблоны - префиксов там действительно нет. Это так и было задумано или все-таки ошибка?
Если ошибка - дайте знать, я для себя буду править отсутствие префиксов в шаблонах и просто залью его тогда пулреквестом
С уважением

@akkrat
Copy link

akkrat commented Jan 21, 2017

Ошибкой в данном случае является добавление префикса к имени файла. Для swift кода общепринятой практикой является неиспользование префиксов https://github.com/raywenderlich/swift-style-guide#class-prefixes. Например, Apple в swift 3 убрала у многих типов префикс NS.

@aknew
Copy link
Author

aknew commented Jan 21, 2017

Вполне возможно, я, если честно, только начинаю работать со свифтом - все как-то больше по старинке на obj-c. Но вот то что названия файлов не совпадают с тем что внутри точно не порядок. Подозреваю что никто просто не пытался мешать вместе модули на свифте и obj-с (уточню - я это сделал забавы ради чтоб потренироваться, в реальном проекте я бы такое делать не стал), вот и не всплыло.

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