search

rage / python-s20

https://python-s20.mooc.fi
1 stars 10 forks source link

teht 11-12 testi luultavasti väärin #145

Closed niinasaarelainen closed 3 years ago

niinasaarelainen commented 3 years ago

image

eli minun koodilla tulee 'programming': 10 ja testi väittää että 'programming': 9 ero tulee siitä että tekstin viimeinen sana on "programming."(loppupiste) Kai pisteet pitäisi poistaa ennen sanojen laskemista eli ei kai ole käytännön järkeä että "programming." ja "programming" ovat 'eri' sanoja ? https://tmc.mooc.fi/paste/8yPCR6jg6UKCvmGpqRvYZQ

sippe2 commented 3 years ago

Tehtävän tarkistus toimii oikein. Kun rivinvaihdon korvaa tyhjällä merkillä, niin edeltävän rivin viimeinen sana ja tulevan rivin ensimmäinen sana eivät yhdisty. Ammuin itseäni jalkaan tämän kanssa, kunnes rupesin vertailemaan siivottua tekstiä sanan "list" kanssa.

sippe2 commented 3 years ago

Tehtävän väri pysyy punaisena läpi menneestä suorituksesta huolimatta.

niinasaarelainen commented 3 years ago

Testi väittää että "list" esiintyy 4 kertaa, vaikka failissa se on 6 kertaa: image

image Noista kahdeksasta 2 on "lists" eli jäljelle jää 6

sippe2 commented 3 years ago

Sitä minäkin ihmettelin. Pitäisikö tehtävänantoa sitten selventää jotenkin, sillä nyt ilmeisemmin haluttiin, että merkkijonon pitää olla täysin sama (esim. "list" versus "lists"). Vastaukseksi sain itse ensin kolme list -sanaa ja sitten kuusi. Lopulta kun katsoin esimerkkitulostusta, niin ymmärsin, että sen mukaan list -sanoja pitäisi olla neljä. Lopulta kun tutkin mitä oma koodini tekee, niin rivinvaihtojen, sekä pilkun ja pisteen poisto yhdisti toisen rivin viimeisen sanan "list" ja kolmannen rivin ensimmäisen "However" sanan, jolloin tulokseksi tuli kolme. Kun korvasin rivinvaihdon tyhjällä merkillä, niin vastaukseksi tuli neljä, jota ilmeisemmin tehtävässä haluttiin. Tällä tavalla itse sain samantuloksen, mikä esimerkissä oli. Olet kyllä varsin oikeassa, että vastaus on tavallaan tehtävänantoon nähden väärä, mutta voin jälleen olla myös karkeasti väärässäkin:

` tiedosto = open(tiedoston_nimi, "r") tmp = "" for rivi in tiedosto: tmp += rivi.replace("\n", " ")

siivous = "".join([merkki for merkki in tmp if merkki not in ".,"]) sanalista = siivous.split() `

Tällä kötöstyksellä tehtävä meni itselläni läpi.

pllk commented 3 years ago

Nyt tehtävänannon pitäisi vastata paremmin testejä. Tuossa list-tapauksessa ei lasketa mukaan esiintymiä, jotka alkavat suurella kirjaimella tai joissa list esiintyy sanan osana (tässä tapauksessa monikossa).

niinasaarelainen commented 3 years ago

vielä tämän voisi fixata ( pun --> vihreä) [image: image.png] [image: image.png]

https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail Ei viruksia. www.avast.com https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail <#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

ma 16. marrask. 2020 klo 14.17 Antti Laaksonen (notifications@github.com) kirjoitti:

Nyt tehtävänannon pitäisi vastata paremmin testejä. Tuossa list-tapauksessa ei lasketa mukaan esiintymiä, jotka alkavat suurella kirjaimella tai joissa list esiintyy sanan osana (tässä tapauksessa monikossa).

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rage/python-s20/issues/145#issuecomment-727942731, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGFHR75AMK54BLLNVS4V4BLSQEJ45ANCNFSM4TMISV4A .

--

t. Niina Saarelainen

pllk commented 3 years ago

Jostain syystä kuvat eivät näy viestissäsi

niinasaarelainen commented 3 years ago

täällä https://python-s20.now.sh/osa-11/2-lisaa-koosteesta

tehtävän status punainen vaikka tein sen 15min sitten (olen kirjautuneena ja on Refreshattu ;-> )

https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail Ei viruksia. www.avast.com https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail <#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

ma 16. marrask. 2020 klo 22.10 Antti Laaksonen (notifications@github.com) kirjoitti:

Jostain syystä kuvat eivät näy viestissäsi

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rage/python-s20/issues/145#issuecomment-728296789, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGFHR73HRXDG6BS7XVWGAATSQGBKNANCNFSM4TMISV4A .

--

t. Niina Saarelainen

pllk commented 3 years ago

Näkyisikö nyt oikein?

sippe2 commented 3 years ago

Mulla näkyy oikein.

niinasaarelainen commented 3 years ago

Toimiii teht 11-12 vihreä

https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail Ei viruksia. www.avast.com https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail <#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

ma 16. marrask. 2020 klo 22.20 Antti Laaksonen (notifications@github.com) kirjoitti:

Näkyisikö nyt oikein?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rage/python-s20/issues/145#issuecomment-728301855, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGFHR774FMUJV3HHWRFJQVTSQGCRHANCNFSM4TMISV4A .

--

t. Niina Saarelainen