I am trying to code a portion of code that gets the elements from two distinct lists and make a match, as you can see below, but for some reason, I keep getting repeated elements on my output lists.
def assign_tasks(operators, requests, current_time):
"""Assign operators to pending requests.
Requires:
- operators, a collection of operators, structured as the output of
filesReading.read_operators_file;
- requests, a list of requests, structured as the output of filesReading.read_requests_file;
- current_time, str with the HH:MM representation of the time for this update step.
Ensures: a list of assignments of operators to requests, according to the conditions indicated
in the general specification (omitted here for the sake of readability).
"""
operators = sorted(operators, key=itemgetter(3, 4, 0), reverse=False)
requests = sorted(requests, key=itemgetter(3), reverse=True)
isAssigned = 0
tasks = []
langr = 0 #Variable that gets the language of the request's file (customer's language)
lango = 0 #Variable that gets the language of the operator's file (operator's language)
for i in range(len(requests)-1):
langr = requests[i][1] #What language does the customer speaks?
for k in range(len(operators)-1):
lango = operators[k][1] #What language does the operator speaks?
if langr == lango: #Do they speak the same language?
for j in range(len(operators[k][2])-1):
if (operators[k][2][j] == requests[i][2]) and (operators[k][4] <= 240): # The operator knows how to solve the client's problem? If yes, then group them together.
a = operators[k][2][j]
b = requests[i][2]
tasks.append([current_time, requests[i][0], operators[k][0]])
operator_time = operators[k][4]
request_time = requests[i][4]
new_operator_time = operator_time + request_time
operators[k][4] = new_operator_time
isAssigned == True
#operators.remove(operators[k])
requests.remove(requests[i])
else:
isAssigned = False
if isAssigned == False:
tasks.append([current_time, requests[i][0], "not-assigned"])
operators = sorted(operators, key=itemgetter(3, 4, 0), reverse=False)
return tasks, operators, requests
My current input is this:
operators = [['Atilio Moreno', 'portuguese', ('laptops',), '10:58', 104], ['Leticia Ferreira', 'portuguese', ('laptops',), '11:03', 15], ['Ruth Falk', 'german', ('phones', 'hifi'), '11:06', 150], ['Marianne Thibault', 'french', ('phones',), '11:09', 230], ['Mariana Santana', 'portuguese', ('phones',), '11:11', 230], ['Beate Adenauer', 'german', ('hifi', 'phones'), '11:12', 140], ['Zdenka Sedlak', 'czech', ('phones',), '11:13', 56], ['Romana Cerveny', 'czech', ('phones',), '11:13', 213]]
requests = [['Christina Holtzer', 'german', 'hifi', 'fremium', 7], ['Andrej Hlavac', 'czech', 'phones', 'fremium', 9], ['Dulce Chaves', 'portuguese', 'laptops', 'fremium', 15], ['Otavio Santiago', 'portuguese', 'laptops', 'fremium', 15], ['Dina Silveira', 'portuguese', 'phones', 'fremium', 9], ['Rafael Kaluza', 'slovenian', 'laptops', 'fremium', 13], ['Sabina Rosario', 'portuguese', 'laptops', 'fremium', 10], ['Nuno Rodrigues', 'portuguese', 'laptops', 'fremium', 12], ['Feliciano Santos', 'portuguese', 'phones', 'fremium', 12]]
current_time = "14:55 06:11:2017"
print(assign_tasks(operators, requests, current_time))
My current output is three lists where, for example, the first one is something like this:
[[11:05, Christina Holtzer, not-assigned],[11:05, Christina Holtzer, Beate Adenauer],[11:05, Andrej Hlavac, not-assigned]]
for i in range(len(x) - 1)
will iterate over all but the last index in some listx
... But I think you are getting repetition because you are looping over the product of your two lists, where I think you want every distinct pair instead...