Le code est plutôt clair est lisible (j'aime pas les #pragma, mais je suis old-school), je ne me suis pas penché sur le design, quelques remarques/questions en vitesse :
Je vois que tu utilises souvent des shared_ptr, je n'ai pas encore bien regardé le besoin à haut niveau, mais parfois, les unique_ptr font l'affaire et te coûteront probablement moins cher ;
Sinon, évite d'utiliser std::endl, '\n' est ce que tu veux utiliser dans 99% des cas (dans ton cas, c'est juste dans le main(), donc ça changera rien mais mieux vaut prendre des bonnes habitudes) ;
Dans assets.cpp, tu dois pouvoir initialiser pathPrefixes_ à l'aide d'une intializer list, ça t'évite d'avoir à ré-allouer. (pareil qu'avant : dans ce contexte c'est du chipotage, mais il s'agit de prendre le pli).
Si j'ai du temps et que je m'ennuie, j'essaierai de prendre du recul et voir comment tout ce tient.
# j'ai vu bien pire
Posté par nazcafan . En réponse au journal Veuillez instancier ce journal avant de le lire. Évalué à 8. Dernière modification le 09 octobre 2014 à 18:50.
Le code est plutôt clair est lisible (j'aime pas les #pragma, mais je suis old-school), je ne me suis pas penché sur le design, quelques remarques/questions en vitesse :
framerate.cpp:utiliser std::max ?
Je vois que tu utilises souvent des
shared_ptr, je n'ai pas encore bien regardé le besoin à haut niveau, mais parfois, lesunique_ptrfont l'affaire et te coûteront probablement moins cher ;Sinon, évite d'utiliser
std::endl,'\n'est ce que tu veux utiliser dans 99% des cas (dans ton cas, c'est juste dans le main(), donc ça changera rien mais mieux vaut prendre des bonnes habitudes) ;Dans
assets.cpp, tu dois pouvoir initialiserpathPrefixes_à l'aide d'une intializer list, ça t'évite d'avoir à ré-allouer. (pareil qu'avant : dans ce contexte c'est du chipotage, mais il s'agit de prendre le pli).Si j'ai du temps et que je m'ennuie, j'essaierai de prendre du recul et voir comment tout ce tient.