30

XPath (v1) contains no way to encode expressions.

If you only have single OR double quotes then you can use expressions such as

//review[@name="Bob's Pizza"]
//review[@name='"Pizza" Pam']

But if you have BOTH e.g [Fred's "Fancy Pizza"] then you have to use something like this Escaping Strings in XPath (C++) to generate

//review[@name=Concat("Fred's ",'"Fancy Pizza"')]

Anyone have a function in c# to do this?

Some links that are close

EDIT: A few answers have suggested escaping ' with ' and " with " but although this makes sense it does not work; try it using the XML fragment:

<review name="Bob's Pizza"/>

and the xpath

//review[@name='Bob&apos;s Pizza']
Jerzy
  • 670
  • 6
  • 12
Ryan
  • 23,871
  • 24
  • 86
  • 132

8 Answers8

17

Wow, you all sure are making this complicated. Why not just do this?

public static string XpathExpression(string value)
{
    if (!value.Contains("'"))
        return '\'' + value + '\'';

    else if (!value.Contains("\""))
        return '"' + value + '"';

    else
        return "concat('" + value.Replace("'", "',\"'\",'") + "')";
}

.NET Fiddle & test

Ryan
  • 23,871
  • 24
  • 86
  • 132
Kaleb
  • 186
  • 1
  • 3
  • Can you leave some comments in the code describing how it works? – Shawn Jul 07 '16 at 20:46
  • Nice! Funny how sometimes you can't see the wood for the trees eh! – Ryan Jul 08 '16 at 13:42
  • 1
    In all fairness, I did notice after posting this that tiwahu had posted an extremely similar answer, but with the more expensive string.Format() method. And his answer even has comments! ;-) – Kaleb Jul 08 '16 at 21:04
  • 1
    Just a heads-up to mention that the "else" statements are not needed. – Plasmabubble Jul 30 '20 at 08:36
10

Though it certainly won't work in all circumstances, here's a way to sidestep the problem:

doc.DocumentElement.SetAttribute("searchName", name);
XmlNode n = doc.SelectNodes("//review[@name=/*/@searchName]");
Robert Rossney
  • 94,622
  • 24
  • 146
  • 218
  • This worked like a charm for me in Java as well, with the exception being I had to add another "/" in the query: `doc.getDocumentElement().setAttribute("searchAttribute", name); String type = xpath.evaluate("//review[@name=//@searchAttribute]/@type", doc);` – Jeff Olson Jul 15 '11 at 20:56
  • Seems like a slick solution... except as far as I can tell it doesn't work under any circumstances. `var n1 = xmlDoc.SelectNodes("//*/*/Credential[@Name='zzz']"); //returns 1 node` `xmlDoc.DocumentElement.SetAttribute("searchName", "zzz");` `var n2 = xmlDoc.SelectNodes("//*/*/Credential[@Name=/@searchName]"); // returns 0 nodes` – mhenry1384 Sep 08 '11 at 04:08
  • 2
    I think that `/@foo` is wrong: to get an attribute on the top-level element, you need to do `/*/@foo`. So the predicate would be `[@Name=/*/@searchName]`. – Robert Rossney Sep 08 '11 at 09:03
  • @RobertRossney That’s kind of bad. That means that it’ll match that attribute regardless of where it is in the document instead of only getting the one you want. Don’t you want something like `@name=/path/to/@searchName`? Seems to me that the answer was better prior to your edit… – binki Nov 11 '19 at 23:57
7

