Usar dos for anidados es inecesario y muy ineficiente, con un solo for e in
hacemos lo mismo pero con menos recursos y más 'pythónico':
texto=raw_input("ingrese un texto: ")
letra=raw_input("ingrese una letra: ")
contador = 0
for palabra in texto.split():
if letra in palabra:
contador += 1
print contador
Otra solución algo más eficiente y con menos código es usar un generador y la función preconstruida sum
:
texto=raw_input("ingrese un texto: ")
letra=raw_input("ingrese una letra: ")
print(sum(letra in palabra for palabra in texto.split()))
La comparativa usando un texto de 80000000 palabras (en mi caso particular y bajo Python 2.7):
2 ciclos for e igualdad: 49.6679999828 segundos.
sum y generador: 16.3229999542 segundos.
1 ciclo for e in: 13.4120001793 segundos.
P.D : Tu código falla por el orden de evaluación de tu sentencia if
, tu línea es:
if letra in caracter==True:
Podría parecer que lo que hace esto es evaluar letra in caracter
(que retorna False
o True
) y luego mira si el resultado es True
. Pero el problema es que esto no es lo que pasa, primero se evalua la igualdad (==
y luego el in
). Imaginemos lo siguiente:
if 'a' in 'a' == True:
print 'Si'
Si lo ejecutamos no imprime nada, el problema es que primero se hace:
'a' == True
Esto lógicamente es False
, el resto ya no tiene sentido por lo que el if
retorna False
. Para solucionarlo podemos añadir un paréntesis para que evalue en el orden que queremos:
if ('a' in 'a') == True:
print 'Si'
En tu caso sería:
if (letra in caracter) == True:
contador=contador+1
Pero esto es redundancia, en Python bastaría:
if letra in caracter:
contador=contador+1
Pero recuerda que esto contaría todas las letras iguales a la introducida presentes en la frase y no las palabras que contienen la letra, la solución es eliminar el segundo for
como te comenté anteriormente.