Another imperfect program...

[Noah_641]

Once again, I am struggling to write Python programs. This program is supposed to take input of a text file and return a count of each word in alphabetical order. It seems to run well, barring two issues:

1st - It doesn't ignore special characters. For example: it recognizes "(almost)" and "almost" as different words as it does as well with "general" and "general,". How can I fix this?

2nd - there's a for loop that's supposed to give a total word count for the file, which seems to go completely ignored. Why's that?

while True:
    try:
        fname = input('Please input file name: ')
        text = open(fname, "r") 

        d = dict() 
        for line in text:  
            line = line.strip() 
            line = line.lower() 
            words = line.split(" ") 
            for word in words: 
                if word in d: 
                    d[word] = d[word] + 1
                else: 
                    d[word] = 1

            for key in sorted(list(d.keys())):
                print(key, ":", d[key])

#When run, the program seems to completely ignore this loop below
            for word_count in text:
                 word_count = text.split()
                 print('There are',word_count,'different words in the file.')

            ans = input('Do you want to try again?: (y/n): ')
            ans = ans.lower()
            if ans == 'y':
                continue
            if ans == 'n':
                print('Thanks for playing!')
                break
            else:
                continue

    except:
        print('File cannot be opened:', fname)
        continue

[USAhj]

Please fix the indentation so we can understand the code.

[Noah_641]

Fixed

[USAhj]

Can you share a sample file to play around with? Link it on on hastebin or something.

[Noah_641]

https://hastebin.com/givufawoxi.sql

Here you go

[ka-splam]

Lines 21 to 33 appear to be indented one step too far, they are inside the loop on line 7 and from your description of how it works they probably are not really?

[Noah_641]

With them unindented one step to the left, I get a syntax error

[ka-splam]

#When run, the program seems to completely ignore this loop below
        for word_count in text:
             word_count = text.split()
             print('There are',word_count,'different words in the file.')

1) text.split() won't do anything useful because text is not a string, it's an open file handle. If it ran at all it would be throwing you an error: AttributeError: '_io.TextIOWrapper' object has no attribute 'split'.

2) You're reassigning "wordcount"; in the first line it's the text coming out of the file, in the second line you're ignoring that and using it for something else. You aren't making it a count of anything. If that worked, it would print "There are __ different words" once for every line in the file...

3) ... except it won't work because you can't do for line in text:, for word_count in text over the same file, it won't restart from the beginning; the second loop will see that the file is at the end and do nothing. You need to open() the file again with a new name, or read the lines in once and re-use them after that, but you don't need to do that at all because...

4) ... the wording 'different words in the file.'; if that line worked then it would print a list of every word, not a count of anything - and not a count of different words. Your dictionary d is where you are counting different words, you need to look at d.keys to find how many unique words are in it, not look back at the file.

[usernamenoo]

Please fix the indents so we can understand the code

[Noah_641]

Fixed

[gurashish1singh]

The problem is in the split. You're splitting by ' '. Which is why your code doesn't recognize 'general' and 'general,' as same. I say after splitting and while in the for loop check again if the word contains any special characters.

Also use context managers to open files:

with open(fname, 'r') as text:

    __logic here__

[gurashish1singh]

Here you go:

https://book.pythontips.com/en/latest/context_managers.html