Processing a company list in which every element is an employee list












0












$begingroup$


Now, there is a company list in which an element is an employee list in every company. I calculate the salary of employees in every company with a RESTful API one by one,. This is very slow, so I join 1000 elements to one list and post it to the service.



import requests


def _calculate_employee_salarices(emploices):
respone = requests.post(url, json={"input": emploices})
salarices = parse_respone(respone)
return salarices


def get_calculate_employee_salarices(company_employees_lis):
begin, step = 0, 1000
company_employee_salarices_lis =
while begin < len(company_employees_lis):
employees =
employee_num_lis =
for comp_employees in company_employees_lis[begin:(begin+step)]:
employees.extend(comp_employees)
employee_num_lis.append(len(comp_employees))
salarices = _calculate_employee_salarices(employees)

idx = 0
for num in employee_num_lis:
company_employee_salarices_lis.append(salarices[idx:(idx + num)])
idx += num
begin += step
return company_employee_salarices_lis


if __name__ == "__main__":
company_employees_lis = [["employees_id_1", "employees_id_2", "employees_id_3"], ["employees_id_4"], ["employees_id_5", "employees_id_6"]]
company_employee_salarices_lis = get_calculate_employee_salarices(company_employees_lis)


I think my code is bad, because I use a variable employee_num_lis to remember the number of employees in every company. And there must be clean, artful and pythonic code. This code is just a sample, not what I should do.










share|improve this question









New contributor




ning chen is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$endgroup$












  • $begingroup$
    Thank you very much!
    $endgroup$
    – ning chen
    4 mins ago
















0












$begingroup$


Now, there is a company list in which an element is an employee list in every company. I calculate the salary of employees in every company with a RESTful API one by one,. This is very slow, so I join 1000 elements to one list and post it to the service.



import requests


def _calculate_employee_salarices(emploices):
respone = requests.post(url, json={"input": emploices})
salarices = parse_respone(respone)
return salarices


def get_calculate_employee_salarices(company_employees_lis):
begin, step = 0, 1000
company_employee_salarices_lis =
while begin < len(company_employees_lis):
employees =
employee_num_lis =
for comp_employees in company_employees_lis[begin:(begin+step)]:
employees.extend(comp_employees)
employee_num_lis.append(len(comp_employees))
salarices = _calculate_employee_salarices(employees)

idx = 0
for num in employee_num_lis:
company_employee_salarices_lis.append(salarices[idx:(idx + num)])
idx += num
begin += step
return company_employee_salarices_lis


if __name__ == "__main__":
company_employees_lis = [["employees_id_1", "employees_id_2", "employees_id_3"], ["employees_id_4"], ["employees_id_5", "employees_id_6"]]
company_employee_salarices_lis = get_calculate_employee_salarices(company_employees_lis)


I think my code is bad, because I use a variable employee_num_lis to remember the number of employees in every company. And there must be clean, artful and pythonic code. This code is just a sample, not what I should do.










share|improve this question









New contributor




ning chen is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$endgroup$












  • $begingroup$
    Thank you very much!
    $endgroup$
    – ning chen
    4 mins ago














0












0








0





$begingroup$


Now, there is a company list in which an element is an employee list in every company. I calculate the salary of employees in every company with a RESTful API one by one,. This is very slow, so I join 1000 elements to one list and post it to the service.



import requests


def _calculate_employee_salarices(emploices):
respone = requests.post(url, json={"input": emploices})
salarices = parse_respone(respone)
return salarices


def get_calculate_employee_salarices(company_employees_lis):
begin, step = 0, 1000
company_employee_salarices_lis =
while begin < len(company_employees_lis):
employees =
employee_num_lis =
for comp_employees in company_employees_lis[begin:(begin+step)]:
employees.extend(comp_employees)
employee_num_lis.append(len(comp_employees))
salarices = _calculate_employee_salarices(employees)

idx = 0
for num in employee_num_lis:
company_employee_salarices_lis.append(salarices[idx:(idx + num)])
idx += num
begin += step
return company_employee_salarices_lis


