Оптимизация кода C # в контроллере MVC

Question

Я делаю несколько различных контроллеров, один из которых относится к каждой хранимой процедуре в базе данных. Они используются только для чтения данных и делают их доступными в формате JSON для JavaScript.

Мой код до сих пор выглядит следующим образом, и мне интересно, упустил ли я какие-либо возможности для повторного использования кода, может быть, сделать несколько классов помощи. У меня слишком мало опыта работы с ООП, поэтому любая помощь и предложения здесь будут очень благодарны.

Вот мой обобщенный код (проверено и работает);

using System;
using System.Configuration;
using System.Web.Mvc;
using System.Data;
using System.Text;
using System.Data.SqlClient;
using Prototype.Models;

namespace Prototype.Controllers
{
    public class NameOfStoredProcedureController : Controller
    {

        char[] lastComma = { ',' };

        String oldChar = "\"";
        String newChar = """;

        StringBuilder json = new StringBuilder();

        private String strCon = ConfigurationManager.ConnectionStrings["SomeConnectionString"].ConnectionString;
        private SqlConnection con;

        public StoredProcedureController()
        {
            con = new SqlConnection(strCon);
        }

        public string do_NameOfStoredProcedure(int parameter)
        {
            con.Open();

            using (SqlCommand cmd = new SqlCommand("NameOfStoredProcedure", con))
            {
                cmd.CommandType = CommandType.StoredProcedure;
                cmd.Parameters.AddWithValue("@parameter", parameter);


                using (SqlDataReader reader = cmd.ExecuteReader())
                {
                    while (reader.Read())
                    {
                        json.AppendFormat("[{0},\"{1}\"],", reader["column1"], reader["column2"]);
                    }
                }
                con.Close();
            }

            if (json.Length.ToString().Equals("0"))
            {
                return "[]";
            }

            else
            {
                return "[" + json.ToString().TrimEnd(lastComma) + "]";
            }
        }


        //http://host.com/NameOfStoredProcedure?parameter=value
        public ActionResult Index(int parameter)
        {
            return new ContentResult
            {
                ContentType = "application/json",
                Content = do_NameOfStoredProcedure(parameter)
            };
        }
    }
}

Answer 1

Я, вероятно, не получал бы прямой доступ к базе данных с контроллера, а скорее абстрагировал бы этот доступ.Не совсем оптимизация производительности, но улучшение дизайна.Поэтому начните с определения модели, которая будет содержать результат хранимой процедуры:

public class MyModel
{
    public string Column1 { get; set; }
    public string Column2 { get; set; }
}

Затем определите интерфейс хранилища, который будет содержать различные операции в этой модели:

public interface IRepository
{
    IEnumerable<MyModel> GetModel(int id);
}

Далеереализовать хранилище:

public class RepositorySql : IRepository
{
    public IEnumerable<MyModel> GetModel(int id)
    {
        using (var conn = new SqlConnection(ConfigurationManager.ConnectionStrings["SomeConnectionString"].ConnectionString))
        using (var cmd = conn.CreateCommand())
        {
            conn.Open();
            cmd.CommandType = CommandType.StoredProcedure;
            cmd.CommandText = "NameOfStoredProcedure";
            cmd.Parameters.AddWithValue("@parameter", id);
            using (SqlDataReader reader = cmd.ExecuteReader())
            {
                while (reader.Read())
                {
                    yield return new MyModel
                    {
                        Column1 = reader["column1"].ToString(),
                        Column2 = reader["column2"].ToString()
                    };
                }
            }
        }
    }
}

Наконец, ваш контроллер будет использовать хранилище:

public class NameOfStoredProcedureController : Controller
{
    private readonly IRepository _repository;
    public NameOfStoredProcedureController(IRepository repository)
    {
        _repository = repository;
    }

    // Warning don't add this constructor. Use a DI framework instead.
    // This kind of constructors are called Poor Man DI (see http://www.lostechies.com/blogs/jimmy_bogard/archive/2009/07/03/how-not-to-do-dependency-injection-in-nerddinner.aspx)
    // for more info on why this is bad.
    public NameOfStoredProcedureController() : this(new RepositorySql())
    { }

    public ActionResult Index(int parameter)
    {
        var model = _repository.GetModel(parameter);
        // Use directly Json, no need to do the serialization manually
        return Json(model);
    }
}

Answer 2

Довольно часто я делаю вещи вручную, но вы уже взглянули на JsonResult ( Пример )?

А также JavaScriptSerializer ?

А также JSON.Net ?