Говнокод

if (count($stores) == 1) {
    foreach ($stores as $store) {
        $storeId = $store->id;
    }
}

Запостил:

jbot, 18.05.2015 (Updated 29.03.2018)

Комментарии (6) RSS

jbot 11 лет ago # 0

Забыл указать: переменная $stores является не просто массивом - она является экземпляром класса, реализующего интерфейсы SeekableIterator и Countable.
Ответить
- pushistayapodmyshka 11 лет ago # 0 ⇈
  
  Это потому, что ПХП – говно, в котором невозможно копошиться без толковой IDE.
  Ответить
  - inkanus-gray 11 лет ago # 0 ⇈
    
    Вы так говорите, как будто IDE пишет код за программиста.
    Ответить
    - kegdan 11 лет ago # 0 ⇈
      
      не пишет, но, сцука, дописывает и периодически неправильно
      Ответить
- bormand 11 лет ago # 0 ⇈
  > она является экземпляром класса, реализующего интерфейсы SeekableIterator и Countable
  Но не является экземпляром ArrayAccess или как там его у вас, чтобы просто достать нужный элемент как $stores[0]?
  
  P.S. Не говнокодная версия будет как-то так?
```
$stores->rewind();
if ($stores->valid())
    $storeId = $stores->current()->id;
```
  Ответить
  - jbot 11 лет ago # 0 ⇈
    
    У Countable есть метод count(), а у SeekableIterator есть метод current(). Я бы написал примерно как у вас, только вместо valid() использовал count() == 1, для ясности.
    Ответить