对我的控制器设计没有 100% 的感觉

Question

基本上，我上传一个 Excel 文件并解析信息，然后在视图中显示解析的内容。

using System.Data;
using System.Data.OleDb;
using System.Web;
using System.Web.Mvc;
using QuimizaReportes.Models;
using System.Collections.Generic;
using System;

namespace QuimizaReportes.Controllers
{
    public class UploadController : Controller
    {
        public ActionResult Index()
        {
            return View();
        }

        [HttpPost]
        public ActionResult Index(HttpPostedFileBase excelFile)
        {
            if (excelFile != null)
            {
                //Save the uploaded file to the disc.
                string savedFileName = "~/UploadedExcelDocuments/" + excelFile.FileName;
                excelFile.SaveAs(Server.MapPath(savedFileName));

                //Create a connection string to access the Excel file using the ACE provider.
                //This is for Excel 2007. 2003 uses an older driver.
                var connectionString = string.Format("Provider=Microsoft.ACE.OLEDB.12.0;Data Source={0};Extended Properties=Excel 12.0;", Server.MapPath(savedFileName));

                //Fill the dataset with information from the Hoja1 worksheet.
                var adapter = new OleDbDataAdapter("SELECT * FROM [Hoja1$]", connectionString);
                var ds = new DataSet();
                adapter.Fill(ds, "results");
                DataTable data = ds.Tables["results"];

                var people = new List<Person>();

                for (int i = 0; i < data.Rows.Count - 1; i++)
                {
                    Person newPerson = new Person();
                    newPerson.Id = data.Rows[i].Field<double?>("Id");
                    newPerson.Name = data.Rows[i].Field<string>("Name");
                    newPerson.LastName = data.Rows[i].Field<string>("LastName");
                    newPerson.DateOfBirth = data.Rows[i].Field<DateTime?>("DateOfBirth");

                    people.Add(newPerson);
                }

                return View("UploadComplete", people);
            }

            return RedirectToAction("Error", "Upload");                      
        }

        public ActionResult Error()
        {
            return View();
        }
    }
}

不那么自信这是最好的方法。 MVC3 资深人士对这位有抱负的高级程序员有什么建议吗？ :)

我是否应该调用另一个操作“UploadComplete”并让其调用 UploadComplete 视图，而不是直接从 [POST]Index 操作调用视图？我什么时候知道是使用一种方法还是另一种方法？

Answer 1

我认为 Post-Redirect-Get 模式非常适合这里。例如，您可以使用 TempData 传递“人员”数据。

Answer 2

我可能会减少索引操作中的代码以简单地保存文件，并重定向到另一个负责从文件中读取数据并显示它的操作。

可以使用路由值的第三个参数调用 RedirectToAction，该路由值可以标识要读取的文件。