
Xenos
08.07.2016
11:58:57
и что ты предлагаешь тогда?

H
08.07.2016
11:59:00
on_page как раз надо вынести в дефолтную переменную

Andrey
08.07.2016
11:59:02
ненужная запятая после self

H
08.07.2016
11:59:03
...

Google

H
08.07.2016
11:59:49
там же тупл! ?

Xenos
08.07.2016
12:00:00
недорефакторил

Andrey
08.07.2016
12:01:05
да и json.loads делай в self.method

Zahar
08.07.2016
12:01:19
Как тюпла на русском?

Andrey
08.07.2016
12:01:28
кортеж

Zahar
08.07.2016
12:01:31
Хотя зачем мне это
А
А, ок

Xenos
08.07.2016
12:01:45
у меня много таких методов, self.method обрабатывает в зависимости от первого аргумента
и в каждом json.loads может по разному обрабатываться, особенно где надо делать несколько запросов за метод

Andrey
08.07.2016
12:02:19
ну ты в этом методе делаешь два раза data = json.loads(response.text)
ну сделай параметром

Kirill
08.07.2016
12:03:09

Google

Kirill
08.07.2016
12:03:14
http://onevpn.co
есть и GUI, и конфиги для желающих

Eugene
08.07.2016
12:04:01

Xenos
08.07.2016
12:04:09
там в response может придти, что есть несколько страниц, и мне нужно будет делать несколько запросов в зависимости от количества страниц. а метод в котором такое может быть - один. т.е. ты предлагаешь и такую логику запрятать в self.method?

Kirill
08.07.2016
12:04:52

Andrey
08.07.2016
12:04:56
я не знаю что тебе надо делать, а сужу только по одному методу

Kirill
08.07.2016
12:04:57
и не будет

Eugene
08.07.2016
12:05:07
устройста

Andrey
08.07.2016
12:05:55
если у тебя в каждом методе есть data = json.loads(response.text), то почему бы не делать это в одном месте

Xenos
08.07.2016
12:06:12
http://pastebin.com/xF3TCSWH на импорты не обращай внимания, рефакторю старую телегу

H
08.07.2016
12:06:40
http://pastebin.com/4xPHzHQC

Xenos
08.07.2016
12:06:42
это часть обертки над yandex domain mail

H
08.07.2016
12:06:45
немного поправил

Xenos
08.07.2016
12:06:58
там к сожалению максимум 100
и всегда нужно максимум
минимум - не нужно

H
08.07.2016
12:07:19
ну и что?
ничего не меняет

Andrey
08.07.2016
12:09:06
requests тебе может сразу json body отдать

Google

H
08.07.2016
12:09:30
хорошее название для метода - метод, который принимает переменную метод
мне нравится ?

Andrey
08.07.2016
12:09:45
ф - фантазия

Xenos
08.07.2016
12:09:54
ато

H
08.07.2016
12:10:15
There are only two hard things in Computer Science: cache invalidation and naming things.
— Phil Karlton

Andrey
08.07.2016
12:10:15
но вообще да, метод то не отражает суть того что делает

H
08.07.2016
12:10:31
как не отражает
он же метод
поэтому и назван метод
если бы была функция, можно было бы написать def function():

Andrey
08.07.2016
12:11:12
тогда странно что остальные не названы methodN

H
08.07.2016
12:11:15
но там то метод
и вообще ненужный класс
ооп головного мозга

Xenos
08.07.2016
12:12:19
так проще говнокодить

H
08.07.2016
12:13:19
как раз таки нет
функциями проще

Xenos
08.07.2016
12:13:26
темболее когда ты не погромист, а сисадмин, и пишешь не для "меющих разбираться в чужом коде"

Akamit
08.07.2016
12:14:03
подскажите пожалуйста, как в jinja2 присвоить значение существующему хешу?
{% set host['name'] = '123' %} - так не прокатывает - ansible ругается AnsibleError: template error while templating string: expected token 'end of statement block'

Google

Xenos
08.07.2016
12:14:29
до этого на функциях было написано, решил в ооп переписать, чтобы при инициализации, моя телега получала все данные от домена, и в любой момент можно было дернуть любую часть. а не вызывать каждый раз нужную функцию

H
08.07.2016
12:16:20
что ты тогда про красоту волнуешься val + 1, когда у тебя код сам говном воняет? :)
дело не в val + 1

Akamit
08.07.2016
12:18:25
просто {% set host = '123' %} работает. видимо хешу как-то по другому присваивать? do host.name = '123' тоже пробовал

H
08.07.2016
12:20:30
ну надо через точку, такую же ошибку выдает?

Admin
ERROR: S client not available

Akamit
08.07.2016
12:21:01

H
08.07.2016
12:23:18
а через set host.name = '123' пробовал?

Akamit
08.07.2016
12:24:32

H
08.07.2016
12:24:55
а do host.update({'name', '123'}) ?
только там дикт

Akamit
08.07.2016
12:25:22
пробовал так host.update({'name'}, {'123'})

H
08.07.2016
12:25:27
нет
отредактировал

Akamit
08.07.2016
12:27:49

H
08.07.2016
12:28:01
скинь пример
на pastebin

Google

Akamit
08.07.2016
12:29:10
это и есть весь пример

H
08.07.2016
12:29:58
In [2]: {% set host = '123' %}
File "<ipython-input-2-8de4fa890913>", line 1
{% set host = '123' %}
^
SyntaxError: invalid syntax
у меня и этот пример не работает тогда

Igor
08.07.2016
12:31:28
?

Akamit
08.07.2016
12:32:36

H
08.07.2016
12:33:06
а, точно
у тебя ж ансибл

Andrey
08.07.2016
12:33:31

H
08.07.2016
12:33:42
у меня нигде не установлен сейчас, напиши в @devops_ru
или @ru_devops

Akamit
08.07.2016
12:34:05
оттуда и отправили)))

H
08.07.2016
12:34:18
скажи что тебя обратно отправили
в jinja2 работает

Andrey
08.07.2016
12:34:33
так а нахрен ты его в консоль питона вставлял
да, я уже понял. это было слишком тонко для меня

H
08.07.2016
12:35:15
ясно

Akamit
08.07.2016
12:48:29
а есть чат по ansible?)

Игорь
08.07.2016
12:49:03

Akamit
08.07.2016
12:49:52