Alexis Nonya - 1 year ago 69

Python Question

I am trying to write a function that takes each element within two strings of equal length and multiply them by each other and then add each product to get a single answer.

vector1 = [1, 2, 3]

vector2 = [4, 5, 6]

For example the above would be (1*4) + (2*5) + (3 * 6) = 32

So far this is what I have:

`vector1 = [1, 2, 3]`

vector2 = [4, 5, 6]

ans = 0

if len(vector1) == 0 or len(vector2) == 0:

print("Invalid vectors")

elif len(vector1) != len(vector2):

print("Invalid vectors")

else:

for i in range (len(vector1)):

temp = vector1(i) * vector2(i)

ans = ans + temp

The code gives me the following error when compiling:

TypeError: 'list' object is not callable

I tried changing the above code to be more like

`vector1 = [1, 2, 3]`

vector2 = [4, 5, 6]

ans = 0

if len(vector1) == 0 or len(vector2) == 0:

print("Invalid vectors")

elif len(vector1) != len(vector2):

print("Invalid vectors")

else:

for i in range (len(vector1)) and i in range(len(vector2)):

temp = vector1(i) * vector2(i)

ans = ans + temp

But that doesn't work either. What am I doing wrong here?

Edit: After running the code through Python Visualizer, the following line is giving me the specific problem here:

temp = vector1(i) * vector2(i)

Recommended for you: Get network issues from **WhatsUp Gold**. **Not end users.**

Answer Source

*zip* and *sum*, you also need to cast to int presuming you have strings of digits:

```
def func(s1, s2):
if len(s1) != len(s2):
raise ValueError("Strings must be the same length")
return sum(int(i) * int(j) for i,j in zip(s1, s2))
```

zip would still work with uneven length strings but if they should be the same length then a ValueError is probably the most appropriate response.

The error in your own code is how you are trying to index:

```
temp = vector1(i) * vector2(i)
```

should be:

```
temp = vector1[i] * vector2[i]
```

`vector1(i)`

is trying to call the list as if it were a reference to a function. The `[i]`

actually indexes the list. This is tailor made for zip but if you were to use a for loop and index, you should use *enumerate:*

```
def func(s1, s2):
if len(s1) != len(s2):
raise ValueError("Strings must be the same length")
total = 0
for ind, ele in enumerate(s1):
total += int(ele) * int(s2[ind])
return total
```

Recommended from our users: **Dynamic Network Monitoring from WhatsUp Gold from IPSwitch**. ** Free Download**