[Resolved] Table help please

Aceman87

My problem is that I want my code to generate a table that will look like this: http://lumi.valkoinen.org/music/as_np_stat.php, so that it will have different colors on rows in turns, by altering the class attributes.

I have tried to do this in my CODE on rows 20-25 and then I have placed the variable onto row 28

<tr class='$x'>

Now the problem is that the code prints out like this: http://koti.mbnet.fi/nanime/parseri2.php
So instead of the table-tumma and table-vaalea taking turns, it only prints out the table-tumma value and even that it prints outside the entire table, when it should be printed inside the <tr> tag, making the headers (e.g. (Oct/16 10:34 AM) [Nanime] Gundam SEED Destiny - 01 (41241597).avi) change colors in turns.

All help would be greatly appreciated !!
Thank you.

tsinka

Hi,

try this modified code:

function endElement($parser, $name) {
  global $insideitem, $tag, $title, $description, $link, $x;

  if ($name == "ITEM") {
    if ($x%2 == 1)
      $strClass = 'table-vaalea';
    else
      $strClass = 'table-tumma';

$x++;

printf("<tr class='%s'><td width=\"620\"><div align=\"left\"><span class=\"style2\"><a href='%s'>%s</a></span></div></td><td width=\"100\"><div align=\"left\"><span class=\"style2\"></span></div></td><td width=\"30\"><div align=\"left\"><span class=\"style2\"></span></div></td></tr>",
$strClass,trim($link),htmlspecialchars(trim($title)));
printf("<tr><td width=\"750\">%s</td></tr>",htmlspecialchars(trim($description)));

$title = "";
$description = "";
$link = "";
$insideitem = false;
  }
}

Thomas

Aceman87

Oh that looks so clean and sparkly, thank you Thomas !
I will try it right away and post how it turns out ^__

tsinka

I forgot one:

You shoud predefine $x at the beginning of the script:

$insideitem = false;
$tag = "";
$title = "";
$description = "";
$link = "";
$x = 0;

Thomas

Aceman87

Oh it WORKS!!! http://koti.mbnet.fi/nanime/parseri3.php
IT REALLY WORKS! 🙂

Thank you, thank you, thank you bow bow bow
After strugling with this for over 60 hours I feel like I could throw my self down and cry of happiness. I will never forget this ! 😃

tsinka

There's something wrong with the table. Not every row contains three tds. Additionally, I'd suggest to remove the div and span tags and change the td tags to e.g.

Thomas

Aceman87

If you mean the TD tags I think you mean, they are suposed to be there. The 2 additional TD´s are the file "size" and "info", when the first one is the name.

That idea about the DIV and SPAN tags is good though, thanks 😃

tsinka

I meant something else. If you look at the output of the parseri3.php script you'll see that there's no border around the last two cells in the rows below the head row. Looking at the source you'll see something like

....
<tr>
  <td>....</td>
  <td>....</td>
  <td>....</td>
</tr>
<tr>
  <td>....</td>
</tr>
<tr>
  <td>....</td>
  <td>....</td>
  <td>....</td>
</tr>
....

This isn't valid since the second row only contains one cell.

Thomas

Aceman87

Hmm, now I get what you mean. Those are the description fields below the headers. They are suposed to be there too, but just until the script is done.

The funny thing is, though I put them on their own TR and TD and told it to be 750 wide, it still just keeps its width at 620, like the name header TD on the TR above it 😕

tsinka

Ok, I got it 🙂

In that case you need to use colspan, example:

....
<tr>
  <td width="100">....</td>
  <td width="300">....</td>
  <td width="350">....</td>
</tr>
<tr>
  <td width="750" colspan="3">....</td>
</tr>
<tr>
  <td width="100">....</td>
  <td width="300">....</td>
  <td width="350">....</td>
</tr>
....

Always make sure to have the same number of cells in the rows of a table.

Thomas

Aceman87

Hmm, thnx. Weird, I have been using tables for ages and I didn't know/remember it :p