I was in need of this so I created this solution, for C#.

    /// <summary>
    /// Returns a valid XPath statement to use for searching attribute values regardless of 's or "s
    /// </summary>
    /// <param name="attributeValue">Attribute value to parse</param>
    /// <returns>Parsed attribute value in concat() if needed</returns>
    public static string GetXpathStringForAttributeValue(string attributeValue)
    {
        bool hasApos = attributeValue.Contains("'");
        bool hasQuote = attributeValue.Contains("\"");

        if (!hasApos)
        {
            return "'" + attributeValue + "'";
        }
        if (!hasQuote)
        {
            return "\"" + attributeValue + "\"";
        }

        StringBuilder result = new StringBuilder("concat(");
        StringBuilder currentArgument = new StringBuilder();
        for (int pos = 0; pos < attributeValue.Length; pos++)
        {
            switch (attributeValue[pos])
            {
                case '\'':
                    result.Append('\"');
                    result.Append(currentArgument.ToString());
                    result.Append("'\",");
                    currentArgument.Length = 0;
                    break;
                case '\"':
                    result.Append('\'');
                    result.Append(currentArgument.ToString());
                    result.Append("\"\',");
                    currentArgument.Length = 0;
                    break;
                default:
                    currentArgument.Append(attributeValue[pos]);
                    break;
            }
        }
        if (currentArgument.Length == 0)
        {
            result[result.Length - 1] = ')';
        }
        else
        {
            result.Append("'");
            result.Append(currentArgument.ToString());
            result.Append("')");
        }
        return result.ToString();
    }
5

This is what I've come up with

public static string EncaseXpathString(string input)
{         
    // If we don't have any " then encase string in "
    if (!input.Contains("\""))
        return String.Format("\"{0}\"", input);

    // If we have some " but no ' then encase in '
    if (!input.Contains("'"))
        return String.Format("'{0}'", input);

    // If we get here we have both " and ' in the string so must use Concat
    StringBuilder sb = new StringBuilder("concat(");           

    // Going to look for " as they are LESS likely than ' in our data so will minimise
    // number of arguments to concat.
    int lastPos = 0;
    int nextPos = input.IndexOf("\"");
    while (nextPos != -1)
    {
        // If this is not the first time through the loop then seperate arguments with ,
        if (lastPos != 0)
            sb.Append(",");

        sb.AppendFormat("\"{0}\",'\"'", input.Substring(lastPos, nextPos - lastPos));
        lastPos = ++nextPos;

        // Find next occurance
        nextPos = input.IndexOf("\"", lastPos);
    }

    sb.Append(")");
    return sb.ToString();
}

Called using something like

