常連

回答者

ベストアンサー

拍手喝采

自己紹介

最初に

閲覧していただきありがとうございます。
C#及びUnity初心者です。
現在２Dゲームを製作しています。
まだ経験が浅くとてもレベルの低い質問で申し訳ございません。

### 前提・実現したいこと

HP,攻撃力,などのステータスやUIが全く違うenemyを作り順番に出現させたいと思っています。

HP,攻撃力,などのステータスやUIが全く違うenemyを作り順番に出現させたいと思っています。
有名どころだとパズドラ、モンストのように１ステージにつき１体のenemyを出現させ、enemyを撃破すると次のステージに進み、また新しいEnemyが１体現れる...

こうしたシステムを実装する際に一つずつGameObjectとScriptを用意するのではなく、一つのScriptで全く違うEnemyを管理して出現させる良い方法はございますでしょうか。

またこのような場合の良い参考資料などを知っておられましたら教えていただけると幸いです。

### 試したこと

ListにEnemyを一括管理しようと思い10種類のEnemyのステータスを入れたLIstを作成しましたがUnity側のGameObjectとの連動の仕方や、UIの管理、連動などがうまくいきませんでした
```C#
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

class Enemeis
{
    public int EnemyHP;
    public string EnemyName;
    public int Enemyatp;
}

public class Enemy : MonoBehaviour
{
    //このカウントでステージに対応したenemyを出そうと思いました。
    public int EnemyCount = 0;

    void Start()
    {
        Enemeis Series1;
        Enemeis Series2;
        Enemeis Series3;
        Enemeis Series4;
        Enemeis Series5;
        Series1 = new Enemeis();
        Series1.EnemyName = "Normal";
        Series1.EnemyHP = 500;
        Series1.Enemyatp = 100;
        Series2 = new Enemeis();
        Series2.EnemyName = "Lilstrong";
        Series2.EnemyHP = 1000;
        Series2.Enemyatp = 199;
        Series3 = new Enemeis();
        Series3.EnemyName = "Strong";
        Series3.EnemyHP = 1000;
        Series3.Enemyatp = 250;
        Series4 = new Enemeis();
        Series4.EnemyName = "Heiger";
        Series4.EnemyHP = 3000;
        Series4.Enemyatp = 200;
        Series5 = new Enemeis();
        Series5.EnemyName = "Boss";
        Series5.EnemyHP = 777;
        Series5.Enemyatp = 7777;


        Enemeis[] i = new Enemeis[4];
        i[0] = Series1;
        i[1] = Series2;
        i[2] = Series3;
        i[3] = Series4;
        i[4] = Series5;
    }
```

多種のステータスを持ったenemyを順番に出現させたい

###困っていること
WindowsFormアプリケーションにてエクセルファイルを選択して、
指定したファイルのデータをDataTableにセットする機能を実装しています。
メモリ不足のエラーが発生するのですが、原因がわからず困っています。

###発生している問題・エラーメッセージ
ファイル選択ボタンをクリックしてエクセルファイルのデータを取得しています。
同じファイルを選択して複数回実行したときに、
1～6回目は処理が成功するのですが、7回目でエラーが発生します。
※ファイルのサイズは522KB(65537行,256列)です。
なお、上記の問題はVisual Studioのビルドのプラットフォームターゲットをx86とし、
64bitのマシン環境で実行すると発生します。
※事情によりプラットフォームターゲットをx86にしています。
プラットフォームターゲットをAny CPU にすると発生しないことまではわかりました。

