Python osaajilta tarvitaan apua

Ippee

Elikkäs olisi pieni pulma ohjelman toimivuuden suhteen:

# -*- coding: latin-1 -*-
lause = "g fmnc wms bgblr rpylqjyrc gr zw fylb. rfyrq ufyr amknsrcpq ypc dmp. bmgle gr gl zw fylb gq glcddgagclr ylb rfyr'q ufw rfgq rcvr gq qm jmle. sqgle qrpgle.kyicrpylq() gq pcamkkclbcb. lmu ynnjw ml rfc spj."
lauselista = list(lause) #pilkotaan lause kirjaimiksi jolloin voidaan käydä ne läpi kirjain kirjaimelta
avain = ["a","b","c","d","e","f","g","h","i","j","k","l","m","n","o","p","q","r","s","t","u","v","x","y","z"]
avain = avain[::-1] #käännetään avain lista nurin
for kirjain in lauselista: #seulotaan lause kirjaimittain läpi
if kirjain in avain: #jos löydetään lauseen kirjain avain listasta
y = avain.index(kirjain) #haetaan sitten kyseisen kirjaimen sijainti avain listasta
y2 = y - 2 #vähennetään kirjaimen sijaintia kahdella että saadaan oikea kirjain g -> i jne
lause = lause.replace(kirjain, avain[y2]) #korvataan lauseen kirjain oikealla kirjaimella joka
#aikaisemmin haettiin
print lause #tulostetaan muunneltu lause oikeilla kirjaimilla


Kyseinen koodi ei ota toimiakseen sitten millään koska se tulostaa aivan väärät kirjaimet mitä olisi tarkoitus vaikka kaiken järjen mukaan sen pitäisi toimia oikein. Olen tätä yrittänyt jo selvitellä useita tunteja ja hakenut tietoa muttei mikään tunnu auttavan joten olisiko kellään tietoa ongelman korjaamiseksi? Olen äskettäin aloitellut ohjelmointia Pythonilla mutta kyseiset 'näkymättömät' ongelmat laittavat seinän vastaan.
Kyseinen koodi liittyy tähän tehtävään:
http://www.pythonchallenge.com/pc/def/map.html

7

