Співробітник перейменував усі мої запити [закрито]


63

Я не знаю, чи слід мене дуже дратувати чи що. Я вручну створив понад 300 запитів для великої бази даних і розробив умову іменування, щоб потім їх міг знайти. Ніхто інший у моєму кабінеті навіть не знає, як скласти запит, але я вчора зайшов, щоб дізнатися, що всі вони були перейменовані. Зараз мені дуже важко знаходити речі, і я намагаюся зрозуміти, що робити.

Я розмовляв із відповідальною людиною, і вона просто перекреслила всю справу. Вона сказала, що перейменувала їх, щоб їх було легше знайти. На жаль, я єдиний, хто знає, як їх будувати, редагувати та підтримувати, і єдиною причиною, яку їй потрібно було знайти, було тестування запитів. Нова конвенція про іменування взагалі не має сенсу, і я відчуваю, що ми зробили крок назад у процесі розробки.

Що я намагаюся з’ясувати:

1) Чи я занадто реагую?

2) Який найкращий спосіб впоратися з цим? Я ненавиджу згадувати це своєму начальнику, але після розмови з моїм колегою вчора я вже можу сказати, що вона відчуває, що вона нічого поганого не зробила.


29
Незважаючи на те, що ми працюємо над командами, існує концепція того, хто чим володіє, і дозволу слід запитувати перед тим, як змінити код, який є автором іншого. Одного разу я це зробив (соромлюсь сказати), і мені дорікали. Коли це трапилось зі мною, я змінив його назад і попросив їх цього не робити.
Майк Данлаве

30
Поєдинок на світанку! ...

46
У вас є резервна копія / SVN? Відновіться до того, як вона зміниться, і розслабтесь.
JYelton

11
якби хтось перейменував мій код, я вирвав би життя з них, як Shang Tsung!
Гленн Феррі

24
Якщо у неї є діти, перейменуйте їх.
Метт

Відповіді:


81
  1. Насправді - це надзвичайно неповажна справа.

  2. Ви розмовляли з нею, а у нас немає, але, здається, ви б в межах своїх прав відновити попередні умови іменування з резервної копії або відновити їх, якщо вони знаходяться в контролі джерела. НЕ повідомляйте свого начальника і колегу, якщо ви це зробите, і поясніть свою причину (Ви не можете підтримувати власну роботу).

Останнє, що ви хочете, - це впоратися назад і вперед над цим питанням, хоча так поводиться, як ситуація здається воєнною, але це має бути принаймні задокументовано, якщо воно стане частиною шаблону неповаги.


81
Як і в сторону, якщо її роль у проекті насправді є "тестером", вона абсолютно не має права змінювати вихідний код. Період. Але я хотів би додати, що для запобігання подібної ситуації вам слід скласти невеликий (кулі!) Документ, який визначає конвенцію про іменування, оскільки в майбутньому ваша компанія може найняти когось для роботи з вами або навіть координувати ви, і було б в його правах змінити назву, якщо немає офіційної конвенції.
Бруно Брант

1
Я згоден з цією відповіддю. Якщо нічого не робити, ви встановлюєте прецедент, який може посилити майбутні розбіжності.
JYelton

2
Поясніть їй, як ви працюєте в конвенції про іменування
Германсон

13
Переконайтеся, що ви забираєте її права на редагування бази даних, поки ви в ній ...
Ant Ant

1
@Ant: Якщо приклад ОП - це вся історія, я думаю, що це може бути дещо суворим. Якщо в ньому є більше (або вони ніколи не повинні мати права редагувати права), ви можете мати рацію.
BCS

117

Чому б вам просто не поводитися з цим, як дорослі: сідайте безконфліктно і не придумайте список плюсів і мінусів для схеми іменування, домовтеся про це і зробіть це офіційним, написавши короткий документ з його описом. Виявляйте справжній інтерес до її вкладу, щоб вона відчувала (і бере участь).

Якщо це здебільшого питання смаку і якщо вона така людина, яка абсолютно повинна мати свої речі, то просто радійте, що ви більша людина, і відпустіть це. Життя занадто коротке, щоб мати шалений конкурс іменних схем.

