Me estaba complicando, hasta que recordé que la función max()
acepta un parámetro key
:D
Yo te proporciono una opción fácil y sin módulos adicionales, esto usando la funcion antes mencionada max()
, pero antes que nada unos detalles con tu código
En primer lugar conviertes el valor obtenido a una lista.
def deteccion_secuencia(v,p,id):
...
for i in p:
secuencia_p1=list(i)
No es necesario convertir el valor obtenido (string) a una lista, pues un string es un iterable, podría considerarse correcto que lo hicieras si no sabes qeu valor te están pasando como parámetro, pero como al final si lo sabes y lo que quieres es iterar en un string no hay necesidad de hacer list(i)
, podría quedar asi:
def deteccion_secuencia(v,p,id):
deteccion=[]
for i in p:
for j in i:
if j not in v:
deteccion.append("Negativo")
elif j in v:
deteccion.append("Positivo")
El ciclo también se podría reducir utilizando funciones como map()
o reduce()
, pero en este caso ya no tocaré nada del ciclo :D.
Otra cosa totalmente inncesaria es tu otro ciclo for
en el que recorres la lista deteccion
donde al final lo unico que haces es escoger la palabra, ¿para que quieres eso? puedes acceder a un elemento de una lista mediante su índice, lo que hice fue eliminar ese ciclo (estorba).
Y ahora si viene una posible respuesta a tu pregunta.
....
count = max(deteccion, key=lambda x: deteccion.count(x))
print(f"el valor que mas se repite es: {count}")
utilizamos la funcion max()
que nos devuelve el máximo de un iterable, pero a esta función le pasamos, en su parámetro key
, una lambda
para decirle que escoja al máximo de acuerdo a la veces que se repita el elemento en la lista Y listo!
Podemos hacer un test para ver que el código funciona.
test = ['Negativo', 'Negativo', 'Positivo', 'Negativo', 'Negativo']
count = max(test, key=lambda x: test.count(x))
print(f"el valor que mas se repite es: {count}")
#salida-> el valor que mas se repite es: Negativo
test = ['Positivo', 'Positivo', 'Positivo', 'Positivo', 'Positivo']
count = max(test, key=lambda x: test.count(x))
print(f"el valor que mas se repite es: {count}")
#salida-> el valor que mas se repite es: Positivo
codigo resultante
def deteccion_secuencia(v,p,id):
#ciclo for para ver si cada caracteres de la cadena p (ej: id=["id_1","id_2"] p=(["i3a2f"],["v89e1"]) esta en la lista designada (ej: v=["v","8","9","e","1","a","4","7"])
deteccion=[]
for i in p:
for j in i:
if j not in v:
deteccion.append("Negativo")
elif j in v:
deteccion.append("Positivo")
result=[]
count = max(deteccion, key=lambda x: deteccion.count(x))
print(f"el valor que mas se repite es: {count}")
empaquetado=zip(id,deteccion)
result=dict(empaquetado)
return result
probamos
res = deteccion_secuencia(v=["v","8","9","e","1","a","4","7"],id=["id_1","id_2"],p=(["i3a2f"],["v89e1"]))
print(res)
resultado
el valor que mas se repite es: Negativo
{'id_1': 'Negativo', 'id_2': 'Negativo'}
La primera linea es el resultado del print(count)
que hicimos y la segunda es el diccionario que retorna la función. No entre en detalles si tu código funciona correctamente o no (para determinar la coincidencia), pero te puedo decir que al ojo se nota que puede tener un problema, pero me puedo equivocar :p
id
yp
con los dev
? o que? Y que le pasa a tu codigo? no funciona o que?id=["id_1","id_2"]
yp=["i3a2f","v89e1"]
son listas. Lo unico que me falta es que enfor k in range(0,len(deteccion),len(secuencia_p1)):
aparezca solo el elemento que más se repite para asignarlo en el diccionario final.p
se encuentra env
y me dicenegativo
. En cua to a obtener el que más se repite lo puedes hacer con 'count`