Grupo tp1g02

Requisitos

• 01_menus 0.8
• 02_autenticacao 0
• 03_administrador 0.3333333333333333
• 04_jogador 0.5
• 05_testes 0
• 06_doc 0.3333333333333333
• 07_modularidade_legibilidade 1

Avaliação

0.42

Comentários

• Avaliado com a última versão do Domingo
• Há um problema relacionado com acentos no dicionário
• Não esquecer de tirar os acentos que estão no dicionário!
• Ao executar "import dicionario" há código que está a correr, e não devia, e o código rebenta
• Não há autenticação
• Falta a possibilidade de voltar atrás nos menus
• Ao começar um jogo, há um erro
• Continuar um jogo funciona mas tem que ser melhorado
• Parece que simplesmente imprime o ficheiro mas com uma linha de intervalo entre cada linha
• Modificação da password tem um bug
• Apagar utilizador tem um bug
• Testes não correm
• Não se podem fazer testes sobre os menus (ver mensagem que eu coloquei sobre isso)
• Há testes cujas funções não começam por test_ e por isso não são corridos:
  • criar_novo_jog
  • delete_jog
• Testes que não fazem sentido:
  • test_menu
  • test_admin
  • test_comeca_jogo
  • test_autentica_jog
• Testes que estão mal escritos:
  • test_escolher_opcao
  • test_ler_jogo
  • test_mudar_password
  • test_admin
  • test_comeca_jogo
  • test_autentica_jog
• Documentação
  • p = str em dicionario
  • Problema frequente:
    • os tipos dos argumentos estão sempre a ser mostrados utilizando a sintaxe errada
    • ver a aula sobre a documentação
    • ver o exemplo da função adiciona_utilizador em https://github.com/rcm/CP2021_Guiao02/blob/main/perfis.py
  • escrever_highscores: scores é lista de quê?
  • Documentação mal escrita:
    • jogo.py: incompleto, e a forma de documentar os parâmetros está errada
    • main.py: não tem documentação
    • menus.py: documentação completa mas mal escrita
    • PERFIS.py: tirando a função adiciona_utilizador (documentada por mim), a sintaxe está errada

Grupo tp1g04

Requisitos

• 01_menus 1
• 02_autenticacao 1
• 03_administrador 0
• 04_jogador 1
• 05_testes 0.2
• 06_doc 0.9
• 07_modularidade_legibilidade 1

Avaliação

0.73

Comentários

• Ao executar o programa através do menus.py:
  • Imprime um jogo qualquer antes dos menus
  • Entrar como administrador é igual a entrar como jogador!
  • Entrar como jogador pede também o nome!
• Funções com nome errado:
  • logon: devia ser register ou create_player
• Ficheiros que não deviam existir:
  • deveriam estar no mesmo ficheiro
    • Leitura_game1.py
    • Leitura_game2.py
• Ficheiros com código que corre ao ser importados:
  • continuar_jogo.py
  • DICIONARIO.py
• Documentação
  • Devia estar no mesmo nível de indentação que o resto do código
  • Tipo devolvido na função ler_jogo1 errado, devia ser list[list[str]]
  • logon: devolve true ou false quando acontece o quê?
  • fileToDic: valor devolvido não é uma string!
  • login: mesmo problema que logon
  • newPlayer: não devolve uma string e a descrição está incorreta
  • newPlayer e modifyPassword: descrição mal escrita, devia ser mais parecida com deleteUser
• Testes
  • Ao correr os testes é impresso o conteúdo de um ficheiro
  • Falha em 4 dos 8 testes
  • test_printScores não faz sentido
  • Outros 3 testes incompletos/não funcionam

Grupo tp1g06

Requisitos

• 01_menus 0.8
• 02_autenticacao 0
• 03_administrador 0
• 04_jogador 0
• 05_testes 0
• 06_doc 0.6428571428571429
• 07_modularidade_legibilidade 1

Avaliação

0.35

Comentários

• Não dá para voltar atrás nos menus
• Não há autenticação!
• Nenhuma das opções do administrador funciona
• Nenhuma das opções do jogador funciona
• Documentação:
  • falta documentação:
    • main.py
    • sopa_de_letras.py
  • tipo de return errado
    • ler_dicionario
    • tem_palavra
    • get_nome
    • get_password
    • ler_perfis: a descrição também está errada
  • Há muitos casos em que o nome do argumento está capitalizado! Por exemplo: Id em vez de id
  • falta descrever o que devolve
    • ler_highscores
  • tipo de argumentos errados / descrição dos argumentos errada
    • tem_palavra
    • escrever_highscores
    • adiciona_score
    • tem_password: dicionario não é uma string
