Issue with List count method

Question

The Program I need help is written below

def countHits(file):    
        f2 = open(file,'rU')    
        l2 = f2.readlines()    
        f2.close()    
        user_input = raw_input("Enter the URL that you wish to chck fr the nmber of Hits")     
        print "The number of HITS for the given STRING/URL is : %s"%(l2.count(user_input))

def main():    
        strin = raw_input("Enter the file name\n")    
        countHits(strin)

if __name__ == '__main__':

   main()

The file given as input contains list of urls (mentioned below):

/RLSsdTfswds/images//innercontainer-collapse.gif    
/RL/css/default.css    
/RLSTsdsdRdsdU/scripts/highslide/graphics/outlines/rounded-white.png    
/RLSsdsdTsdsRsddU/scripts/highslide/graphics/zoomout.cur    
/RLS/css/highslide/highslide/graphics/loader.white.gif    
/RL/css/default.css    
/RLST/rws/scripts/processschScript.js    
/RLSR/scripts/NumberFormat.js    
/RL/css/default.css

My query is that when I try to find url "/RL/css/default.css", the program doesn't gives me the count. Help me where I'm making a mistake ?

maybe list element contains \n or another endline and your string don't so they aren`t actuually equal — Darth Kotik
– Darth Kotik, Commented Nov 7, 2014 at 13:03
@ Darth Kotik I tried printing list elements individually and they don't show a "\n" at the end. — harveyD
– harveyD, Commented Nov 7, 2014 at 13:05
@Harvey, You could do this for addressed comment: i.imgur.com/8eIrY0D.png — user1630938
– user1630938, Commented Nov 7, 2014 at 14:20

Johannes S. · Accepted Answer · 2014-11-07 14:16:00Z

2

The following lines can be found in the documentation:

f.readline() reads a single line from the file;
a newline character (\n) is left at the end of the string,
and is only omitted on the last line of the file if the file doesn’t end in a newline.
This makes the return value unambiguous;
if f.readline() returns an empty string,
the end of the file has been reached,
while a blank line is represented by '\n',
a string containing only a single newline.

Though, you need to "sanitize" every line which is being read with file.readlines(),
like this for instance:

with open(file, 'r') as f :
    data = [x.strip() for x in f.readlines()]

And data will contain the list of lines, without tabs, spaces or newlines.

edited Nov 7, 2014 at 14:16

Johannes S.

4,66628 silver badges42 bronze badges

answered Nov 7, 2014 at 13:43

dcexcal

1951 silver badge7 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Darth Kotik · Accepted Answer · 2014-11-07 13:20:09Z

1

Possible solution:

def countHits(file):    
    f2 = open(file,'rU')    
    l2 = f2.readlines()    
    f2.close()
    l2 = [l.strip() for l in l2]
    user_input = raw_input(
       "Enter the URL that you wish to chck fr the nmber of Hits"
    )   
    print "The number of HITS for the given STRING/URL is : %s"% (
       l2.count(user_input)
    )

I guess this will work faster.

def countHits(file):    
    f2 = open(file,'rU')    
    l2 = f2.readlines()    
    f2.close()
    l2 = map(lambda x: x.strip(), l2)
    user_input = raw_input(
       "Enter the URL that you wish to chck fr the nmber of Hits"
    )   
    print "The number of HITS for the given STRING/URL is : %s"% (
       l2.count(user_input)
    )

edited Nov 7, 2014 at 13:20

answered Nov 7, 2014 at 13:05

Darth Kotik

2,3811 gold badge23 silver badges29 bronze badges

Comments

admdrew · Accepted Answer · 2014-11-07 15:09:24Z

0

Using List Comprehension is faster than lambda function.

 def countHits(file):
        f2 = open(file,'rU')
        l2 = f2.readlines()
        f2.close()
        user_input = raw_input(
          "Enter the URL that you wish to chck fr the nmber of Hits"
        ) 

        lst = [(s.strip()) for s in l2]
        print "The number of HITS for the given STRING/URL is : %s" % (
           lst.count(user_input.strip())
        )

def main():
        strin = raw_input("Enter the file name\n")
        countHits(strin)

if __name__ == '__main__':
   main()

edited Nov 7, 2014 at 15:09

admdrew

3,9444 gold badges30 silver badges41 bronze badges

answered Nov 7, 2014 at 13:27

Piyush Jajoo

1,0932 gold badges18 silver badges27 bronze badges

Comments

admdrew · Accepted Answer · 2014-11-07 15:09:29Z

0

Try to remove trailing whitespace from the input.

print "The number of HITS for the given STRING/URL is : %s" % (
   l2.count(user_input.strip())
)

edited Nov 7, 2014 at 15:09

admdrew

3,9444 gold badges30 silver badges41 bronze badges

answered Nov 7, 2014 at 13:04

user1907906

Collectives™ on Stack Overflow

Issue with List count method

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related