| TOTAL | CREATIVITY | USEFULNESS | DESIGN | SPEED | FUNCTIONALITY | 24 | 2 | 8 | 4 | 4 | 6 | | out of 33 | out of 3 | out of 10 | out of 6 | out of 4 | out of 8 |
| 1lann
A good entry, although you're not the first to implement something like this, so a mark was deducted for creativity. 2 marks were dropped for design due to some code quality issues such as having too many indentation levels. Try to create a separate function, or make use of guard code (Google it if you don't know what a guard code is) when your indentation is going too far. Some code can also be shortened. For example:
if eventData[2] == 54 then rShiftPressed = true end
can be shortened to
rShiftPressed = eventData[2] == 54
The implementation itself isn't the most efficient, and could be made shorter and simpler. The program can be quite useful, so it scored 8/10. Functionality wise it was okay, although more features could definitely be added. A command to download files from the remote computer to the local computer like "scp" would be cool, or even remote screen control over HTTP would have scored you higher marks. Your implementation is fast though with you buffering the screen, and I credit you for that. I also credit you for your user experience choices, which were nice. Unfortunately I did encounter some bugs with your program, such as being unable to select the difficulty when running worm through the remote desktop due to the enter event being fired at different times. It also unfortunately failed the "Firewolf screen test" (which is admittedly quite hard to pass), where I attempt to use Firewolf through your remote desktop program. Unfortunately it caused the host and client computer to freeze to what seems like an infinite loop of events. Overall though, you have a good submission. I hope my score doesn't discourage you, I think you really do have potential to make awesome programs in the not so distant future. |