[eddychamoun]

hello ,
In my form I have 1 textbox , 1 labels and 1 button + I've added a resource file "sites.txt"
Now in the textbox i should enter the URL of the site and when I click the button , the application should search in the resource file "sites.txt" for the URL of the site , if the site was found then in the label the word "INFECTED" must appear and if not the word "NOT INFECTED" must appear in the label .
My code is :

Label1.Text = ""

        Dim FileContents() As String = IO.File.ReadAllLines("sites.txt")
        
        For I = 0 To FileContents.Length - 1

            Dim LineParts As String = FileContents(I)

            If TextBox1.Text = LineParts Then

                Label1.Text = "Infected"
                
            Else : Label1.Text = "Not Infected"

            End If

        Next

But my problem is when the URL of a site is found in "sites.txt" , the word is always "Not Infected"
Any suggestions would be appreciated

[AdamSpeight2008]

Of course it is, the label will be what the last condition in the the for-loop results in choosing.

Accessing Resource in vb.net is simple as;-
My,Resources.

[Ionut]

Here I strongly suggest you to put a breakpoint and debug your code. My guess is that at the end of every LineParts value is a CR_LF(Carriage return and Line feed = Enter). A breakpoint will help you to see the value of that variable at each step.
Also, you may find Contains method more useful

if LineParts.Contains(TextBox1.Text) then 
   'set infected
else 
   'set not infected
end if 

Furthermore, you can shorten your code by using IIf

Label1.Text = IIf(LineParts.Contains(TextBox1.Text), "Infected", "Not infected")

[AdamSpeight2008]

IFF is a throwback to vb6, and has to preserve vb6 semantics. It has an issue that it evaluates both potential return result before returning one.

Use: If( {Boolean_Condition}, {IsTrue}, {IsFalseResult})

[eddychamoun]

AdamSpeight2008, on 08 July 2012 - 01:59 AM, said:

Of course it is, the label will be what the last condition in the the for-loop results in choosing.

Accessing Resource in vb.net is simple as;-
My,Resources.

So in my case what should I do in order to solve my problem - If the site was included in the "sites.txt" the label should be Infected and if the site wasn't included in the "sites.txt" the label should be Not Infected -

Ionut, on 08 July 2012 - 02:06 AM, said:

Here I strongly suggest you to put a breakpoint and debug your code. My guess is that at the end of every LineParts value is a CR_LF(Carriage return and Line feed = Enter). A breakpoint will help you to see the value of that variable at each step.
Also, you may find Contains method more useful

if LineParts.Contains(TextBox1.Text) then 
   'set infected
else 
   'set not infected
end if 

Furthermore, you can shorten your code by using IIf

Label1.Text = IIf(LineParts.Contains(TextBox1.Text), "Infected", "Not infected")

thanks Ionut you solved my problem

[AdamSpeight2008]

Ionut may have provide a solution to your problem, but I reckon you still don't comprehend the reason why your original don't act as you thought.