Tibor Drèvis ha detto:

Più per curiosità che per altro ho pensato sarebbe stato carino creare un programma di criptaggio per i messaggi e magari automatizzarlo etc. Ho scritto questi due codici in una mezz'oretta circa quindi cioé, può essere migliorato molto. Non sono riuscito a rendere criptabili anche i simboli, però mi sarebbe piaciuto parecchio, se avete qualche idea su come rendere questo programma un po' più fluido fatemelo sapere, so che ruby di norma è parecchio lento, ma questo l'ho trovato esagerato.


Qui di seguito l'encrypter:

Ruby:

puts("Seleziona la chiave personalizzata che vuoi utilizzare, e ricorda! Devono essere tutte le lettere dell'alfabeto, e non usare né numeri né simboli.")
p_key=gets.chomp.to_s
puts("============================================================================")
puts("Dopodiché inserisci il messaggio che vuoi criptare con la chiave selezionata")
msg=gets.chomp.to_s
msg=msg.downcase
alphabet="abcdefghijklmnopqrstuvwxyz"
encr_msg=""

for char in (0..msg.length-1)
    if (msg[char]==" ")
        encr_msg<< " "
    end
    for x in (0..alphabet.length-1)
        if (msg[char]==alphabet[x])
            encr_msg<<p_key[x]
        end
    end
end



puts(encr_msg)
puts("Non scordare la chiave per decriptare il messaggio: ", p_key)

e qui il decrypter:

Ruby:

print("Inserisci il messaggio che desideri descriptare: ")
encr_msg=gets.chomp.to_s
encr_msg=encr_msg.downcase
puts("Inserisci la chiave per decriptare il messaggio! E ricorda, solo lettere.")
p_key=gets.chomp.to_s
decr_msg=""
alphabet="abcdefghijklmnopqrstuvwxyz"

for char in (0..encr_msg.length-1)
    if (encr_msg[char]==" ")
        decr_msg << " "
    end
    for x in (0..p_key.length-1)
        if (encr_msg[char]==p_key[x])
            decr_msg<<  alphabet[x]
        end
    end
end

puts("Il messaggio decriptato è: ", decr_msg)

Fatemi anche sapere se potete se funziona alla perfezione!! Ho notato che a volte ripete qualche lettera, ma non sono sicuro se fosse dovuto alla chiave che ho scelto o altro.

Clicca per espandere...

Tibor Drèvis ha detto:

Sì, ero a conoscenza dell'each loop, avevo provato inizialmente ad usarlo perché mi sembrava più semplice, poi però ho cambiato idea perché per qualche ragione non mi riusciva, allora ho risalito la strada diciamo più ''ostica'' provando con i for loop, tenendo a mente che sarei riuscito facilmente anche a tradurlo in Python, perché più o meno era anche questo il mio scopo. Comunque sì, ho iniziato relativamente da poco, circa da 2 mesi con un tutor

Clicca per espandere...

TheSeeker ha detto:

Non so se hai iniziato Ruby da poco ma ci sono alcune cose, dal punto di vista stilistico che puoi migliorare:

1. for non viene usato, si usa invece each:

Codice:

(A..Z).each do |letter|
    puts letter
end

Inoltre, come hai visto dall'esempio precedente, Ruby permette di definire una lista delle lettere dell'alfabeto come un range (non c'è bisogno di scriverle una ad una).

2. Le parentesi intorno alla condizione del if non si mettono mai siccome non sono richieste.

3. Invece di 0..msg.length - 1 puoi fare:

Codice:

0.upto(message.size) do |index|
    # ...
end

Ti consiglio di guardarti qualche styleguide: https://github.com/bbatsov/ruby-style-guide

Clicca per espandere...

Tibor Drèvis ha detto:

Ho notato che a volte ripete qualche lettera, ma non sono sicuro se fosse dovuto alla chiave che ho scelto o altro.

Clicca per espandere...

for char in (0..msg.length-1)
    if (msg[char]==" ")
        encr_msg<< " "
    end
    for x in (0..alphabet.length-1)
        if (msg[char]==alphabet[x])
            encr_msg<<p_key[x]
        end
    end
end

nullptr ha detto:

1. Il for viene pure utilizzato nel Ruby, ma l'each viene considerato il più peculiare al linguaggio. Considererei di utilizzare comunque l'each principalmente perchè l'indice è persistente solo in quella porzione del codice (nel contesto del loop).
2. Omettere le parentesi non fa nè caldo nè freddo. Io considererei di lasciarle stare.
3. Il tuo consiglio non migliorerebbe nulla, stai solo provando a rendere più elegante il codice inutilmente. Se fosse un po' troppo sporca la soluzione dal punto di vista di sintassi, allora passo a trovarmi un'alternativa che non cambi funzionalmente, ma il modo di scrivere il codice.

Clicca per espandere...

Tibor Drèvis ha detto:

Non è quello il problema

Clicca per espandere...

Tibor Drèvis ha detto:

Non è quello il problema, cioé, non riuscivo già da prima a criptare numeri e simboli

Clicca per espandere...

Tibor Drèvis ha detto:

esattamente lunga quanto la variabile alphabet="abcdefghijklmnopqrstuvwxyz" , senza nessuna ripetizione di lettere.

Clicca per espandere...

Tibor Drèvis ha detto:

Ho pubblicato il codice proprio per ricevere consigli, esponimi la tua soluzione!

Clicca per espandere...

if (msg[char]==" ")
        encr_msg<< " "
    end

if (msg[char]==" ")
        encr_msg<< " "
        next
    end

nullptr ha detto:

p.s: il seguente codice nel tuo primo loop:

Ruby:

if (msg[char]==" ")
        encr_msg<< " "
    end

prosegue inutilmente anche se la condizione è vera. Tu non noterai niente, ma contemporaneamente fai un altro loop inutilmente pur sapendo che lo spazio non è presente nel tuo charset. Dovrebbe essere:

Ruby:

if (msg[char]==" ")
        encr_msg<< " "
        next
    end

per passare subito alla successiva iterazione.

Clicca per espandere...

if msg[char] >= 'a' and msg[char] <= 'z'
  # codice per cifrare
else
  encr_msg << msg[char]
end

if alphabet.include? msg[char]
  # codce per cifrare
else
  encr_msg << msg[char]
end

St3ve ha detto:

O ancora meglio:

Ruby:

if msg[char] >= 'a' and msg[char] <= 'z'
  # codice per cifrare
else
  encr_msg << msg[char]
end

Clicca per espandere...

St3ve ha detto:

È poco utile perfezionare qualcosa di così corto e semplice, se inizi a fare qualcosa di un po' più intricato se ne riparla.

Clicca per espandere...