timoheijne/ma-expanding-space

Expanding Space Project

This is a game made for a school assignment called "Expanding Space" we made this game in Unity. Also ignore the contributions page. We made this project in unity using unity collaborate so all our commits are not shown here.

The controls of the game are:

• WASD : Walking
• Arrow Keys : Shooting
• Mouse : Menu Navigation

---

Made by "group 6":

• Developers:
  • [GD1A] Timo Heijne
  • [GD1A] Wessel Vink
• Artists
  • [GA1A] Alan Reulen Solano
  • [GA1A] Lois Dobbelaar

---

Downloads & Links

The Game [Windows Build | Mac Build | Linux Build | Web Build]

Trello

Technisch Onderzoek - GDocs | Pipeline - LucidChart

ma-expanding-space's Issues

Copypasta

Je hebt feedback gekregen van JelmerDeV
URL: https://github.com/lghenk/MA-Expanding-Space/blob/master/Assets/Scripts/Map Generator

Feedback: Als je code direct kopieert, vermeld dan de bron bovenaan je script. Natuurlijk kun je niet alles uit je hoofd maken maar probeer wel je eigen draai er aan te geven.

Duidelijkheid

Je hebt feedback gekregen van JelmerDeV
URL: https://github.com/lghenk/MA-Expanding-Space/blob/master/Assets/Scripts/Gameplay/PauseGame.cs

Feedback: Vermeld in het menu of ergens in het begin van het spel de controls zodat de speler weet hoe hij bijvoorbeeld het spel pauzeert.

Mappen structuur

Je hebt feedback gekregen van JelmerDeV
op:

    get { return m_NavMesh; }
        set { m_NavMesh = value; }
    }

    [SerializeField]
    bool m_FollowTransform;
    public bool followTransform {
        get { return m_FollowTransform; }
        set { SetFollowTransform(value); }
    }

    NavMeshDataInstance m_Instance;

    // Position Tracking
    static readonly List<NavMeshPrefabInstance> s_TrackedInstances = new List<NavMeshPrefabInstance>();
    public static List<NavMeshPrefabInstance> trackedInstances {get {return s_TrackedInstances; }}
    Vector3 m_Position;
    Quaternion m_Rotation;

    void OnEnable() {
        AddInstance();

        if (m_Instance.valid && m_FollowTransform)
            AddTracking();
    }

    void OnDisable() {
        m_Instance.Remove();
        RemoveTracking();
    }

    public void UpdateInstance() {
        m_Instance.Remove();
        AddInstance();
    }

    void AddInstance() {
#if UNITY_EDITOR 

URL: https://github.com/lghenk/MA-Expanding-Space/blob/master/Assets

Feedback: Als je mappen aanmaakt voor scripts, art ect. maak hier dan ook gebruik van en zet niet losse bestanden buiten de mappen.

Public variables

Je hebt feedback gekregen van JelmerDeV
URL: https://github.com/lghenk/MA-Expanding-Space/blob/master/Assets/Scripts

Feedback: Het is handiger om zo min mogelijk 'public' variables te gebruiken, maar om gebruik te maken van private variables.

Onnodige bestanden

Je hebt feedback gekregen van JelmerDeV
URL: https://github.com/lghenk/MA-Expanding-Space/blob/master/

Feedback: Als je klaar bent met het project, verwijder dan alle ongebruikte(development) scenes en ongebruikte art zodat je meer overzicht hebt in wat waar voor gebruikt wordt. En het ziet er wat professioneler uit.

Meer overzicht in je code

Je hebt feedback gekregen van JelmerDeV
op:

GameObject.FindGameObjectWithTag("Player").GetComponent<PlayerPickup>();
        animContro 

URL: https://github.com/lghenk/MA-Expanding-Space/blob/master/Assets/Scripts/Gameplay/ChestPickup.cs

Feedback: Maak gebruik van comments tussen je code om sneller te kunnen navigeren door je code, ook voor andere developers die misschien met je scripts verder werken. Dit geldt ook voor de andere scripts natuurlijk.