String Concatenation in VB.Net

Question

0.00/5 (No votes)

See more:

Hi Expert

I had a for loop and i am getting one string value at all the time of execuing the string value.
How to append the string. Code pasted below. Kindly provide your solutions.

VB

For Each row As GridViewRow In gvmstrgrid.Rows
                Dim chk As CheckBox = DirectCast(row.FindControl("chkbx1"), CheckBox)
                If chk.Checked = True Then
                    Dim strbr As New StringBuilder
                    Dim final As String
                    Dim str1 As String
                    str1 = DirectCast(row.FindControl("lblamc"), Label).Text

                End If

From the above code i want the str1 to be concatenate and show the final result in another string.

Rgds
Jagadesh

[edit]Removed shouting in subject and added code formatting.[/edit]

Posted 24-Jun-13 22:31pm

Kumar from madras

Updated 24-Jun-13 22:34pm

Pheonyx

v2

Add a Solution

3 solutions

Solution 3

VB

Dim str1 As String = string.Empty
For Each row As GridViewRow In gvmstrgrid.Rows
    Dim chk As CheckBox = DirectCast(row.FindControl("chkbx1"), CheckBox)
    If chk.Checked = True Then
        Dim strbr As New StringBuilder
        Dim final As String
        str1 &= DirectCast(row.FindControl("lblamc"), Label).Text & VBNewLine 'or & ", "
    End If
Next

Happy Coding!
:)

Posted 24-Jun-13 22:42pm

Aarti Meswania

Comments

Thomas Daniels 25-Jun-13 4:47am

Reason for my vote of 1: string concatenation should be avoided. Use StringBuilder instead:
What's so bad about string concatenation?

Solution 4

use

VB

str1 += DirectCast(row.FindControl("lblamc"), Label).Text

and finally, you will be able to find string concatanation in this

VB

str1

string type.

Posted 24-Jun-13 22:43pm

ashriv

Comments

Thomas Daniels 25-Jun-13 4:48am

Reason for my vote of 1: string concatenation should be avoided. Use StringBuilder instead:
What's so bad about string concatenation?

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Pheonyx · Accepted Answer · 2013-06-24T22:38:00

I would use a string builder object to achieve this:

VB

Dim s As New StringBuilder
Dim final As String
For Each row As GridViewRow In gvmstrgrid.Rows
     Dim chk As CheckBox = DirectCast(row.FindControl(&quot;chkbx1&quot;), CheckBox)
     If chk.Checked = True Then

        Dim str1 As String
        str1 = DirectCast(row.FindControl(&quot;lblamc&quot;), Label).Text
        s.Append(str1)

      End If
Next

final = s.ToString()

Something like that I believe will do what you are after.
Please note where I moved your declarations of the string builder and the final string to.