feature-duder #9

Merged
zephyr merged 6 commits from feature-duder into main 2023-08-04 18:37:13 -05:00
2 changed files with 27 additions and 0 deletions
Showing only changes of commit 5920e63d0c - Show all commits

16
Assets/Scripts/Heroism.cs Normal file
View File

@ -0,0 +1,16 @@
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
/// <summary>
/// Component that makes a duder a duder.
/// Manages class, skills, level, attrs, and AI.
/// </summary>
public class Heroism : MonoBehaviour
zephyr marked this conversation as resolved Outdated

Could we name this like HeroUnit or something

HeroUnit

EnemyUnit

Could we name this like `HeroUnit` or something `HeroUnit` `EnemyUnit`
{
[SerializeField] private string Name;
[SerializeField] private HeroClassSO Class;
[SerializeField] private long Level;
[SerializeField] private long XP;
zephyr marked this conversation as resolved Outdated

dont change this now, but it would be nice in the future if we could encapsulate the concepts of level and xp in a single class. It might contain a XP -> level curve and then we can jsut .addXp and it will figure everything out. Would be useful if we have anywhere else that needs the same concepts

dont change this now, but it would be nice in the future if we could encapsulate the concepts of level and xp in a single class. It might contain a XP -> level curve and then we can jsut .addXp and it will figure everything out. Would be useful if we have anywhere else that needs the same concepts
[SerializeField] private HeroAttributesSO Attrs;
}

11
Assets/Scripts/Heroism.cs.meta generated Normal file
View File

@ -0,0 +1,11 @@
fileFormatVersion: 2
guid: 583304b24f3e6394f9e4d49747e22193
MonoImporter:
externalObjects: {}
serializedVersion: 2
defaultReferences: []
executionOrder: 0
icon: {instanceID: 0}
userData:
assetBundleName:
assetBundleVariant: