K
request.newBuilder()
new Request(…)Size: a a a
2020 July 22
AU
в конструктор кидать то, что мы в билдере используем?
QH
в конструктор кидать то, что мы в билдере используем?
ну да, не оставляя возможности сделать
new Builder().build() и свалиться, или написать глупость вроде new Builder().url(…).url(…).url(…)AU
но усложнять сбор запроса, если нужно чуть больше параметров воткнуть
K
А почему не использовать старый реквест, модифицировав его через .newBuilder()?
AU
так он не модифицируется, а мы через новый билдер создадим новый реквест
QH
А почему не использовать старый реквест, модифицировав его через .newBuilder()?
а где в условии значится, что какой-то реквест уже существует?)
AD
А почему не использовать старый реквест, модифицировав его через .newBuilder()?
val newRequest = oldRequest.copy(path = ''books/like'')
Kd
билдер тебе дает возможность передать недостроеный объект по цепочке каким то другим обработчикам чтобы они что нибудь туда добавили.
с конструктором такое не получится
с конструктором такое не получится
Kd
Konstantin dmz9
билдер тебе дает возможность передать недостроеный объект по цепочке каким то другим обработчикам чтобы они что нибудь туда добавили.
с конструктором такое не получится
с конструктором такое не получится
о глядите да это же chain of responsibility 😂
AD
Konstantin dmz9
билдер тебе дает возможность передать недостроеный объект по цепочке каким то другим обработчикам чтобы они что нибудь туда добавили.
с конструктором такое не получится
с конструктором такое не получится
выглядит, как отличное способ создать кучу ошибок 🙄
Kd
выглядит, как отличное способ создать кучу ошибок 🙄
выглядит как будто кто то не писал сложные программы )
QH
Konstantin dmz9
выглядит как будто кто то не писал сложные программы )
выглядит как бессмысленный переход на личности
Kd
выглядит как бессмысленный переход на личности
ну, кстати да, так и есть, микроагрессия какая то даже
AD
Konstantin dmz9
выглядит как будто кто то не писал сложные программы )
val builder = Builder().url(''https://example.com'')
val newClient1 = newClient1(builder)
val newClient2 = newClient2(builder)
fun newClient1(builder: Builder) = builder.url(''https://kek.lol'').build()
fun newClient2(builder: Builder) = builder.path(''/main'').build()а потом сиди и чеши репу, кто тебе по яйцам с вертухи дал
Kd
val builder = Builder().url(''https://example.com'')
val newClient1 = newClient1(builder)
val newClient2 = newClient2(builder)
fun newClient1(builder: Builder) = builder.url(''https://kek.lol'').build()
fun newClient2(builder: Builder) = builder.path(''/main'').build()а потом сиди и чеши репу, кто тебе по яйцам с вертухи дал
есть 3 метода эту проблему решить
1. бросай исключение когда кто то пытается поменять уже установленый урл
2. создается новый билдер внутри с новым объектом когда вызывается .url
3. сделай урл параметром конструктора билдера чтобы его нельзя было изменить
1. бросай исключение когда кто то пытается поменять уже установленый урл
2. создается новый билдер внутри с новым объектом когда вызывается .url
3. сделай урл параметром конструктора билдера чтобы его нельзя было изменить
AD
1. лови краши в проде (или не лови)
2. да, но можно же просто client.copy(url = 'http://kek.lol')
3. смотри второй пункт, это уже куча «если» к привычному всем билдеру, которые ставят под вопрос его необходимость
2. да, но можно же просто client.copy(url = 'http://kek.lol')
3. смотри второй пункт, это уже куча «если» к привычному всем билдеру, которые ставят под вопрос его необходимость
Kd
1. лови краши в проде (или не лови)
2. да, но можно же просто client.copy(url = 'http://kek.lol')
3. смотри второй пункт, это уже куча «если» к привычному всем билдеру, которые ставят под вопрос его необходимость
2. да, но можно же просто client.copy(url = 'http://kek.lol')
3. смотри второй пункт, это уже куча «если» к привычному всем билдеру, которые ставят под вопрос его необходимость
если ты делаешь запрос на какой то эндпойнт - нельзя его давать менять тем кто дописывает в билдер что либо, т.к. это нелогично по сути.
хочешь обратиться на новый урл - делай новый объект запроса через билдер
хочешь обратиться на новый урл - делай новый объект запроса через билдер
Kd
типа тебе дали допилить фичу, а ты стал пилить другую фичу - это вообще то не то что от тебя хотели
QH
Konstantin dmz9
если ты делаешь запрос на какой то эндпойнт - нельзя его давать менять тем кто дописывает в билдер что либо, т.к. это нелогично по сути.
хочешь обратиться на новый урл - делай новый объект запроса через билдер
хочешь обратиться на новый урл - делай новый объект запроса через билдер
или не через билдер
короче, не давайте никому мутабельные объекты вроде билдеров
короче, не давайте никому мутабельные объекты вроде билдеров