я получил неверную попытку вызвать чтение, когда читатель закрыт при использовании sqldatareader, как решить эту проблему в трехуровневой архитектуре

Это действительно проблема дизайна - вы не можете вернуть «живой» считыватель данных, если вы намереваетесь разрушить соединение, SqlDataReader зависит от него, т.е.

SqlDataReader dr;
using (SqlConnection con = ConnectionManager.GetDatabaseConnection())
{
    SqlCommand cmd = new SqlCommand("getInfo", con);
    cmd.CommandType = CommandType.StoredProcedure;
    cmd.Parameters.Add("@query", SqlDbType.VarChar).Value = sql;
    cmd.Connection = con;
    dr = cmd.ExecuteReader();
} // the SqlConnection is disposed here
return dr; // dr is now invalid

Вдобавок ко всему, вы передаете детали реализации своим слоям BLL/UI, возвращая SqlDataReader. Вы должны читать данные, пока соединение активно, и вместо этого возвращать фактические данные, чтобы все было хорошо и чисто, например.

public class Download
{
     public string Name { get; set; }
     public byte[] Data { get; set; }
}
...
private Download getDownload(string sql)
{
    using (SqlConnection con = ConnectionManager.GetDatabaseConnection())
    using (SqlCommand cmd = new SqlCommand("getInfo", con))
    {
        cmd.CommandType = CommandType.StoredProcedure;
        cmd.Parameters.Add("@query", SqlDbType.VarChar).Value = sql;
        con.Open();
        Using (SqlDataReader dr = cmd.ExecuteReader())
        {
            while (dr.Read())
            {
                return new Download
                {
                    Name = (string)dr["mfile_name"],
                    Data = (byte[])dr["file_data"]
                };
            }
        }
    }
}

Это проблема дизайна с вашим уровнем доступа к данным. Это не отделяет приложение от базы данных, как предполагалось. это просто другой интерфейс к базе данных, чем использование SQL. Вам нужно полностью их разъединить. То есть вам нужно вернуть объекты данных в приложение. Затем эти объекты данных должны быть заполнены в DAL с помощью средства чтения.

Итак, ваш метод будет примерно таким:

private MyFile getDownload(string sql)
{
    SqlDataReader dr;
    using (SqlConnection con = ConnectionManager.GetDatabaseConnection())
    {
        SqlCommand cmd = new SqlCommand("getInfo", con);
        cmd.CommandType = CommandType.StoredProcedure;
        cmd.Parameters.Add("@query", SqlDbType.VarChar).Value = sql;
        cmd.Connection = con;
        dr = cmd.ExecuteReader();


        return new MyFile {
                  file_name = dr["mfile_name],
                  file_data = dr["file_data]
        }
    }
}

Оператор using размещает объект в контексте (в данном случае это SqlConnection) сразу после выполнения последнего оператора в блоке using. Это причина, по которой вы получаете сообщение об ошибке «читатель закрыт ...», когда вы обращаетесь к нему вне функции DAL.

Либо вы должны удалить блок using и обрабатывать открытие/закрытие соединения на более высоком уровне, либо, что еще лучше, вы должны преобразовать SqlDataReader в объект или DataTable внутри метода DAL и вместо этого вернуть его. Пример с DataTable:

private DataTable getDownload(string sql)
{
    SqlDataReader dr;
    DataTable dt;
    using (SqlConnection con = ConnectionManager.GetDatabaseConnection())
    {
        SqlCommand cmd = new SqlCommand("getInfo", con);
        cmd.CommandType = CommandType.StoredProcedure;
        cmd.Parameters.Add("@query", SqlDbType.VarChar).Value = sql;
        cmd.Connection = con;
        dr = cmd.ExecuteReader();
        dt = new DataTable();
        dt.Load(dr);
    }
    return dt;
}

вы должны изменить/добавить метод

public DataTable getDownload(string sql)
{
    using (SqlConnection con = new SqlConnection(yourconstring))
    {
        SqlDataAdapter dap = new SqlDataAdapter(sql,con);
        DataTable dt = new DataTable();
        dap.Fill(dt);
        return dt;
    }
}

затем в методе загрузки нажмите

DataTable dt = MclsAssignment.getDownload(field);
        if (dt.Rows.Count> 0)
        {
            Response.AddHeader("Content-Disposition", "attachment;filename=\"" + (string)dt.Rows[0]["mfile_name"] + "\"");
            Response.BinaryWrite((byte[])dt.Rows[0]["file_data"]);
            Response.End();
        }