【エラー発生箇所】
DaoExcelFileクラスのReadFileメソッド
```
【エラーメッセージ】
プログラムを続行するための十分なメモリがありませんでした
```
###該当のソースコード
```ここに言語を入力
    public partial class Form1 : Form
    {
        private UserInfoFile userFile;

        public Form1()
        {
            InitializeComponent();
        }
        //ファイル選択ボタンクリック
        private void button1_Click(object sender, EventArgs e)
        {
            var selectFileFormat = "Excel";
            //ユーザーファイルを選択してデータを読み込み
            FileReadWriteController.ReadUserInfoFile(selectFileFormat, out userFile);
        }
        //ファイルからデータを読み込み、書込みをコントロールするクラス
        public static class FileReadWriteController
        {
            ///ユーザー情報ファイルを選択して読み込み、
            ///読み込んだデータをユーザー情報ファイルオブジェクトのDataTabaleにセットする
            public static bool ReadUserInfoFile(
                string selectFileFormat, out UserInfoFile userInfoFile)
            {
                OpenFileDialog ofd = new OpenFileDialog();
                if (ofd.ShowDialog() == DialogResult.OK)
                {
                    userInfoFile = new UserInfoFileExcel();
                    userInfoFile.FilePath = ofd.FileName;
                    userInfoFile.ReadData();
                    return true;
                }
                else
                {
                    userInfoFile = null;
                    return false;
                }
            }
        }
        //ファイルの抽象クラス
        abstract public class MyFile
        {
            public string FilePath;
            public DataTable FileData;
            public MyFile()
            {
                FileData = new DataTable();
            }
            public abstract void ReadData();
        }
        //ユーザー情報情報ファイルの抽象クラス
        abstract public class UserInfoFile : MyFile
        {
            //省略
        }
        //Excel形式のユーザー情報ファイル
        public class UserInfoFileExcel : UserInfoFile
        {
            public override void ReadData()
            {
                using (var excelApp = new DaoExcelFile())
                {
                    excelApp.ReadFile(FilePath, FileData, "Info");
                    excelApp.Quit();
                }
            }
        }
        //Excelクラス
        public class DaoExcelFile : IDisposable
        {
            private object xlsApplication = null;
            private object xlsBooks = null;
            protected object XlsApplication
            {
                get
                {
                    // 存在しない場合は作成する
                    if (xlsApplication == null)
                    {
                        Type classType = Type.GetTypeFromProgID("Excel.Application");
                        xlsApplication = Activator.CreateInstance(classType);
                    }
                    return xlsApplication;
                }
            }
            
            // Workbooksオブジェクト
            protected object Workbooks
            {
                get
                {
                    if (xlsBooks == null)
                    {
                        xlsBooks = XlsApplication.GetType().InvokeMember("Workbooks", BindingFlags.GetProperty, null, XlsApplication, null);
                    }
                    return xlsBooks;
                }
            }
            //Dispose
            public void Dispose()
            {
                ReleaseComObject(xlsBooks);
                ReleaseComObject(xlsApplication);
            }

            // COMオブジェクトのリリース
            public void ReleaseComObject(object target)
            {
                try
                {
                    if ((target != null))
                    {
                        System.Runtime.InteropServices.Marshal.ReleaseComObject(target);
                    }
                }
                finally
                {
                    target = null;
                }
            }

            // Excel終了
            public void Quit()
            {
                XlsApplication.GetType().InvokeMember("Quit", BindingFlags.InvokeMethod, null, XlsApplication, null);
            }

            // 対象ファイルの対象シートのデータをデータテーブルにセットする
            public void ReadFile(string filePath, DataTable fileData, string sheetName)
            {
                object book = null;
                object sheets = null;
                object sheet = null;
                object cells = null;
                object cella = null;
                object cellb = null;
                object targetRange = null;
                object[,] obj1 = null;

                try
                {
                    book = GetOpenFile(filePath);
                    sheets = GetSheets(book);
                    sheet = GetSheet(sheets, 1);
                    int lastRow = 65537;
                    int lastColumn = 256;
                    cells = GetCells(sheet);
                    cella = GetCell(cells, 1, 1);
                    cellb = GetCell(cells, lastRow, lastColumn);
                    targetRange = GetRange(sheet, cella, cellb);

                    //*********ここでエラーが発生する
                    //*********エラー内容:"プログラムを続行するための十分なメモリがありませんでした"
                    //オブジェクトの2次元配列へセット
                    //以降この配列にアクセスすることで処理を高速化
                    obj1 = (System.Object[,])targetRange.GetType().InvokeMember("Value2", BindingFlags.GetProperty, null, targetRange, null);
                }
                finally
                {
                    //オブジェクトの解放
                    ReleaseComObject(targetRange);
                    ReleaseComObject(cellb);
                    ReleaseComObject(cella);
                    ReleaseComObject(cells);
                    ReleaseComObject(sheet);
                    ReleaseComObject(sheets);
                    ReleaseComObject(book);
                }
            }

            // Excelファイルを開いてオブジェクトを返す
            public object GetOpenFile(string xlsFilePath)
            {
                object[] parameters = new object[15];
                parameters[0] = xlsFilePath;
                parameters[1] = Type.Missing;
                parameters[2] = Type.Missing;
                parameters[3] = Type.Missing;
                parameters[4] = Type.Missing;
                parameters[5] = Type.Missing;
                parameters[6] = Type.Missing;
                parameters[7] = Type.Missing;
                parameters[8] = Type.Missing;
                parameters[9] = Type.Missing;
                parameters[10] = Type.Missing;
                parameters[11] = Type.Missing;
                parameters[12] = Type.Missing;
                parameters[13] = Type.Missing;
                parameters[14] = Type.Missing;
                object openFile = new object();
                openFile = Workbooks.GetType().InvokeMember("Open", BindingFlags.InvokeMethod, null, Workbooks, parameters);
                return openFile;

            }

            // Sheetsを取得
            public object GetSheets(object book)
            {
                return book.GetType().InvokeMember("Worksheets", BindingFlags.GetProperty, null, book, null);
            }
            // シートインデックスからSheetを取得
            public object GetSheet(object sheets, int index)
            {
                object[] parameters = new object[1];
                parameters[0] = index;
                return sheets.GetType().InvokeMember("Item", BindingFlags.GetProperty, null, sheets, parameters);
            }
            // Cellsを取得
            public object GetCells(object sheet)
            {
                return sheet.GetType().InvokeMember("Cells", BindingFlags.GetProperty, null, sheet, null);
            }
            // Cellを取得
            public object GetCell(object cells, int row, int column)
            {
                object[] parameters = new Object[2];
                parameters[0] = row;
                parameters[1] = column;
                return cells.GetType().InvokeMember("Item", BindingFlags.GetProperty, null, cells, parameters);
            }
            // Rangeを取得
            public object GetRange(object sheet, object cell1, object cell2)
            {
                object[] parameters = new Object[2];
                parameters[0] = cell1;
                parameters[1] = cell2;
                return sheet.GetType().InvokeMember("Range", BindingFlags.GetProperty, null, sheet, parameters);
            }
        }
    }
```

