Test in_coppia e3 HW3

L
Lorespo (610 points)
0 13 17
asked Dec 11, 2018 in Es3 by Lorespo (610 points)
recategorized Dec 11, 2018 by andrea.sterbini

Nella funzione in_coppia, nel caso in cui l'argomento partner passato sia None, restituisco un set di tuple disposte (femmina, maschio, numero film insieme che deve essere maggiore di uno). Il problema è che quando faccio i test ricevo l'errore AssertionError: 'F' != 'M' . Ho controllato all'interno di test_03.py e nel test 17 mi sembra che consideri il primo membro della tupla di genere maschile (l'istruzione di test è: 

incoppia = attore.in_coppia()

etc..

for t in incoppia:

etc..

male, female, Nf = t

Credo mi stia sfuggendo qualcosa ma proprio non riesco a capire cosa

1 Answer

_andrea_ (45670 points)
2 39 297
answered Dec 11, 2018 by _andrea_ (45,670 points)
L'ordine giusto è prima maschio e poi femmina, è un errore del testo
L
Lorespo (610 points)
0 13 17
commented Dec 11, 2018 by Lorespo (610 points)
Adesso mi da come errore che ci sono delle terne in più. Non può essere per il problema dei remake giusto? Il professore aveva detto di collegare in ogni caso i film vecchi agli attori quindi non dovrebbero alterare il conteggio delle volte che due attori hanno recitato insieme
_andrea_ (45670 points)
2 39 297
commented Dec 11, 2018 by _andrea_ (45,670 points)
Non saprei. Tuple in più forse può essere che hai messo il minimo per l'inserimento a 1? Cioè un film girato insieme, mentre l'esercizio dice più di un film, quindi almeno 2
L
Lorespo (610 points)
0 13 17
commented Dec 11, 2018 by Lorespo (610 points)
Infatti pare strano pure a me. Il minimo è due e le tuple in più hanno 2 come numero di volte
L
Lorespo (610 points)
0 13 17
commented Dec 11, 2018 by Lorespo (610 points)
Ho capito l'errore, avevo usato un if al posto di un elif nella parte in cui incrementavo il valore nel dizionario degli attori che avevano fatto coppia e così veniva incrementato 2 volte. Grazie per il supporto ;)
_andrea_ (45670 points)
2 39 297
commented Dec 11, 2018 by _andrea_ (45,670 points)
Ah ecco, bene così