yeet size_t

QUESTIONS.md

@@ -1,7 +1,7 @@
 Questions que je (Thomas Rubini) voudrais poser
 
 - Est-on obligé d'utiliser size_t quand on sait que la taille du vecteur ne dépassera jamais concrètement la taille d'un int (cas précis : taille de 100 maximum, est-on obligé d'utiliser size_t de 8 bytes ?)
-* Non
+* Non, on peut utiliser unsigned si on est sur qu'il n'y aura pas d'overflow
 - Que pensez-vous de la sémantique de déplacement, plutot que la référence constante ?
 * on peut utiliser la sémantique de déplacement au besoin
 - Est-ce qu'on doit forcément utiliser const pour des valeurs primitives (int, float...) qu'on ne touche pas en paramètres de fonction ?

headers/menu.h

@@ -28,7 +28,7 @@ struct Menu{
      /*!
 	 * @brief index of currently selected menu option 
 	 */
-    size_t currentValue = 0;
+    unsigned currentValue = 0;
 
     /*!
 	 * @brief color of currently selected menu option

src/configManagement.cpp

@@ -281,7 +281,7 @@ void ConfigBuilder::getColor(const configKey& key, nsGraphics::RGBAcolor& color)
 }
 
 void ConfigBuilder::getList(const configKey& key, vector<string>& toPopulate) const {
-	size_t i=0;
+	unsigned i=0;
 	string fullKey = key+".0";
 	if(!internalValues.contains(fullKey))throw config_error("Non-existent list key requested : "+key);
 

src/drawMenu.cpp

@@ -30,8 +30,8 @@ void PixelManager::displayMenu(const Position& pos, Menu& currentMenu){
 	drawSprite(menuBackground);
 	drawSprite(logo,Position(100,50));
 	drawText(Position(1150, 700), "version 1.0.0");
-	size_t margin = 0;
-	size_t cpt = 0;
+	unsigned margin = 0;
+	unsigned cpt = 0;
 	for(string& value : currentMenu.entries ){
 		displayButton(Position(0,0+margin)+ pos, value, (currentMenu.currentValue == cpt) ? currentMenu.selectedColor : currentMenu.unSelectedColor );
 		++cpt;
@@ -78,14 +78,14 @@ PlayMode PixelManager::showInitialMenu(){
 
 void PixelManager::askPlayerNameMenu(playerID pID, unsigned score, string& name) {
 	name = string(6, 'A');
-	size_t currentSelected = 0 ;
+	unsigned currentSelected = 0 ;
 	chrono::milliseconds waitTime = chrono::milliseconds(100);
 	while (window.isOpen()){
 		startFrame();
 		drawSprite(menuBackground);
 		drawText(Position(600, 100), "Nom du joueur " + to_string(pID + 1));
 		drawText(Position(600, 150), "Score : " + to_string(score));
-		for (size_t i = 0; i < name.size(); ++i){
+		for (unsigned i = 0; i < name.size(); ++i){
 			drawText(Position(600 + 30 * i, 200), string(1, name[i]),
 					 (i == currentSelected) ? nsGraphics::KRed : nsGraphics::KWhite);
 		}

src/game/display.cpp

@@ -41,7 +41,7 @@ void Game::displayAll() const {
 		pm.drawText(Position(pm.getScreenWidth()-200, 20), "FPS : "+to_string(fps), nsGraphics::KWhite, Font::BITMAP_8_BY_13);
 	)
 
-	for(size_t i=0;i<players.size();++i){
+	for(unsigned i=0;i<players.size();++i){
 		if(!players[i].isEliminated()){
 			if(players[i].deathAnimCounter%2==0){
 				pm.drawPlayer(players[i].x, confData.playersWidth, confData.playerDefs[i].color);

src/game/gameManagers.cpp

@@ -166,7 +166,7 @@ void Game::checkMissilesAndPlayers() {
 bool Game::checkTorpedosAndInvaders() {
 	auto tor_ite = torpedos.begin();
 	while(tor_ite!=torpedos.end()){
-		size_t i=0;
+		unsigned i=0;
 		for(;i<grid.size();++i){
 
 			unsigned alienIndex = grid[i].getOutterInvader();

src/main.cpp

@@ -16,7 +16,6 @@
 #include "game.h"
 using namespace std;
 
-// TODO changer tout les unsigned par des size_t dans les boucles
 int main(){
 	DEBUG_MSG("Starting program")
 	srand(time(NULL));