Чи проблема в схемі називання чи ви вважаєте, що не отримуєте поваги? Якщо так, можливо, ви можете попрацювати над своїми робочими стосунками. Якщо ви вважаєте, що це не варто, то навіщо вам все одно, що вона думає? :) Ще одним варіантом може бути те, що вона справді не відчуває, що це велика справа, і якщо ви добре поясните, що у вас проблеми з пошуком речі, можливо, ви можете змінити її назад.


2
Я збирався сказати змінити її назад і попросити її не робити цього, але ваша відповідь ще краща.
Майк Данлаве

Це гарне місце.
Уейн Моліна

20
А потім

7
@konrad, ви не помічаєте одне найважливіше питання - тестер не має ані авторитету, ані будь-якого бізнесу, який займається подібним редагуванням . @anon є відповідальним програмістом, іменування конвенцій належним чином прийняте його рішення, і вони не підлягають голосуванню в комітеті, не кажучи вже про односторонні зміни того, кому ніколи не доведеться потім налагоджувати код.
Шадур

36
  1. Дизайн бази даних включає дозволи (GRANT та REVOKE).
  2. Тестування включає дозволи на тестування.
  3. Порівняно небагато людей повинні мати дозвіл на перейменування об'єктів бази даних.
  4. Ваш колега - не один з небагатьох.

Це дуже хороший момент. Мені було цікаво, чому тестер взагалі мав би доступ до цього в першу чергу.
Джастін Омс

Тому що це Access. В доступі немає жодної ГРАНТИ та відмови. Дозволів взагалі немає.
Kibbee

7
Насправді в Access є дозволи, але я ще не зустрів когось, хто розуміє, як їх слід використовувати (не кажучи вже про їх реалізацію)
Mchl

1
Тому що, без сумніву, ця сама компанія, мабуть, навіть не має систем управління джерелами, не кажучи вже про кваліфіковану DBA, яка заблокувала Access. До речі, ви повинні подивитися, це не дуже важко зробити.
Анонімний тип

21

"Нова угода про іменування взагалі не має сенсу" звучить так, як це може бути і так:

  1. Вона застосувала до них якусь норму компанії. Вони часто використовуються для того, щоб переконатися, що код є принаймні послідовним, і в кращих випадках може допомогти периферійним речам, таким як невеликі спеціальні сценарії, легко знайти код. У цьому випадку вам потрібно зрозуміти норму і чому вона "не має сенсу" у вашій ситуації. Якщо ви все ще вважаєте, що краще всі розробники залишити його таким, яким він був, поясніть їм, чому саме ваш метод перевершує, і запитайте, чи можете ви змінити норму (напевно, гаразд, якщо вони згодні, що це вище) або відмовитись у вашому випадку (малоймовірно і безладно в довгостроковій перспективі).
  2. Вона вигадала власний стандарт на місці і застосувала це. Це більше ймовірно, якщо вона нова, і їй слід пояснити своє обґрунтування. Ви можете чомусь навчитися, і / або вона може чомусь навчитися, якщо потім поясніть своє обгрунтування.

Важливим моментом є те, що це не ваш (єдиний) код, якщо що-небудь йому належить і буде модифікований цілою групою. Жодна критика щодо коду ніколи не повинна зосереджуватися на тому, хто його написав.


2
+1 хороший бал. всі інші, хто відповідає, передбачають, що запитуючий має дійсну схему іменування. Що робити, якщо він / вона насправді є компаніями "інформаційною скарбницею". Можливо, його тестер, який насправді полегшує всім іншим (і будь-кому новому) у компанії пошук запитів.
Анонімний тип

Хороші бали. Якщо схема називання хороша, обидві сторони зможуть використовувати її (як тільки вони звикнуть), незалежно від того, хто її придумав.
BCS

2
@bcs Навіть у такому випадку правильним способом зробити це було б сповістити програміста ПЕРШИХ і попросити його / її змінити конвенцію про іменування самі, а не вкрадати її і чекати, коли він / вона прийде на роботу наступного дня і відкрийте для себе всю його структуру.
Шадур

16

Я розмовляв із відповідальною людиною, і вона просто перекреслила всю справу.


Тоді я скажу вам без зусиль:

Скасувати зміни.

Ведіть цю війну. Ваш менеджер повинен підтримати вас і зміцнити свої повноваження.


