第一项的 SQL 数组值被第二项覆盖

Question

我正在尝试构建一个系统，其中一个字母是从 2 个表中自动生成的。一张表格包含了这封信的所有内容，而另一张表格则列出了接收特定信件的人员。可能有多个人收到同一个字母，所以我尝试进行连接和循环以生成每个字母。

我的问题是我遇到从数组读取时返回的第一个值是数据库中的第二个条目。 IE 我应该有 (Rob, Jess, Alex) 输出，但我得到的是 (Jess, Jess, Alex)。

我不确定我的问题出在哪里。

SQL = "SELECT * FROM S.dbo.letters l INNER JOIN 
       S.dbo.ds d ON l.sID= d.sID
       WHERE l.id=@primaryID";

 SqlConnection dbconn = new SqlConnection(connStr);
 SqlCommand cmd = new SqlCommand(SQL, dbconn);

 dbconn.Open();
 cmd.Parameters.AddWithValue("@primaryID", 1);//Session["primIDedit"]

 SqlDataReader myReader = cmd.ExecuteReader();
 ArrayList letterInfo = new ArrayList();
 ArrayList dFname = new ArrayList();
 myReader.Read();

 while (myReader.Read())
 {

     dFname.Add(myReader["dfname"]);
     foreach (string i in dFname)
     {
     Label txtLabel1 = new Label(); //Generating a label
     txtLabel1.Text = i.ToString();
     ph1.Controls.Add(txtLabel1); //writing to a placeholder on the aspx page
     //for now I'm just wanting to return all values.
     }

     sName.Text = myReader[1].ToString() 
                  + " " + myReader[3].ToString() + " " + 
                  myReader[2].ToString();
     paragraph1.Text = myReader[9].ToString();
     paragraph2.Text = myReader[10].ToString();
     paragraph3.Text = myReader[11].ToString();
     signoff.Text = myReader[12].ToString();
}

Answer 1

我在这里做了假设，但我会先创建两个类...

public class LetterInfo
{
   public List<Recipient> Recipients {get; set;}
   public string Paragraph1 {get; set;}
   public string Paragraph2 {get; set;}
   public string Paragraph3 {get; set;}
   etc

   public LetterInfo(){
    Recipients = new List<Recipient>();
   }
}

public class Recipient
{
   public string FirstName {get; set;}
   public string LastName {get; set;}
   public string MiddleName {get; set;}
}

然后，将您的数据库代码更改为如下所示...

using (Sqldbconn dbconn = new Sqldbconn(connStr))
    {
        string sql = "SELECT l.Field, d.field FROM S.dbo.letters l 
                INNER JOIN S.dbo.ds d ON l.sID= d.sID WHERE l.id=@primaryID";

        SqlCommand cmd = new SqlCommand(sql.ToString(), dbconn);
        cmd.Parameters.AddWithValue("@primaryID", 1);//Session["primIDedit"]

        dbconn.Open();

        LetterInfo letterInfo = new LetterInfo();
        var counter = 0;

        using (var rdr = command.ExecuteReader())
            {
                while (rdr.Read())
                {
                 var recipient = new Recipient();
                 recipient.FirstName = myReader["dfname"].ToString()
                 recipient.LastName = myReader["..."].ToString()
                 ....
                 letterInfo.Recipients.Add(recipient);

                 //only need to populate the letterinfo once.
                 if (counter == 0){
                     //use field name here, not the ordinal number
                     letterInfo.Paragraph1 = myReader[9].ToString();
                     letterInfo.Paragraph2 = myReader[10].ToString();
                     letterInfo.Paragraph3 = myReader[11].ToString();
                     ....
                  }

                     counter++;
                  }                   
            }
    }

然后您可以使用letterInfo 对象来填充您的表单。

你也可以把它移到某种数据帮助类中，然后从你的代码隐藏中调用 if...

public LetterInfo GetLetterInfo(int primaryId){
  //the code above in here
}

您的代码可能看起来像...

LetterInfo letter = dbHelper.GetLetterInfo(1);

p.s.我是用 Notepad++ 写的，所以我预计会出现编译错误 :)

Answer 2

在 while 循环之前删除这一行。

myReader.Read();

并在调试时使用断点检查 myreader 对象的下标值

 sName.Text = myReader[1].ToString() 
              + " " + myReader[3].ToString() + " " + 
              myReader[2].ToString();
 paragraph1.Text = myReader[9].ToString();
 paragraph2.Text = myReader[10].ToString();
 paragraph3.Text = myReader[11].ToString();
 signoff.Text = myReader[12].ToString();