search

acmfi / discord-bot

Discord bot developed during summer 2020 as a summer project :D
1 stars 0 forks source link

Issue 3 polls #26

Closed onmax closed 3 years ago

onmax commented 4 years ago

ANTES DE MERGEAR: Resolver esto

Ya está la estructura básica para las encuestas con la arquitectura que hemos decidido y con los comentarios. El código se puede ver en la issue 3.

@JustAntoRS Ya está listo para mergear. Creearé un nuevo issue con una lista de bugs y mejoras (como la de los gráficos). Pero esta primera versión está bastante completa para versión alpha del bot.

JustAntoRS commented 4 years ago

He visto que estas lanzando excepciones en varias partes del codigo:

  1. Tendria sentido crear nuestras propias excepciones? Igual es mas facil de hacer debug con excepciones propias
  2. No he visto que las trates (en los test si), si se lanza una de las excepciones en la ejecucion del bot que ocurre? Se para la ejecucion y es el administrado el que tiene que ver que ha ocurrido? O la ejecucion continua pero esa operacion falla?
JustAntoRS commented 4 years ago

Hecho en falta comentarios en las funciones, igual deberiamos usar algun sistema estandar de documentacion?

Yo he usado sphinx y a mi me gusta mucho, opiniones?

onmax commented 4 years ago

He visto que estas lanzando excepciones en varias partes del codigo:

  1. Tendria sentido crear nuestras propias excepciones? Igual es mas facil de hacer debug con excepciones propias
  2. No he visto que las trates (en los test si), si se lanza una de las excepciones en la ejecucion del bot que ocurre? Se para la ejecucion y es el administrado el que tiene que ver que ha ocurrido? O la ejecucion continua pero esa operacion falla?
  1. Creo que ahora he creado para todos los casos de error excepciones propias. Aunque seguro que se pueden añadir más en algún sitio
  2. Si una excepción ocurre es muy posible que sea culpa del usuario por lo que se le envía la ayuda del comando y la ejecución continua. Puede darse el caso de que salte una excepción ajena al usuario y este reciba la ayuda incluso habiendo invocado al comando correctamente, pero esto creo que puede ocurrir un porcentaje muy bajo de las veces.
JustAntoRS commented 4 years ago 
1. Creo que ahora he creado para todos los casos de error excepciones propias. Aunque seguro que se pueden añadir más en algún sitio

2. Si una excepción ocurre es muy posible que sea culpa del usuario por lo que se le envía la ayuda del comando y la ejecución continua. Puede darse el caso de que salte una excepción ajena al usuario y este reciba la ayuda incluso habiendo invocado al comando correctamente, pero esto creo que puede ocurrir un porcentaje muy bajo de las veces.

Okay, thanks por la explicacion, no habia visto que cuando una excepcion saltara se le enviara ayuda al usuario, parece correcto por mi lado.