VeronikaDolan / python-kurz-2022-ukoly

0 stars 0 forks source link

Odevzdáni úkolu č.6 #6

Closed VeronikaDolan closed 2 years ago

VeronikaDolan commented 2 years ago

Ahoj,

odevzdávám úkol č. 6, včetně csv souborů.

https://github.com/VeronikaDolan/python-kurz-2022-ukoly/blob/78bb641f1ab77a11d6fd87dd9c39ef9ce71b03fc/ukol-6.py4.py https://github.com/VeronikaDolan/python-kurz-2022-ukoly/blob/78bb641f1ab77a11d6fd87dd9c39ef9ce71b03fc/znamky_za_semestr.csv https://github.com/VeronikaDolan/python-kurz-2022-ukoly/blob/78bb641f1ab77a11d6fd87dd9c39ef9ce71b03fc/znamky_za_semestr_upraveno.csv

Hezký den :-)

Verča Dolanová

pmoravkova commented 2 years ago

Ahoj. Mas to spravne, to rozhodne a ukol mas uznany. Priznavam, ze mne by se libilo, kdybys zkusila vymyslet, jak se vyhnout tehle silene casti kodu, a vzhledem, ze jsme se to ucili, tak ti verim, ze to das snadno. Je to latka ze slovniku, prochazeni slovniku a prirazeni ke klici zmenenou hodnotu. Kdybys nevedela, tak treba zde druha odpoved, prvni cast kodu, to je presne to, co se ti bude hodit. Tehle casti se nevyhnes .replace('1', 'A').replace('2', 'B').replace('3', 'C').replace('4', 'D').replace('5', 'E'), ale staci to tam bohate napsat jen jednou, ne pro kazdy test zvlast. Nedejboze, kdyby tech testu nebylo 6, ale treba uz 30:

for grade in students:  #nahrazeni cisel za pismena
    grade['Test 1'] = grade['Test 1'].replace('1', 'A').replace('2', 'B').replace('3', 'C').replace('4', 'D').replace('5', 'E')
    grade['Test 2'] = grade['Test 2'].replace('1', 'A').replace('2', 'B').replace('3', 'C').replace('4', 'D').replace('5', 'E')
    grade['Test 3'] = grade['Test 3'].replace('1', 'A').replace('2', 'B').replace('3', 'C').replace('4', 'D').replace('5', 'E')
    grade['Test 4'] = grade['Test 4'].replace('1', 'A').replace('2', 'B').replace('3', 'C').replace('4', 'D').replace('5', 'E')
    grade['Test 5'] = grade['Test 5'].replace('1', 'A').replace('2', 'B').replace('3', 'C').replace('4', 'D').replace('5', 'E')
    grade['Test 6'] = grade['Test 5'].replace('1', 'A').replace('2', 'B').replace('3', 'C').replace('4', 'D').replace('5', 'E')
VeronikaDolan commented 2 years ago

Ahoj, dekuji za zpetnou vazbu. Osobne se mi to take moc nelibilo, ale nebyla jsem schopna nic lepsiho vymyslet... Vecer se na to urcite jeste podivam. Diky.

pmoravkova commented 2 years ago

Je mi to jasny. Promin, ja byla zas o vikendu pryc a pak nic nestiham, tak jsem na to nekoukla driv. Jeste jedna vec, zkuste se vyhybat natvrdo psanym hodnotam. Jakmile to muzete odnekud vzit ze zdrojovych dat, tak pokud na tom nestoji cely kod, je dobry to tak udelat, protoze proste zdrojovy data se predpoklada, ze se budou menit. Ted narazim na tohle: fieldnames = ['Příjmení', 'Jméno', 'Test 1', 'Test 2', 'Test 3', 'Test 4', 'Test 5', 'Test 6'] #nazvy v header. Zkus i tohle nevypisovat, ale vzit z kodu, ze zdroje, protoze predstav si, ze delas obecny kod, kdy jeden ucitel bude mit 6 testu, dalsi 20 a oba budou chtit pouzit stejny kod.

VeronikaDolan commented 2 years ago

Ahoj, tak pokus č.2 :-)

https://github.com/VeronikaDolan/python-kurz-2022-ukoly/blob/620dc80ed69058951f3fc849462b375bcb031496/ukol-6.py https://github.com/VeronikaDolan/python-kurz-2022-ukoly/blob/620dc80ed69058951f3fc849462b375bcb031496/znamky_za_semestr.csv https://github.com/VeronikaDolan/python-kurz-2022-ukoly/blob/620dc80ed69058951f3fc849462b375bcb031496/znamky_za_semestr_upraveno.csv

Díky za trpělivost.

pmoravkova commented 2 years ago

Super! diky moc. jeste pred produkci schovat kontrolni printy a proste super.