Домовились. Спершу переконайтесь, що немає вагомих причин, чому нові імена кращі. Якщо не просто змінити його назад.
Річард

11
Коли "Веди цю війну" колись корисна порада?
Sverre Rabbelier

3
@Sverre Rabbelier: ... Коли n00b намагається встановити неоптимальні методи програмного забезпечення. ОП намагалася з нею міркувати. Тепер саме час виправити проблему. Обговорення рекламного музею з n00b про щось настільки очевидне - непродуктивний і марний рух.
Джим Г.

4
@Jim Можливо, OP є новим?
Джо Філіпс

3
Якщо вона не є вашим начальником, змініть це лайно і не озирайтеся. Я не кажу про ведення війни, але впевнено не дозволяйте комусь діяти так, як їм належить. Керівники проектів - це ті, хто повинен застосовувати стандарти кодування. Мова йде не про повноваження, а про досвід, порядок та продуктивне робоче середовище.
Джонатан Хенсон

10

1) Ні, ти не надто реагуєш. Хтось змінив вашу роботу, не розповідаючи вам, і викреслив її, коли ви запитали їх, чому. Це вкрай неповажно і грубо імхо.

2) Ви є офіційним представником справ або, принаймні, особою, яка перетворилася на зберігача БД? Якщо так, змініть імена та напишіть документ про звичайні умови для того, як ви робите справи. Крім того, напишіть документ у стилі "Посібник користувача", щоб, якщо комусь доведеться зайти в БД і знайти щось, що вони можуть.

Я б надіслав це групі, не вказуючи жодними пальцями, з корисною відміткою про те, що ви з радістю будете сідати і ходити за людьми через деякі нюанси структури.

Якщо ні, то придумайте команди як команди та дотримуйтесь їх як команди.

Зі сторони, для когось, кому довелося щось перевірити, щоб змінити назви 300+ запитів, це здається досить по-дитячому. Скільки часу вона витрачала на це, і лише для того, щоб вона могла знайти речі? Замість того, щоб просто поїхати і попросити когось про допомогу, вона витратила свій час, свій час та час компанії. Не кажучи вже про код, який, ймовірно, зламався, коли вона це зробила, тим самим витрачаючи час іншого члена команди.

Якби я був ти, я б зачекав, поки ти трохи охолоне, спробуй ще раз поговорити з нею. Якщо це не допоможе, візьміть це з начальником. Такий менталітет ковбоя зрештою поставить всю команду.


8

Перейменування випадково в базі даних може легко призвести до погіршення виробничого середовища. Якщо на ці процедури посилатися десь у коді, це може мати серйозні наслідки. Ви можете зробити відкат, але якщо тестер, подібний до цього, справді не знає, що вона робить, це не так далеко, щоб побачити тестер внести деякі зміни у виробництво. Це може означати втрату бізнесу, саме тому вам слід спробувати реалізувати окремі ролі користувачів для розробників та тестерів. Ми робимо це разом з нашими тестерами, і це чудово працює. Тестери часто це цінують, оскільки їм не доведеться жити, боячись викрутити живі дані.


5

Це, схоже, не торкається в іншому місці, але будь-яке джерело (наприклад, запит), розміщене в громадському місці, повинно знаходитися під системою контролю версій.

Потім, якщо колега змінить вашу схему іменування, ви можете легко повернутись до вашої робочої схеми (і переглянути їх зміни; і, можливо, повернути назад, якщо необхідно). Ви також прив’язуєте зміни до конкретних користувачів, щоб ви могли бачити, хто зіпсував речі.


2

Не дивись подарунковій коні в рот.

По-перше, колективне володіння кодом - вони не повинні бути "вашими".

По-друге, якщо вони перейменували їх, то запитайте міркування щодо нової схеми іменування. Або вони використовують запити - у цьому випадку це свого роду їх дзвінок; або це перший крок у них, починаючи надати вам певну допомогу в їх підтримці.

Якщо всі думають, що вони "твої", ти їх ніколи не позбудешся, і перейдеш до чогось нового.a


2

