c＃System.ArgumentOutOfRangeException

Question

我正在制作发送带有一些数据的电子邮件的程序。 我知道System.ArgumentOutOfRangeException异常意味着数组/列表中的数字不存在，但是我不知道我写错了什么。

这是一个代码：

public void SendMail()
{
    StreamReader sr = new StreamReader(path1);
    var lineCount = File.ReadLines(path1).Count();
    List<string> data = new List<string>();
    for (int i = 0; i < lineCount; i++)
    {
       data[i] = sr.ReadLine(); //Error Comes here.
    }
    string finaldata = data[0] + "/n" + data[1] + "/n" + data[2] + "/n" + data[3] + "/n" + data[4] + "/n" + data[5] + "/n" +       
    data[6] + "/n" + data[7] + "/n" + data[8] + "/n" + data[9] + "/n" + data[10];
    var fromAddress = new MailAddress("tutorialvideohd@gmail.com", "From Name");
    var toAddress = new MailAddress("tutorialvideohd@example.com", "To Name");
    const string fromPassword = "*****";
    string subject = "Some Users Data.";
    string body = finaldata;

    var smtp = new SmtpClient
    {
       Host = "smtp.gmail.com",
       Port = 587,
       EnableSsl = true,
       DeliveryMethod = SmtpDeliveryMethod.Network,
       UseDefaultCredentials = false,
       Credentials = new NetworkCredential(fromAddress.Address, fromPassword)
    };

    using (var message = new MailMessage(fromAddress, toAddress)
    {
        Subject = subject,
        Body = body
    })
    {
        smtp.Send(message);
    }
}

Answer 1

当您这样做时，它只会创建一个空列表：

List<string> data = new List<string>();

因此，如果您尝试使用索引分配值，则尝试将值分配给不存在的对象。

相反，您应该如下所示Add() ：

data.Add(sr.ReadLine());

---

编辑：

另外，与提出的问题无关，但我看不到您关闭了已打开的StreamReader ，这始终是一个坏习惯。 我建议改为使用using语句，该语句将为您处理StreamReader的打开和关闭。 另外，获取lineCount是多余的，您可以利用这样的事实来进行以下操作：不需要预先设置列表中的项目数。

List<string> data = new List<string>();
using (StreamReader sr = new StreamReader(path1))
{
    while(!sr.EndOfStream)
        data.Add(sr.ReadLine()); 
}

Answer 2

首先，您有一个空字符串列表，您无法通过访问索引来填充data ，因为这些索引尚不存在。 您必须使用data.Add(sr.ReadLine())创建新索引并在其中添加值。

string finaldata = data[0] + "/n" + data[1] + "/n" + data[2] + "/n" + data[3] + "/n" + data[4] + "/n" + data[5] + "/n" +       
data[6] + "/n" + data[7] + "/n" + data[8] + "/n" + data[9] + "/n" + data[10];

硬编码ID意味着您至少需要11项，如果数量少了，它将中断。 您为什么不这样做呢？

string finalData = String.Join("/n", data);

这样，它使用换行符作为分隔符来连接您的字符串列表，并且列表中是否有更多项都无关紧要。

Answer 3

您正在数组列表中设置一个尚未初始化的项目。 在引发异常的位置使用以下行。

data.Add(sr.ReadLine());

Answer 4

您在做同一件事两次。

此代码块：

StreamReader sr = new StreamReader(path1);
// ... you had some code here, but not relevant to the point...
for (int i = 0; i < lineCount; i++)
{
    data[i] = sr.ReadLine(); //Error Comes here.
}

用一串字符串填充列表data 。

此代码块：

var lineCount = File.ReadLines(path1).ToList();

精确地使用您要填充到data中的字符串列表填充lineCount 。

因此，只需执行以下操作：

StreamReader sr = new StreamReader(path1);
var lineCount = File.ReadLines(path1).Count();
List<string> data = lineCount;

并摆脱以下代码块：

for (int i = 0; i < lineCount; i++)
{
    data[i] = sr.ReadLine(); //Error Comes here.
}

并注意现在如何正确填充data 。您确实也不需要该StreamReader ，但是，嘿，我现在不想重写整个代码。

关键要点：阅读文档，尝试了解每个函数调用的功能，尤其是.NET Framework的功能。