動的に生成したボタンを横に並べ方と関数呼び方

Question

現状

リソースから取り出した３つのボタンを画面に生成するが全てのオブジェクトが重なってしまう
ボタンを押しても関数が設定されていないので何も起きない

実現したいこと

取り出したボタンを順に横に並べたい
動的に生成したボタンの関数を指定したい(インベントリにあるたくさんのボタンから選択したボタンに対応するボタンの色を明るくしてボタンを有効にして押せるようにする)

コード
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class Player : MonoBehaviour
{
    //
    public float moveSpeed;
    private Vector3 velocity;
    public Joystick joystick;
    public Animator animator;
    public Image Aboid;
    public float avoidtime = 5;
    public float invincible = 2;
    public float deltainvincible = 0;

    //
    public float span = 2;
    public float delta = 0;
    private GameObject Magic;
    private GameObject nearObj;
    public List<string> magic = new List<string>();
    public int i;
    public List<string> unLearnedMagic = new List<string>();
    int count = 3;
    public GameObject parentChoice;

    //
    int maxHp = 100;
    int currentHp;
    int maxMana = 100;
    int currentMana;
    public Slider hpslider;
    public Slider manaslider;
    private GameObject damageUI;


    //
    public GameObject Controller;
    public GameObject Menu;

/////////////////////////////////////////////////////////////
    public void Awake()
    {
        unLearnedMagic.Add("MagicButton/FireBallButton");
        unLearnedMagic.Add("MagicButton/IceBallButton");
        unLearnedMagic.Add("MagicButton/EathBallButton");
        while (count-- > 0)
        {

            int index = Random.Range(0, unLearnedMagic.Count);

            string ransu = unLearnedMagic[index];
            GameObject selectmagic= (GameObject)Resources.Load(ransu);//
            GameObject childmagic=Instantiate(selectmagic, transform.position, Quaternion.identity);
            childmagic.transform.parent = parentChoice.transform;
            Debug.Log(ransu);
        }
    }
////////////////////////////////////////////////////

// クリックした魔法名を引数で渡す
    public void OnRegisterMagic(string magicName)
    {
        // 既に登録済みの魔法を選択した場合はリストから削除する
        foreach (var v in magic)
        {
            if (v == magicName)
            {
                magic.Remove(v);
                return;
            }
        }

        // セット数は三つまで
        if (magic.Count < 3)
        {
            magic.Add(magicName);
        }
    }
}

Answer 1

取り出したボタンを順に横に並べたい

親にしているparentChoiceに予めコンポーネントとしてHorizontal Layout Groupを設定しておくと子にするだけで自動で整列されます。
将来的に2列3列と増えていきそうならばGrid Layout Groupの方が良いです。

動的に生成したボタンの関数を指定したい

onClick.AddListenerを使います。
下記の様にInstantiateで生成した後におこなって下さい。

    GameObject childmagic = Instantiate(selectmagic, transform.position, Quaternion.identity);
    childmagic.GetComponent<Button>().onClick.AddListener(() => OnRegisterMagic(ransu));

Comments

[退会済みユーザー]

すみません
書き方が分からなくてうまくいってないです

public void Awake()
{
unLearnedMagic.Add("MagicButton/FireBallButton");
unLearnedMagic.Add("MagicButton/IceBallButton");
unLearnedMagic.Add("MagicButton/EathBallButton");
while (count-- > 0)
{

int index = Random.Range(0, unLearnedMagic.Count);
string ransu = unLearnedMagic[index];
GameObject selectmagic = (GameObject)Resources.Load(ransu);//
GameObject childmagic = Instantiate(selectmagic, transform.position, Quaternion.identity);
childmagic.transform.parent = parentChoice.transform;
////////////////////////// 55行目　///////////////////////////////
childmagic.GetComponent<Button>().onClick.AddListener(() => OnRegisterMagic(ransu));
/////////////////////////////////////////////////////////////

Debug.Log(ransu);
}
}
こう書いたんですがうまくいかないです。
ボタンを押しても何の反応もないです
NullReferenceException: Object reference not set to an instance of an object
Player.Awake () (at Assets/Scripts/Player/Player.cs:55)

[退会済みユーザー]

すみません
反応するようにはなりました！
二つ以下の場合どうすればいいんでしょうか

後ボタン一つ選んで押した時にそのキャンバスを閉じたいんですが新しくキャンバスを閉じる関数を作ってonClick.AddListener(()の様にしないといけないんでしょうか関数が多くなりすぎてもいいのかも知れないですが悪いと思っているので関数作る以外に方法があるなら教えて頂きたいです