Я не знаю, чи це запитували, але яка конвенція про іменування є офіційною версією? Якщо Ваша версія офіційна, то я скажу це питання з точки зору. Тож замість того, щоб сказати, "Person X відмовив усі мої зміни", просто скажіть: "Person X вніс зміни, які суперечать офіційним умовам іменування". Якщо офіційної конвенції немає, то я пропоную їй повідомити, що ви не цінуєте внесені зміни без попередньої консультації.

В будь-якому випадку я думаю, що "війна" не є відповіддю. Навіть якщо ви виграєте, ви програєте.


Це єдина правильна відповідь. Якщо існує стандарт іменування, то імена повинні відповідати стандарту, і той, хто хоче, щоб вони мали інші імена, неправильно. Якщо немає стандарту іменування, має бути стандарт іменування - команда, або технічний керівник, або хто - повинен написати його. Такі умови кодування є тьмяними, але вони є важливою частиною соціальної тканини, яка дозволяє команді працювати.
Том Андерсон

1

Це жахлива поведінка. Це здається, що вона не шкодує, тому візьміть це до свого начальника і зробіть випадок, щоб її скасувати доступ до тих пір, поки вона не зможе переконатись, що не возитися.

Якщо ваш начальник не технічний, поясніть це термінами, які вони зрозуміють. Уявіть, що ви починаєте роботу в поштовій кімнаті, де посада сортується в отвори голубів, готові до доставки. Ви вирішуєте в односторонньому порядку сортувати отвори голубів за підлогою, потім прізвище замість поточної системи відділу, а потім підлоги. Це може полегшити ваше життя за короткий термін, але вас вб'є інший персонал поштової кімнати.

Це поза грубістю. Я був би розлючений.


1

Крім встановлення дозволів на зупинку випадкових людей, що їх змінюють, ви також повинні пояснити, як це її робота - перевірити функціональність, ви не можете дати жодної гарантії надійності, якщо випадкові люди вносять зміни в код.


1

Як усі казали, вона не мала цього робити, якби тільки з поваги до вас, оскільки ви є творцем, який підтримує ці запити.

Сказавши це, я не бачу, щоб хтось згадував той факт, що якщо вона перейменувала ваші запити в першу чергу, це було тому, що вона не могла осмислити вашу угоду про іменування.
Таким чином, проблему можна було б легко вирішити, задокументувавши свою конвенцію про іменування та забезпечивши доступ співробітників до документа та змогу знайти те, що їм потрібно.

Ви також повинні бути обережними та враховувати, як інші люди будуть знаходити та використовувати ваші запити: якщо ваша конвенція про іменування не дозволяє їм ефективно виконувати свою роботу, вам, ймовірно, потрібно вести більш ретельний список ваших запитів, використовуючи, можливо, теги та узгоджені ключові слова, щоб інші могли знайти те, що шукають.

Головне, тут я думаю, що ніхто не працює ізольовано, і найкращий спосіб уникнути наступання пальців ніг один на одного - це спілкування та узгодження загальних принципів.


0

Я відповів би таким же чином - оскаржуючи ваше рішення повернути все назад. Просто відновіть її зміни та напишіть справді короткий електронний лист вашим колегам:

"Змінено rXXXX на даний момент, оскільки я не зрозумів, що це умовне іменування. Дякую, що спробував. :)"


0

Так, ви надмірно реагуєте.

Існує щось, що називається контролем версій, що, крім усього іншого, його застосовують для того, щоб не потрібно вибивати $ #! 7 з колег, коли вони возиться з вашими речами. Просто поверніть на попередню версію і заблокуйте файл, тому дозвольте їй впоратися з гнівом. Це відкриє вам можливість пояснити, що виконувати радикальні зміни коду, які залежать від чужих речей без поважних причин і не вимагати спочатку, це не просто неправильно, вкрай непрактично і практично гріх.

Звичайно, це передбачає, що ваша умова іменування краща, ніж її, і що ви можете насправді створити резервну копію цього рішення грунтовними об'єктивними аргументами, якщо це не так, наймудріше, що потрібно робити, почати змінювати свій код, як тільки ви зможете впоратися зі змінами та спробуйте придумати кращу конвенцію про іменування наступного разу.

Не приймайте це до свого начальника, зрілий спосіб вирішити це безпосередньо зі своїм колегою, після цього вам доведеться працювати з нею, щоб нерозумно пошкодити стосунки за легко вирішувану сварку.

Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.