649

    Vastaukset

    Anonyymi (Kirjaudu / Rekisteröidy)
    5000
    • Ippee
    • 223123132

      Tuo koodi nyt ei pysty toimimaan. Ajatus sinänsä ihan hyvä että teet taulukon kirjaimien kääntämistä varten, toteutus vain tökkii eikä ole kauhean pythonmaista.

      Ehkä helpointa olisi käyttää dictionary-objektia käännöstaulukkona. Teet ohjelmanpätkän joka muodostaa "taulukon" esim:
      d = {'a': 'c', 'b': 'd', 'c': 'e' .. jne} (tosin, älä tee tätä taulukkoa käsin vaikka se niin toimisikin =)

      Kun "taulukon" olet luonut voit kääntää salatun viestin vaikka näin:
      for kirjain in lause:
      ....print d[kirjain]

      Ja se taulukon luominenkaan ei ole kovin monimutkaista.. siihenkin on kovin monta tapaa.
      Yksi helppo tapa olisi vaikka aloittaa jotenkin näin - tämä on puutteellinen tarkoituksella, jääpähän vielä jotain mietittävääkin:
      d = {}
      for i in xrange(len(aakkoset)): # silmukoidaan i:n arvoilla 0,1,2,3,4 kirjainten lukumäärän mukaan
      ....d[aakkoset[i]] = aakkoset[i 2] # talletetaan taulukkoon kirjaimesta - tässä menee viduralleen tiiätkös miksi?

      Kysessähän on siis rot13 salauksen muoto jossa kirjaimia käännetäänkin vaan se 2 merkkiä. Rot13 hakemalla löytynee moniakin toteutustapoja jos tästä ei meinaa päästä eteenpäin millään.

      Onnea vaan!

      ps. "import this" ;)

      • Ippee

        [quote]d[aakkoset[i]] = aakkoset[i 2] # talletetaan taulukkoon kirjaimesta - tässä menee viduralleen tiiätkös miksi? [/quote]
        Ainakin koska se yrittää hakea kahdella viimeisellä kerralla tietoa taulukon ulkopuolelta ja pukkaa vaan erroria. Kyseistä menetelmää en saanut toimimaan vaikka laitoin rajoitukset siihen koska se pukkasi kirjaimet tyyliin a:c c:c c:e tai vastaavaa elikkä vain ensimmäinen pari meni oikein eikä hajuakaan miksi :)

        Noh nyt kumminkin hulppeat 8 tuntia myöhemmin sain sen vihdoinkin toimimaan, sitkeys palkitaan:

        import string
        kirjaimet = string.ascii_lowercase
        avain = 2
        kaanna = kirjaimet[avain:] kirjaimet[0:avain]
        swap = string.maketrans(kirjaimet,kaanna)
        text = "g fmnc wms bgblr rpylqjyrc gr zw fylb. rfyrq ufyr amknsrcpq ypc dmp. bmgle gr gl zw fylb gq glcddgagclr ylb rfyr'q ufw rfgq rcvr gq qm jmle. sqgle qrpgle.kyicrpylq() gq pcamkkclbcb. lmu ynnjw ml rfc spj."
        print text.translate(swap)

        Kiitoksia paljon avusta vaikkei se paksuun kallooni uponnutkaan, sainpahan ainakin pohdiskella tovin.
        Aamuun asti meni joten tästä varmaan joutais jo pitkälleen :)


      • sdfsdfsf
        Ippee kirjoitti:

        [quote]d[aakkoset[i]] = aakkoset[i 2] # talletetaan taulukkoon kirjaimesta - tässä menee viduralleen tiiätkös miksi? [/quote]
        Ainakin koska se yrittää hakea kahdella viimeisellä kerralla tietoa taulukon ulkopuolelta ja pukkaa vaan erroria. Kyseistä menetelmää en saanut toimimaan vaikka laitoin rajoitukset siihen koska se pukkasi kirjaimet tyyliin a:c c:c c:e tai vastaavaa elikkä vain ensimmäinen pari meni oikein eikä hajuakaan miksi :)

        Noh nyt kumminkin hulppeat 8 tuntia myöhemmin sain sen vihdoinkin toimimaan, sitkeys palkitaan:

        import string
        kirjaimet = string.ascii_lowercase
        avain = 2
        kaanna = kirjaimet[avain:] kirjaimet[0:avain]
        swap = string.maketrans(kirjaimet,kaanna)
        text = "g fmnc wms bgblr rpylqjyrc gr zw fylb. rfyrq ufyr amknsrcpq ypc dmp. bmgle gr gl zw fylb gq glcddgagclr ylb rfyr'q ufw rfgq rcvr gq qm jmle. sqgle qrpgle.kyicrpylq() gq pcamkkclbcb. lmu ynnjw ml rfc spj."
        print text.translate(swap)

        Kiitoksia paljon avusta vaikkei se paksuun kallooni uponnutkaan, sainpahan ainakin pohdiskella tovin.
        Aamuun asti meni joten tästä varmaan joutais jo pitkälleen :)

        Tapoja on monia kuten näet :) maketrans on tosin pitkälti valmis ratkaisu, mutta pätevä juuri tuollaiseen

        Tässä vielä tämä mikä itellä tuli ensiksi mieleen (jos kiinnostusta enää riittää).. korvaa rivin alussa olevat .... asianmukaisella sisennyksellä niin toimii paremmin ;)

        import string
        letters = string.lowercase
        d = {}
        for i in xrange(0, len(letters)):
        ....d[letters[i]] = letters[(i 2) % len(letters)] #jakojäännös estää indeksin vuotamasta yli

        text = "g fmnc wms bgblr rpylqjyrc gr zw fylb. rfyrq ufyr amknsrcpq ypc dmp. bmgle gr gl zw fylb gq glcddgagclr ylb rfyr'q ufw rfgq rcvr gq qm jmle. sqgle qrpgle.kyicrpylq() gq pcamkkclbcb. lmu ynnjw ml rfc spj."
        l = [] # lista johon kirjaimet lisätään
        for c in text:
        ....l.append(d.get(c, c)) # d[c] tai c jos taulukossa ei kyseistä merkkiä
        print ''.join(l)

        En tehnyt tuosta mitään extrakompaktia. Ehkä tuosta jotain apua jollekin on :)


      • sdfsfsfsfd
        sdfsdfsf kirjoitti:

        Tapoja on monia kuten näet :) maketrans on tosin pitkälti valmis ratkaisu, mutta pätevä juuri tuollaiseen

        Tässä vielä tämä mikä itellä tuli ensiksi mieleen (jos kiinnostusta enää riittää).. korvaa rivin alussa olevat .... asianmukaisella sisennyksellä niin toimii paremmin ;)

        import string
        letters = string.lowercase
        d = {}
        for i in xrange(0, len(letters)):
        ....d[letters[i]] = letters[(i 2) % len(letters)] #jakojäännös estää indeksin vuotamasta yli

        text = "g fmnc wms bgblr rpylqjyrc gr zw fylb. rfyrq ufyr amknsrcpq ypc dmp. bmgle gr gl zw fylb gq glcddgagclr ylb rfyr'q ufw rfgq rcvr gq qm jmle. sqgle qrpgle.kyicrpylq() gq pcamkkclbcb. lmu ynnjw ml rfc spj."
        l = [] # lista johon kirjaimet lisätään
        for c in text:
        ....l.append(d.get(c, c)) # d[c] tai c jos taulukossa ei kyseistä merkkiä
        print ''.join(l)

        En tehnyt tuosta mitään extrakompaktia. Ehkä tuosta jotain apua jollekin on :)

        sama lyhempänä :)
        from string import lowercase as x
        print ''.join([dict([(x[i], x[(i 2) % len(x)]) for i in xrange(len(x))]).get(c, c) for c in "g fmnc wms bgblr rpylqjyrc gr zw fylb. rfyrq ufyr amknsrcpq ypc dmp. bmgle gr gl zw fylb gq glcddgagclr ylb rfyr'q ufw rfgq rcvr gq qm jmle. sqgle qrpgle.kyicrpylq() gq pcamkkclbcb. lmu ynnjw ml rfc spj."])

        Eri asia tekeekö tämmöisellä sitten enää mitään..


      • Ippee
        sdfsfsfsfd kirjoitti:

        sama lyhempänä :)
        from string import lowercase as x
        print ''.join([dict([(x[i], x[(i 2) % len(x)]) for i in xrange(len(x))]).get(c, c) for c in "g fmnc wms bgblr rpylqjyrc gr zw fylb. rfyrq ufyr amknsrcpq ypc dmp. bmgle gr gl zw fylb gq glcddgagclr ylb rfyr'q ufw rfgq rcvr gq qm jmle. sqgle qrpgle.kyicrpylq() gq pcamkkclbcb. lmu ynnjw ml rfc spj."])

        Eri asia tekeekö tämmöisellä sitten enää mitään..

        Ahaa jakojäännöksellä saa korjattua tuon tsydeemin, hyvä tietää.

        Tuo jälkimmäinen metodi hieman epäselvä ainakin aloittelijan kalloon mutta hyvä kuitenkin tietää miten eri ratkaisut voidaan toteuttaa. Kiitoksia jälleen :)


    • anonyymi koodari

      Ihan vinkkinä voit käyttää jatkossa vaikka pastebiniä tai muuta vastaavaa. Ne osaavat myös koodin värjäyksen.

    Ketjusta on poistettu 0 sääntöjenvastaista viestiä.

    Luetuimmat keskustelut

    1. Nasima löi Jussille luun kurkkuun

      Nasima kertoi ettei Jussi sovi puhemieheksi, koska sallii rasismin. Mihin toimiin perussuomalaiset ryhtyvät? Kuka nouse
      Maailman menoa
      461
      8288
    2. Razmyar on säälittävä - puhemiehellä ei ole mitään syytä ottaa kantaa tähän "silmäkohuun"

      jonka toimittajat sai aikaan. Asia ei kuulu puhemiehelle millään lailla. Razmyar haluaa taas vaan huomiota. Mutta jos r
      Maailman menoa
      116
      5707
    3. Pitkän linjan yritysjohtaja: Suomen tapahtumat eivät ole Aasian kärkiuutinen

      Viimeaikaista kohua on käytetty sisäpoliittisena lyömäaseena, Thomas Zilliacus sanoo. – Koska asiaa kuitenkin kommentoi
      Maailman menoa
      116
      4711
    4. Miten Eerolan silmäkuvat voivat levitä muutamassa tunnissa ympäri maailmaa?

      Seuraako koko maailma persujen ja erityisesti Eerolan somea reaaliajassa? Edes kansanedustajan itsemurha eduskuntatalos
      Maailman menoa
      228
      4628
    5. Siviilipalvelusmies Halla-aho normalisoi rasismin perussuomalaisissa

      SMP:n tuhkille perustettu puolue ei ollut ihmisiä vastaan, vaan instituutiokriittinen. "Missä EU - siellä ongelma", oli
      Perussuomalaiset
      36
      4529
    6. Suomalaisilta vaaditaan valtavasti suvaitsevaisuutta - miksi sitä ei vaadita muslimeilta

      Suomalaisilta vaaditaan kaikkea, pitää olla suvaitsevainen ja hyväksyä vieraiden tavat, rasisti ei saa olla jne. Miksi s
      Maailman menoa
      55
      4033
    7. Juuri Suomen valtamedian toimittajat teki "silmävääntelystä" sen kohun

      ja ilmeisesti ottivat yhteyttä myös ulkomaisiin medioihin, että katsokaas tätä. Mutta Japanin medioissa on asiaan suhta
      Maailman menoa
      160
      3900
    8. Demarit ovat oikeasti törppöjä eli heikkoälyisiä

      ja todistavat sitä päivittäin täällä lapsellisilla jutuillaan. Kauheasti on hehkuttaneet kuinka demarien kannatus vaan
      Maailman menoa
      79
      3772
    9. Perussuomalaiset eivät harrasta maalittamista kuten vasemmisto

      Huomaa tässä keinotekoisessa "rasismi"-kohussakin kuinka vasemmistolaiset maalittavat Perussuomalaisia. Me emme vastaava
      Maailman menoa
      45
      3715
    10. SDP:n kansanedustaja Marko Asell: Suomen myönnettävä maahanmuuton ongelmat

      Hänen mielestään Suomen pitää pyrkiä rajoittamaan jyrkästi turvapaikanhakijoiden pääsyä maahan ja hän arvioi, että maaha
      Maailman menoa
      129
      3608
    Aihe