06-01-2015, 11:33 PM
Nice project. Looking at the commits there is hard coding or spaghetti code can be cleaned up. The fromArray method can just parse XML and put the values into variables (named by the tag names) rather than parsing the textual XML tags. If the servers should change, editing only a text file that contains the list of servers would allow for modular changes instead of editing hard coded servers inside this method. The load method could read from a text file rather than hard code the urls inside the code. You can edit the block inside updateServerData which checks the selected frame against the name of the selection by matching the selection against the value of the name.
Voted Friendliest NordInvasion Player
Quote Board is on Discord (last updated 2025.07.24)
"When I feel bad, I read your quote board." - Corndog
Nka: You’re playing it smart, always [using] logic, and they get pissed.
Kaasovic: God, this guy is unbeatable. I feel defeated in every verbal argument we end up in. Good job, Sir.
Odi: Hasn’t [PCK beaten] the game at this point?
Quote Board is on Discord (last updated 2025.07.24)
"When I feel bad, I read your quote board." - Corndog
Nka: You’re playing it smart, always [using] logic, and they get pissed.
Kaasovic: God, this guy is unbeatable. I feel defeated in every verbal argument we end up in. Good job, Sir.
Odi: Hasn’t [PCK beaten] the game at this point?