Hangman

[metro17]

def split(word): 
    return list(word)

def cw():
    n = 0
    print("Welcome to Hangman !")
    print("You have 8 guesses left")
    Alphabets = ['a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z']
    TAlphabets = ['a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z']*5
    print("Available letters are:",Alphabets)
    word = input("Please store word: ")
    lettersinword = split(word)
    print(lettersinword)
    print("The word in list format is", lettersinword)
    print(type(lettersinword))
    wordlength = len(lettersinword)
    print("Length of the word is: ",wordlength)
    print("Length of word as alphabets in list is: ",wordlength)
    InitialEmptyList = []*wordlength
    print(type(InitialEmptyList))
    print("The length of the new list is: ", len(InitialEmptyList))
    print("This is the list where the guessed letters will be filled in :",len(InitialEmptyList))
    Failedattempts =  8
    ConInitialEmptyList=''
    print("Number of failed attempts allowed is :",Failedattempts)
    while n < Failedattempts and ConInitialEmptyList!=word:
        letter = input("Please guess the letter :")
        if letter not in (lettersinword):
            print("Letter not in word :", letter)
            n = n+1
            print("You have exhausted",n,"out of",Failedattempts,"attempts")
        else:
            print("Letter in word :", letter)
            x = lettersinword.index(letter)
            print("Position of the provided letter is :",x)
            TAlphabets.remove(letter)
            print("Available letters are: " ,TAlphabets)
            InitialEmptyList.insert(x,letter)
            print("You have guessed word:",InitialEmptyList)
            ConInitialEmptyList = ''.join(InitialEmptyList)
            print(type(ConInitialEmptyList))
            print(ConInitialEmptyList)
            print("The word you have guessed is :",ConInitialEmptyList)

I am trying build an easy version for hangman.I had to make 5 lists of alphabets to make
sure I have enough alphabets
for words having repeating letters.
Once I guess a letter, I should be able to identify all indexes of same letter and
insert all the repeatitions of the letter in InitialEmptyList as above in a single guess.
How do I do this.
Thanks

[ichabod801]

When you start InitialEmptyList, an empty list times 5 is just an empty list. I would do something like ['-'] * len(word), so you can later assign by index.

I'm not sure why you have lettersinword. The 'in' operator and the index method both work fine on strings, you don't need to convert the word to a list to use them.

Put those two points together, and when you find a word it can be assgined with InitialEmptyList[word.index(letter)] = letter. That part should be in a loop though, so you can find multiple instances of the letter. Note that index takes a start parameter, which tells it where in the sequence to start looking. The loop would look like:

start = 0
while letter in word[start:]:
    location = word.index(letter, start)
    InitialEmptyList[location] = letter
    start = letter + 1

Your variable names are confusing. For one thing, some are all lower case, and others are camel case. Pick one and stick with it. While it's good that you are using descriptive variable names, the descriptions are confusing. Frex, InitialEmptyList is the guessed word so far, so I would call it guessed_word, or even guess.

[metro17]

This is what I ended up with and it works good.
Please advise more efficient ways.

Appreciate the responses.

def cw():
n = 0
print("Welcome to Hangman !")
print("You have 8 guesses left")
word = input("Please store word: ")
lettersinword = split(word)
print("The letters in the word are: ",lettersinword)
wordlength=len(lettersinword)
InitialEmptyList = ['']*wordlength
print("The length of the new list is :",len(InitialEmptyList))
print(InitialEmptyList)
Failedattempts = 8
ConInitialEmptyList=''
print("Number of failed attempts allowed is :",Failedattempts)
while n < Failedattempts and ConInitialEmptyList!=word:
letter = input("Please guess the letter :")
if letter not in (lettersinword):
print("Letter not in word :", letter)
n = n+1
print("You have exhausted",n,"out of",Failedattempts,"attempts")
else:
print("Letter in word :", letter)
x = lettersinword.index(letter)
print("Position of the provided letter is :",x)
lettersinword[x]=''
print("Available letters in the word are :",lettersinword)
InitialEmptyList[x]=letter
print("Filled letters in InitialEmptyList :",InitialEmptyList)
ConInitialEmptyList = ''.join(InitialEmptyList)
print(type(ConInitialEmptyList))
print(ConInitialEmptyList)
print("The word you have guessed is :",ConInitialEmptyList)

[buran]

You have been repeatedly advised to use proper tags when post code, traceback, output, etc.
See BBcode help for more info.

[perfringo]

This is what I ended up with and it works good.
Please advise more efficient ways.
Appreciate the responses.

You say that it works good but i have doubts. Starting from line lettersinword = split(word). What is 'split'?

I concur with buran - if you are looking for help you should put your code into appropriate tags.

If you expect responses then people have to read your code. Therefore you should follow conventions set in PEP8 >>> Function and Variable Names:

Function names should be lowercase, with words separated by underscores as necessary to improve readability.

Variable names follow the same convention as function names.

mixedCase is allowed only in contexts where that's already the prevailing style (e.g. threading.py), to retain backwards compatibility.

I personally dislike 'ConInitialEmptyLis'. I don't use things that I dislike and in current context it means that I am less inclined to help than otherwise (but it just me).

However, I hope that you post runnable code in correct tags for review. I promise to give my feedback.

Meanwhile I can give you some ideas how to avoid long typing while creating list of alphabet characters (which was in your initial code).

(1) Save strokes on quotes by splitting string:

>>> 'a b c d e'.split()
['a', 'b', 'c', 'd', 'e']

(2) Save strokes by using codepoints:

>>> [chr(i) for i in range(97, 123)]
['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z']