• Não foram feitos testes nesta etapa

Grupo tp1g11

Requisitos

• 01_menus 0.8
• 02_autenticacao 0.8
• 03_administrador 0.5333333333333333
• 04_jogador 0.8
• 05_testes 0
• 06_doc 0
• 07_modularidade_legibilidade 0.5

Avaliação

0.49

Comentários

• Eu sei que vocês trabalharam
• Mas não se consegue avaliar o último commit do Domingo
• Problema grave:
  • muitos ficheiros com inúmeras versões, por exemplo:
    • existem 3 versões dos menus
    • existem talvez 3 versões dos highscores
  • não se consegue avaliar porque não se percebe qual é a versão correta
• Há código que será corrido ao importar ficheiros
• A maioria do código não está documentada
• Não há testes

Grupo tp1g13

Requisitos

• 01_menus 0
• 02_autenticacao 0
• 03_administrador 0
• 04_jogador 0
• 05_testes 0
• 06_doc 0.8571428571428571
• 07_modularidade_legibilidade 1

Avaliação

0.27

Comentários

• Fatam algumas coisas para fazer a avaliação das funcionalidades:
  • Não há perfis
  • Há pelo menos uma função que tem o nome errado: foi definida com o nome utilizador mas usada com o nome pede_utilizador
  • Há um erro na função ler_perfis na linha 166: o split devia receber como argumento ":" e não " : "
• Documentação
  • Devia existir um espaço entre o nome do argumento e o tipo
  • Ficheiros não documentados:
    • main.py
  • Falta documentar argumentos:
    • autenticar
  • Tipo devolvido errado:
    • criar_novo_utilizador
    • modificar_password_utilizador
    • jogar
    • check_password
• Testes não correm e estão extremamente incompletos

Grupo tp1g14

Requisitos

• 01_menus 0.8
• 02_autenticacao 0.0
• 03_administrador 0.0
• 04_jogador 0.0
• 05_testes 0.0
• 06_doc 0.0
• 07_modularidade_legibilidade 0.7

Avaliação

0.21

Comentários

• Não dá para voltar para trás nos menus
• Não há autenticação
• Nada funciona no administrador
• Nada funciona no jogador
• Há código fora das funções:
  • continuacao_jogos.py
  • Dicionario.py
  • Palavras_in_fdic.py
• Não há documentação
• Testes:
  • Só há o teste que eu sugeri

Grupo tp2g01

Requisitos

• 01_menus 0.9
• 02_autenticacao 0.0
• 03_administrador 0.9
• 04_jogador 1
• 05_testes 0.0
• 06_doc 0.125
• 07_modularidade_legibilidade 1

Avaliação

0.56

Comentários

• Sair do admin não funciona

• Sair no jogador sai do programa

• Não há autenticação

• Criar utilizador funciona mas não valida se utilizador existe
• Alterar password funciona mas não valida se utilizador existe
• Apagar utilizador funciona mas não valida se utilizador existe
• Código fora das funções:
  • highscores.py
• Documentação
  • Ficheiros onde falta documentação:
    • autenticacao.py
    • Autenticacao2.py
    • DICIONARIO.py
    • main.py
    • perfis.py
  • Documentação mal escrita ou nos locais errados:
    • highscores.py
    • Dicionario (2).py
      • Ficheiro com nome errado, que não devia existir, provavelmente substitui o outro
      • A documentação é nas funções!
• Testes:
  • Não há testes
  • Unico teste que criaram que talvez faça sentido: test_alterar_pwd

Grupo tp2g03

Requisitos

• 01_menus 0
• 02_autenticacao 0
• 03_administrador 0
• 04_jogador 0
• 05_testes 0
• 06_doc 0.16666666666666666
• 07_modularidade_legibilidade 1

Avaliação

0.17

Comentários

• Autenticação não está a funcionar
• Não se consegue testar
• Documentação
  • Ficheiros sem documentação:
    • Autenticação.py
    • Dicionário.py
    • Highscores.py
    • leitura do jogo1.py
    • main.py
  • Funções mal documentadas:
    • escolher_opcao: argumento com nome errado
    • pedir_id: não tem parâmetros
    • verifica_password: não documentada
    • autenticacao:
      • parece que há confusão entre parâmetros e inputs
      • só se documentam os parâmetros!
      • O parâmetro não foi documentado
    • criar_novo_utilizador: ver anterior
    • mudar_password: idem
    • admin: idem
    • jogador: idem

