у нас правило задавать по дефолту хотя бы null возведено в абсолют, думаю чуваки на что-то напарывались

может просто стоит делать поменьше пропертей в классах?)

if (array_key_exists('key3', $config) && is_null($config['key3'])) { echo '$key3 = null' . PHP_EOL; }

как-то слишком сложно

их немного

может просто стоит делать поменьше пропертей в классах?)

а при росте числа пропертей, можно во что-то влететь?)

типа если больше 65535 пропертей?)

а при росте числа пропертей, можно во что-то влететь?)

ну тип если их штук 40-50

то с большой долей вероятностью можно сказать что что-то не так с декомпозицией задачи

это да

но нулы-то

суть в том, что тебе пхп все равно не обещает на 100% дефолтное значение, которое ты ожидаешь

class Poinless { private $pointless = null; }

?

тип вот так?

да

нуууу... это немного... бред но ладно)

senseless

ну в общем да)

как-то слишком сложно

Ну так получилось в итоге, is_null не нужен public function unset_key($key) { if (array_key_exists($key, $this->config)) { unset($this->config[$key]); } }

Ну так получилось в итоге, is_null не нужен public function unset_key($key) { if (array_key_exists($key, $this->config)) { unset($this->config[$key]); } }

зачем ты нулы вобщем удаляешь?)

тут блин пропертю у класса можно с пустым именем сделать, к которой нельзя обратиться, ничему верить нельзя

Конкретно удалить нулл задача не стоит. НО если он будет в конфиге и будет необходимость удалить его, то iiset не сработает.

удалить переменную по ключу*

попробуй взять какую-то реальную задачку и сделать как получиться

а потом порефакторить

тут блин пропертю у класса можно с пустым именем сделать, к которой нельзя обратиться, ничему верить нельзя

такое очень легко можно трекать на код ревью

например типичный e-commerce

+1

такое очень легко можно трекать на код ревью

я про верить пхп, а ревью да

я про верить пхп, а ревью да

просто не надо писать говно

и все будет не так плохо)

тут блин пропертю у класса можно с пустым именем сделать, к которой нельзя обратиться, ничему верить нельзя

Это для чего и как это делается?

НЕ РАССКАЗЫВАЙТЕ ЕМУ

))))

Это для чего и как это делается?

это не те дроиды которых вы ищите

Я вычислю! ?

@lokin_roman

попробуй взять какую-то реальную задачку и сделать как получиться

а потом порефакторить

например типичный e-commerce

В любом случае есть же property_exists(). ?

ох уж эти пустые проперти)))

Так зачем мне абстрактные задачи? У меня есть мой же говнокод, который я перевожу в более-менее читаемый вид. У меня есть конфиги в json'ах, нужно удалять переменную по ключу, и там теоретически может быть null, что я делаю не так? ?

а хдебаг его вообще дико выводит

зачем тебе из конфига удалять поля?

задачи они для того, чтобы учиться

Это просто key => value хранилище с настройками, там хоть что может быть. Теоретически null'у там делать нечего, но если есть способ обезопасить себя простой функцией то почему бы и нет?

так может типизируй? Сделай так, чтобы там было не хоть что, а все очень конкретное?

чтобы это было не key-value

Да, возможно. ??

Ещё вопрос, есть метод get_key(), который проверяет наличие ключа в массиве, и есть метод unset_key(), который удаляет ключ, но он сначала проверяет на наличие ключа. Получается снова дублирование кода. Так как нет возможности вернуть корректный ответ из get_key(), так как return null или return false не корректны, так как ключ может принимать такое значение, даже если забыть про null, false это же классический способ вернуть ошибку из метода.

Если нагляднее, то хочу так:

public function get_key($key) { if (array_key_exists($key, $this->config)) { return $this->config[$key]; } throw new Exception("[Config] Key '{$key}' not found.", 1); } public function unset_key($key) { if ($this->get_key($key)) { unset($this->config[$key]); } }

А приходится так:

public function get_key($key) { if (array_key_exists($key, $this->config)) { return $this->config[$key]; } throw new Exception("[Config] Key '{$key}' not found.", 1); } public function unset_key($key) { if (array_key_exists($key, $this->config)) { unset($this->config[$key]); } throw new Exception("[Config] Key '{$key}' not found.", 1); }