###試したこと
COMオブジェクトが解放されていないのかとも思いましたが、
ファイル選択ボタンのクリック時の処理が終了すれば、
タスクマネージャーにEXCEL.EXEプロセスは残っていない状態にはなっています。

###補足情報(言語/FW/ツール等のバージョンなど)
C#
NET Framework4.0
Microsoft Visual Studio 2015

C#でエクセルファイルのデータ取得時にメモリ不足が発生する

他の人に作ってもらった、C#で作成したソフトがあります。これを保守しているのですが、大きな配列をクラス内で定義しているとき、時折、配列内の中が中途半端になることがあります。例えば、30万行の文字列を記憶させたのが、あるとき、1万行しかない状態になることがあります。

今まで、この現象をGCの余計な作業によるものと思っていましたが、最近怪しいかなと思うようになっています。

そこで、基本に立ち返って、staticなクラス内に定義した動的な配列はGCの対象でないかどうか教えていただけませんでしょうか？通常、static宣言した変数はGCの対象から外れるので、データが勝手にクリアーされることがないのは確かと思いますが、staticなクラス内の動的な変数に対してまで適用されるのかどうかについては結論がでていません。
論理的な説明で結構ですので回答よろしくお願いします。

static内の動的な変数はGCの対象外でしょうか？

管理者と普通のユーザーに別の権限を与えるにあたって、

実現方法１：
User データベースと Adminデータベースを作る

実現方法２：
User モデルに、is_adminカラムを作り、adminの場合はtrue にする。

などがあると思いますが、
大規模アプリケーションにおける２のデメリットを伺いたく存じます。
自分はずっと、２のやり方はあまり好ましいものではないのだと思い込んで来ましたが、その具体的なデメリットが分からなくなって来ました。

たとえば、多くの人が使っているwordpressでは２の方法が取られていると思います。
実装的には２の方が簡単です。
どちらの方が広く使われている手法なのか？大規模アプリケーション（１万人以上が登録利用する想定）ではどうか？など伺いたく存じます。