1974

Что думаете, коллеги?

Вступление.

Привет. Я тут новенький. Работаю в геймдеве уже более 10 лет. Последние лет 5  - мобильные игры. В основном С++, а так же немного луа и питон. Есть опыт в наставничестве и хочется развиваться в этом направлении.


Цель.

Я типа добрый самаритянин. Есть желание развиваться как наставник. Делиться опытом по мере сил/возможностей  и тем самым помогать начинающим, либо более опытным программистам. Возможности диктуют в среднем час в день. Разумеется бесплатно.


Способ.

Как именно помогать, пока не в знаю. Предложения в комментах. Оптимальный способ на мой взгляд - кодревью. Возможно созвоны.


PS.

Если пост зайдёт, значит востребован. Если нет - минусуйте на здоровье.

Лига программистов

2.1K постов11.9K подписчиков

Правила сообщества

- Будьте взаимовежливы, аргументируйте критику

- Приветствуются любые посты по тематике программирования

- Если ваш пост содержит ссылки на внешние ресурсы - он должен быть самодостаточным. Вариации на тему "далее читайте в моей телеге" будут удаляться из сообщества

Вы смотрите срез комментариев. Показать все
1
Автор поста оценил этот комментарий

А можно мне код-ревью?

https://github.com/nazarpunk/cheapack

раскрыть ветку (12)
долболед
Автор поста оценил этот комментарий

У вас там идеи в игноре. Вы хоть делитись ими между собой?))

Иллюстрация к комментарию
раскрыть ветку (11)
1
Автор поста оценил этот комментарий

Я так понимаю, продуктами от JetBrains вы не пользуетесь?

раскрыть ветку (9)
долболед
Автор поста оценил этот комментарий

Нет. Так понимаю,  IDE предоставляет возможность делать заметки и дельные мысли?

раскрыть ветку (8)
0
Автор поста оценил этот комментарий

В этой папке всего лишь хранятся настройки проекта. Заметки и дельные мысли правда тоже там хранятся.

Иллюстрация к комментарию
раскрыть ветку (7)
0
долболед
Автор поста оценил этот комментарий

Бегло посмотрел реализацию собиратора. В целом ничё так. В первую очередь смотрел читабельность кода. Есть пару замечаний и предложение.


1) Некоторые имена функций неочевидны. Например tableInsertReadStr

https://github.com/nazarpunk/cheapack/blob/fc4cee334698e0a07...


2) "main"-функция слишком громоздкая.

https://github.com/nazarpunk/cheapack/blob/fc4cee334698e0a07...


3) Предлагаю внести мини-фикс. Доступа к репе нету, поэтому приготовил патч

https://www.dropbox.com/s/aogecphl10q1ila/patch_read_me.diff...

раскрыть ветку (6)
0
Автор поста оценил этот комментарий
Некоторые имена функций неочевидны. Например tableInsertReadStr

У меня не хватило фантазии по другому назвать функцию, которая «добавляет в таблицу прочитанную строку».


"main"-функция слишком громоздкая

Код из этой функции не будет переиспользован, так зачем создавать лишние функции?


Доступа к репе нету, поэтому приготовил патч

Можно же форкнуть, запушить коммит и сделать pull-request.

раскрыть ветку (5)
2
Автор поста оценил этот комментарий

У меня не хватило фантазии по другому назвать функцию, которая «добавляет в таблицу прочитанную строку».


Одна функция - одно смысловое действие. Тогда и с неймингом проблем не будет: TableInsert(ReadStr());
раскрыть ветку (1)
0
Автор поста оценил этот комментарий

Не нужно плодить сущности сверх меры.

0
долболед
Автор поста оценил этот комментарий
У меня не хватило фантазии по другому назвать функцию, которая «добавляет в таблицу прочитанную строку».

Это да, извечная проблема, как назвать функцию. Но есть нюанс. Если для функции вот никак не получается придумать имя, то может функция делает что-то не то.


