subString问题，超急，在线等~~~

悬赏园豆：20 [已解决问题] 解决于 2008-08-25 11:31

我的代码不知道为什么不能return, 别人说，可能是substring改变了自负串的长度 但是，我查过，并没有改变，但是我去掉了subString的部分，果然可以retun，郁闷 谢谢你们了，以下是我的代码 <DIV class=cnblogs_code> 1<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/None.gif" align=top>        public static ArrayList getUrl(string Html, string linkStart, string linkEnd)  2<IMG id=Codehighlighter1_94_1493_Open_Image onclick="this.style.display='none'; document.getElementById('Codehighlighter1_94_1493_Open_Text').style.display='none'; document.getElementById('Codehighlighter1_94_1493_Closed_Image').style.display='inline'; document.getElementById('Codehighlighter1_94_1493_Closed_Text').style.display='inline';" src="http://space.cnblogs.com/Images/OutliningIndicators/ExpandedBlockStart.gif" align=top><IMG id=Codehighlighter1_94_1493_Closed_Image style="DISPLAY: none" onclick="this.style.display='none'; document.getElementById('Codehighlighter1_94_1493_Closed_Text').style.display='none'; document.getElementById('Codehighlighter1_94_1493_Open_Image').style.display='inline'; document.getElementById('Codehighlighter1_94_1493_Open_Text').style.display='inline';" src="http://space.cnblogs.com/Images/OutliningIndicators/ContractedBlock.gif" align=top>        <SPAN id=Codehighlighter1_94_1493_Closed_Text style="BORDER-RIGHT: #808080 1px solid; BORDER-TOP: #808080 1px solid; DISPLAY: none; BORDER-LEFT: #808080 1px solid; BORDER-BOTTOM: #808080 1px solid; BACKGROUND-COLOR: #ffffff"><IMG src="http://space.cnblogs.com/images/dot.gif">{  3<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>            ArrayList urlList = new ArrayList();  4<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>            string tempLinkStart = "{$theLinkStart}";  5<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>            string tempLinkEnd = "{$theLinkEnd}";  6<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>            Html = Html.Replace(linkStart, tempLinkStart);  7<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>            Html = Html.Replace(linkEnd, tempLinkEnd);  8<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>  9<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>            for (int i = 0; i < Html.Length; i++) 10<IMG id=Codehighlighter1_426_1454_Open_Image onclick="this.style.display='none'; document.getElementById('Codehighlighter1_426_1454_Open_Text').style.display='none'; document.getElementById('Codehighlighter1_426_1454_Closed_Image').style.display='inline'; document.getElementById('Codehighlighter1_426_1454_Closed_Text').style.display='inline';" src="http://space.cnblogs.com/Images/OutliningIndicators/ExpandedSubBlockStart.gif" align=top><IMG id=Codehighlighter1_426_1454_Closed_Image style="DISPLAY: none" onclick="this.style.display='none'; document.getElementById('Codehighlighter1_426_1454_Closed_Text').style.display='none'; document.getElementById('Codehighlighter1_426_1454_Open_Image').style.display='inline'; document.getElementById('Codehighlighter1_426_1454_Open_Text').style.display='inline';" src="http://space.cnblogs.com/Images/OutliningIndicators/ContractedSubBlock.gif" align=top>            <SPAN id=Codehighlighter1_426_1454_Closed_Text style="BORDER-RIGHT: #808080 1px solid; BORDER-TOP: #808080 1px solid; DISPLAY: none; BORDER-LEFT: #808080 1px solid; BORDER-BOTTOM: #808080 1px solid; BACKGROUND-COLOR: #ffffff"><IMG src="http://space.cnblogs.com/images/dot.gif">{ 11<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>                if (Html.Substring(i, tempLinkStart.Length) == tempLinkStart)       //判断是否查找到tempLinkStart({$theLinkStart}) 12<IMG id=Codehighlighter1_568_1327_Open_Image onclick="this.style.display='none'; document.getElementById('Codehighlighter1_568_1327_Open_Text').style.display='none'; document.getElementById('Codehighlighter1_568_1327_Closed_Image').style.display='inline'; document.getElementById('Codehighlighter1_568_1327_Closed_Text').style.display='inline';" src="http://space.cnblogs.com/Images/OutliningIndicators/ExpandedSubBlockStart.gif" align=top><IMG id=Codehighlighter1_568_1327_Closed_Image style="DISPLAY: none" onclick="this.style.display='none'; document.getElementById('Codehighlighter1_568_1327_Closed_Text').style.display='none'; document.getElementById('Codehighlighter1_568_1327_Open_Image').style.display='inline'; document.getElementById('Codehighlighter1_568_1327_Open_Text').style.display='inline';" src="http://space.cnblogs.com/Images/OutliningIndicators/ContractedSubBlock.gif" align=top>                <SPAN id=Codehighlighter1_568_1327_Closed_Text style="BORDER-RIGHT: #808080 1px solid; BORDER-TOP: #808080 1px solid; DISPLAY: none; BORDER-LEFT: #808080 1px solid; BORDER-BOTTOM: #808080 1px solid; BACKGROUND-COLOR: #ffffff"><IMG src="http://space.cnblogs.com/images/dot.gif">{ 13<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>                    for (int j = i + tempLinkStart.Length; j < Html.Length; j++)    //从tempLinkStart({$theLinkStart})的后一位开始循环 14<IMG id=Codehighlighter1_716_1309_Open_Image onclick="this.style.display='none'; document.getElementById('Codehighlighter1_716_1309_Open_Text').style.display='none'; document.getElementById('Codehighlighter1_716_1309_Closed_Image').style.display='inline'; document.getElementById('Codehighlighter1_716_1309_Closed_Text').style.display='inline';" src="http://space.cnblogs.com/Images/OutliningIndicators/ExpandedSubBlockStart.gif" align=top><IMG id=Codehighlighter1_716_1309_Closed_Image style="DISPLAY: none" onclick="this.style.display='none'; document.getElementById('Codehighlighter1_716_1309_Closed_Text').style.display='none'; document.getElementById('Codehighlighter1_716_1309_Open_Image').style.display='inline'; document.getElementById('Codehighlighter1_716_1309_Open_Text').style.display='inline';" src="http://space.cnblogs.com/Images/OutliningIndicators/ContractedSubBlock.gif" align=top>                    <SPAN id=Codehighlighter1_716_1309_Closed_Text style="BORDER-RIGHT: #808080 1px solid; BORDER-TOP: #808080 1px solid; DISPLAY: none; BORDER-LEFT: #808080 1px solid; BORDER-BOTTOM: #808080 1px solid; BACKGROUND-COLOR: #ffffff"><IMG src="http://space.cnblogs.com/images/dot.gif">{ 15<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>                        if (Html.Substring(j, tempLinkEnd.Length) == tempLinkEnd)   //判断是否查找到tempLinkEnd({$theLinkEnd}) 16<IMG id=Codehighlighter1_862_1287_Open_Image onclick="this.style.display='none'; document.getElementById('Codehighlighter1_862_1287_Open_Text').style.display='none'; document.getElementById('Codehighlighter1_862_1287_Closed_Image').style.display='inline'; document.getElementById('Codehighlighter1_862_1287_Closed_Text').style.display='inline';" src="http://space.cnblogs.com/Images/OutliningIndicators/ExpandedSubBlockStart.gif" align=top><IMG id=Codehighlighter1_862_1287_Closed_Image style="DISPLAY: none" onclick="this.style.display='none'; document.getElementById('Codehighlighter1_862_1287_Closed_Text').style.display='none'; document.getElementById('Codehighlighter1_862_1287_Open_Image').style.display='inline'; document.getElementById('Codehighlighter1_862_1287_Open_Text').style.display='inline';" src="http://space.cnblogs.com/Images/OutliningIndicators/ContractedSubBlock.gif" align=top>                        <SPAN id=Codehighlighter1_862_1287_Closed_Text style="BORDER-RIGHT: #808080 1px solid; BORDER-TOP: #808080 1px solid; DISPLAY: none; BORDER-LEFT: #808080 1px solid; BORDER-BOTTOM: #808080 1px solid; BACKGROUND-COLOR: #ffffff"><IMG src="http://space.cnblogs.com/images/dot.gif">{ 17<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>                            //把tempLinkStart({$theLinkStart})与tempLinkEnd({$theLinkEnd})之间的内容添加到urlList里 18<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>                            urlList.Add(Html.Substring(i + tempLinkStart.Length, j - (i + tempLinkStart.Length)).ToString()); 19<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>                            //MessageBox.Show(Html.Substring(i + tempLinkStart.Length, j - (i + tempLinkStart.Length)).ToString()); 20<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>                            break; 21<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/ExpandedSubBlockEnd.gif" align=top>                        } 22<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/ExpandedSubBlockEnd.gif" align=top>                    } 23<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/ExpandedSubBlockEnd.gif" align=top>                } 24<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>                MessageBox.Show(Html.Length.ToString());        //我这里为了测试，出数html的长度，但是没有问题~~不变的！但是不知道为什么return不了 25<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/ExpandedSubBlockEnd.gif" align=top>            } 26<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top> 27<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/InBlock.gif" align=top>            return urlList; 28<IMG src="http://space.cnblogs.com/Images/OutliningIndicators/ExpandedBlockEnd.gif" align=top>        }</DIV>

