rage / ohjelmointi-21

https://ohjelmointi-21.mooc.fi
0 stars 17 forks source link

globaalin muuttujan käyttö funktion sisällä -varoituksen toisto kolme kertaa #79

Open viipu opened 3 years ago

viipu commented 3 years ago

Varoitusteksti globaalien muuttujien käytöstä funktioissa toistuu kolme kertaa käytännössä identtisenä. Jotain karsintaa pitäisi ehkä harrastaa. @pllk ?

Loppupuolella, eri koodiesimerkki kuin muissa: https://github.com/rage/ohjelmointi-21/blob/master/data/osa-3/4-omat-funktiot.md Alkupuolella: https://github.com/rage/ohjelmointi-21/blob/master/data/osa-4/6-lisaa-rakenteista.md Alkupuolella, vähän eri otsikolla mutta muuten lähes täsmälleen sama kuin edellinen: https://github.com/rage/ohjelmointi-21/blob/master/data/osa-5/1-lisaa-listoista.md

mluukkai commented 3 years ago

kyllä tämä on ihan harkittua. siperia opetti

viipu commented 3 years ago

Jäkätän vielä mut näin kääntäjän -> ylläpidettävyyden näkökulmasta järjetöntä että käytännössä sama teksti on monta kertaa eri paikoissa, ja tosiaan osaan on tehty kielenhuoltoa ja osaan ei. Joku kommentti edes tiedostoon että tarkista nämä muutkin kohdat jos muokkaat jotain?

mluukkai commented 3 years ago

siellä on lisää toistoa vielä luvassa... ja joo, kommentit ois tietysti hyvä idea, muuta nyt se on myöhästä, kukaan ei enää muista missä ja mitä kaikkea on toistettu

pllk commented 3 years ago

Olen samaa mieltä Virpin kanssa: ei ole hyvä että materiaalissa on samaa copy-pastetettua tekstiä moneen kertaan. Se on hankalasti ylläpidettävä ja käännettävä eikä ole muutenkaan ammattimaista. Yksi mahdollinen parempi ratkaisu olisi, että tuo olisi kerran hyvin tehtynä esimerkkinä ja muissa kohdissa olisi viittaus siihen. En ehdi ihan lähipäivinä miettiä tätä, mutta hyvä että tämä issue on täällä.