Post
Topic
Board Announcements (Altcoins)
Re: [ANN] [ARO] | Arionum | CPU+GPU+Masternode | PHP Based |Decentralized Revolution
by
ConcertBil
on 04/10/2018, 10:10:35 UTC
Quick Code Review and issues:

1.Telegram?

Type your direct Telegram account

2.No framework / No MVC / No .htaccess etc

Idea is great! And it is a great quick start!

It is time to start refactoring and we have a problem:

Routes like /mine.php are already in use by other nodes and clients  Sad

So for by backward compatibility we will have quirks here.

BTW it is not only PHP coin, it is coin based on HTTP protocol, so we can develop services based on NodeJS, Django, Go, etc

3.Not enough documentations how does it works

Type algorithms somewhere

4.Directly including config.php - bad way

config files should be in .gitignore file

5.And many more

It is quick code review, can do better. Donate.

My Telegram: @vvmspace

My WhatsApp: +79530011203

Donate for more Code Review and contributions:

iumf9ADadSYULPJE8t9UWyFs56SY53F3QN48hG6boMHxnvhmoXy5PnaqKxwhGNukceXLStPhQurG868 ULMArg1M