Вот как бы избавиться от дублирования? Как методу unset_key() узнать о том что ключа нету именно от метода get_key()?

Всё, решил задачу. ??

ну и кто так делает, задачу решил, а в чат не написал

видимо он понял, что надо сделать hasKey метод

к слову, isset, будет работать быстрей, чем array_key_exists... но это выйгрыш совсем не значительный

O(n) O(1)

задолбало отсутсвие нормальных библиотек логирования для по ха пе

Monolog?

Выйгрыш незначительный, пока n незначительна

да, а что не так с monolog?

ну может человек не знал про монолог)

O(n) O(1)

раскрой тему, почему isset/array_key_exists должны иметь разную сложность?

ERROR: S client not available

?

:D

гуглить я и так умею

гуглить я и так умею

?

ну и кто так делает, задачу решил, а в чат не написал

Сам такое не люблю, но зачем вам, профи, мои дилетанские решения? ?

видимо он понял, что надо сделать hasKey метод

Именно так.

public function get_key($key) { if ($this->key_exists($key)) { return $this->config[$key]; } } public function unset_key($key) { if ($this->key_exists($key)) { unset($this->config[$key]); } } private function key_exists($key) { if (array_key_exists($key, $this->config)) { return true; } throw new Exception("[Config] Key '{$key}' not found.", 1); return false; }

:)

public function get_key($key) { if ($this->key_exists($key)) { return $this->config[$key]; } } public function unset_key($key) { if ($this->key_exists($key)) { unset($this->config[$key]); } } private function key_exists($key) { if (array_key_exists($key, $this->config)) { return true; } throw new Exception("[Config] Key '{$key}' not found.", 1); return false; }

return false никогда не будет работать

нужно удалить throw

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

+

ведь экзепшен это исключительная ситуация, когда все, пришел пиздец

А как тогда во внешнем коде узнать что ключа нету? Не пойму как правильно эксепшенами пользоваться.

Да, понял что не вернётся false никогда, потому что поток прерывается.

false - нет ключа

Но с другой стороны, если прервётся стандартный поток, то на и false не нужен?

Обработаем эксепшин и всё.

В общем, пойду дальше книжку читать. ?

@lokin_roman если без ключа невозможна дальнейшая работа — исключение. Если можно что-то дальше делать — просто false.

ну метод же называется key_exists, который из названия ну никак не должен кидать экзепшены, а просто рассказывать, есть или нет ключ, а уже дальше пользователь этого метода может кидать что захочет, если это обусловлено ситуацией

так а ключа со значением false быть не может?

так а ключа со значением false быть не может?

Может, но он же не передаётся напрямую. Возвращается только наличи или отсутствие ключа.

ну метод же называется key_exists, который из названия ну никак не должен кидать экзепшены, а просто рассказывать, есть или нет ключ, а уже дальше пользователь этого метода может кидать что захочет, если это обусловлено ситуацией

Согласен в общем, но если ключа нету, а его кто-то запрашивает и хочет с ним работать, то это и есть исключительная ситуация и потому эксепшин.

да, но это должен решать тот, кто запрашивает, а метод с таким названием должен просто рассказывать, есть или нет

тебе скажут спасибо те, кто после тебя подддерживать будут

Если у кого есть образцовый класс для загрузки конфига из JSON/XML, покажите его. Я посмотрю на то КАК это должно быть. ?

Ну или не конфига а вообще key => value.

как минимум где-то в компонентах симфони или в ларавеле есть такой класс)

Отличные у них названия, главное что отличаются ?: JsonEncode.php JsonEncoder.php

как минимум где-то в компонентах симфони или в ларавеле есть такой класс)

Ларавель: /** * Get an item from an array using "dot" notation. * * @param array $array * @param string $key * @param mixed $default * @return mixed */ public static function get($array, $key, $default = null) { if (is_null($key)) { return $array; } if (isset($array[$key])) { return $array[$key]; } foreach (explode('.', $key) as $segment) { if (! is_array($array) || ! array_key_exists($segment, $array)) { return value($default); } $array = $array[$segment]; } return $array; }