Чистый код в SIL

[aleme]

Если будет вот такой код:

data += "new data";

data это string или string []? Непонятно. Нужно будет смотреть объявление переменной и тратить на это лишнее время. Если таких строк будет много, то это будет отвлекать.
Если будет вот такая запись:

data = addElement(data, "new data");

сразу понятно, что data это string [].
Конечно, можно договориться, что всем переменным типа массив добавлять Arr (dataArr) или добавлять s в конец (datas такого нет, поэтому придется что-то другое придумать). Но мне кажется, проще просто сделать код прозрачным. Да мы напишем чуть больше, но будет понятнее.

[aleme]

Любой рефакторинг субъективен. Из моего опыта для прозрачности нужно использовать addElement.

[ratnikov-in]

Красавчик! Только первичное авторство кода бы написал, что первый день с SIL

[aleme]

)))))

[million]

Для запроса и ответа создания страницы в Confluence можно использовать одну и туже структуру. Только в запросе полей меньше заполнять. А из ответа можно получить полезные вещи: код страницы и ссылку на страницу, чтобы потом можно было в задаче линк сделать.

[aleme]

Если использовать одну структуру, то нужно ей дать какое-то название, из которого будет понятно, что это и запрос и ответ.

[million]

Просто ConfluenceApiData если любите длинные названия