Skip to main content
se añadieron 66 caracteres en el cuerpo
Origen Enlace
ArianJM
  • 3.4k
  • 10
  • 22

Según el nombre de la función, y los comentarios que hay en el código, lo que hace la función es decirte si el número pasado es una potencia de 2 o no.

En tu código, hay algunos errores de lógica. En tu while, por ejemplo, tienes:

while n % 2 == 0:
    n = 0
    break

Eso lo único que hace es que si el número es par (n % 2 == 0), entra en el bucle y asigna n = 0. Luego el break termina el bucle, esto en sí no te sirve de mucho para saber si el número es una potencia de 2. De hecho, sin el break sería bucle infinito porque 0 % 2 == 0 es True.

En tu while quieres:

while n % 2 == 0:
    n = n / 2

Ahora, mientras el número sea par, lo dividirá en 2, y guardará el resultado, si sigue siendo par, lo divide en dos, etc. Hasta que sea impar.

Lo demás estaría bien, aunque el final puedes reducirlo, y quitar los if, si pones: return n == 1


Dicho esto, la función no hace lo que tú dices que debería hacer:

estoy desarrollando un ejercicio en el que hay una funcion que lee un número y si este es divisible entre 2 devuelve como resultado "Positivo"

Para hacer eso, solo necesitas comprobar lo que tienes en el while, y devolver "Positivo" en caso de ser par:

def es_pardivisible_entre_dos(n):
    returnif n % 2 == 0:
        return 'Positivo'

Según el nombre de la función, y los comentarios que hay en el código, lo que hace la función es decirte si el número pasado es una potencia de 2 o no.

En tu código, hay algunos errores de lógica. En tu while, por ejemplo, tienes:

while n % 2 == 0:
    n = 0
    break

Eso lo único que hace es que si el número es par (n % 2 == 0), entra en el bucle y asigna n = 0. Luego el break termina el bucle, esto en sí no te sirve de mucho para saber si el número es una potencia de 2. De hecho, sin el break sería bucle infinito porque 0 % 2 == 0 es True.

En tu while quieres:

while n % 2 == 0:
    n = n / 2

Ahora, mientras el número sea par, lo dividirá en 2, y guardará el resultado, si sigue siendo par, lo divide en dos, etc. Hasta que sea impar.

Lo demás estaría bien, aunque el final puedes reducirlo, y quitar los if, si pones: return n == 1


Dicho esto, la función no hace lo que tú dices que debería hacer:

estoy desarrollando un ejercicio en el que hay una funcion que lee un número y si este es divisible entre 2 devuelve como resultado "Positivo"

Para hacer eso, solo necesitas comprobar lo que tienes en el while:

def es_par(n):
    return n % 2 == 0

Según el nombre de la función, y los comentarios que hay en el código, lo que hace la función es decirte si el número pasado es una potencia de 2 o no.

En tu código, hay algunos errores de lógica. En tu while, por ejemplo, tienes:

while n % 2 == 0:
    n = 0
    break

Eso lo único que hace es que si el número es par (n % 2 == 0), entra en el bucle y asigna n = 0. Luego el break termina el bucle, esto en sí no te sirve de mucho para saber si el número es una potencia de 2. De hecho, sin el break sería bucle infinito porque 0 % 2 == 0 es True.

En tu while quieres:

while n % 2 == 0:
    n = n / 2

Ahora, mientras el número sea par, lo dividirá en 2, y guardará el resultado, si sigue siendo par, lo divide en dos, etc. Hasta que sea impar.

Lo demás estaría bien, aunque el final puedes reducirlo, y quitar los if, si pones: return n == 1


Dicho esto, la función no hace lo que tú dices que debería hacer:

estoy desarrollando un ejercicio en el que hay una funcion que lee un número y si este es divisible entre 2 devuelve como resultado "Positivo"

Para hacer eso, solo necesitas comprobar lo que tienes en el while, y devolver "Positivo" en caso de ser par:

def divisible_entre_dos(n):
    if n % 2 == 0:
        return 'Positivo'
Origen Enlace
ArianJM
  • 3.4k
  • 10
  • 22

Según el nombre de la función, y los comentarios que hay en el código, lo que hace la función es decirte si el número pasado es una potencia de 2 o no.

En tu código, hay algunos errores de lógica. En tu while, por ejemplo, tienes:

while n % 2 == 0:
    n = 0
    break

Eso lo único que hace es que si el número es par (n % 2 == 0), entra en el bucle y asigna n = 0. Luego el break termina el bucle, esto en sí no te sirve de mucho para saber si el número es una potencia de 2. De hecho, sin el break sería bucle infinito porque 0 % 2 == 0 es True.

En tu while quieres:

while n % 2 == 0:
    n = n / 2

Ahora, mientras el número sea par, lo dividirá en 2, y guardará el resultado, si sigue siendo par, lo divide en dos, etc. Hasta que sea impar.

Lo demás estaría bien, aunque el final puedes reducirlo, y quitar los if, si pones: return n == 1


Dicho esto, la función no hace lo que tú dices que debería hacer:

estoy desarrollando un ejercicio en el que hay una funcion que lee un número y si este es divisible entre 2 devuelve como resultado "Positivo"

Para hacer eso, solo necesitas comprobar lo que tienes en el while:

def es_par(n):
    return n % 2 == 0