Организация пространства имён

dionys

2015-02-04 20:36

Привет.

В модуле Business::CPI вышла путаница с использованием пространства имён. Она писана в соответствующем баге на GitHub.

Вкратце суть в следующем. Есть базовый модуль драйвера Business::CPI::Gateway::Base и базовые модули для различных сущностей используемых драйвером Business::CPI::Base::Buyer, Business::CPI::Base::Card и т. п. (Уже тут видно, что единый стиль именования не выдержан.) Далее конкретный драйвер с именем, допустим, MyDriver предлагается размещать в модуле Business::CPI::Gateway::MyDriver, а вот если нужно отнаследовать другие выше названные сущности, они попадают в Business::CPI::MyDriver::Buyer и т. д.

Как бы вы организовали именование модулей? Желательно при этом не засорять пространство имён Business::CPI.

vti

2015-02-04 21:48 #

Мне кажется, что удобнее придерживаться компоненто-подобной системы, такой как Business::CPI::MyDriver::*, но насколько я вижу из обсуждения тикета, сошлись на использовании другого namespace ;)
Содержимое

Предпросмотр

или CTRL+Enter формат
Содержимое

Предпросмотр

Мне кажется, что удобнее придерживаться компоненто-подобной системы, такой как `Business::CPI::MyDriver::*`, но насколько я вижу из обсуждения тикета, сошлись на использовании другого namespace ;)

или CTRL+Enter формат
- dionys → vti
  
  2015-02-04 22:10 #
  
  Можно отвлечься от конкретно этого модуля и обсудить абстрактно.
  Предложенный тобой вариант имеет один, на мой взгляд, большой минус — засоряется основное пространство имён модуля. Мне этот подход и в DBIx::Class не нравится.
  Содержимое
  
  Предпросмотр
  
  или CTRL+Enter формат
  Содержимое
  
  Предпросмотр
  
  Можно отвлечься от конкретно этого модуля и обсудить абстрактно. Предложенный тобой вариант имеет один, на мой взгляд, большой минус — засоряется основное пространство имён модуля. Мне этот подход и в module:DBIx::Class не нравится.
  
  или CTRL+Enter формат
- - vti → dionys
    
    2015-02-04 22:16 #
    
    Ну, т.е. ::Gateway:: можно туда тоже втиснуть. Я пропустил видимо.
    Содержимое
    
    Предпросмотр
    
    или CTRL+Enter формат
    
    Содержимое
    
    Предпросмотр
    
    Ну, т.е. `::Gateway::` можно туда тоже втиснуть. Я пропустил видимо.
    
    или CTRL+Enter формат
  - - dionys → vti
      
      2015-02-05 12:11 #
      
      Это вариант, но часто он получается слишком длинным.
      
      Содержимое
      
      Предпросмотр
      
      или CTRL+Enter формат
      
      Содержимое
      
      Предпросмотр
      
      Это вариант, но часто он получается слишком длинным.
      
      или CTRL+Enter формат
vti

2015-02-05 09:04 #
Еще интересный вопрос насчет фабрик. Например, есть у нас такая иерархия:
```
├── Foo
│   ├── Bar.pm
│   └── Base.pm
└── FooFactory.pm
```
Тут проблема, что фабрика реализована как обычно require "$namespace\::$class". И поэтому можно создать экземпляр Base. С этим, конечно, можно жить, но несколько раздражает. Пока что я пробовал нечто вроде:
```
├── Foo
│   └── Bar.pm
├── FooBase.pm
└── FooFactory.pm
```
Но оно как-то не интуитивно. Можно, конечно, в фабрику добавить проверку на Base...
А может все это бред и не стоит заморачиваться :D
Содержимое

Предпросмотр

или CTRL+Enter формат
Содержимое

Предпросмотр

Еще интересный вопрос насчет фабрик. Например, есть у нас такая иерархия: ``` ├── Foo │ ├── Bar.pm │ └── Base.pm └── FooFactory.pm ``` Тут проблема, что фабрика реализована как обычно `require "$namespace\::$class"`. И поэтому можно создать экземпляр `Base`. С этим, конечно, можно жить, но несколько раздражает. Пока что я пробовал нечто вроде: ``` ├── Foo │ └── Bar.pm ├── FooBase.pm └── FooFactory.pm ``` Но оно как-то не интуитивно. Можно, конечно, в фабрику добавить проверку на `Base`... А может все это бред и не стоит заморачиваться :D

или CTRL+Enter формат
- dionys → vti
  
  2015-02-05 09:18 #
  А почему бы не назвать базовый класс просто Foo?
```
├── Foo
│   └── Bar.pm
├── Foo.pm
└── FooFactory.pm
```
  Содержимое
  
  Предпросмотр
  
  или CTRL+Enter формат
  Содержимое
  
  Предпросмотр
  
  А почему бы не назвать базовый класс просто `Foo`? ``` ├── Foo │ └── Bar.pm ├── Foo.pm └── FooFactory.pm ```
  
  или CTRL+Enter формат
- - vti → dionys
    
    2015-02-05 09:21 #
    
    Foo подразумевает, что его можно сразу использовать. Поэтому я обычно Base ассоциирую с абстрактным классом.
    Содержимое
    
    Предпросмотр
    
    или CTRL+Enter формат
    
    Содержимое
    
    Предпросмотр
    
    `Foo` подразумевает, что его можно сразу использовать. Поэтому я обычно `Base` ассоциирую с абстрактным классом.
    
    или CTRL+Enter формат
  - - dionys → vti
      
      2015-02-05 12:10 #
      
      Если класс совсем абстрактный, то можно реализовать его в виде роли, тогда и имя у него будет типа Role::Foo.
      
      Содержимое
      
      Предпросмотр
      
      или CTRL+Enter формат
      
      Содержимое
      
      Предпросмотр
      
      Если класс совсем абстрактный, то можно реализовать его в виде роли, тогда и имя у него будет типа `Role::Foo`.
      
      или CTRL+Enter формат

Чтобы ответить, залогиньтесь или зарегистрируйтесь.

Вход или Регистрация

@user
```
_italic_
```
→
italic
```
**bold**
```
→
bold
```
[PP](http://pragmaticperl.com)
```
→
PP
```
<http://pragmaticperl.com>
```
→
http://pragmaticperl.com
```
module:Plack
```
→
Plack
```
release:URI
```
→
URI
```
author:VTI
```
→
VTI
```
`my $foo = 'bar'`
```
```
```
my $multi;
$line;
```
```

Форум

Организация пространства имён