funcion facil no funciona :S

[mds90]

Gente aver si me pueden ayudar, ya que soy bastante nuevo en php... me hice una funcion para ver si existe caracter eligido dentro de string eligido (ya se que existe esa funcion, pero lo pago para practicar) este es el codigo:

Código PHP:

  function siElSimboloExiste ($string, $simbolo){
    
            for($i = 0; $i <= 100; $i ++){
        
                if($string{$i} == $simbolo){
                    return true;
                } 
 
                if($string{$i} != $simbolo){        
                    return false;
                }
            }
} 
   

gracias :)

[Sourcegeek]

1- En el for, por qué lo haces hasta 100? Qué tal si el string que le pasarás es de 20 caracteres o de 200? En el primer caso, será una cuenta innecesaria. En el segundo, faltarán caracteres... Entonces, usa mejor strlen checando la longitud del string
2- Por qué usas $string{$i} en lugar de $string[$i]??? No confundas corchetes con llaves
3- Por qué usas un segundo if en lugar de else/elseif?

Necesitas aprender más sobre PHP amigo..

Saludos!

[s00rk]

Ademas de lo que ya te comentaron en cuanto no encuentra el caracter regresa un false lo cual esta mal deves hacer que al terminar el for si continua que ahi regrese el false

ejemplo:

envio palabra y simbolo
...
hago el for hasta el total de caracteres que contiene palabra dentro del for
busco por posicion el simbolo si es == regreso true, en ese instante sino continuo, termina for y como no lo encontro regreso false.

Y ya ahora si tendrias una funcion para buscar un caracter en una cadena.

[Sourcegeek]

Ah, cierto, algo como:

Código PHP:

  function siElSimboloExiste ($string, $simbolo){
            $foo = false;
            for($i = 0; $i <= 100; $i ++){
        
                if($string[$i] == $simbolo){
                    $foo = true;
                }
            }
            return $foo;
} 
   

Claro, corrige lo que te dije anteriormente

[mds90]

gracias a todos, ya me funciono y ya encontre mis errores. Sourcegeek ya se que tengo que aprender mas de PHP para eso estoy practicando ;)
lo del for ya se que no es correcto hacer 100 iteraciones cuando no es nesesario, lo que pasa que este codigo lo hice en 1 min, y no pensaba en esos detalles. Lo de corchetes no es confundirme, es que para caracretes dentro del string uso llaves, y para array uso corchetes

[GrupoC]

¿Y porqué limitas a 100 los caracteres del string? Sería strlen($string).

Saludos!

[IEKK]

Es cierto que estás aprendiendo. Sería bueno que le echaras un ojo al manual de php. Sólo hay que buscar manual php en google.

En la parte de manejos de strings tienes la función strpos
Por tanto... es innecesario hacer un bucle en la cadena para saber si contiene un caracter.

Código PHP:

  echo buscarChar('Este es un ejemplo', 'w'); // false
echo buscarChar('Este es un ejemplo', 'e'); // true
 
function buscarChar($str, $char)
{
strpos($str, $char)? $value='el caracter '.$char.' existe': $value='el caracter '.$char.' NO existe';
return $value.'<br>';
} 
   

[Sourcegeek]

GrupoC: Ya le sugerí eso
IEKK:

(ya se que existe esa funcion, pero lo hago para practicar)

[IEKK]

Iniciado por Sourcegeek

GrupoC: Ya le sugerí eso
IEKK:

Si te fijas no indica que se refiriese al strpos por eso dije que evitase los bucles . Ahora bien a lo mejor era los for lo que practicaba yo supuse que eran las funciones pasando parametros y su correspondiente return