LINUX.ORG.RU

История изменений

Исправление panter_dsd, (текущая версия) :

Тем что никто кроме тебя не понимает как это работает. Ты пишешь код не для себя, а для тех людей которые его будут потом поддерживать (в т.ч. прямо сейчас на форуме мы это делаем). Код гораздо больше читают чем пишут.

Что конкретно непонятно в этом коде? Названия методов/параметров описывают свое назначение. Алгоритм понять тоже просто - сейчас непредвзято перечитал. Да, хранить в таске реквест было некрасивым решением, это я исправлю.

Полная неразбериха что откуда берётся для человека который видит это в первый раз. Дело не в формальном соотв. требованиям («коллизий нет»). Не гонись за формальными показателями, гораздо важнее на сколько удобно с кодом работать людям. Поэтому лучше писать сразу правильно.

Если бы мы говорили про c++, то да. Но в Пайтоне есть self при обращении к мемберу/методу класса, поэтому не понять что откуда достаточно сложно.

В данном случае можно тупо сделать на одной очереди (массиве). Если данные не смогли отправиться то тупо возвращаешь запрос в хвост очереди или же в отдельную очередь недоставленных запросов. Кода меньше, читать проще, производительность выше, тестировать легче.

Если бы все было так просто, сделал бы проще, но сам алгоритм работы сложнее.

Исправление panter_dsd, :

Тем что никто кроме тебя не понимает как это работает. Ты пишешь код не для себя, а для тех людей которые его будут потом поддерживать (в т.ч. прямо сейчас на форуме мы это делаем). Код гораздо больше читают чем пишут.

Что конкретно непонятно в этом коде? Названия методов/параметров описывают свое назначение. Алгоритм понять тоже просто - сейчас непредвзято перечитал. Да, хранить в таске реквест было некрасивым решением, это я исправлю.

Полная неразбериха что откуда берётся для человека который видит это в первый раз. Дело не в формальном соотв. требованиям («коллизий нет»). Не гонись за формальными показателями, гораздо важнее на сколько удобно с кодом работать людям. Поэтому лучше писать сразу правильно.

Если бы мы говорили про c++, то да. Но в Пайтоне есть self при обращении к мемберу/методу класса, поэтому не понять что откуда достаточно сложно.

Исправление panter_dsd, :

Тем что никто кроме тебя не понимает как это работает. Ты пишешь код не для себя, а для тех людей которые его будут потом поддерживать (в т.ч. прямо сейчас на форуме мы это делаем). Код гораздо больше читают чем пишут.

Что конкретно непонятно в этом коде? Названия методов/параметров описывают свое назначение. Алгоритм понять тоже просто - сейчас непредвзято перечитал. Да, хранить в таске реквест было некрасивым решением, это я исправлю.

Полная неразбериха что откуда берётся для человека который видит это в первый раз. Дело не в формальном соотв. требованиям («коллизий нет»). Не гонись за формальными показателями, гораздо важнее на сколько удобно с кодом работать людям. Поэтому лучше писать сразу правильно.

Если бы мы говорили про c++, то да. Но в Пайтоне есть self при обращении к мемберу/методу класса, поэтому непонять что откуда достаточно сложно.

Исходная версия panter_dsd, :

Тем что никто кроме тебя не понимает как это работает. Ты пишешь код не для себя, а для тех людей которые его будут потом поддерживать (в т.ч. прямо сейчас на форуме мы это делаем). Код гораздо больше читают чем пишут.

Что конкретно непонятно в этом коде? Названия методов/параметров описывают свое назначение. Алгоритм понять тоже просто - сейчас непредвзято перечитал. Да, хранить в таске реквест было некрасивым решением, это я исправлю.

Полная неразбериха что откуда берётся для человека который видит это в первый раз. Дело не в формальном соотв. требованиям («коллизий нет»). Не гонись за формальными показателями, гораздо важнее на сколько удобно с кодом работать людям. Поэтому лучше писать сразу правильно.