search

alessandro-bugatti / ask_mvc

Piccolo framework MVC di esempio
0 stars 2 forks source link

SaveAnswer fix #36

Closed alessandro-bugatti closed 4 years ago

alessandro-bugatti commented 4 years ago

Trasforma la funzione saveAnswer in un metodo statico della classe QuestionRepository (issue #34 )

AndreaBegni commented 4 years ago

mi sorgono due dubbi, il primo è perchè il metodo saveAnswer sia pubblico dato che dovrebbe essere usato solo dall'interno e secondo la chiamata al metodo saveAnswer non dovrebbe essere preceduta dal QuestionRepository:: dal momento che è diventato un metodo statico?

alessandro-bugatti commented 4 years ago

@AndreaBegni Sono d'accordo con entrambe le osservazioni, tra l'altro il codice non funzionava, mi sembra strano perchè anche se non abbiamo una struttura di test formale, faccio sempre qualche prova prima della commit, boh. Comunque ho corretto come hai indicato, prova a dare un'occhiata poi chiudi pure e fai il merge.