Python：无法从函数范围正确返回变量

Question

I am trying to have the user input two names into variables. These variables will be mixed with scramble-function. Then I want to have them seperated again with the unscramble function but I want the unscramble function to recognise which of the two words was the first input and which the second input.. but I am stuck with scope and have no clue how to solve this correctly.

I appreciate any solution!

a = raw_input("First word please?")
b = raw_input("Second word please?")

def scramble(word1, word2):

    if len(word1) > len(word2):
        a1 = word1
        a2 = word2
    else:
        a1 = word2
        a2 = word1

    maxlength = max( len(word1),len(word2) )
    magicWord = ""

    for x in range(0,maxlength):

        magicWord += a1[x]

        if x < len(a2)-1:
            magicWord += a2[x]

        elif x == len(a1)-1:
            fin = len(a2)-1
            magicWord += a2[fin]

        else:
            magicWord += "0"

    return magicWord

magicWord = scramble(a,b)
print "\nYour magic word is: \""+magicWord+"\"\n"

magicSize = len(magicWord)



def unscramble(magicWord):

    z = 0

    if z == magicSize:
        return False

    else:
        while unscramble(magicWord):

            word1 = ""
            word2 = ""

            if z % 2 == 1:
                if magicWord[z] == "0":
                    z+= 1
                else:
                    word2 += magicWord[z]
                    z += 1
            else:
                if magicWord[z] == "0":
                    z+= 1
                else:
                    word1 += magicWord[z]
                    z += 1

            return word1

if word1 > word2:
    print "First input: " + word1
    print "Second input: " + word2
else:
    print "First input: " + word2
    print "Second input: " + word1

Answer 1

After

 if len(word1) > len(word2):
     a1 = word1
     a2 = word2
 else:
     a1 = word2
     a2 = word1

the original order is lost. Without this it's quite easy to reconstruct the order, as your scramble function always puts the first letter of the first word first.

Then change your unscramble function to return word1, word2 and you get the words back in the original order.

By the way, you should have a look at itertools.zip_longest() . It would make your scramble function easier to read. for char1, char2 in itertools.zip_longest(word1, word2, fillvalue='0') loops through all letters in both words and uses '0' for the missing letters in the shorter word.

Answer 2

You never got rid of a or b , so you can use them to know which word was first and which was second. There's no need to manually unscramble them. I would use a tuple to store them after scramble , so instead of:

magicWord = scramble(a,b)

I would just do:

magicWord_tuple = (a, b, scramble(a,b))

Then you just pull out whichever word you want when you need it.

firstword = magicWord_tuple[0]
secondword = magicWord_tuple[1]
magicword = magicWord_tuple[2]

Answer 3

I'm sorry, but your unscramble function cannot recover the words any more, let alone tell which one was entered first. You've destroyed information in your scramble function.

In scramble, your first few lines destroy the ordering; since you never again refer to word1 and word2 (the only indicator of order), that information is lost. Perhaps you would want to use a different separator (where you insert a "0") to denote which word is which. This insertion will let the unscramble routine differentiate.

if len(word1) > len(word2):
    a1, a2 = word1, word2
    sep = '1'
else:
    a1, a2 = word2, word1
    sep = '2'
...
   else:
        magicWord += sep

There are several other coding improvements we could make to be more "Pythonic", but we're doing functionality right now; code review goes into another group.

Answer 4

maybe you can have an additional variable saying you which word enters first in the magic word, and pass this variable to the unscramble function to do the separation of the words. You can change the scramble word like this:

if len(word1) > len(word2):
    flag = 1
    a1 = word1
    a2 = word2
else:
    flag = -1
    a1 = word2
    a2 = word1

...

return magicWord, flag

and the unscramble will take the flag to recover the correct order of the words. This is the only solution, since when you create the magicword you loose the information about which is the first and which is the second word. This means that your unscramble function cannot work without this information.