ansaurus

Question

If Html File Has No Ending "/tr" Tag OR "/td" Tag Then HTML Agility Pack Does Not Read That Information Perfectly.

Answer 1

A:

I haven't tried this, but does it help to add this code?

HtmlNode.ElementsFlags.Add("tr", HtmlElementFlag.Closed);
HtmlNode.ElementsFlags.Add("td", HtmlElementFlag.Closed);

Jeremy Stein 2010-03-19 15:43:37

@Jeremy Stein..It adds end tags but not at the appropriate location If there is a table like `<tr><td>firstrow</td> <tr><td>secondrow</td>` then and we do like `HtmlNode.ElementsFlags.Add("tr",HtmlElementFlag.Closed);` then the table will `<tr></tr><td>firstrow</td> <tr></tr><td>secondrow</td>` instead of `<tr><td>firstrow</td></tr> <tr><td>secondrow</td></tr>`

Harikrishna 2010-03-20 04:50:41

Thanks for following up. Sorry I wasted your time on that idea.

Jeremy Stein 2010-03-22 14:24:34

@Jeremy Stein..It's ok.

Harikrishna 2010-03-23 04:43:03

Answer 2

+1 A:

Since you tested my other idea and it didn't work, I think you have only two options:

Modify HTML Agility Pack to handle your case, or
Fill in the missing </tr>s yourself.

Here's a regex that might fill in the missing </tr>s for you:

html = Regex.Replace(html, "<tr[^>]*>(?:(?!</?tr>|</tbody>|</table>).)*?(?=<tr[^>]*>|</tbody>|</table>)", "$&</tr>", RegexOptions.Singleline | RegexOptions.IgnoreCase);

(If someone can improve my regex, please feel free.)

Jeremy Stein 2010-03-22 14:35:51

@Jeremy Stein..Thanks for the answer..Above regex works for small table but does not work for another table like I have put the table in the question which I had changed.

Harikrishna 2010-03-23 08:22:56

OK, I edited my answer to allow attributes on the `tr` tag.

Jeremy Stein 2010-03-23 16:35:18

@Jeremy Stein.Thanks for the answer.Above regex works for that but I found another table in which this regex does not work.So I found html tidy as a solution.Thank you very much again for helping..

Harikrishna 2010-03-24 12:13:24

@Jeremy Stein,I was using html tidy pack and it inserts the missing closing tag but it also does other formatting what I don't want.I want to insert the missing tag only in the html file where necessary.So I want to use regex for that because only regex will fulfill my need but regex does not insert the </tr> for above table which I have put in the question.

Harikrishna 2010-03-31 05:10:26

@Jeremy Stein,Thank You Very Much for helping..Your regex is very good and works very good...

Harikrishna 2010-04-01 08:41:22

@Jeremy Stein,If the starting tag is missing then which regex should be there ?

Harikrishna 2010-05-21 12:44:21

Give me an example.

Jeremy Stein 2010-05-21 15:47:58

http://stackoverflow.com/questions/1732348/regex-match-open-tags-except-xhtml-self-contained-tags/1732454#1732454

RCIX 2010-05-22 07:05:27

@Jeremy Stein,I have update my question with that example in which opeing tr tag is missing

Harikrishna 2010-05-24 05:29:40

It's unreasonable to expect to be able to parse such invalid HTML. I suggest you switch to a regex to pull the data you need.

Jeremy Stein 2010-05-25 17:11:00

@Jeremy Stein,It's good feeling to get you back after many days,because I have this problem and I can not get it solved.But I can not switch to regex because it is almost to complete that project with parser and in only one file there is a problem like that.And html file is displayed properly of example which with I had updated my question which has missing opening tag.So if it is possible I want to use here regex to insert missing opening tags.

Harikrishna 2010-05-26 04:20:14

@Jeremy Stein, Is it possible with regex ?

Harikrishna 2010-05-27 08:53:33

Sure. Download the evaluation version of RegexBuddy, copy in your HTML, and tweak a regex until it works.

Jeremy Stein 2010-05-27 15:30:07

@Jeremy Stein, Will you please help to make the regex for inserting opening tags in html file ?

Harikrishna 2010-05-29 05:56:17

You may want to do something like this: `html = Regex.Replace(html, "(?<=</tr>[^<]*)<td", "<tr>$`

Jeremy Stein 2010-06-01 15:43:27

ansaurus

tags:

views:

answers:

If Html File Has No Ending "/tr" Tag OR "/td" Tag Then HTML Agility Pack Does Not Read That Information Perfectly.

related questions