You are viewing a single comment's thread from:

RE: Snake3D: Released Version 1.1 containing an upgrade of the GUI.

in #utopian-io6 years ago

Thank you for your contribution.

  1. You could use the custom sorting in Java instead of writing your own sorting here
  2. You might use Arrays.fill instead in here if the array is continous.
  3. duplicate code could be refactored: here and here

Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.

To view those questions and the relevant answers related to your post, click here.


Need help? Chat with us on Discord.

[utopian-moderator]

Sort:  

Thanks for your review.

  1. It is hard to sort an array if the array itself does not also contain the data that it should be sorted by.
  2. Yes I could do that, but that wouldn't improve readability (I could only replace the innermost loop which doesn't make a big difference in readability), but it would also mean an unnecessary drop in performance based on the data mentioned here in the second answer.
  3. You are right. I was thinking about making a new class "Button", but that has some complications so I decided to keep it as it is for now(until the next update).

Thank you for your review, @justyy! Keep up the good work!

#utopian-io has been a gift. We would like to keep its curation-efforts alive here on 'marlians.com'. We want to curate and reward 'contributions to open-source projects' with MARLIANS on the the marlians.com tribe, a SCOT-enabled steem condenser. Contributions can include suggestions, graphics, bug-finds, code etc. You can simply add in #marlians to your #utopian-io posts and it will appear on https://www.marlians.com/created/utopian enabling you to earn some MARLIANS along with steem/sbd. You can also post directly to steem via 'marlians.com'. We have some overseers who curate and who can render you help too. You can find them enlisted on https://www.marlians.com/created/utopian

Coin Marketplace

STEEM 0.09
TRX 0.31
JST 0.033
BTC 110045.53
ETH 3881.23
USDT 1.00
SBD 0.58