You are viewing a single comment's thread from:

RE: DTube App for Android & iOS

in #utopian-io6 years ago (edited)

Thanks for the contribution, @bostrot. As @reazuliqbal has already mentioned some things to keep in mind here and I have already left a comment here I won't go over the same things again in this comment.

Instead I will focus on reviewing this commit, which is what I'll base your score off of. For future reference it would've been better if the development contribution covered and explained the feature(s) added in the above commit instead of the entire project (for project introductions you can use the blog category instead).

Anyway, let's have a look shall we?

  • I'm not familiar with Dart and since there are no comments it's not really clear to me where the magic numbers in this come from. Wouldn't it be better to use constants declared somewhere else for this?
  • The amount of characters per line is pretty high in some places. Not sure if there aren't some best practices to keep the line width below a certain number, but I think it would definitely improve the readability of the code (had to scroll horizontally in some places).
  • There aren't really that many comments (function declarations would be nice at least) and there is quite a lot of commented out code left in there. In my opinion it would be better to remove those unused snippets, but if you might need them in the future then I understand.

Other than that everything seems to look fine to me and I don't have any other remarks - great work! I will once again (it's that good) recommend you read this amazing guide on how to achieve a higher score in the development category before creating another contribution, as it will definitely help you receive a higher reward (it's exponential and not linear).

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? Write a ticket on https://support.utopian.io/.
Chat with us on Discord.
[utopian-moderator]

Sort:  

Thank you for your review, @amosbastian!

So far this week you've reviewed 3 contributions. Keep up the good work!

Well thank you again for this detailed anwer. I will add more comment in the future. The magic numbers you mentioned is a function that returns an Array (in Dart: list) of possible video qualities. This happens with checking whether the desired quality exists in json_metadata.
Wrapping lines can be done, though Dart especially Flutter is known for those heavy indents. But its true I could have avoided a few of these.
Some of the unused parts are important and some not. Will sort this out.

Coin Marketplace

STEEM 0.30
TRX 0.12
JST 0.034
BTC 64136.70
ETH 3128.20
USDT 1.00
SBD 3.94