Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding "planets" #8

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Adding "planets" #8

wants to merge 4 commits into from

Conversation

mygizli04
Copy link

It's a concept i've been thinking about, since we gave the AI "eyes" and they aren't smart enough to discover light without light, i've added 2 other "planets" ( like think of like earth but it's only purpose is to make light ). To stop glitches from happening the "planets" are also inhabitable ( That's why they have so little attributes! ) unless a glitch gets found that let's them find a way to live there for long periods of "time". There's definetly more to come and this's just a concept!

Seems like humans have a hard time without light, so this is just a temporary inclusion until we find a permanent solution.
It's a new concept and it's really early for now. Basically, the idea is for this to be a natural source of light to humans since they're not smart enough to create light without light. Very temporary.
It's another concept i've been thinking about. I've done some tests with "The Sun" and it seems like people aren't smart enough to find their way in complete dark, even though they should've learned that! Also some AI ignore "the light" completely and for some reason die for a stupid reason anyway.
@mygizli04
Copy link
Author

mygizli04 commented Feb 21, 2019

( Also this is not real code and some random code i threw since i don't know scala sorry about that :/ )

@YHWH-333
Copy link
Owner

I appreciate you submitting these as a proof of concept; I'll work on the syntax issues at a later time.

I did have class files for heavenly bodies aside from Earth, they just got…lost in the company filesystem. I'm totally supportive of rewriting them anyway. It's good practice.

One suggestion I do have: the Sun class should not be a class nor trait of its own, but actually an instance of a class called Star. Each star should have a collection of orbiting planets as one of its properties, like so:

class Star {
    var orbitingPlanets: Array[Planet];
}

Then in the main program:

var sol: Star = new Star();
sol.orbitingPlanets = {
    mercury,    // Also note that all of the planets must be declared/initiated beforehand.
    venus,
    earth,
    mars, 
    jupiter,
    saturn,
    uranus,
    neptune
}
// add pluto if you'd like

@mygizli04
Copy link
Author

mygizli04 commented Feb 21, 2019

Haha, yeah i did it just for fun really. I didn’t even know there was a language called “Scala” :P. I’m not expecting this to be merged or anything but thanks for the tip!

Edit: The proof of concept thing was also just a joke since i realized there weren’t any planets and stuff

Edit2: I answered without reading and didn’t realize it wasn’t a tip but just putting it under a Sun class :

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants