- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
- 33
- 34
- 35
- 36
- 37
- 38
- 39
public function getAddressInfo($address) {
if (!$address instanceof SalesOrderAddressShipping) return '';
$country = $address->getCountry();
$region = $address->getRegion();
$city = $address->getCity();
$zipCode = $address->getZipCode();
$street = $address->getStreet();
$houseNumber = $address->getHouseNumber();
$building = $address->getBuilding();
$appartment = $address->getAppartment();
$addressInfo = '';
$addressInfo .= $city . ', ' . $street . ' ' . $houseNumber;
$addressInfo .= !empty($building) ? ', корп. ' . $building : '';
$addressInfo .= !empty($appartment) ? ', кв. ' . $appartment : '';
return $addressInfo;
}
public function getAddressInfoWithoutApartment($address) {
if (!$address instanceof SalesOrderAddressShipping) return '';
$country = $address->getCountry();
$region = $address->getRegion();
$city = $address->getCity();
$zipCode = $address->getZipCode();
$street = $address->getStreet();
$houseNumber = $address->getHouseNumber();
$building = $address->getBuilding();
$appartment = $address->getAppartment();
$addressInfo = '';
$addressInfo .= $city . ', ' . $street . ' ' . $houseNumber;
$addressInfo .= !empty($building) ? ', корп. ' . $building : '';
//$addressInfo .= !empty($appartment) ? ', кв. ' . $appartment : '';
return $addressInfo;
}
а сам класс называется AdminOrderListContentView, ебиеговрот....
interested 31.07.2012 09:11 # 0
guest 31.07.2012 21:52 # +2
interested 31.07.2012 22:01 # +1
Этот код точно, ясно и верно выполняет свою работу.
И о локалях тут даже ошмётки не брякали.
Какой коллектив -- такой код.
SET 31.07.2012 22:11 # 0
interested 31.07.2012 22:21 # 0
Здесь всё ясно. Было спрошено с программиста, чтобы в вид возвращалась форматированная строка. Это и было сделано. Формотирование меняется -- меняется код. И что там будет послезавтра -- неизвестно. Вот и висят куски.
Говнокод, говногод...
Как маленький ребёнок, честное слово...
Или, как говорится, тролль.
SET 01.08.2012 11:21 # +3
interested 01.08.2012 11:56 # 0
Две функции с близкими названиями совершают близкие действия.
Если бы было две функции с непонятными и разными названиями, а они делали бы одно и то же -- это говноконструирование.
Если бы было две функции с близкими названиями, а дела ли бы они совершенно разное -- говноконструирование.
Если здесь есть говно- то это говноменеджмент, который привёл к тому, что из MVC выпал кусок. И этот выпавший кусок ещё достаточно прилично сделан!
Гораздо ужаснее было бы, если бы этот кусок вообще был прямым текстом размещён посреди "бизнес-логики". Так он хоть упакован отдельно: с глаз долой, из сердца вон.
Двайте уж, пишите сразу имена фамили, юр. лица. фотографии. Чтобы никто к этим людям уже работать не приходил.
Vasiliy 01.08.2012 12:02 # 0
interested 01.08.2012 12:07 # 0
roman-kashitsyn 01.08.2012 12:19 # +4
interested 01.08.2012 12:22 # 0
Бессмысленно говорить "о хорошем коде", когда здесь дыра между разными людьми. Говно в человеческом факторе.
Форматер сработал бы, как, например, доп. тег в JSP. Но подобные коды, как выше, появляются именно от того, что кто-то не хочет делать форматирование, а кто-то не хочет его поддерживать.
defecate-plusplus 01.08.2012 12:46 # +2
представляю себе глаза проффессионнала-пхпшника, которому ради всего 2 вариантов форматирования, вместо 30 секунд работы "добавить уже сраный аргумент bool в эту сраную функцию", тимлид предложит написать небольшой такой парсер-форматтер
Vasiliy 01.08.2012 12:53 # 0
roman-kashitsyn 01.08.2012 12:56 # 0
Vasiliy 01.08.2012 13:18 # +3
Vasiliy 31.07.2012 11:36 # 0
interested 31.07.2012 12:41 # 0
Когда алгоритм устроен так, что наличие реального объекта не является обязательным условием, возвращают Null object -- дырку без поведения, в данном случае пустую строку.
Это проблема не PHP и даже не конкретного программиста, данная проблема имеет более глубокие и исторические образующие причины.
roman-kashitsyn 31.07.2012 11:51 # 0
rat4 31.07.2012 12:03 # +3
Vasiliy 31.07.2012 12:08 # 0
bormand 31.07.2012 12:37 # 0
guest 31.07.2012 14:37 # −6
roman-kashitsyn 31.07.2012 23:48 # +4
wvxvw 01.08.2012 10:35 # −2
roman-kashitsyn 01.08.2012 10:48 # +1
interested 31.07.2012 12:43 # 0
Vasiliy 31.07.2012 12:03 # +1
rat4 31.07.2012 12:21 # 0
Vasiliy 31.07.2012 12:41 # 0
roman-kashitsyn 31.07.2012 12:44 # +2
Vasiliy 31.07.2012 12:50 # 0
rat4 31.07.2012 13:11 # 0
Vasiliy 31.07.2012 13:31 # 0
roman-kashitsyn 31.07.2012 13:51 # 0
interested 31.07.2012 12:55 # 0
Гораздо проще создать несколько динамических страничек на PHP, чем на java. Всё же, java для web в своей основе более сложное с инженерной точки зрения сооружение.
Фреймворки тоже не помощники. Нужно ещё разбираться как они работают, как они устанавливаются.
Только по мере роста проектов становится ясно, что "по-простому" не выйдет, а "по-сложному" -- проще на других инструментах.
roman-kashitsyn 31.07.2012 12:27 # +4
sayidandrtfm 31.07.2012 12:17 # +4
interested 31.07.2012 12:44 # +4
Новый девайс от Apple? :)
sayidandrtfm 31.07.2012 13:28 # +1
interested 31.07.2012 15:19 # −2
Vasiliy 31.07.2012 15:55 # 0
sayidandrtfm 01.08.2012 14:02 # 0
roman-kashitsyn 01.08.2012 15:01 # +1
sayidandrtfm 01.08.2012 15:05 # 0
roman-kashitsyn 01.08.2012 15:07 # +1
sayidandrtfm 01.08.2012 15:22 # 0
< Все «хотелки» должны быть формализованы предварительно, желательно.
Вопрос сводиться к написанию автоматически конфигурируемого парсера выхлопа.
Без спецификации формат выхлопа - это не серьёзный разговор.
Кстати говоря. Вместо '1' в $addressInfo могут содержаться индексы формата.
т. е. если не ноль, то $func[$key][$val]