Parsing a string to find element

Question

#!/usr/bin/python

def split(x):
    return [char for char in x]
def func(x):
    a = []
    a.append(split(x))
    for i in a:
        if a[i]== "\"":
            print ("hw")
str = "string\""
print (str)
func(str)

Trying to implement a function which finds " in a string and prints "hw" when finds it. What could be the problem?

In terms of your question, you're using `append` instead of `extend` to add a collection of elements to your list (amongst other things..) — Sayse, Oct 15 '19 at 13:02
@Sayse if a[i]== '"': TypeError: list indices must be integers, not list — ccvhd, Oct 15 '19 at 13:04
When you do `for i in a` you are iterating over a "list" (for lack of a better name) of strings. — Dani Mesejo, Oct 15 '19 at 13:06

Siro · Accepted Answer · 2019-10-15T13:09:45.910

list indices must be integers, not list.

I modify your code, so, try code below:

def split(x):
    return [char for char in x]
def func(x):
    a = split(x)  # notice this line
    for i in range(len(a)):
        if a[i]== "\"":
            print ("hw")
str1 = "string\""
print (str1)
func(str1)

@See this Ouput Demo

Or:

def split(x):
    return [char for char in x]
def func(x):
    a = split(x)
    for i in a:
        if i == "\"":
            print ("hw")
str1 = "string\""
print (str1)
func(str1)

Or:

str1 = "string\""
test = ["hw" for i in str1 if i == '"']
print(test[0])

score 0 · Answer 2 · edited Jun 20 '20 at 09:12

0

UPDATE

Actually it is much simpler than you think. First of all split is a built-in function. Secondly the for i in a checks every character of string a you do not need an index. Try the following:

s = "string\""
print("hw" if "\"" in s else None)

if you need to also get the indexes of the found characters you can do this:

s = "string\"\""
print([(i, "hw") for i, j in enumerate(s) if j == "\""])

edited Jun 20 '20 at 09:12

Community

1
1

answered Oct 15 '19 at 13:04

Kostas Charitidis

2,991
1
12
23

How can i get the index of an element? – ccvhd Oct 15 '19 at 13:10
@dhvcc updated the answer so you can get the indexes too. – Kostas Charitidis Oct 15 '19 at 13:26

Parsing a string to find element

2 Answers2

UPDATE