XmlNode node = doc.SelectSingleNode("//review[@name=" + EncaseXpathString("Fred's \"Fancy Pizza\"" + "]")

So we get the following results

EncaseXpathString("Pizza Shed") == "'Pizza Shed'";
EncaseXpathString("Bob's pizza") == "\"Bob's Pizza\"";
EncaseXpathString("\"Pizza\" Pam" == "'\"Pizza\" Pam'";
EncaseXpathString("Fred's \"Fancy Pizza\"") == "concat(\"Fred's \",'\"',\"Fancy Pizza\",'\"')";

So it's only using concat when its necessary (both " and ' in string)

The last result show the concat operation is not as short as it could be (see question) but its close enough and anything more optimal would be very complex as you would have to look for matching pairs of " or '.

Ryan
  • 23,871
  • 24
  • 86
  • 132
  • 3
    Your code assumes that the last double quote is at the end of the input string. To cover the case where the last double quote is somewhere else, the line `sb.Append(")");` should be written as `sb.AppendFormat(",\"{0}\")", input.Substring(lastPos));`. Otherwise, good job, thanks for a working example! – herzbube Sep 10 '13 at 16:28
  • Hi I tried your code it did not worK I am looking for **/Math/Show[Quiz = 'This 'wont hurt']** I have tried calling it the following ways but I still get an error saying has an **invalid token** `var node = xd.XPathSelectElements("/Math/Show[Quiz = " + EncaseXpathString("'This 'wont hurt'"+ "]"));` `var node = xd.XPathSelectElements("/Math/Show[Quiz = " + EncaseXpathString("This 'wont hurt"+ "]"));` – justinf Sep 22 '13 at 21:36
  • 1
    Something is broken here. EncaseXpathString( "'foo\"bar\"baz") returns concat("'foo",'"',"bar",'"') – Gigo May 07 '15 at 15:29
4

I've had problems with all solutions so far. One has extra text sections (e.g. '"' or "'") which breaks what you're looking for. One drops all text after the last quote/dblquote which breaks as well.

This is a dumb and quick solution from a dumb vb developer:

Function ParseXpathString(ByVal input As String) As String
    input = Replace(input, "'", Chr(1))
    input = Replace(input, """", Chr(2))
    input = Replace(input, Chr(1), "',""'"",'")
    input = Replace(input, Chr(2), "','""','")
    input = "concat('','" + input + "')"
    Return input
End Function

Usage (same as previous examples):

x.SelectNodes("/path[@attr=" & ParseXpathString(attrvalue) & "]")
Eric
  • 41
  • 2
2

I needed to do this in XSLT itself, so came up with the following based on the answers on this page:

<xsl:template name="escape-string">
  <xsl:param name="string"/>
  <xsl:param name="concat" select="true()"/>
  <xsl:variable name="quote">"</xsl:variable>
  <xsl:variable name="apos">'</xsl:variable>
  <xsl:choose>
    <xsl:when test="not(contains($string, $apos))">'<xsl:value-of select="$string"/>'</xsl:when>
    <xsl:when test="not(contains($string, $quote))">"<xsl:value-of select="$string"/>"</xsl:when>
    <xsl:otherwise>
      <xsl:if test="$concat">concat(</xsl:if>
      <xsl:call-template name="escape-string">
        <xsl:with-param name="string" select="substring-before($string, $apos)"/>
        <xsl:with-param name="concat" select="false()"/>
      </xsl:call-template>
      <xsl:text>, "'", </xsl:text>
      <xsl:call-template name="escape-string">
        <xsl:with-param name="string" select="substring-after($string, $apos)"/>
        <xsl:with-param name="concat" select="false()"/>
      </xsl:call-template>
      <xsl:if test="$concat">)</xsl:if>
    </xsl:otherwise>
  </xsl:choose>
</xsl:template>
Laurence Rowe
  • 2,909
  • 17
  • 20
2

Another variation...my concat() part is a little lazy, but at least it uses the whole value.

    /// <summary>
    /// Returns an XPath string literal to use for searching attribute values (wraped in apostrophes, quotes, or as a concat function).
    /// </summary>
    /// <param name="attributeValue">Attribute value to encode and wrap.</param>
    public static string CreateXpathLiteral(string attributeValue)
    {
        if (!attributeValue.Contains("\""))
        {
            // if we don't have any quotes, then wrap string in quotes...
            return string.Format("\"{0}\"", attributeValue);
        }
        else if (!attributeValue.Contains("'"))
        {
            // if we have some quotes, but no apostrophes, then wrap in apostrophes...
            return string.Format("'{0}'", attributeValue);
        }
        else
        {
            // must use concat so the literal in the XPath will find a match...
            return string.Format("concat(\"{0}\")", attributeValue.Replace("\"", "\",'\"',\""));
        }
    }
tiwahu
  • 356
  • 3
  • 5
1

Join in the fun

public string XPathLiteral(string text) {

    const string APOS = "'";
    const string QUOTE = @"""";

    int pos = 0;

    int posApos;
    int posQuote;

    posQuote = text.IndexOf(QUOTE, pos);

    if (posQuote < 0) {
        return QUOTE + text + QUOTE;
    }//if

    posApos = text.IndexOf(APOS, pos);

    if (posApos < 0) {
        return APOS + text + APOS;
    }//if

    bool containsApos = posApos < posQuote;

    StringBuilder sb = new StringBuilder("concat(", text.Length * 2);

    bool loop = true;
    bool comma = false;

    while (loop) {

        if (posApos < 0) {
            posApos = text.Length;
            loop = false;
        }//if

        if (posQuote < 0) {
            posQuote = text.Length;
            loop = false;
        }//if

        if (comma) {
            sb.Append(",");
        } else {
            comma = true;
        }//if

        if (containsApos) {
            sb.Append(QUOTE);
            sb.Append(text.Substring(pos, posQuote - pos));
            sb.Append(QUOTE);
            pos = posQuote;
            if (loop) posApos = text.IndexOf(APOS, pos + 1);
        } else {
            sb.Append(APOS);
            sb.Append(text.Substring(pos, posApos - pos));
            sb.Append(APOS);
            pos = posApos;
            if (loop) posQuote = text.IndexOf(QUOTE, pos + 1);
        }//if

        // Toggle
        containsApos = !containsApos;

    }//while

    sb.Append(")");

    return sb.ToString();

}//method
Jack
  • 11
  • 1