Grupo tp1g02

Requisitos

• 01_impressao 1
• 02_val_coord 0.75
• 03_val_linha 0.7
• 04_val_pal 0.7
• 05_doc 0.8
• 06_testes 0.4
• 07_mod_leg 1

Avaliação

0.76

Comentários

Comentários globais sobre o programa

• Há muito código desenvolvido
• A maioria do código é coerente e está relativamente bem escrito
• Há uma grande preocupação de perceber código e de o escrever
• Há um problema relativamente frequente:
  • dificuldade em perceber como combinar várias funções para resolver um problema
  • há uma confusão entre a função e a sua invocação

Eis um exemplo do que falo:

    buscar_palavra(sopa, L1, C1, L2, C2)
    pertence_ao_dicionario(palavra)

    if pertence_ao_dicionario == False:

• Este extrato de código foi buscar a palavra mas não a armazenou em nenhuma variável
• O argumento passado à função pertence_ao_dicionario devia ter sido o resultado da função buscar_palavra
• A invocação da função pertence_ao_dicionario não é utilizada e por isso não é útil
• O if está a comparar a função pertence_ao_dicionario com o valor falso, o que dá sempre falso porque uma função não é igual a um valor verdadeiro ou falso

O que faria sentido neste caso era algo do género:

    palavra = buscar_palavra(sopa, L1, C1, L2, C2)
    if not pertence_ao_dicionario(palavra):

Avaliação

• Validação das coordenadas tem um bug (uma linha que estraga tudo)
• A função que invoca a função de validação está a passar mal o nº de linhas e de colunas
• Mesma linha não está a ser invocado corretamente
• Falta colocar o main.py como a função principal, neste momento é o menus.py

• Documentação

  • Todo o código tem a documentação mas há muitos erros
  • A sintaxe correta para os argumentos é palavra: str e não palavra = str

  • Ficheiros nos quais a documentação antiga não foi corrigida:

    • highscores.py
    • menus.py
    • PERFIS.py

  • Erros com a descrição de argumentos e do valor devolvido e/ou do seu tipo

    • imprimir_highscores
    • ler_jogo
    • criar_sopa
  • Erros com a descrição de argumentos e/ou do seu tipo
    • escrever_highscores
    • gravar_nlinhas_ncolunas
    • gravar_sopa
    • imprimir_sopa
    • gravar_jogo
    • valida_coord
    • pede_coord
    • mesma_linha
    • buscar_palavra
    • buscar_palavra_horiz
    • buscar_palavra_vert
    • buscar_palavra_diagED
    • buscar_palavra_diagDE
    • insere_hor_inv
    • insere_hor
    • insere_ver_inv
    • insere_ver
    • insere_diag_decres
    • insere_diag_cres
    • insere_diag_decres_inv
    • insere_diag_cres_inv
    • pede_palavra
  • Erros com a descrição do valor devolvido e/ou do seu tipo
    • ler_highscores
  • Testes
  • Testes de valida_coord não estão corretos:
    • test_VC1, etc deviam estar ao nível anterior e não devia existir a função test_valida_coord
    • tirando o test_VC1, que foi sugerido por mim, os outros testes estão mal feitos
    • O objetivo era ver se o texto introduzido pelo utilizador correspondia a uma coordenada válida e não a colocar valores que não faziam sentido no nº de linhas e nº de colunas
    • test_mesma_linha só está a testar casos em que as coordenadas pertencem à mesma direção
    • test_buscar_palavra não faz sentido visto que não está a testar a função corretamente, nota-se que a pessoa que escreveu os testes não percebeu como as funções devem ser utilizadas

Grupo tp1g04

Requisitos

• 01_impressao 1
• 02_val_coord 0.9
• 03_val_linha 1
• 04_val_pal 0.3
• 05_doc 0.9
• 06_testes 0.5
• 07_mod_leg 0.9

Avaliação

0.79

Comentários

• Função validateCoords assume que a sopa é quadrada!
• Parte de ir buscar a palavra só está parcialmente implementada

• continuar_jogo.py não faz sentido e não deveria estar no programa, se calhar até nem pertence

• Documentação

  • Erros com a descrição de argumentos e/ou do seu tipo
    • validateCoords
    • buscar_palavra_horizontal
    • buscar_palavra_vertical
    • buscar_palavra_diagonal
  • Sem documentação
    • pede_palavras
