staticとstructを一緒に使うとうまくいきません

Question

ゲームデータのセーブ用スクリプトを実装したいと思い。保持したい変数をstructな変数にまとめ各セーブデータから読み込めるような形の実装をしたいと思いまして、以下のようなスクリプトを組みました。

class save(){
public struct S{
int life;
float mp;
}
}


public static class savemanager(){
static save[5];
void set(save s){
s.life=10;
s.mp=100;
}
}

しかし、この形ではエラーが出ます、エラーの対処法を色々調べたのですが
そもそも根本的な実装の仕方が違うのではと思い質問させていただきました

Answer 1

こんにちは。

C#は進歩が早いので私がついて行っていないだけかもしれませんが、下記構文はないのでは？

class save()
{
    クラス定義
}

私の知る限り()が余分と思います。

public static class savemanager(){

も()が余分と思います。

static save[5];

は変数名がないですよ。

void set(save s){

s.life=10;
s.mp=100;
}

saveクラスにlifeやmpは定義していないように見えます。
これらを定義しているのはS構造体では？

Comments

[Panzer_vor]

> Chironianさん
空で書き直してる時に見落としてましたが、
確かにクラス名横にカッコがついているのも文法的に誤りですね・・・^^;

当方の回答にもフィードバックさせていただきます。

Answer 2

C# はやっていないので詳しいことはわかりませんが、

構造体は、

class内じゃなく、classのように書くのでは?

構造体オブジェクトを生成する場合は、

構造体名 オブジェクト名;

とするはずです。

なので、

struct save{
       int life;
        int mp;
}

とあったら、

save s[10];

のようにします。

staticをつけてもいい。

つまり構文間違い。

また、他の方もおっしゃっているように、

名前の付け方の決まり ( 命名規則っていいます。 ) があります。

基本的に構造体はすべて大文字。

struct save{

ではなく

struct SAVE{

です。

クラス名は 大文字から初めて、後は基本的に小文字。
二語以上でなる名前なら、それぞれの頭の文字は大文字、それ以外は小文字。
...

という風に。

Answer 3

先ず1つ目はコーディングのお作法的なお話になります。
変数名と区別する意味でもクラス名は単語単位で大文字を使う方が良いです。
それとインデントをつけるようにするとソースのメリハリがつき可読性が上がるので、
これも普段から行うように心がけると良いでしょう。
(整形機能のあるソースエディタだと勝手にやってくれたりしますが。)

次に2つ目ですがsaveクラスにスコープ指定してませんがこれは意図的にそうしているのでしょうか？
入れ子していないクラスの場合は省略するとスコープはinternalとなります。
これは同一プロジェクトないのモジュールに対しては公開、
別プロジェクトからは非公開という公開範囲となります。
アクセス修飾子はいい加減に覚えても良いことはないので、
もしあまり意識していない場合は、
意識するようにしていきましょう。

ここからが本題ですが、
savemanagerクラスのstaticメンバ変数の書き方がおかしいです。

static save[5];
これだとクラスを配列にしてるように見受けられるけど、
実は変数名も指定されておらず宣言としては完全に誤りです。
更に言うとsaveクラスに入れ子となる形で構造体を宣言しているため、
他のクラスから構造体を利用する際は、save.Sという感じで型を宣言しなくてはなりません。
あえて入れ子としているというよであれば良いですが、
そうでないなら構造体単体で宣言した方がシンプルかもしれません。

これらを踏まえて以下にサンプルを記載します。
空で書いてるのでミスしてるかもですが・・・

// 入れ子が不要なら構造体を単独で定義
// class Save(){
public struct Save{
    // インデントをつけると見やすいかと
    int life;
    float mp;
}
//}

// クラス名は単語単位で大文字に
// public static class SaveManager(){
public static class SaveManager{
    //static save[5];
    static Save[] saves = new Save[5];
    // 蛇足だけど配列よりはListの方が使い勝手は良い。
    // 最初のusingでSystem.Collections.Genericを参照してると使える
    // static List<Save> saves = new List<Save>();

// C#では慣習的にはメソッド・プロパティ名は大文字始まり
    void Set(Save s){
        s.life=10;
        s.mp=100;

        // 引数に値を設定してるけどこれを配列に保存したい？？その場合は以下が処理が必要
        // 要素番号指定がいる場合
        // saves[0] = s;
       
        // そもそも要素番号が大事でなければListでは以下のように書ける
        // ただし下のように書くとのメソッドを呼ぶたびに要素数が増え、List末尾に新しく保存されます
        // saves.Add(s);
    }
}

#追記
Chironianさんのご指摘にある通り、
クラス定義時にはクラス名横の()は不要でした。

当方の回答も誤ったままなので、
ソース訂正をさせていただきます。

ちなみにアスセス修飾子 クラス名(){}の書き方は、
クラスのコンストラクタで使用します。
(※カッコ内には必要であれば引数を指定できます。)