如何基于单元格内容定位表格中符合条件的首个Review按钮？

Hey there! I see you're trying to click the "Review" button in the first cell of a table row, but only when the 4th cell's <a> tag has text equal to "0". Let's fix up your code and make this work smoothly.

Option 1: Use a Direct XPath to Locate the Button (Cleaner Approach) #

Instead of looping through all rows, you can craft a precise XPath that directly targets the button you need. This is more efficient and reduces code complexity:

try
{
    // Directly find the Review button where the 4th td's a tag has text "0"
    IWebElement reviewButton = Browser.Driver.FindElement(By.XPath("//*[@id='summaryGrid']/table/tbody/tr[td[4]/a[text()='0']]/td[1]/button[text()='Review']"));
    
    // Click the button if it exists and is interactable
    if (reviewButton.Displayed && reviewButton.Enabled)
    {
        reviewButton.Click();
    }
}
catch (NoSuchElementException)
{
    // Handle the case where no matching row/button is found
    Console.WriteLine("No matching Review button found for rows with 4th cell value '0'");
}

Why this works: #

• The XPath tr[td[4]/a[text()='0']] filters rows to only those where the 4th <td> contains an <a> tag with text "0".
• We then target the <button> with text "Review" in the first <td> of that filtered row.
• Added error handling to catch cases where the element doesn't exist (so your script doesn't crash unexpectedly).

Option 2: Complete Your Existing Loop Logic #

If you prefer to keep the row-by-row approach (maybe you need to do additional checks or actions on the row), here's how to finish your code:

IWebElement baseTable = Browser.Driver.FindElement(By.XPath("//*[@id='summaryGrid']/table/tbody"));
ICollection<IWebElement> rows = baseTable.FindElements(By.TagName("tr"));

foreach (var row in rows)
{
    try
    {
        // Get the 4th cell's a tag text, trim to avoid whitespace issues
        IWebElement fourthCellLink = row.FindElement(By.XPath("./td[4]/a"));
        string linkText = fourthCellLink.Text.Trim();

        // Check if the link text matches "0"
        if (linkText == "0")
        {
            // Find and click the Review button in the first cell
            IWebElement reviewButton = row.FindElement(By.XPath("./td[1]/button[text()='Review']"));
            if (reviewButton.Displayed && reviewButton.Enabled)
            {
                reviewButton.Click();
                // Uncomment below if you only need to click the first matching button
                // break;
            }
        }
    }
    catch (NoSuchElementException)
    {
        // Skip rows that don't have the required elements (e.g., missing a tag or Review button)
        continue;
    }
}

Key Improvements here: #

• Added Trim() to the link text to handle accidental whitespace (super common in web tables).
• Wrapped each row's element lookup in a try-catch to skip rows that don't match the expected structure (so the loop doesn't fail mid-way).
• Added checks to ensure the button is visible and enabled before clicking (prevents click errors from non-interactable elements).
• Included an optional break if you only need to click the first matching button instead of all of them.

Pro Tips: #

• For better reliability, use explicit waits (like WebDriverWait) to wait for the table to load fully before interacting with it. This avoids race conditions where elements aren't ready yet.
• If the text "0" is loaded dynamically or hidden, try using fourthCellLink.GetAttribute("textContent") instead of .Text (though .Text works for visible text in most cases).

内容的提问来源于stack exchange，提问作者user1279586