asp.net mvc listのmodelの表示の際のエラー

Question

listのmodelの表示について。商品名を取得して、要素に設定したい。
id=P1からP20までの要素にそれぞれ名称を入れたい。
現状1個しか入れることができていない。こんな感じ↓

Index.cshtml

$('#P1').attr('data-title', '@Model.GoodsMeisho.First().goodsmeisho');

そこでとりあえず名称を出そうと思い、下記のようにscriptを書いてみた。
ビルドは通り、デバッグ実行すると「重大な JavaScript エラーが検出されました\n\nSCRIPT1002: 構文エラーです。」と表示される。動的scriptが表示され、名称が取得できていることは確認できるが、エラーが出てしまう。
idと名称をそれぞれforeachで回して設定したいのです。
お力をお貸しねがいます。

View

 @foreach(var item in Model.GoodsMeisho)
            {
                <p>maisho = @item.goodsmeisho</p>
            }

//<summary>
        ///グッズ名称取得
        ///</summary>
        private IEnumerable<GoodsModel.GoodsMeishoViewModel> GetGoodsName(NpgsqlConnection dataConnector)
        {
            List<GoodsModel.GoodsMeishoViewModel> GoodsMeishos = new List<GoodsModel.GoodsMeishoViewModel>();


       //テスト用の値
            string Type_cd = "A";
           

            string sql = @"select box_tsuban, goods_tsuban, goods_meisho"
+ " from master.グッズ名称マスタ where Room_cd = :RoomCd and Type_cd = : TypeCd and box_tsuban = :Box_tsuban and goods_tsuban = :Goods_tsuban";

        NpgsqlCommand cmd = new NpgsqlCommand(sql, dataConnector);
            cmd.Parameters.Add(new NpgsqlParameter("kikishitsuCd", NpgsqlDbType.Text));

            cmd.Parameters.Add(new NpgsqlParameter("TypeCd", NpgsqlDbType.Text));
            cmd.Parameters.Add(new NpgsqlParameter("Box_tsuban", NpgsqlDbType.Integer));
            cmd.Parameters.Add(new NpgsqlParameter("Goods_tsuban", NpgsqlDbType.Integer));
            cmd.Prepare();
            cmd.Parameters["RoomCd"].Value = Session["RoomCode"].ToString();
            cmd.Parameters["TypeCd"].Value = Type_cd;
            cmd.Parameters["Box_tsuban"].Value = goods_tsuban;
            cmd.Parameters["Goods_tsuban"].Value = goods_tsuban;

            using (NpgsqlDataReader dr = cmd.ExecuteReader())
            {
               while (dr.Read())
                {
                    GoodsMeishos.Add(new GoodsModel.GoodsMeishoViewModel()
                    {
                        boxtsuban = (int)dr["box_tsuban"],
                        goodstsuban = (int)dr["goods_tsuban"],
                        goodsmeisho = dr["goods_meisho"].ToString()

                    });
                }
                return GoodsMeishos;
            }

model

public class GoodsModel : 〇〇ModelBase
    {
 ///<summary>
        ///グッズ名称をセット
        ///</summary>
        public IEnumerable<GoodsMeishoViewModel> GoodsMeisho { get; set; }

        public class GoodsMeishoViewModel
        {
            //box通番
            public int boxtsuban { get; set; }

            //グッズ通番
            public int goodstsuban { get; set; }

            //グッズ名称
            public string goodsmeisho { get; set; }
        }

 public ActionResult Index()
        {
            var model = this.CreateModel<GoodsModel>();
            // セッションよりNpgsqlConnectionを取得
            NpgsqlConnection dataConnector = (NpgsqlConnection)Session["dataConnector"];

            // DBオープン
            dataConnector.Open();


            model.GoodsMeisho = GetGoodsName( dataConnector);

            dataConnector.Close();

            return View(model);

        }

Comments

[退会済みユーザー]

質問する際は開発環境を書くようにしてください。OS, .NET, MVC, Visual Studio のバージョン、テンプレートは何を使ったかなどです。前のスレッドで書いたのかもしれませんが、ここしか見てない人も多々いるはず。

[退会済みユーザー]

定義不明の変数、メソッドを含めたコードは書かないようにしてください。前にもお願いしたはずです。前のスレッドで書いたのかもしれませんが、ここだけ見れば分かるようにしてください。

[退会済みユーザー]

もう一つお願いです。あなたのデーターベースは第三者には誰にもわからないということを認識してください。GetGoodsName メソッドで何が取得できるかなんてことは誰にもわかりません。そのメソッドが返すオブジェクトのサンプルを書いてく、それをベースに話をしてください。

[sanezane]

すみません。以後気をつけます。

Answer 1

下記、Viewの記述（htmlの生成）で、pタグに連番のP1から始まるidをふっています。（P1～Pnになりますが・・・）

@foreach (var item in Model.GoodsMeisho.Select((val, index) => new { Val = val, Idx = index }))
{
    <p id=@string.Format("P{0}", item.Idx + 1)>maisho = @item.Val.goodsmeisho</p>
}

Comments

[sanezane]

ありがとうございます。期待する動作を実現できました。