Another Challenge...

DeadlySin3

Well, for me anyway.

As i've mentioned before, i'm writing a Tag Board script. What i'd like to do, is for every other post, I want to add a table around it, and highlight it to a light gray/silver - i'm unsure how to even do this. Any links to resources or code snippets are greatly appreciated!

Btw.. I'm using a flat .txt file for data storage, not a database, if that makes any difference at all

edit: i'm thinking I could use a counter that counts up to two, then after the second execution, it resets back to one. How can this be done?

DeadlySin3

This was explained to me:

Just use a counter as you loop through the records. If the counter is even, display one bgColor, if the counter is odd, display a different one.
For example, if you're using a for-loop to display the table rows, and the for-loop counter is $i, you could say:

<tr bgColor="<?=($i%2)?"#ffffff":"#efefef";?>">

In English that's saying, if the remainder of $i divided by 2 equals 1, make the bgColor white, otherwise, make it grey.
That's usually how I do it.

It does make sense to me.. somewhat. I'm just unsure how to play with for loops - i can make a basic counter using them, but other then that, i'm lost. Can anyone else re-iterate this in a more understandable fashion? :x

ednark

yes... however % is kinda expensive for what you are using...

you just want some sort of toggle or on-off switch... on for white off for grey

try this

    $toggle = TRUE;
    for {
        $toggle = !$toggle;
    }

here is an example

<table>
<?php
$toggle = TRUE;
while( 1 ) {
     $color = ( $toggle ) ? "#DDDDDD" : "#EEEEEE";
    print <<<EOS 
        <tr bgcolor="{$color}"><td>ROW</td></tr>
EOS;
    $toggle = !$toggle;
}
?>
</table>

DeadlySin3

hmm.. i'm getting a parse error when I do that. Trying the other method, w/the %... It was working, but only printing one color, instead of alternating. 🙁

Here's what i'm doing :

$fp = fopen ("$filename", "w+"); // opens $filename for reading & writing
        for ( $i=0; $i<=1; $i++ ) {
        // Displays alternate table row colors 

    $bg1 = "#efefef"; // color one     
    $bg2 = "#ffffff"; // color two 
    $table1 = "<tr></tr><td bgcolor=\"$bg1\">";
    $table2 = "<tr></tr><td bgcolor=\"$bg2\">";
    $endtab = "</td></tr>";
    }
    if ( $i%2 ) {
    $tags ="$table1<span class=\"tagit\"><b><a href=\"$url\" target=\"_blank\" class=\"tag\" onMouseover=\"showtip(this,event,'Posted: $tagdate')\" onMouseout=\"hidetip()\">$name</a></b>: $comment</span><br>$endtab\r\n$oldtags";
    }
    else {
    $tags ="$table2<span class=\"tagit\"><b><a href=\"$url\" target=\"_blank\" class=\"tag\" onMouseover=\"showtip(this,event,'Posted: $tagdate')\" onMouseout=\"hidetip()\">$name</a></b>: $comment</span><br>$endtab\r\n$oldtags";
    }
    flock( $fp, "2" );
fwrite ( $fp, $tags); // writes post to $filename
    flock( $fp, "3" ); // Unlocks File for all processes again.
fclose ( $fp );
header("Location: tagit.php");

This displays one color - it does not alternate between the two. I've been up for over 36 hours; thats probably why i'mhaving so much trouble w/this. btw... the opening <table> and closing </table> tags are echoed in the file that reads the $filename. Thats probably irrelevant, but eh. thought i'd add it.

ednark

     for ( $i=0; $i<=1; $i++ ) {
        // Displays alternate table row colors 
        $bg1 = "#efefef"; // color one     

        $bg2 = "#ffffff"; // color two 
        $table1 = "<tr></tr><td bgcolor=\"$bg1\">";
        $table2 = "<tr></tr><td bgcolor=\"$bg2\">";
        $endtab = "</td></tr>";
     }

if ( $i%2 ) {
    .... $table1 code ...
} else {
    .... $table2 code ...
}

1) your for loop.... your if $i%2 is not inside your for loop.... so i don't see how this would work

2) your <tr></tr><td> doesn't make sense

3) you are setting tag each time... so every loop... if were happening.. you would obverwrite your old data you want to use

$tags .= not $tags =

4) you can use the more friendly FLOCK_EX and FLOCK_UN syntax... don't have to but you should not use the string rep of "2" at least use 2

flock( $fp, FLOCK_EX );

my code:

there was a small type-o in my code: an extra space after the EOS that was invisible, that and my while(1) were lethal....

here is a correct version of my code that i actually tested

<table>
<?php
    $toggle = TRUE;
    for ( $i=0; $i<10; $i++ ) {
        $color = ( $toggle ) ? "#efefef" : "#ffffff";
        print <<<EOS
            <tr bgcolor="{$color}"><td>ROW {$i} </td></tr>
EOS;
        $toggle = !$toggle;
    }
?>
</table>

your code:

here is what i believe to be a fixed version of your code useing your method:


<table>
<?php

$url     = 'http://fakeurl';
$tagdate = '3000-02-03';
$name    = 'ednark';
$comment = 'hope this works';
$oldtags = '<tr><td>OLD TAGS</td></tr>';

$bg1 = "#efefef"; // color one     

$bg2 = "#ffffff"; // color two 

//$fp = fopen ("$filename", "w+"); // opens $filename for reading & writing

for ( $i=0; $i<10; $i++ ) {
    $bg = ($i%2) ? $bg1 : $bg2;
    $tags .= <<<END_TAGS
        <tr>
            <td bgcolor="{$bg}">
                <span class="tagit">
                    <b><a href="{$url}" target="_blank" class="tag" onMouseover="showtip(this,event,'Posted: {$tagdate}')" onMouseout="hidetip()">{$name} $i</a></b>: {$comment}
                </span>
                <br>
            </td>
        </tr>
        {$oldtags}
END_TAGS;
}

echo $tags;
//flock( $fp, "2" );
//fwrite ( $fp, $tags); // writes post to $filename
//flock( $fp, "3" ); // Unlocks File for all processes again.
//fclose ( $fp );

?>
</table>

DeadlySin3

o.O

thanks for the reply!

I know the way i had the table rows was wrong. but thats the only way it would display properly before.

I pretty much have it working the way I want - I'm coming up with a can't send headers, headers already sent error, but i'll figure that one out when i'm actually awake!

Thanks again!