0

estaba realizando este ejercicio, sin embargo, al ejecutarlo no me aparece el numero de palabras que contienen tal letra, sólo aparece un cero, ¿ en que estoy mal? De antemano muchas gracias

texto=raw_input("ingrese un texto: ")
letra=raw_input("ingrese una letra: ")
contador=0
textoLista=texto.split()
for palabra in textoLista:
    for caracter in palabra:
        if letra in caracter==True:
            contador=contador+1
print contador
0

2 respuestas 2

1

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.

7
  • wow muchisimas gracias amigo. Tu respuesta está muy completa y precisamente es lo que buscaba: un programa que utilizara funciones básicas . Muchas gracias por haberte dado el tiempo de responderme y a la vez explicarme el error de mi programa. Ahora sí que se despejaron todas mis dudas y a la vez, se respondieron todas mis necesidades, muchas gracias y saludos!! Commented el 13 feb. 2017 a las 20:42
  • pero sabes, me quedó una duda. Por qué sólo basta usar un solo for-in?? Se supone que al utilizar solo un for-in, se recorrería solamente las palabras y no las letras presentes en cada palabra, no entiendoo :( , ¿ explicame por favor? Por cierto, probé tu programa y funciona perfectamente, pero es que me quedó dando vueltas esa duda del for-in. Saludos Commented el 13 feb. 2017 a las 20:51
  • @WhiteMamba como bien dices con un solo for iteras sobre las palabras. Lo que ocurre realmente es que la claúsulain actua como un segundo for y es ella la que itera (eficientemente) sobre cada palabra. Esto es muy genérico, la realidad interna es más compleja pero básicamente a in b lo que hace es comprobar si a es miembro del objeto b. En el caso de cadenas retorna True si y solo si a es una subcadena de b. Coloquialmente podemos decir que in lo que hace es preguntar : ¿Está a dentro de b?
    – FJSevilla
    Commented el 13 feb. 2017 a las 21:31
  • @WhiteMamba no se si te habré dejado medianamente claro como funciona, te dejo la documentación oficial sobre el operador inen el siguiente enlace por si quieres echarle un ojo.
    – FJSevilla
    Commented el 13 feb. 2017 a las 21:41
  • Dejame ver si entendí, el primer for-in permite iterar sobre las palabras del texto, luego el in que incluyes dentro del ciclo permite iterar sobre las LETRAS de cada palabra ? O en ningun momento se itera sobre las letras?? Jajaja parece que me confundí Commented el 13 feb. 2017 a las 23:05
1

Tu último if está mal formado porque mezclas la sintaxis del for in con la sintaxis del if, deberías comparar si palabra y caracter son iguales, así:

texto=raw_input("ingrese un texto: ")
letra=raw_input("ingrese una letra: ")

contador=0
textoLista=texto.split()

for palabra in textoLista:
    for caracter in palabra:
        if letra == caracter:
            contador=contador+1
            break
print contador

Nota que la línea que cambia es esta:

if letra == caracter:

Y añadimos un break para que abandone la palabra si ya hemos encontrado la letra que buscábamos en ella.

Ahora, como mencionas en los comentarios, si deseas evitar el uso de un break, entonces deberás usar una variable auxiliar, así (nota la variable encontrado):

texto=raw_input("ingrese un texto: ")
letra=raw_input("ingrese una letra: ")

contador=0
textoLista=texto.split()

for palabra in textoLista:
    encontrado = False
    for caracter in palabra:
        if (letra == caracter and encontrado == False):
            contador=contador+1
            encontrado = True

print contador

Sin embargo, no recomiendo esta segunda solución porque es más lenta. Tiene que recorrer todas las letras de cada palabra aunque ya hayas encontrado lo que buscas.

6
  • muchas gracias por tu tiempo y ayuda, ese break cumple con el objetivo de mi programa :D . Sin embargo, tengo dos preguntas mal. ¿ Por qué mi if está mal ? Por qué mi contador no cuenta la cantidad de palabras que contienen la letra elegida?? Mi otra pregunta es: ¿ existe una alternativa que no utilice el break como funcion?? Digamos, que utilice funciones más basicas? Es que a veces no me permiten usar ciertas funciones ni metodos :( Commented el 13 feb. 2017 a las 2:31
  • 1
    Fue sin querer @Mariano, tuve que eliminar una de las dos. Commented el 13 feb. 2017 a las 2:35
  • @WhiteMamba ya he puesto la razón por la que tu if estaba malo, déjame ver si encuentro una forma de no usar break y lo agrego a la respuesta. Aunque ahora mismo no se me ocurre nada más básico que un break, eso está bastante cerca del lenguaje máquina. Commented el 13 feb. 2017 a las 2:40
  • vale muchas gracias, estare atento a tu respuesta Commented el 13 feb. 2017 a las 2:46
  • 1
    muchas gracias, me sirvio muchisimo tu segunda solucion . Saludos y gracias por dedicarle tiempo a mi consulta :D Commented el 13 feb. 2017 a las 3:08

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.