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

643

    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. MIksi suurin osa persuista on alkoholisteja?

      Tätä jäin pohdiskelemaan....
      Maailman menoa
      94
      4165
    2. 67
      3421
    3. Putin lähti takki auki sotaan....

      Luuli, että kolmessa päivässä hoidetaan, nyt on mennyt 3,5 vuotta eikä voitosta tietoakaan. Kaiken lisäksi putin luuli,
      Maailman menoa
      72
      3064
    4. SDP ylivoimainen ykkönen

      En ole koskaan viitsinyt käydä äänestämässä, mutta nyt SDP:n etumatka on niin kutkuttava, että pakkohan se on vaivautua.
      Maailman menoa
      81
      2539
    5. Suomalainen lähti ilman takkia sotaan

      Malli oli "kajander", johon kuului pelkästään kokardi, ja sotilaan itse metsästä katkoma puukeppi ryssien huitomiseen.
      Maailman menoa
      5
      2346
    6. Polttomoottoriauto on köyhän merkki

      Kun ei ole varaa ostaa sähköautoa, niin joutuu köyhän autoa käyttämään.
      Maailman menoa
      203
      2126
    7. Patteriauton ovia ei saatu auki - kuljettaja koki hirvittävän kuoleman!

      ”Oviongelma johti kuskin kuolemaan kolarissa – tämä ratkaisu saatetaan kieltää kokonaan Sivulliset pyrkivät tempomaan a
      Maailman menoa
      30
      2061
    8. Mies älä

      Odota enää vaan toimi. Pieni vinkkivitonen 🫰💥
      Ikävä
      41
      860
    9. Harmittaa ettei paluuta entiseen enää ole

      Paluuta entiseen ei ole koska pilasit kaiken.
      Ikävä
      73
      781
    10. Mä tiedän

      Että sä tiedät, että mä tiedän, että sä tiedät. Me molemmat tiedetään. Onko näin?
      Ikävä
      52
      756
    Aihe