.NET技术 C#

Richard Law | 初学一级 | 园豆：0
提问于：2008-08-25 00:28

< >

最佳答案

建议你使用 Html.Indexof(tempLinkStart, i) 来进行判断,而不要使用SubString 而且你用这个for 循环，效率太低，应使用 while 循环， while (i < Html.Length) { int pos = Html.Indexof(tempLinkStart, i); if (pos < 0) {//未找到tempLinkStart break; } else { i = pos + tempLinkStart.Length; } } 这样做效率要高很多。

eaglet | 专家六级 |园豆：17139 | 2008-08-25 08:39

其他回答(3)

=。= 估计是超出索引界限了我记得js的substring和。net的substring处理方式不一样一个是从索引a到b 一个是从索引a开始的b个字符的长度建议你try catch下把for循环try下看会不会到catch里你这段代码如果不发生死循环且不抛异常是不会没有retrun的

小眼睛老鼠 | 园豆：2731 (老鸟四级) | 2008-08-25 01:20

你是在做循环标签替换嘛？改为正则表达式方面多了.

snryang | 园豆：24 (初学一级) | 2008-08-25 09:05

赞同eaglet 的作法：说下你这个代码： #1 效率低是一定的了，因为你这个要遍历每一个字符 #2 代码我测试过这个不存在return不了的问题，应该是在11行时“if(Html.Substring(i, tempLinkStart.Length) == tempLinkStart)” 这个出现问题，因为当i 本身小于html.length，但是当它再加上一个值时，就很可能出现超出html.length的长度，这样就出现异常了实质上是Html.Substring(i, tempLinkStart.Length)出现异常

程序员老李 | 园豆：790 (小虾三级) | 2008-08-25 10:07

清除回答草稿

您需要登录以后才能回答，未注册用户请先注册。

欢迎，请先 登录 或者 注册 。

subString问题，超急，在线等~~~

欢迎，请先登录或者注册。