current community

your communities

more stack exchange communities

Stack Exchange
tour help
stack overflow careers
Take the 2-minute tour ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 0 down vote favorite

This code works:

$row = array(5,6,89,97,101);
$found = array();
$post = 89;
$count_row = count($row);
for($i = 0; $i < $count_row; $i++){
    if($row[$i]==$post){
        $found[] = $row[$i-2];
        $found[] = $row[$i-1];
        $found[] = $row[$i];
        $found[] = $row[$i+1];
        $found[] = $row[$i+2];
        var_dump($found);
    }
}

And this does not, probably doing something wrong in the mysql_fetch_array;

$found = array();
        $q = "SELECT object_id FROM wp_term_relationships WHERE term_taxonomy_id='$term_tax_id'";
        $rs = mysql_query($q) or die(mysql_error());
        $row = mysql_fetch_array($rs, MYSQL_NUM);
        $count_row = count($row);
        for($i = 0; $i < $count_row; $i++){
        print_r($row);
            if($row[$i]==$post){
                $found[] = $row[$i-2];
                $found[] = $row[$i-1];
                $found[] = $row[$i];
                $found[] = $row[$i+1];
                $found[] = $row[$i+2];
                var_dump($found);
            }
        }

Nothing is shown if post is more than 1. Anyone know a way to solve this problem?

share|improve this question
1  
Try print_r($row): what is the output? More: in second example I don't see $found and $post initialization... –  Marco Aug 29 '11 at 12:04
    
Define "does not work", please? –  deceze Aug 29 '11 at 12:06
    
@Marco Array ( [0] => 1 ) –  Muazam Aug 29 '11 at 12:06
    
@deceze Nothing is shown. –  Muazam Aug 29 '11 at 12:07
1  
So, your query is wrong or doesn't return what you're expecting I think... If you want us to help you, show us some code !!! –  Marco Aug 29 '11 at 12:09

2 Answers 2

up vote 1 down vote accepted

The select statement should be rewritten so as to return only the values you're looking for. One way to do this is with a UNION of two selects, one returning lesser object IDs and one returning greater. You should also make use of WordPress's wpdb class. For one thing, a site admin may change the table prefix from "wp_" to something else; $wpdb->term_relationships will give the correct name for the table.

$statement = $wpdb->prepare(
   "  (SELECT object_id 
        FROM $wpdb->term_relationships 
        WHERE term_taxonomy_id= %d 
          AND object_id <= %d
        ORDER BY object_id DESC 
        LIMIT 3)
    UNION
      (SELECT object_id 
        FROM $wpdb->term_relationships 
        WHERE term_taxonomy_id= %d
          AND object_id > %d
        ORDER BY object_id ASC 
        LIMIT 2)
      ORDER BY object_id", $term_tax_id, $post, $term_tax_id, $post);
$found = $wpdb->get_results($statement);

This also has the advantage of working when there are less than two terms relationships before or after the center (the object with id $post).

share|improve this answer
    
It seems to work, but I how do I loop them? Thanks! I tried echo $found[0]['object_id']; –  Muazam Aug 29 '11 at 13:22
    
No need for the loop to build the result array. wpdb::get_results returns all the results as an array. By default, the items in this array are objects. Set the second arguments of get_results to one of the constants mentioned in the documentation for a different item type. In this specific example, you could instead use wpdb::get_col –  outis Aug 29 '11 at 13:26
    
Thanks for helping, I appreciate it a lot! –  Muazam Aug 29 '11 at 13:57
up vote 4 down vote

mysql_fetch_array() fetches one single row, which, in your case, contains exactly one column.

You should either query all rows from the result set at once (which I don't know how that works), or you should do this:

$q = "SELECT object_id FROM wp_term_relationships WHERE term_taxonomy_id='$term_tax_id'";
$rs = mysql_query($q) or die(mysql_error());
$data = array();
while ($row = mysql_fetch_array($rs, MYSQL_NUM)) {
    $data[] = $row[0];
}
for($i = 2; $i < count($data) - 2; $i++){ // adjusted boundaries
    if($data[$i]==$post){
        $found[] = $data[$i-2];
        $found[] = $data[$i-1];
        $found[] = $data[$i];
        $found[] = $data[$i+1];
        $found[] = $data[$i+2];
        var_dump($found);
    }
}

}

I as well have adjusted the boundaries: if you are interested in the range $i-2 to $i+2, you only should run from 2 to end-2.

share|improve this answer
    
The code works almost like I want it. Nothing is shown if the post is less than 4. do you know how to fix that? –  Muazam Aug 29 '11 at 12:31
    
Instead of adjusting the boundaries, do a boundary check inside the loop so if you're at $i == 0 don't add $data[$i-2] and $data[$i-1] etc. –  jli Aug 29 '11 at 12:55

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.