Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people, just like you, are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
Solved

For Each  - Xpath /VB.Net

Posted on 2009-04-09
6
751 Views
Last Modified: 2013-11-11
Hi,
I am having difficulty getting this to work.  

If I run it with the return value string I get 2 results (which is right) but they both formatted according to the first match.  It I run it without the returnVal then I only get the first match.  Either way its not working as I hoped.

What am I doing wrong?

Snippet:
<products>
  <product code="av" name="AV Training">
      <duration>4 Days</duration>
      <prices>
        <price vehicle="av">2395</price>
        <price vehicle="avno">1875</price>
      </prices>
    </product>
</products>


Happy to try linq - but don't know it very well.
Cheers
Tania
Public Function getPrice ()  As String 
 
   Dim sHireType As String = XPath("prices/price/@vehicle")
   Dim sHire4x4 As String = "(includes use of a 4wd training vehicle)"
   Dim sBYO4x4 As String = "(if you bring your 4wd)"
   Dim sHireAV As String = "(includes use of a 4wd AV)"
   Dim sByoAV As String = "(if you bring your 4wd AV)"
   Dim sBYOMO As String = "<br />(you will need to bring your own motorcycle)"
			  
   Dim sPriceFormat As String = "<li>Price: US$ {0} per Student {1}.</li>" 
   
   Dim MyList As New ArrayList
          MyList = XpathSelect("prices/*")
 
   Dim sPrice As String = XPath("prices/*")
				 
   Dim a as XmlNode
   Dim ReturnVal as String 
   
        For Each a In MyList
     	
             Select Case sHireType
                   Case "av"
		 	'Return String.Format(sPriceFormat, sPrice, sHire4x4)
      		         ReturnVal &= " " & String.Format(sPriceFormat, sPrice, sHireAV)
       		   Case "avno"
       			ReturnVal &= " " & String.Format(sPriceFormat, sPrice, sByoAV)
		       'Return String.Format(sPriceFormat, sPrice, sByoAV)
                  Case "yes"
       			ReturnVal &= " " & String.Format(sPriceFormat, sPrice, sHire4x4)
		       'Return String.Format(sPriceFormat, sPrice, sHire4x4)
    	       End Select
    	  Next a
 	 Return ReturnVal        
  End Function
 
 
 
  <asp:Repeater id="repeater_price" DataSourceId="XmlSource1"  runat="server">
            <ItemTemplate> 
                <%#getDuration()%>
             <%#  getPrice() %>
            </ItemTemplate>  
          </asp:Repeater>

Open in new window

0
Comment
Question by:Tania_Farmer
  • 4
  • 2
6 Comments
 
LVL 39

Expert Comment

by:abel
ID: 24114623
The code is already very helpful, but I'm not sure I understand your question well, can you give the response that you expect from the repeater (i understand you'll have to type it by hand) alongside with the wrong response you're having now?
0
 
LVL 39

Expert Comment

by:abel
ID: 24114727
Think I see your problem (just ran it locally). Your sPrice is the price tag and is a string which is declared like so:

  Dim sPrice As String = XPath("prices/*")

this string is then used inside the for-each and contains the string value of the first element that the xpath matched, which is 2395. To get the price of each node while going through the prices, you can use:
  For Each a In MyList       sPrice = a.FirstChild.Value

which will give you the value of the (first and only) text node inside <price>.

May I ask you why you are using an asp:Repeater and then do the repetition inside a method? Or is it part of a larger whole?

-- Abel --
0
 

Author Comment

by:Tania_Farmer
ID: 24114873
Hi,
Thanks - it works.  It shows:
# Price: US$ 2395 per Student (includes use of a 4wd AV).
# Price: US$ 1875 per Student (includes use of a 4wd AV).

How do I adjust it so the vehicle info is also updated:
# Price: US$ 2395 per Student (includes use of a 4wd AV).
# Price: US$ 1875 per Student (if you bring your 4wd AV).  

<price vehicle="av">2395</price>
<price vehicle="avno">1875</price>

I used the method so I could format and expand the results based on the return.  Its a control and used on several pages.  

I have :

<asp:Repeater id="repeater_price" DataSourceId="XmlSource1"  runat="server">
            <ItemTemplate>
                    <%#getDuration()%>
                    <%#  getPrice() %>
            </ItemTemplate>            
</asp:Repeater>

It returns the 2 prices and the course duration.

T
0
Master Your Team's Linux and Cloud Stack

Come see why top tech companies like Mailchimp and Media Temple use Linux Academy to build their employee training programs.

 
LVL 39

Accepted Solution

by:
abel earned 500 total points
ID: 24114998
> How do I adjust it so the vehicle info is also updated:

same story there. You'll have to put that inside the loop (and your can remove the initialization of the variables):

 For Each a In MyList
     sPrice = a.FirstChild.Value
     sHireType = a.Attributes("vehicle").Value


On the other point: if you meant to use the repeater for looping through the prices, you can make it much easier in another way. If you wanted to use the repeater to loop through the products, than the current method is probably the easiest.

-- Abel --
0
 

Author Comment

by:Tania_Farmer
ID: 24115432
Hi,
Thanks  that works great.

By the way what is better to use?  If there is a difference?

Dim myList As IEnumerable = xpathselect("prices/*")

or

Dim myList As New ArrayList
      myList = XpathSelect("prices/*")

How do I make it easier?  Its looping through the prices.  The product info is set and displayed by xmldatasource on the relevent page and then the control is pulled to display the price.

T
0
 
LVL 39

Expert Comment

by:abel
ID: 24118879
> Thanks  that works great.

you're welcome

> By the way what is better to use?  If there is a difference?

in your case, you do not need to use either. You can simply put the XPathSelect inside the for-each, this will only be evaluated once.

> How do I make it easier?  Its looping through the prices.

my personal thought when looking at your code was that you wanted to use the repeater to "repeat" through the prices. Now you are doing that in the code behind by hand, making the repeater effectively redundant (unless it is part of a large whole, i.e., if the repeater is used to loop through the products).

I can help you set it up for the repeater to loop through the prices, either here or, if you prefer (since it is technically a new thread/question) on another question.

-- Abel --
0

Featured Post

Free Tool: Postgres Monitoring System

A PHP and Perl based system to collect and display usage statistics from PostgreSQL databases.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

1.0 - Introduction Converting Visual Basic 6.0 (VB6) to Visual Basic 2008+ (VB.NET). If ever there was a subject full of murkiness and bad decisions, it is this one!   The first problem seems to be that people considering this task of converting…
Microsoft Reports are based on a report definition, which is an XML file that describes data and layout for the report, with a different extension. You can create a client-side report definition language (*.rdlc) file with Visual Studio, and build g…
Microsoft Active Directory, the widely used IT infrastructure, is known for its high risk of credential theft. The best way to test your Active Directory’s vulnerabilities to pass-the-ticket, pass-the-hash, privilege escalation, and malware attacks …
A short tutorial showing how to set up an email signature in Outlook on the Web (previously known as OWA). For free email signatures designs, visit https://www.mail-signatures.com/articles/signature-templates/?sts=6651 If you want to manage em…

790 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question