• Testes
  • Faltam testes à mesma linha em casos em que as duas coordenadas não pertençam à mesma linha
  • Testes a coordenadas válidas errados porque as condições testadas são sempre verdadeiro, o que não pode ser
  • Pelos comentário, este caso: validateCoords("25 25", 25,25)) devia dar verdadeiro e não falso!
  • Todos os testes a buscar paravra estão mal escritos

Grupo tp1g06

Requisitos

• 01_impressao 1.0
• 02_val_coord 0.0
• 03_val_linha 1.0
• 04_val_pal 0.0
• 05_doc 0.5
• 06_testes 0.0
• 07_mod_leg 0.8

Avaliação

0.47

Comentários

• Má organização: há funções tais como buscar_pal que estão no ficheiro main.py

• Documentação

  • Ficheiros nos quais a documentação antiga não foi corrigida:
    • codigo_dicionario.py
    • codigo_Highscores.py
  • Ficheiros sem documentação:
    • ler_jogo.py
  • Erros com a descrição dos argumentos e/ou do seu tipo
    • autenticar
  • Erros com a descrição do valor devolvido e/ou do seu tipo
    • apagar_utilizador
    • voltar_menu
    • ler_perfis
    • tem_password
  • Funções sem documentação
    • ler_perfis

Grupo tp1g11

Requisitos

• 01_impressao 1
• 02_val_coord 0.0
• 03_val_linha 0.9
• 04_val_pal 0.9
• 05_doc 0.9
• 06_testes 0.0
• 07_mod_leg 1

Avaliação

0.67

Comentários

Avaliação síncrona

Grupo tp1g13

Requisitos

• 01_impressao 1.0
• 02_val_coord 1.0
• 03_val_linha 0.8
• 04_val_pal 0
• 05_doc 0.7
• 06_testes 0.5
• 07_mod_leg 0.7

Avaliação

0.67

Comentários

• Parece haver vários ficheiros com o mesmo código
• Alguns estão documentados, outros não
• Documentação:
  • Erros com a descrição do valor devolvido e/ou do seu tipo
    • read_highscores
    • jogar
    • encriptar
    • check_password
    • buscar_palavra
  • Erros com a descrição dos argumentos e/ou do seu tipo
    • write_highscores
  • Ficheiros/funções sem documentação
    • main.py
    • insere_horizontalDE
    • insere_verticalBC
    • insere_obliquaED_BC
    • insere_obliquaDE_CB
    • insere_obliquaDE_BC
  • Má descrição
    • print_sopa

Grupo tp1g14

Requisitos

• 01_impressao 1.0
• 02_val_coord 0
• 03_val_linha 0
• 04_val_pal 0
• 05_doc 0.2
• 06_testes 0
• 07_mod_leg 0.7

Avaliação

0.27

Comentários

• Má organização do código
• Há vários módulos demasiado pequenos e que não fazem muito sentido
• Exemplo: introdução_das_coordenadas.py tem funções como imprime_sopa e comecar_jogo
• Documentação:
  • Ficheiros sem documentação ou com pouca documentação:
    • Dicionario.py
    • highscores.py
    • introdução_das_coordenadas.py
    • continuação_jogos.py
    • administrador.py
    • sopa.py
    • perfis.py
    • Palavras_em_fdic.py
    • main.py
    • menu.py

Grupo tp2g01

Requisitos

• 01_impressao 1.0
• 02_val_coord 0.0
• 03_val_linha 0.5
• 04_val_pal 0.0
• 05_doc 0.8
• 06_testes 0
• 07_mod_leg 0.6

Avaliação

0.41

Comentários

• Há vários ficheiros que não fazem sentido existirem
  • Não possuem código útil (isto é, não tem funções)
• As funções jogar_sopaA e jogar_sopaB não fazem sentido porque abrem o ficheiro para leitura mas não fazem nada com ele
• As funções cont_sopaA e cont_sopaB sofrem do mesmo tipo de estranheza
• Há código na raiz, por exemplo no ficheiro autenticacao.py e Autenticacao2.py quando só o ficheiro main.py deveria ter o programa principal
• Mesmo problema no ficheiro guião4.py (e.g., linha 14)
• Documentação
  • Há funções no ficheiro guião4.py sem documentação
    • diagonal_DE
    • buscar_palavra

Grupo tp2g03

Requisitos

