Контрактники...
Этот один уникальный Патрик-ПМ знает
-----
Увы - такого нету... и тебе об этом сказали нормальным русским языком...
Последним частично знающим был не Патрик, а Фридрих...
Но когда Фридрих попытался вместить необходимые знания предмета в свою голову для анализа чего и как - у него поехала крыша и поехал он с белым билетом по больничкам...
Так никто ж не знает, что там заказал заказчик
-----
Заказчик - знает... и дока от него на заказ - есть...
Нет - детального понимания процесса перехода от доков заказчика в цифирьку-буковку в Р20.
правильность кода на реальном станке
-----
Но продукция то идет... по моей вине за 6 лет простой был минут 20-ть...
Я так понимаю, что критерием приема была успешная компиляция.
-----
Я не знаю что было критерием - Я их не нанимал...
Как без тестов оценить качество перевода SQL запросов?
-----
Получением одинаковых результатов выполнения Р20-х из двух разных баз...
Сейчас, кстати, будет сложнее - базы уже месяц как рассинхронизированы...
Рассказывай, как ты собираешься это делать.
-----
Никак. Вообще не моя задача.
Моя задача - когда все примут - сопровождать и чинить то что не работает...
Написано же черным по белому
-----
Что если Я не починю... или починю не правильно... на выходе будет не то...
и в лучшем случае - просто простой завода, а в худшем - запоротый материал и куча зря потраченного рабочего времени.
И то, и другое - потерянные деньги.
Потому у меня ответа "это соседний отдел и МОЙ код для них не пойдет" - просто не существует.
Последним частично знающим был не Патрик, а Фридрих...
Да хоть Ваня! Кто-то сказал ему как формируется поле Р20.
Нет - детального понимания процесса перехода от доков заказчика в цифирьку-буковку в Р20.
Ну что тут сказать? Так вам раздолбаем и надо :)
Получением одинаковых результатов выполнения Р20-х из двух разных баз...
Это будет уже после релиза. А баблосики заплатить надо до релиза. Так что не канает такая оценка :D Думай дальше :)
Моя задача - когда все примут - сопровождать и чинить то что не работает...
Ну вот и делай свою работу. А консалтеры свою работу сделали. И акт приема-передачи подписан. Значит работа исполнена в точности с требованием заказчика :) Работаешь на идиотов? - Страдай :) Тем более, что страдать тебе осталось уже немного :)
Работаешь на идиотов? - Страдай :)
-----
Страдай - переводе с языка страны исхода - работай... :)
А баблосики
-----
Это не ко мне - свои Я получаю во время...
По сути проблемы - у тебя рецепта решения имеющихся проблем НЕТ.
У меня - тоже. Остается - разгребать спагетти...
Следующая задача - в Р20 посчитанная сумма какой-то фигни не совпадает с ожидаемой... либо исходные не правильные, либо фильтрация... бо, суммирование идет стандартной функцией...
А консалтеры пока могут подумать над
ERROR: 42703: column "rownum" does not exist
отчетик по выполнению "переведенного" скл у меня пришел - есть инфо
- откуда коннектились
- куда коннектились
- в какой момент возникла ошибка
- сама ошибка
- выполнявшийся скл...
все то, что консалтеры похерили в своей переработке...
если есть code style guide, то от куда взялись несоответствующие им имена?
Пока этим не будет заниматься прога, никакого толка не будет.
но в этом случае code style guide относится к новому коду.
А старый получается пусть нечитаемым спагетти так и остается.
Во всяком случае, переформатирование кода не должно, ну абсолютно никак отражаться на его работоспособности.
С переименованием, да можно отхватить. Хотя хороший код должен быть резистент к переименованию или хотя бы иметь коммент, что данный класс имеет неявные связи с тем то и тем то.
применять новые правила к старому коду - может обернуться очень большими затратами
Так Вы же кричали, что спагетти нужно обязательно рефакторить, а "правила оформления" в это значит не входят?
Хотя хороший код должен быть резистент к переименованию или хотя бы иметь коммент, что данный класс имеет неявные связи с тем то и тем то.
-----
У меня голубая мечата не резистентный к переименованиям код, а хотя бы возможность начать искать тупые дубли кода в уже вынутом из спагетти...
Пока этим не будет заниматься прога, никакого толка не будет.
Ну как минимум человечество придумало code review :)
И таки да, можно прикрутить статический анализатор кода. Не вижу никакой проблемы.
А старый получается пусть нечитаемым спагетти так и остается.
Старый может заменять на новый по мере исправления ошибок. Не обязательно все делать одним махом.
Так Вы же кричали, что спагетти нужно обязательно рефакторить, а "правила оформления" в это значит не входят?
Рефакторить нужно то, что мешает.
Рефакторить нужно то, что мешает.
А Вам значит не мешает, грубо говоря, код без отступов?
Или то что в каждом новом файле оформление кода тоже совершенно различное.
Ну как минимум человечество придумало code review
Да много еще чего интересного есть 
Так хоть бы код нормальный был
Не обязательно все делать одним махом.
-----
Я выше спрашивал что делать с переработанным каленадариком смен...
И как-то внятно не прозвучало что делать с устраивающим кодом...
И таких "календариков" уже поднакопилось с вагончик и тележку...
и встроить их НЕЛЬЗЯ... не потому что работают не так, а потому что где-то в спагетти... а то еще и в работающем коде без исходников - такое тоже ест'... может быть кусок работающий со старой версией...
Это же СПА-ГЕТ-ТИ...
Блин, ближе к голубой мечте...
прожка посчитала: - £27620.81
бух. сказала надо: - £26936.61
Ну и что с этим делать?
Обрезал весь обвес в имеющемся скле - оставил без чего не посчитать...
То без чего не посчитать - разобрал на составлающие с контролем каждого этапа фильтрации...
Проверил каждую подвыборку - вроде все нормально...
161 ордер
1462 позиции...
Сумма... £27620.81 - хи-хи (нервное)...
Пар уже кончился, но еще немного дыма в черепушке есть - не все остатки мозгов дотлелi...
Завтра, однако, продолжу потрошить код - есть маленький шанс что где-то не так вычислили минус-единицу коэффициента-множителя...
А откуда бухгалтер знает сколько должно быть?
Не может ли она это сказать для одного заказа (для 10)?
Ты не пробовал сделать excel файл со всеми твоими цифрами и ей показать?
У неё же нюх на цифры должен быть.
Ты своей логикой неделю будешь искать, а она должна сразу сказать что вот тута таких цифр не бывает. При этом, почему, она не знает. Ну не было таких никогда.
А откуда бухгалтер знает сколько должно быть?
-----
НЕЗ-ЗНА-ЮЮЮ...
Не может ли она это сказать для одного заказа (для 10)?
-----
Р20 - не по заказам, а по периодам.
Дополнительную фильтрацию по клиенту в скле Я делал - моя цифирька не поменялась.
Но идея попросить по-заказные цифирьки - здравая - добавить фильтрацию по заказу и сверить не сложно.
Ты не пробовал сделать excel файл со всеми твоими цифрами и ей показать?
-----
Ну так когда Я починил ИменаКлиентов ей пересчетный файлик пересылали - эта цифирька - ответ на ту пересылку...
Ты своей логикой неделю будешь искать
-----
Ну так это означает что Я буду на неделю ближе к дембелю...
При этом, почему, она не знает. Ну не было таких никогда.
-----
Это Я знаю и потому не тороплюсь - мне надо:
- знать что и как считается в Р20
- понимать где подкручивать
- иметь минимальный инструментарий для быстрой проверки.
Пока Я разобрал спагетти на классы и написал минимальные тесты.
По ним проверяется сколько есть по факту в выборке.
Вроде как правильно - т.е. шарповый код работает как должен.
Значит проблема либо в данных в базе, либо в переведенном и починенном скле...
Данные - живые - по ним выполнены/выполняются реальные заказы...
Р20 - не по заказам, а по периодам.
ну попроси по дням разбивку. Хотя бы пару дней.
какие-то заказы могут совпадать, а какие то нет. Будет хоть от чего оттолкнуться
ей пересчетный файлик пересылали
А там есть Raw дата? И просили ее глянуть на валидность остальных полей?
знать что и как считается в Р20
ну сам же сказал, что никто не знает. Но раз клиентам счета выставляют, значит что то откуда берут. Тащи бухгалтера на полиграф 
А Вам значит не мешает, грубо говоря, код без отступов?
Ты это серьезно? Во-первых, студия сама устанавливает отступы. Во-вторых, настройки студии должны быть у всех одинаковые. В-третьих, если я не правлю код, то я функцию не трогаю и отступы мне не мешают.
Или то что в каждом новом файле оформление кода тоже совершенно различное.
Не мешает.
Мешает, если кто-то решит, что в каком-то файле неправильное оформление и решит исправить оформление на свой вкус. Вот это пипец как мешает.
ну попроси по дням
-----
Нее, оказалось есть стандартная возможность получить отдельный заказ в деталях из системы... Вот сижу, жду...
А там есть Raw дата? И просили ее глянуть на валидность остальных полей?
-----
Есть там детальки.
А вот буха трогать не надо... это Я давно и крепко усвоил... надо решать проблему, а не дергать занятых людей...
ну сам же сказал, что никто не знает.
-----
Да.
Не было.
Но ведь Я его уже недельку ковыряю - кое-что начал понимать...
По крайней мере нет проблемы прочесть КАК считается... и можно додумкать что в результате должно быть...
Тащи бухгалтера на полиграф
-----
Ты таки хочешь чтобы меня уволили раньше срока...
Прогонять балду - могу - ИБД и никаких претензий. Особливо на фоне заваливающихся контрактников...
А вот если отсвечивать "не-по-де-лу" - пинка можно отхватить не вовремя... Ну его нафиг...
если я не правлю код, то я функцию не трогаю и отступы мне не мешают.
-----
Певерь простой штуке - загрузив код в Студию ты уже меняешь его.
Ничего не меняя в логике, но дописав комментарий и, по случаю, убив и поставив обратно последнюю скобку получаешь переформатированный текст. Мой код станет менее понятен после таких ничего не меняющих манипуляций.
Не мешает.
-----
Бред.
В 20% случав "невозможности решения" задачи у студентов реальная проблема становилась понятной студенту после "стандартного" форматирования кода...
Ты это серьезно?
ну не надо понимать всё так буквально.
Что сейчас может делать студия не имею никакого понятия, так как уж фиг сколько лет стоит решарпер и все правила вбиваются в него.
если я не правлю код, то я функцию не трогаю
ну это если команда хоть какая есть. А Мурка то один можно сказать.
А мне вот мешает что неправильно отформатировано 
. Но да, в команде приходится не трогать, пока тикет на это не взял.
и решит исправить оформление на свой вкус.
Ну так вроде для это и есть правила.