(no subject)
Dec. 8th, 2005 02:56 pm![[personal profile]](https://www.dreamwidth.org/img/silk/identity/user.png){kind=small}
jackofallgeeksFrom our professor:
"I reviewed your TicTacToe submission. One note: you read the user list with each iteration through the 'for loop' contained in the 'while loop.' This is an excessive number of file reads which could dramatically impact the performance. Unless you expect the file to be modified during execution, it would be better to read it once and refer to the copied version."
A part of me is feeling rather smug about this, as I suggested the same thing when we were coding the thing, but I was overruled because 'it doesn't matter that much.'
"I reviewed your TicTacToe submission. One note: you read the user list with each iteration through the 'for loop' contained in the 'while loop.' This is an excessive number of file reads which could dramatically impact the performance. Unless you expect the file to be modified during execution, it would be better to read it once and refer to the copied version."
A part of me is feeling rather smug about this, as I suggested the same thing when we were coding the thing, but I was overruled because 'it doesn't matter that much.'