• 01_impressao 0.0
• 02_val_coord 0.0
• 03_val_linha 0.0
• 04_val_pal 0.0
• 05_doc 0.0
• 06_testes 0.0
• 07_mod_leg 0.0

Avaliação

0.00

Comentários

• Avaliação nula devido a código igual em vários grupos

• Casos de funções demasiado específicas:

  • não se conseguem generalizar
  • são cópias umas das outras
  • jogo_1 e jogo_2
  • continuar_jogo1 e continuar_jogo2

• Ficheiros leitura1.py e leitura2.py não fazem sentido

• Não se criam ficheiros com uma única função e a função em causa tem um argumento que não é utilizado
• Documentação:
  • Há casos onde simplesmente se coloca informação sobre parâmetros que não existem na função
  • Problemas na documentação
    • ler_dicionario
    • Ficheiros onde se descrevem argumentos que não pertencem à função em causa:
      • Autenticação.py
      • Highscores.py
    • Ficheiros onde se descrevem o que devolvem funções que não devolvem nada:
      • guiao 4.py (ex: imprimir_jogo, gravar_jogo)
      • menu.py
    • Má descrição do que devolvem (ex: valida_coords)

Grupo tp2g05

Requisitos

• 01_impressao 0.0
• 02_val_coord 0.0
• 03_val_linha 0.0
• 04_val_pal 0.0
• 05_doc 0.0
• 06_testes 0.0
• 07_mod_leg 0.0

Avaliação

0.00

Comentários

• Avaliação nula devido a código igual em vários grupos

• Há funções repetidas nos vários ficheiros, isso gera vários tipos de problemas

• Documentação:

  • Não se percebe pela descrição das funções, qual é a diferença entre as várias funções que tratam as diagonais
  • Coloquei só alguns exemplos porque há muitos
  • Erro na descrição ou tipo do valor devolvido por
    • tem_palavra
    • ler_perfis
    • get_nome get_password
    • adiciona_utilizador
    • pede_utilizador
  • Erro na descrição ou tipo de argumentos
    • tem_password
    • escolher_opcao
    • menu
    • autenticar
    • criar_utilizador

Grupo tp2g08

Requisitos

• 01_impressao 0.0
• 02_val_coord 0.0
• 03_val_linha 0.0
• 04_val_pal 0.0
• 05_doc 0.0
• 06_testes 0.0
• 07_mod_leg 0.0

Avaliação

0.00

Comentários

• Avaliação nula devido a código igual em vários grupos

• Documentação

  • Ficheiros sem documentação

    • main.py

  • Em alguns casos falta descrever o tipo que a função devolve (ex: encriptar)

  • Noutros casos, descreve-se que uma função devolve algo quando não o faz (ex: escrever_highscores)
  • Também acontece descrever mal o argumento ou o seu tipo (ex: escolher_opcao, imprime_jogo)
  • Descrever argumentos que não existem (ex: criar_utilizador)

Grupo tp2g10

Requisitos

• 01_impressao 0.0
• 02_val_coord 0.0
• 03_val_linha 0.0
• 04_val_pal 0.0
• 05_doc 0.0
• 06_testes 0.0
• 07_mod_leg 0.0

Avaliação

0.00

Comentários

• Avaliação nula devido a código igual em vários grupos
• Qual é a razão da existência do funcoes.py, ficheiro.py ou dicionario.py?

• Vários ficheiros com código executável, ex: dicionario.py ou continuarjogo.py

• Código mal estruturado (ver ficheiros acima)

• Código sem documentação ou com documentação mal escrita
  • lerperfis.py
  • HIGHSCORES.py
  • main.pys
• Exemplo de funções mal documentadas:
  • ler_game
  • imprime_game
  • ler_jogo

Grupo tp2g12

Requisitos

• 01_impressao 1.0
• 02_val_coord 0.0
• 03_val_linha 0.0
• 04_val_pal 0.0
• 05_doc 0.2
• 06_testes 0.0
• 07_mod_leg 0.7

Avaliação

0.27

Comentários

Fazer perguntas sobre o código, há código estranho feito de uma maneira diferente da que foi ensidada menu- guiao3.py: menu_admnistrador: 154

• Código hard-coded para casos específicos

• sopa_letra guião 4.py: 102

• Documentação muito incompleta

• Muitos ficheiros sem documentação
• Os ficheiros que estão documentados não seguem as regras
  • Faltam os tipos