.joindre une sortie de tableau

Question

Je suis en train de créer un chiffreur de base qui est sorti dans un tableau et non une chaîne. Je suppose que je dois utiliser la méthode .join mais pour la vie de moi ne peut pas trouver où, sans obtenir une erreur.

class Encryptor 
    def cipher 
    {'a' => 'n', 'b' => 'o', 'c' => 'p', 'd' => 'q', 
    'e' => 'r', 'f' => 's', 'g' => 't', 'h' => 'u', 
    'i' => 'v', 'j' => 'w', 'k' => 'x', 'l' => 'y', 
    'm' => 'z', 'n' => 'a', 'o' => 'b', 'p' => 'c', 
    'q' => 'd', 'r' => 'e', 's' => 'f', 't' => 'g', 
    'u' => 'h', 'v' => 'i', 'w' => 'j', 'x' => 'k', 
    'y' => 'l', 'z' => 'm'} 
    end 

    def encrypt_letter(letter) 
    lowercase_letter = letter.downcase 
    end 

    def encrypt(string) 
    letters = string.split("") 

    letters.collect do |letter| 
     encrypted_letter = encrypt_letter(letter) 
    end 
    end 

end 

Answer 1

Vous pouvez resserrer votre méthode encrypt_letter en se souvenant que la dernière valeur évaluée dans la méthode est également la valeur de retour.

def encrypt_letter(letter) 
    cipher[letter.downcase] 
end 

Encryptor.new.encrypt_letter('h') #=> "u" 

En outre, la méthode collect retourne en fait un tableau de toutes les valeurs renvoyées par le bloc (la dernière valeur évaluée par le bloc) donc il n'y a pas besoin d'assigner à une variable dans le bloc. Puisque vous avez le tableau de collect (qui est seulement toutes les lettres chiffrées, appelez join sur ce (et puisque c'est l'évaluation finale de la méthode, il est la valeur de retour).

def encrypt(string) 
    letters = string.split("") 

    letters.collect {|letter| encrypt_letter(letter) }.join 
end 

Encryptor.new.encrypt("Hello") #=> "uryyb" 

Techniquement, vous pourriez même juste enlever la variable letters et de le faire en une seule ligne mais je pense personnellement qu'il est un peu plus lisible de cette façon

à mon humble avis.

vous pourriez probablement faire toutes les méthodes de la classe des méthodes puisque vous ne stockent aucun variables d'instance et il ne semble pas y avoir de raison de le garder en dehors de simplement chiffrer une chaîne.

Answer 2

class Encryptor 
    def cipher 
    {'a' => 'n', 'b' => 'o', 'c' => 'p', 'd' => 'q', 
     'e' => 'r', 'f' => 's', 'g' => 't', 'h' => 'u', 
     'i' => 'v', 'j' => 'w', 'k' => 'x', 'l' => 'y', 
     'm' => 'z', 'n' => 'a', 'o' => 'b', 'p' => 'c', 
     'q' => 'd', 'r' => 'e', 's' => 'f', 't' => 'g', 
     'u' => 'h', 'v' => 'i', 'w' => 'j', 'x' => 'k', 
     'y' => 'l', 'z' => 'm'} 
    end 

    def encrypt_letter(letter) 
    lowercase_letter = cipher[letter.downcase] #each letter passed is crypted here 
    end 

    def encrypt(string) 
    letters = string.split("") 

    encrypted_letter = [] #define an array to store each encrypted char 

    letters.collect do |letter|   
     encrypted_letter << encrypt_letter(letter) #accumulate encrypted chars in the array 
    end 
    encrypted_letter.join #now time to use join to form a string and return it 
    end 
end 

Encryptor.new.encrypt("something") #=> "fbzrguvat"