The daily Axapta WTF

[Vadik]

Цитата:

Сообщение от kashperuk

Нуу, работать то будет. Чуть странно с concurrencyModel, первый раз такое видел в коде. А так - ну, подумаешь, indentation плохой.

Там все чуть более запущено чем просто проблемы с отступами - обрати внимание на первичный ключ на табличке

[kashperuk]

Цитата:

Сообщение от Vadik

Там все чуть более запущено чем просто проблемы с отступами - обрати внимание на первичный ключ на табличке

RecId. Но и поиск тоже по RecId. То есть с точки зрения моделирования данных - все ОК.
Другое дело, что в этой табличке надо бы еще другой уникальный индекс создать. Но видимо табличка так часто используется, что пока не понадобилось

[Vadik]

Цитата:

Сообщение от kashperuk

RecId. Но и поиск тоже по RecId. То есть с точки зрения моделирования данных - все ОК

Да как сказать.. И рядом с ОК не стояло, если честно - см. на EDT, связь с другими таблицами не по RecId, а по Code строится. Представь на секунду, что ты в запросе связываешь клиентов и группы классификации - что получится?

Цитата:

Другое дело, что в этой табличке надо бы еще другой уникальный индекс создать. Но видимо табличка так часто используется, что пока не понадобилось

Это не другой уникальный индекс, это первичный ключ. Его обычно сразу (при проектировании) создают, а не "когда понадобится"

Я в принципе догадываюсь, зачем эти неиспользуемые find() и exist() методы создавались - чтобы тупо пройти формальную проверку на их наличие. Вопрос в том, как такое сумели в SYS слой закоммитить и на кой ляд вводили разрекламированные юнит-тесты