Grupo tp2g05

Requisitos

• 01_menus 0.6400000000000001
• 02_autenticacao 0.8
• 03_administrador 0.8
• 04_jogador 0.8
• 05_testes 0.0
• 06_doc 0.5
• 07_modularidade_legibilidade 1

Avaliação

0.65

Comentários

• O código não corre e por isso não pode ser avaliado
• Penso que há problemas de funções indentadas e falta invocar a função main
• Documentação:
  • Ficheiros não documentados:
    • main.py
  • DICIONARIO.py: falta tem_palavra
  • Documentam-se os parâmetros da função
  • Não se documenta o que é lido ou o que a função faz
  • A linha de descrição deve ser sucinta. Exemplo: ler_highscores:
    • esta função lê os highscores do ficheiro e devolve-os
    • No returns: esta função devolve uma lista ordenada com os highscores
    • Se devolve uma lista, não é str, é list[(int, str)]
  • Funções não documentadas, ou mal documentadas:
    • adiciona_score
    • main
    • autenticar: parâmetros
    • ler_jogo: o tipo devolvido
    • imprime_sopa: o argumento
    • continuar_jogo: o argumento
    • encriptar: documentação no local errado
    • ler_perfis: documentação no local errado
    • tem_password
    • autenticacao
• Testes
  • Não se testam menus
  • Testes mal escritos

Grupo tp2g08

Requisitos

• 01_menus 0.6400000000000001
• 02_autenticacao 0.8
• 03_administrador 0.8
• 04_jogador 0.8
• 05_testes 0
• 06_doc 0.5
• 07_modularidade_legibilidade 1

Avaliação

0.65

Comentários

• Não se consegue correr porque há erros
• Documentação
  • Ficheiros sem documentação:
    • main.py
    • MENU.py
    • perfis.py
  • verificar_palavra: falta descrever o que é devolvido
• Não há testes novos

Grupo tp2g10

Requisitos

• 01_menus 0
• 02_autenticacao 0
• 03_administrador 0
• 04_jogador 0
• 05_testes 0.0
• 06_doc 0.2
• 07_modularidade_legibilidade 0

Avaliação

0.03

Comentários

• Não se pretendia que se criasse a sopa da maneira como foi feito em sopadeletras
• Não se consegue correr o código
• Há código fora das funções que causa erros
• Testes
  • Alguns testes não fazem sentido
  • Não se testam as funções com menus, ver o que eu escrevi no canal #projeto sobre este assunto
• Documentação
  • dicionario.py não tem funções
  • ficheiro.py
    • escolher_option: falta argumento
  • funcoes.py não tem funções
  • HIGHSCORES
    • Faltam os argumentos de todas as funções
    • Falta o que devolvem
  • lerperfis: 1/3
  • lersopadeletras: faltam os argumentos em todas as funções
  • sopadeletras: faltam os argumentos em todas as funções
  • Ficheiros sem documentação
    • main
    • Menu
      • Tem funções escondidas! Tem que passar para o nível de indentação de fora ou não são visíveis
    • perfis

Grupo tp2g12

Requisitos

• 01_menus 0.8
• 02_autenticacao 0.0
• 03_administrador 1
• 04_jogador 0
• 05_testes 0
• 06_doc 0.16666666666666666
• 07_modularidade_legibilidade 1

Avaliação

0.42

Comentários

• Não dá para voltar atrás nos menus
• Falta autenticação como administrador
• Falta a parte de jogar ou continuar um jogo
• ficheiro DICIONARIO deveria ser DICIONARIO.py
• Documentação mal feita porque:
  • Deve estar imediatamente depois do def (da linha que define a função)
  • Faltam argumentos
  • Falta o que é devolvido
  • Ver exemplo de como se faz na função adiciona_utilizador em https://github.com/rcm/CP2021_Guiao02/blob/main/perfis.py
  • No ficheiro dos menus:
    • se não há parâmetros, não é preciso colocar a secção
• Não se conseguem correr os testes (acho que foi da tentativa de colocar a documentação)
• Eu diria que os testes não correm por causa de alguns indícios:
  • Há funções que estão a ser invocadas com o nº errado de argumentos