if __name__ == "__main__":
company_employees_lis = [["employees_id_1", "employees_id_2", "employees_id_3"], ["employees_id_4"], ["employees_id_5", "employees_id_6"]]
company_employee_salarices_lis = get_calculate_employee_salarices(company_employees_lis)


I think my code is bad, because I use a variable employee_num_lis to remember the number of employees in every company. And there must be clean, artful and pythonic code. This code is just a sample, not what I should do.










share|improve this question









New contributor




ning chen is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$endgroup$




Now, there is a company list in which an element is an employee list in every company. I calculate the salary of employees in every company with a RESTful API one by one,. This is very slow, so I join 1000 elements to one list and post it to the service.



import requests


def _calculate_employee_salarices(emploices):
respone = requests.post(url, json={"input": emploices})
salarices = parse_respone(respone)
return salarices


def get_calculate_employee_salarices(company_employees_lis):
begin, step = 0, 1000
company_employee_salarices_lis =
while begin < len(company_employees_lis):
employees =
employee_num_lis =
for comp_employees in company_employees_lis[begin:(begin+step)]:
employees.extend(comp_employees)
employee_num_lis.append(len(comp_employees))
salarices = _calculate_employee_salarices(employees)

idx = 0
for num in employee_num_lis:
company_employee_salarices_lis.append(salarices[idx:(idx + num)])
idx += num
begin += step
return company_employee_salarices_lis


if __name__ == "__main__":
company_employees_lis = [["employees_id_1", "employees_id_2", "employees_id_3"], ["employees_id_4"], ["employees_id_5", "employees_id_6"]]
company_employee_salarices_lis = get_calculate_employee_salarices(company_employees_lis)


I think my code is bad, because I use a variable employee_num_lis to remember the number of employees in every company. And there must be clean, artful and pythonic code. This code is just a sample, not what I should do.







python multithreading asynchronous






share|improve this question









New contributor




ning chen is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.











share|improve this question









New contributor




ning chen is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.









share|improve this question




share|improve this question








edited 31 mins ago









Jamal

30.3k11116226




30.3k11116226






New contributor




ning chen is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.









asked 1 hour ago









ning chenning chen

1




1




New contributor




ning chen is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.





New contributor





ning chen is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.






ning chen is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.












  • $begingroup$
    Thank you very much!
    $endgroup$
    – ning chen
    4 mins ago


















  • $begingroup$
    Thank you very much!
    $endgroup$
    – ning chen
    4 mins ago
















$begingroup$
Thank you very much!
$endgroup$
– ning chen
4 mins ago




$begingroup$
Thank you very much!
$endgroup$
– ning chen
4 mins ago










0






active

oldest

votes











Your Answer





StackExchange.ifUsing("editor", function () {
return StackExchange.using("mathjaxEditing", function () {
StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix) {
StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
});
});
}, "mathjax-editing");

StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");

StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "196"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});

function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});


}
});






ning chen is a new contributor. Be nice, and check out our Code of Conduct.










draft saved

draft discarded


















StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f211891%2fprocessing-a-company-list-in-which-every-element-is-an-employee-list%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown

























0






active

oldest

votes








0






active

oldest

votes









active

oldest

votes






active

oldest

votes








ning chen is a new contributor. Be nice, and check out our Code of Conduct.










draft saved

draft discarded


















ning chen is a new contributor. Be nice, and check out our Code of Conduct.













ning chen is a new contributor. Be nice, and check out our Code of Conduct.












ning chen is a new contributor. Be nice, and check out our Code of Conduct.
















Thanks for contributing an answer to Code Review Stack Exchange!


  • Please be sure to answer the question. Provide details and share your research!

But avoid



  • Asking for help, clarification, or responding to other answers.

  • Making statements based on opinion; back them up with references or personal experience.


Use MathJax to format equations. MathJax reference.


To learn more, see our tips on writing great answers.




draft saved


draft discarded














StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f211891%2fprocessing-a-company-list-in-which-every-element-is-an-employee-list%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown





















































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown

































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown







Popular posts from this blog

Сан-Квентин

8-я гвардейская общевойсковая армия

Алькесар