如何避免循环中断

我有一个web浏览器在Windows窗体项目。它导航表元素中的所有链接。它工作得很好;然而，它在循环中使用break。如何在没有break语句的情况下实现此功能?

注意:在我的实际场景中，如果我们使用该链接的导航命令，所有链接将重定向到登录页面。所以，存储所有的url和做导航之后将不适合我，在我的实际情况。

c#代码

public partial class Form1 : Form
{
string websiteUrl = @"C:Samples_LMyTableTest.html";
List<string> visitedUrls = new List<string>();
string currentUrl = String.Empty;
private void ExerciseApp(object sender, EventArgs e)
{
    Thread.Sleep(1000);
    if (currentUrl != websiteUrl)
    {
        currentUrl = websiteUrl;
        wb.Navigate(websiteUrl);
    }

    HtmlElement tableElement = wb.Document.GetElementById("four-grid");
    if (tableElement != null)
    {
        foreach (HtmlElement e1 in tableElement.All)
        {
            string x = e1.TagName;
            String idStr = e1.GetAttribute("id");
            if (!String.IsNullOrWhiteSpace(idStr))
            {
                if (idStr.Contains("catalogEntry_img"))
                {
                    string url = e1.GetAttribute("href");
                    if (!visitedUrls.Contains(url))
                    {
                        currentUrl = url;
                        visitedUrls.Add(url);
                        e1.InvokeMember("Click");
                        //Use break when the first match is found
                        break;
                    }
                }
            }
        }
    }
}

private System.Windows.Forms.WebBrowser wb = null;
private Button button1 = null;
private ListBox listBox1 = null;
public Form1()
{
    // button1
    button1 = new Button();
    button1.Location = new Point(20, 430);
    button1.Size = new Size(90, 23);
    button1.Text = "Load and Test";
    button1.Click += new EventHandler(this.button1_Click);
    // listBox1
    listBox1 = new ListBox();
    listBox1.Location = new Point(10, 460);
    listBox1.Size = new Size(460, 200);
    // Web Browser
    wb = new WebBrowser();
    wb.Location = new Point(10, 10);
    wb.Size = new Size(1000, 400);
    //Subscribing for the Document Completed Event
    wb.DocumentCompleted += new WebBrowserDocumentCompletedEventHandler(ExerciseApp);
    // Form1
    this.Text = "Web Browser Test";
    this.Size = new Size(5000, 7100);
    this.Controls.Add(wb);
    this.Controls.Add(button1);
    this.Controls.Add(listBox1);
    currentUrl = websiteUrl;
}
private void button1_Click(object sender, EventArgs e)
{
    listBox1.Items.Add("Loading Web app under test into WebBrowser control");
    wb.Url = new Uri(websiteUrl);
}

}

HTML使用

<html>
<head>
    <style type="text/css">
        table {
            border: 2px solid blue;
        }
        td {
            border: 1px solid teal;
        }
    </style>
</head>
<body>
    <table id="four-grid">
        <tr>
            <td>
                <a href="https://stackoverflow.com/users/696627/lijo" id="catalogEntry_img63664" class="itemhover"
                    onfocus="showPopupButton('category_63664');"
                    onkeydown="shiftTabHidePopupButton('category_63664',event);">
                    <img src="ssss"
                        alt="G" width="70" />
                </a>
            </td>
            <td>
                <a href="http://msdn.microsoft.com/en-US/#fbid=zgGLygxrE84" id="catalogEntry_img63665" class="itemhover"
                    onfocus="showPopupButton('category_63665');"
                    onkeydown="shiftTabHidePopupButton('category_63665',event);">
                    <img src="ssss"
                        alt="Y" width="70" />
                </a>
            </td>
        </tr>
        <tr>
            <td>
                <a href="https://www.wikipedia.org/" id="catalogEntry_img63666" class="itemhover"
                    onfocus="showPopupButton('category_63666');"
                    onkeydown="shiftTabHidePopupButton('category_63666',event);">
                    <img src="ssss"
                        alt="B" width="70" />
                </a>
            </td>
            <td>
                <a href="http://www.keralatourism.org/" id="catalogEntry_img63667" class="itemhover"
                    onfocus="showPopupButton('category_63667');"
                    onkeydown="shiftTabHidePopupButton('category_63667',event);">
                    <img src="ssss"
                        alt="A" width="70" />
                </a>
            </td>
        </tr>
    </table>
</body>
</html>

参考

下一步使用LINQ方法

您可以将其重构为查询，使代码的意图更清晰:

var nextElement = tableElement.All
    .Where(element => element.GetAttribute("id") != null &&
        element.GetAttribute("id").Contains("catalogEntry_img") &&
        !visitedUrls.Contains(element.GetAttribute("href")))
    .FirstOrDefault();
if(nextElement != null)
{
    visitedUrls.Add(nextElement.GetAttribute("href"));
    nextElement.InvokeMember("Click");
    currentUrl = nextElement.GetAttribute("href");
}

我还建议将visitedUrls更改为HashSet，而不是列表，因为它是一个更有效的数据结构，可以简单地确定一个项目是否在一组项目中。

如果您遍历一系列值，并希望在找到满足条件的值时立即退出，则通常使用break:

foreach(var a in list) {
   if(test(a)) {
       // use a
       break;
   }
}

有时可能不需要，你可以这样做:

bool found = false;
foreach(var a in list) {
   if(found && test(a)) {
       // use a
       found = true;
   }
}

甚至:

bool found = false;
foreach(var a in list) {
   if(test(a)) {
       if(found) {
           // use a
           found = true;
       }
   }
}

第一个版本可以在找到合适的元素后跳过迭代。另外两个变体继续迭代甚至检查所有元素。在几乎所有情况下，这只是浪费处理能力。但是看看这里，您可能需要使用第三种变体:http://en.wikipedia.org/wiki/Timing_attack

相关内容

最新更新

热门标签：