@phpgeeks
F01134H31.03.2017
15:50:19
15:50:19
ага)
da horsie31.03.2017
15:50:21
15:50:21
Утрочат
f4rt~31.03.2017
15:50:26
15:50:26
приветик коняша
F01134H31.03.2017
15:50:27
15:50:27
ватсо
GoogleF01134H31.03.2017
15:50:42
15:50:42
я вообще не уверен, нужны ли там тесты
da horsie31.03.2017
15:50:44
15:50:44
Завтра примерно в это время хенгаутс :)
f4rt~31.03.2017
15:53:39
15:53:39
я код ревью делаю
F01134H31.03.2017
15:53:51
15:53:51
да они не нужны там прост
f4rt~31.03.2017
15:54:13
15:54:13
мусор от тестов тож тогда не нужн)
F01134H31.03.2017
15:54:38
15:54:38
а че за scrutinizer?
f4rt~31.03.2017
15:54:54
15:54:54
геттеры сеттеры такое
F01134H31.03.2017
15:55:37
15:55:37
ты о чем?
GoogleF01134H31.03.2017
15:55:45
15:55:45
а
да не, нормально
не люблю когда свойства объекта трогают без гетов и сетов
f4rt~31.03.2017
15:56:44
15:56:44
геттеры ломают инкапсуляцию же
f4rt~31.03.2017
15:57:46
15:57:46
открывают внутреннее состояние обьекта
F01134H31.03.2017
15:58:21
15:58:21
?
f4rt~31.03.2017
15:58:22
15:58:22
с их использованием логика размазана получается
F01134H31.03.2017
15:58:27
15:58:27
чо эт
f4rt~31.03.2017
15:58:32
15:58:32
ping @f3ath
Если ты НЕ юзаешь геттеры/сеттеры то логика работы реализовывается прямо в классе;
ее потом легко найти, понять что происходит, исправить/модифицировать.
Если же ИСПОЛЬЗОВАТЬ getter/setter - логика работы окажется где-то за пределами класса, и потом сложно контролировать где и что происходит.
F01134H31.03.2017
16:00:13
16:00:13
у меня логика работы и должна выходить за пределы класса, это же пакет
f4rt~31.03.2017
16:01:17
16:01:17
ты ебанулся штоль?
ничего я не ебанулся, если тебе нужны геттеры сеттеры, велика( я не говорю что 100%) вероятность, что ты что то делаешь не такF01134H31.03.2017
16:01:24
16:01:24
эм
F01134H31.03.2017
16:01:42
16:01:42
редко такое бывает, но я согласен с скопуком)
f4rt~31.03.2017
16:01:58
16:01:58
мде
Dmitriy31.03.2017
16:02:00
16:02:00
сетеры не нужны
GoogleF01134H31.03.2017
16:02:21
16:02:21
Все зависит от конкретной реализации
Dmitriy31.03.2017
16:02:43
16:02:43
сеттры ? не а
v31.03.2017
16:02:51
16:02:51
да а
Dmitriy31.03.2017
16:03:27
16:03:27
http://martinfowler.com/bliki/TellDontAsk.html
v31.03.2017
16:04:05
16:04:05
и?
Dmitriy31.03.2017
16:04:26
16:04:26
Вмест того что бы у новостей поставить setPublish и setDatePublish
можно просто publish() который все проставит сам
f4rt~31.03.2017
16:04:27
16:04:27
они нужны в осномном зачем?
Что бы читать и устанавливать свойства, а это делается что бы управлять объектом как марионеткой, идея в том, что бы сам объект был живой, сам принимал решения и содержал логику;
f4rt~31.03.2017
16:04:52
16:04:52
v31.03.2017
16:04:58
16:04:58
конвертировать дату при присвоении
Dmitriy31.03.2017
16:05:13
16:05:13
Дата обязательный эллемент ?
v31.03.2017
16:05:25
16:05:25
ага
Dmitriy31.03.2017
16:05:46
16:05:46
тогда new Post(text,data)
Dmitriy31.03.2017
16:06:19
16:06:19
че это ?
Googlev31.03.2017
16:06:33
16:06:33
или вот например
Dmitriy31.03.2017
16:06:51
16:06:51
Как это не выйдет. если дата обязательный элемент
v31.03.2017
16:06:54
16:06:54
Timer timer = new Timer(1000);
timer.Start();
timer.Stop();
tiemr.SetЕшьу(100);
timer.Start();
надо было новый таймер создавать?
F01134H31.03.2017
16:07:54
16:07:54
вот и я о том
AdminERROR: S client not available
v31.03.2017
16:08:11
16:08:11
Как это не выйдет. если дата обязательный элемент
так, не выйдет. Я Post создал уже, с датой. зачем мне его пересоздавать с новой датой?Dmitriy31.03.2017
16:08:11
16:08:11
timer.date(100)->start()
Sergey31.03.2017
16:08:55
16:08:55
/**
* Create a new converter
* @param ApiConnection $connection
* @param String $currency - current currency
*/
public function __construct(ApiServiceInterface $connection)
{
$this->connection = $connection;
}
Dmitriy31.03.2017
16:08:57
16:08:57
setDate = что-то такое
Sergey31.03.2017
16:09:00
16:09:00
Лишни параметр в доке
Googlef4rt~31.03.2017
16:09:22
16:09:22
@pronin86 за геттеры/сеттеры м?
Dmitriy31.03.2017
16:09:25
16:09:25
так, не выйдет. Я Post создал уже, с датой. зачем мне его пересоздавать с новой датой?
У тебя дата создание может поменятся ?Sergey31.03.2017
16:09:32
16:09:32
public function convert($currentCurrency, $value, $currency)
{
$allCurrencies = $this->connection->getAllCurrencies();
$val = ConverterLogic::convert(
$allCurrencies[$currentCurrency],
$allCurrencies[$currency],
$value
);
return $val;
}
Sergey31.03.2017
16:09:37
16:09:37
Вообще нет доки
v31.03.2017
16:09:46
16:09:46
это просто дата
F01134H31.03.2017
16:10:32
16:10:32
Сделаю конеш
f4rt~31.03.2017
16:10:37
16:10:37
ты не считаешь что они нарушают в основном принцип инкапсуляции?
v31.03.2017
16:10:41
16:10:41
Dmitriy31.03.2017
16:11:09
16:11:09
сеттеры ломают инвариант объекта
Открыть в Telegram