Old question, please pass by to see

saxonn

public void CreatTable ()
    {
        upfiles = new DataTable ("upfile");
        DataColumn [] keys = new DataColumn [1];
        keys [0] = upfiles.Columns.Add ("filename", typeof (string));
        keys [0] .AllowDBNull = false;
        upfiles.PrimaryKey = keys;
        upfiles.Columns.Add ("filestream", typeof (Stream));
        fname.Text = "";
    }
    /// <summary>
    /// add attachment to list
    /// </ summary>
    private void AddFile ()
    {
        if (upfiles == null)
        {
            CreatTable ();
        }
        if (myFile.PostedFile.FileName! = "")
        {
            string nam = myFile.PostedFile.FileName;
            int i = nam.LastIndexOf ("\\");
            string filename = nam.Substring (i + 1);
            if (upfiles.Rows.Find (filename) == null)
            {
                Stream fstream = myFile.PostedFile.InputStream;
                DataRow dr = upfiles.NewRow ();
                dr [0] = filename; // Get the file name
                upfiles.Rows.Add (dr); // increase
                ArrayList dc = new ArrayList ();
                for (int j = 0; j <upfiles.Rows.Count; j ++)
                {
                    DataRow df = upfiles.Rows [j];
                    dc.Add (df [0] .ToString ());
                }
                files.DataSource = dc;
                files.DataBind ();
            }
        }
    }
    /// <summary>
    /// remove attachment from list
    /// </ summary>
    private void RemoveFile ()
    {
        if (files.SelectedItem.Value! = null)
        {
            string fname = files.SelectedItem.Value;
            DataRow foundRow = upfiles.Rows.Find (fname);
            if (foundRow! = null)
                upfiles.Rows.Remove (foundRow);
            ArrayList dc = new ArrayList ();
            for (int j = 0; j <upfiles.Rows.Count; j ++)
            {
                DataRow df = upfiles.Rows [j];
                dc.Add (df [0] .ToString ());
            }
            files.DataSource = dc;
            files.DataBind ();
        }
    }
    /// <summary>
    /// send email
    /// </ summary>
    private void UploadFile ()
    {
        string userId = this.GetSessionState ("userId");
        if (upfiles.Rows.Count> 0)
        {
            fname.Text = "";
            for (int j = 0; j <upfiles.Rows.Count; j ++)
            {
                DataRow dr = upfiles.Rows [j];
                string topath = Server.MapPath ((this.GetSessionState ("userId") + "/") + dr [0]); // Set attachment path
                fname.Text + = topath.ToString () + ";";
                myFile.PostedFile.SaveAs (topath); // Save attachment

    }

When I insert "multi-attachment upload address" fname.Text into the database, the following happens:
For example: When I upload 3 files at a time, 3 rows of records are generated in the database:
A: / aaa;
A: / aaa; B: / bbb;
A: / aaa; B: / bbb; C: / ccc;
And what I want is to generate only one row of records in the database for each upload, such as:
A: / aaa; B: / bbb; C: / ccc;
Please help me.

苹果小刀

Can't see the statement you inserted into the record

saxonn

Just a sentence of insert, I did not post it, multi-attachment upload address =fname.Text

paoloaxm

for (int j = 0; j <upfiles.Rows.Count; j++)
                {
                    DataRow df = upfiles.Rows[j];
                    dc.Add(df[0].ToString());
                }

Construct a string inside the loop, increase outside the loop

shenmue024

Just a sentence of insert, I did not post it, multi-attachment upload address =fname.Text


Update outside the loop.

That is, all events are finished. Just update the database.

crabhero

For example: When I upload 3 files at a time, 3 rows of records are generated in the database:
A:/aaa;
A:/aaa;B:/bbb;
A:/aaa;B:/bbb;C:/ccc;
What I want is to generate only one row of records in the database for each upload, such as:
A:/aaa;B:/bbb;C:/ccc;
------------------------------------
When you traverse to the last attachment, the record inserted into the database is correct. Presumably you made the insertion inside the loop. In fact, you only need to construct the path string (A:/aaa; B:/bbb; C:/ccc; ) Insert just outside the loop, you can look at the code of your database operation

sabs_nn

Use stringbuilder.appendtext to append filename every time.

dr[0] = filename, filename should be stringbuilder.tostring

yjycoral

Yes, operate the database outside the loop, add a judgment in front, and add it outside the FOR

saxonn

Thank you, brother, solved