Obviously we do extensive exception handling. I'm not certain why the Slack code didn't throw an exception. I'm guessing the I/O stream locked up somewhere, possibly while trying to get the contents of the error stream when it didn't get a 200 status code. Nothing was immediately apparent when I did a review earlier and also now.
I'll move the scorebox display after the game being stored, though. No reason not to. Well, actually, some reason not to; this code is ancient and written almost exclusively by computer science students and amateur enthusiasts who are still learning how to code. So it doesn't exactly follow best practices. Sometimes a method that indicates it's only displaying scores might actually modify things ... goofy stuff like that. In this case it looks to be safe, though.
I'll move the scorebox display after the game being stored, though. No reason not to. Well, actually, some reason not to; this code is ancient and written almost exclusively by computer science students and amateur enthusiasts who are still learning how to code. So it doesn't exactly follow best practices. Sometimes a method that indicates it's only displaying scores might actually modify things ... goofy stuff like that. In this case it looks to be safe, though.
Comment