Ayuda! Arithmetic Formatter, final

1

Hola!
No sé si hay mucha gente de habla hispana por acá, pero hago el intento.
Estoy teniendo problemas con el primer proyecto.

Este es mi código.

def arithmetic_arranger(problems,est = False):
    # Variables a usar
    todos = []  # todos los listados
    resultados = []
    primero = []
    segundo = []
    operador = []
    longitud = []
    relleno = " "
    check2 = "Error: Operator must be '+' or '-'."
    # check1 :5 problemas
    if len(problems) > 5:
        # máximo de 5 problemas
        return "Error: Too many problems."

    for k in problems:
        # separar por espacio y agrego a una lista con valores
        oper = k.split(' ')
        primero.append(oper[0])
        segundo.append(oper[2])
        operador.append(oper[1])
    #agrego a un conjunto
    def conjunto(listado):
        return todos.append(listado)
    # obtengo todos los primeros términos de cada operación
    conjunto(primero)
    conjunto(operador)
    conjunto(segundo)
    # print(todos)
    toditos = list(zip(primero, operador, segundo))
    # print(toditos)
    for i in operador:
        # check2: operadores + o -
        if i == "+":
            pass
        elif i == "-":
            pass
        else:
            #arranged_problems = print(check2)
            #print("Error: Operator must be '+' or '-'.")
            return "Error: Operator must be '+' or '-'."
    # check4: hasta 4 digitos
    def largo (entrada):
        if len(entrada) > 4:
            #arranged_problems = print('Error: Numbers cannot be more than four digits.')
            return 'Error: Numbers cannot be more than four digits.'
    for i in toditos:
        # check4: hasta 4 digitos
        if len(i[0]) > 4:
            #arranged_problems = print('Error: Numbers cannot be more than four digits.')
            #print('Error: Numbers cannot be more than four digits.')
            return 'Error: Numbers cannot be more than four digits.'
        if len(i[2]) > 4:
            #arranged_problems = print('Error: Numbers cannot be more than four digits.')
            #print('Error: Numbers cannot be more than four digits.')
            return 'Error: Numbers cannot be more than four digits.'
        #check enteros a cada valores
        try:
            a = int(i[0])
            b = int(i[2])
        except:
            #arranged_problems = print('Error: Numbers must only contain digits.')
            #exit()
            return 'Error: Numbers must only contain digits.'
        # check2: operadores + o - y hace la operatoria
        if i[1] == '+':
            suma = a+b
            resultados.append(suma)
            # print(f'Sumando: {suma}')
        elif i[1] == '-':
            resta = a-b
            resultados.append(resta)
            #print(f'Restando: {resta}')
        else:
            #arranged_problems = print(check2)
            return "Error: Operator must be '+' or '-'."
        longitud.append(max(len(i[0]), len(i[2])))
    # print(resultados)
    toditos = list(zip(primero, operador, segundo, resultados, longitud))
    conjunto(resultados)
    conjunto(longitud)

    reng1 = ""  # primer numero
    reng2 = ""  # simbolo mas segundo numero
    divisor = ""  # lineas horizontales
    reng3 = ""  # resultados
    for i in toditos:
        # componentes: (primero,operador, segundo,resultados,longitud)
        reng1 += "  " + i[0].rjust(i[4], relleno) + "    "
        reng2 += i[1] + " " + i[2].rjust(i[4], relleno) + "    "
        divisor += "-" * len("  " + i[0].rjust(i[4], relleno)) + "    "
        reng3 += str(i[3]).rjust(len("-" * len("  " + i[0].rjust(i[4], relleno))), relleno) + "    "
    if est == True:
        #arranged_problems = print(f'{reng1.rstrip()}\n{reng2.rstrip()}\n{divisor.rstrip()}\n{reng3.rstrip()}')
        arranged_problems = print(reng1.rstrip()+"\n"+reng2.rstrip()+"\n"+divisor.rstrip()+"\n"+reng3.rstrip())
    else:
        #arranged_problems = print(f'{reng1.rstrip()}\n{reng2.rstrip()}\n{divisor.rstrip()}')
        arranged_problems = print(reng1.rstrip() + "\n" + reng2.rstrip() + "\n" + divisor.rstrip())
    return arranged_problems

Y estos son los errores:

======================================================================
FAIL: test_arrangement (test_module.UnitTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/boilerplate-arithmetic-formatter-3/test_module.py", line 10, in test_arrangement
    self.assertEqual(actual, expected, 'Expected different output when calling "arithmetic_arranger()" with ["3 + 855", "3801 - 2", "45 + 43", "123 + 49"]')
AssertionError: None != '    3      3801      45      123\n+ 855 [56 chars]----' : Expected different output when calling "arithmetic_arranger()" with ["3 + 855", "3801 - 2", "45 + 43", "123 + 49"]

======================================================================
FAIL: test_solutions (test_module.UnitTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/boilerplate-arithmetic-formatter-3/test_module.py", line 39, in test_solutions
    self.assertEqual(actual, expected, 'Expected solutions to be correctly displayed in output when calling "arithmetic_arranger()" with arithemetic problems and a second argument of `True`.')
AssertionError: None != '   32         1      45      123\n- 698 [90 chars] 172' : Expected solutions to be correctly displayed in output when calling "arithmetic_arranger()" with arithemetic problems and a second argument of `True`.

----------------------------------------------------------------------

No entiendo cuál es el error
Ya pasé por varias revisiones que fui solucionando con este foro, pero aquí quedo trabado
Busqué en las líneas que indica, pero no logro descrifrar. Supongo que debe ser algo a la hora de armar el formato.

1 Like
2

print() no es necessario en estes codigos. print() revuelva None a arranged_problems aqui. Cuando los remuevo, el programa funciona perfectamente y se pasa las pruebas.

Lo siento por el espanol horrible. Cuando estudiaba en la universidad, solamente hablabamos sobre ordenadores de vez en cuando.

2 Likes
3

muchas gracias!
Probé eso y funcionó perfecto. Lo hice para todos los ítems anteriores, pero no para ese. Es decir, todos los check basados en las reglas, pero no para el resultado final.
tu español está muy bien. igual que mi inglés: se logra entender muy bien!

again, in english.
thank you very much! i write this originally in spanish to test my luck. And it worked!
my english is not perfect, so…

getting to the point, I tried that way and it worked! I had fixed that for all checks based on the rules, but not on the result.

4

Hola saludos, he notado algunos detalles en su codigo,
1- La validación del mensaje : " Error: Too many problems.", se refiere a que usted
debe estar contabilizando los numeros de errores que se producen durante la ejecución de la función, más no de los problemas aritmeticos que se le ingresan como argumentos.
2- La validación de las operaciones aritmeticas + y - puede realizarlas en un solo if
con el operador logico (or)
3- La validación de la longitud debe ser menor e igual a 4, usted esta excluyendo longitud 4 y puede validarlos en un mismo if empleando el operador lógico (and)

Saludos desde México !!!

5

Buenas,

Pues diría que el error de “Error:Too many problems” si que se refiere a las operaciones, las trata como “problems” o problemas matemáticos a resolver. Y es más, te lo indica en el “README” que como máximo se le pueden introducir 5 problemas a resolver.

Solo era comentar esto, por si alguien lo lee y se confunde luego.

Un saludo.