Z
Z
ZvookieJoo2016-01-15 09:35:09
C++ / C#
ZvookieJoo, 2016-01-15 09:35:09

Who can review the Windows desktop application project?

Hello colleagues. I am a novice programmer and I would like one of you to evaluate my code and application in general. In short: I wrote a small utility to drain music content from an Apple iPod 7 Nano. This is my first experience of programming desktop applications on Windows and I would like to get feedback from more experienced colleagues.
Repository with MS Visual Studio 2008 project on github: https://github.com/zvookiejoo/ipod-downloader

Answer the question

In order to leave comments, you need to log in

1 answer(s)
S
so-olitary, 2016-01-26
@so-olitary

Looked. I don't have an ipod.
I liked the neat design of the repository.
Good structuring of the program - everything is divided into files, handlers and functions according to the meaning.
The interface is meaningful, does not hang or flicker.
Visual programming method learned.
Well done.
Few comments in the code at all.
Lots of global variables - you can pack everything into classes, IMHO.
Interfering with malloc & new is bad practice. By specification, they are not required to be compatible.
this is optional.
Error processing? Are errors ignored ( if (!text) return; ) a little less than completely?
The question of how to interfere with C & C ++ code remains open for me - with each new version they become less and less compatible ... It's better to avoid it altogether.
The question remains only in the coverage of the program with tests.

Didn't find what you were looking for?

Ask your question

Ask a Question

731 491 924 answers to any question