Код из этой функции не будет переиспользован, так зачем создавать лишние функции?

Дело в другом. Сейчас для вас этот код вполне очевиден. Но что будет через год, два, пять лет? Сможете ли еог с легкостью прочитать. Это с одной стороны. С другой, при работе в команде с вашим кодом будут работать другие люди. Далеко не факт, что они сходу вникнут в вашу логику. Да и не факт, что вникнут.


В продолжение обсуждения апру вопросов. Доводилось ли вам работать в команде из 20, 30 или более человек? Доводилось ли вам работать с чужим кодом, вне вашего проекта?


Можно же форкнуть, запушить коммит и сделать pull-request.

Про форк что-то там слышал, но не знал до сего момента, что это такое и с чем его едят. Спасибо за подсказку. Изучу.

раскрыть ветку (2)
0
Автор поста оценил этот комментарий
Если для функции вот никак не получается придумать имя, то может функция делает что-то не то.

Она делает ровно то, что нужно - читает строку и добавляет её в таблицу.

Но что будет через год, два, пять лет? Сможете ли еог с легкостью прочитать.

Для этого в коде комментарии оставлены.

С другой, при работе в команде с вашим кодом будут работать другие люди. Далеко не факт, что они сходу вникнут в вашу логику. Да и не факт, что вникнут.

Можете показать код, который не открытое API, в логику которого можно сразу вникнуть?

Доводилось ли вам работать в команде из 20, 30 или более человек?

Максимум до пяти доходило.

Доводилось ли вам работать с чужим кодом, вне вашего проекта?

Мне часто приходилось работать с говнокодом вне своего проэкта.

Про форк что-то там слышал, но не знал до сего момента, что это такое и с чем его едят.

Удобная штука. Единственный способ внести изменения в чужой репозиторий не являясь мэйнтейнером.

раскрыть ветку (1)
0
долболед
Автор поста оценил этот комментарий
Она делает ровно то, что нужно - читает строку и добавляет её в таблицу

Не совсем. Она пытается считать строку. Если попытка удалась, то строка добавляется в таблицу и функция возвращает строку. Если не удалось, то таблица не меняется и на выход получаем nil.


Можете показать код, который не открытое API, в логику которого можно сразу вникнуть?

https://code.videolan.org/videolan/vlc/-/blob/master/src/pla...

По началу смутило именование, пока не понял, что Dup - это Duplicate, а priv это private

0
Автор поста оценил этот комментарий
Это локальные настройки IDE, их лучше игнорить.
Вы смотрите срез комментариев. Чтобы написать комментарий, перейдите к общему списку

Темы

Политика

Теги

Популярные авторы

Сообщества

18+

Теги

Популярные авторы

Сообщества

Игры

Теги

Популярные авторы

Сообщества

Юмор

Теги

Популярные авторы

Сообщества

Отношения

Теги

Популярные авторы

Сообщества

Здоровье

Теги

Популярные авторы

Сообщества

Путешествия

Теги

Популярные авторы

Сообщества

Спорт

Теги

Популярные авторы

Сообщества

Хобби

Теги

Популярные авторы

Сообщества

Сервис

Теги

Популярные авторы

Сообщества

Природа

Теги

Популярные авторы

Сообщества

Бизнес

Теги

Популярные авторы

Сообщества

Транспорт

Теги

Популярные авторы

Сообщества

Общение

Теги

Популярные авторы

Сообщества

Юриспруденция

Теги

Популярные авторы

Сообщества

Наука

Теги

Популярные авторы

Сообщества

IT

Теги

Популярные авторы

Сообщества

Животные

Теги

Популярные авторы

Сообщества

Кино и сериалы

Теги

Популярные авторы

Сообщества

Экономика

Теги

Популярные авторы

Сообщества

Кулинария

Теги

Популярные авторы

Сообщества

История

Теги

Популярные авторы

Сообщества