Кто может проверить проект десктопного приложения для Windows?

Question

[ZvookieJoo]

Здравствуйте, коллеги. Я - начинающий программист и хотел бы, чтобы кто-то из Вас оценил мой код и приложение в целом. Если вкратце: я написал небольшую утилиту, для слива музыкального контента из Apple iPod 7 Nano. Это - мой первый опыт программирования десктопных приложений под Windows и я хотел бы получить обратную связь от более опытных коллег.

Репозиторий с проектом MS Visual Studio 2008 на гитхабе: https://github.com/zvookiejoo/ipod-downloader

Answer 1

[so-olitary]

Посмотрел. ipod'а у меня нет.
Понравилось аккуратное оформление репозитория.
Хорошее структурирование программы - всё разбито на файлы, обработчики и функции по смыслу.
Интерфейс осмысленный, не виснет и не мерцает.
Метод визуального программирования усвоен.
Молодец.

Мало комментариев в коде совсем.
Много глобальных переменных - можно упаковать всё в классы, ИМХО.
Мешать в программе malloc & new - bad practice. По спецификации они не обязаны быть совместимыми.
this - не обязательно указывать.
Обработка ошибок? Ошибки игнорируются ( if (!text) return; ) чуть менее чем полностью?
Вопрос как мешать код С & C++ для меня остаётся открытым - с каждой новой версией они становятся всё менее совместимыми... Лучше избегать совсем.

Вопрос остаётся только в покрытии программы тестами.

Comments

[ZvookieJoo]

Спасибо! Согласен насчёт обработки ошибок: я ещё не определился, как бы их по уму обрабатывать. Насчёт malloc и new - запомню, спасибо. This использовал для себя - чтобы не забывать, что обращаюсь к члену класса, а не к одноимённому входному параметру метода.