Appending Quations to a string

PIEBALDconsult

Replace perhaps? But why create the string with the @s rather than the characters you want? Creating the string the way you want shouldn't be a problem. Perhaps you should explain what you want and show the code you have so far.

Arun Philip Reynolds

try
{

        //OleDbCommand oleDbCmd = new OleDbCommand("Select \* From \[Sheet1\]", oleDbCon);            
        //oleDbDa.SelectCommand = oleDbCmd;            
        oleDbDa.Fill(ds,"ServerName");
        DataTable dt = ds.Tables\["ServerName"\];
        //string str1stValue = ds.Tables\[0\].Rows\[0\]\[0\].ToString();
        foreach(DataRow dr in dt.Rows)
        {
            foreach (DataColumn dc in dt.Columns)
                strServerName.Append(dr\[dc\].ToString()).Append("\\",\\"");
                //strServerName.AppendFormat("\\",\\"", dr\[dc\].ToString());
        }
       //strFinalList = strFinalList.AppendFormat("\\",\\"", strServerName);
        Response.Write(strServerName);
    }

But here again i need to get rid of of the quotations at the end of the string and need to add it before the string. The result that is get is, abc","def","ghi","jkl"," As I had mentioned i'd like to get the result as, "abc","def","ghi","jkl"

PIEBALDconsult

How about:

foreach(DataRow dr in dt.Rows)
{
foreach (DataColumn dc in dt.Columns)
strServerName.AppendFormat( "\"{0}\"," , dr[dc] );
}
strServerName.Length-- ;

wizardzz

Based on your posts below, have you tried using string.Join?

string example = String.Join(", ", string[] exList);

"I have a theory that the truth is never told during the nine-to-five hours. " — Hunter S. Thompson

Luc Pattyn

What Piebald said; or else what I often do when I do not want the StringBuilder:

string result="";
string sep="";
foreach(string s in strings) {
result+=sep+"\""+s+"\"";
sep=",";
}

:)

Luc Pattyn [My Articles] Nil Volentibus Arduum

PIEBALDconsult

I'd still use String.Format -- I find it easier to read than a bunch of concatenations.

Arun Philip Reynolds

Thanks PIEBALDconsult for the help. The solution is just perfect, but there is an extra comma at the end of the string, "abc","def","ghi","jkl", How could I get rid of the last comma?

Arun Philip Reynolds

I'd like to go with PIEBALDconsult on this one, but thanks for the help :)

Luc Pattyn

I use it sparingly, string.Format I mean. :)

Luc Pattyn [My Articles] Nil Volentibus Arduum

PIEBALDconsult

You probably left off the decrement of the Length.

Arun Philip Reynolds

oops!!! I did miss that one out. After I included the decrement of the length, it is removing all the comma's from the string. I need the comma that is in between the strings but it should not add the comma at the end of the string. It is just like the SQL statement that we write. e.g. Select * from table where name like ('abc','def','ghi') I am trying to send the variable in this format to the stored procedure.

PIEBALDconsult

It should be outside the loop, as in the example I posted.

PIEBALDconsult

Arun Philip Reynolds wrote:

It is just like the SQL statement that we write.
e.g. Select * from table where name like ('abc','def','ghi')

If you are using SQL Server (and you should), then how about... SQL Server 2008 User Defined Table Types and Table-Valued Parameters[^]

PIEBALDconsult

I use it whenever I can -- muaa haa ha ha! :cool:

Arun Philip Reynolds

Thanks, but I just tried this,

strHolderString = strServerName.ToString();
strHolderString = strHolderString.TrimEnd(',');
strFinalList = new StringBuilder(strHolderString);

here the strHolderString is a string variable and I am getting the result that I was looking for. Thanks once again to everyone. :-D

Arun Philip Reynolds

strHolderString = strServerName.ToString();
strHolderString = strHolderString.TrimEnd(',');
strFinalList = new StringBuilder(strHolderString);

I just tried the above and it works for me. Here the strHolderString is a string variable and I used the TrimEnd function to get rid of the trailing comma. The strFinalList is a StringBuilder variable. Thanks once again to all

PIEBALDconsult

:sigh: Just decrement the Length. Why create two Strings and another StringBuilder?

Subin Mavunkal

You can use SmartPaster addin for visual studio.Copy the text you needed and paste it as string builder.Here is the code and installer http://www.martinwilley.com/blog/2010/06/06/SmartPasteIn2010.aspx[^]

BobJanova

There's pretty much no reason to avoid StringBuilder, it's in the core Framework and the memory footprint of creating a new object is likely to be less than all the wasted strings you make in a concatenation session.

Pete OHanlon

An alternative approach is to use a list of strings instead of a StringBuilder to build your list and then concatenate them as you need. Here's one way to do this:

public class FormattedStrings : List<string>()
{
public override string ToString()
{
StringBuilder sb = new StringBuilder();
bool started = false;
foreach (string item in this)
{
if (started)
{
sb.AppendFormat(",\"{0}\"", item);
}
else
{
started = true;
sb.AppendFormat("\"{0}\"", item);
}
}

return sb.ToString();

}
}

I am assuming that you have assigned the @ symbol when you built your StringBuilder.

*pre-emptive celebratory nipple tassle jiggle* - Sean Ewington

"Mind bleach! Send me mind bleach!" - Nagy Vilmos

My blog | My articles | MoXAML PowerToys | Mole 2010 - debugging made easier - my favourite utility