problema foreach en un retun

[raid_]

hola, cuando meto un foreach en un retun me da este error
"Parse error: syntax error, unexpected T_FOREACH in"

estoy utilizando este codigo

Código PHP:

  function Print_price_fileds($cnt, $p = null) {
if ($p === null){
    $a = $b = $c = '';
}else{
    $a = $p['n'];
    $b = $p['d'];
    $c = $p['p'];
}
 
 
return '
<li>
    <label>Nr :</label> 
     <select name="price_data[$cnt][n] id="gender" >
 
            
            ' foreach (get_terms ( 'cat', $args ) as $tag)
            {
            $ciudad1 = get_option('price_data');;
    
            if( $ciudad1 == $tag->name   ) {
            echo '<option value="'. $tag->name .'"  selected="slelected"  >'.$tag->name.'</option>';
            } else {
            echo '<option value="'. $tag->name .'">'.$tag->name.'</option>';
            }
            } '
 
        
        </select>
    
</li>
 
    <label>Description :</label>
    <input type="text" name="price_data[$cnt][d]" size="50" value="$b"/>
 
    <label>Price :</label>
    <input type="text" name="price_data[$cnt][p]" size="20" value="$c"/>
    <span class="remove">Remove</span>
</li>
'
;
}
} 
   

y por mas que pruebo y miro no encuentro el error,

gracias

[Perr0]

tienes que tomarte la molestia de revisar tu código...
por ejemplo aquí:

Código PHP:

<select name="price_data[$cnt][n] id="gender" >
 

te fijas donde está el error ?
por mi parte no uso comilla simple ' prefiero la doble " y para escapar a ella \"
ademas que no te olvides de concatenar con puntos y los return los trabajo por lo general linea a linea, asi...

Código PHP:

$retu .= "<li>";
$retu .= "<label>Nr :</label>";
$retu .= "<select name=\"price_data[$cnt][n]\" id=\"gender\" >";
$retu .= .............
 
return $retu;
 

salu2

[raid_]

Gracias por la ayuda, y perdona mi torpeza, pero sigo teniendo el mismo problema

este es el codigo

Código PHP:

function Print_price_fileds($cnt, $p = null) {
if ($p === null){
    $a = $b = $c = '';
}else{
    $a = $p['n'];
    $b = $p['d'];
    $c = $p['p'];
}
   $retu .= "  <label>Nr :</label> ";
   $retu .= " <select name=\"price_data[$cnt][n]\" id=\"gender\" > ";
   $retu .= foreach (get_terms( "category", $args ) as $tag)
            {
            $practicas1 = esc_attr( get_the_author_meta( 'practicas', $user->ID ) );
    
            if( $practicas1 == $tag->name   ) {
            echo '<option value="'. $tag->name .'"  selected="slelected"  >'.$tag->name.'</option>';
            } else {
            echo '<option value="'. $tag->name .'">'.$tag->name.'</option>';
            }
            };
        $retu .= "</select>";
    $retu .= " </li>";
    $retu .= " <label>Description :</label>";
        $retu .= "  <input type=\"text\" name=\"price_data[$cnt][d]\" size=\"50\" value=\"$b\"/>";
 
return $retu;
}
 

de esta forma da error,
si pongo el foreach entre comillas, en el select sale el codigo, (mirando el codigo fuente), he probado a poner las comillas como dijiste, a las simples......
y siempre me da el mismo error unexpected T_FOREACH

Gracias

[pateketrueke]

1. Un bloque tipo if, for, etc. no es una cadena, por ende no la puedes ir concatenando
2. Lo que debes concatenar son los resultados (cadenas) generados dentro del bloque, algo así...

Código PHP:

  $foo = '';

$foo .= 'bar';

for (/* ... */) {
  $foo .= $candy;
}

$foo .= 'does'; 
   

[antoniopol]

Como bien dice pateketrueke, tienes concatenar en este caso, los echos de dentro del while así quedaria

Código PHP:

$retu .= "  <label>Nr :</label> ";
   $retu .= " <select name=\"price_data[$cnt][n]\" id=\"gender\" > ";
    foreach (get_terms( "category", $args ) as $tag)
            {
            $practicas1 = esc_attr( get_the_author_meta( 'practicas', $user->ID ) );
    
            if( $practicas1 == $tag->name   ) {
            $retu .= '<option value="'. $tag->name .'"  selected="slelected"  >'.$tag->name.'</option>';
            } else {
            $retu .= '<option value="'. $tag->name .'">'.$tag->name.'</option>';
            }
            };
        $retu .= "</select>";
    $retu .= " </li>";
    $retu .= " <label>Description :</label>";
        $retu .= "  <input type=\"text\" name=\"price_data[$cnt][d]\" size=\"50\" value=\"$b\"/>";
 
return $retu;
 

[raid_]

Muchisimas gracias por su ayuda, ya consegui solucionarlo son su ayuda.