Вопросы
John Smith
360

Что это значит

Читая очередную вакансию попадается такая фраза

опыт и умение разбираться в чужом коде, оптимизация существующего кода. Насчёт оптимизации ясно , но что значит опыт разбираться в чужом коде? Если я работаю с фреймворками , это считается ? Там ведь чужой код ?

Материал опубликован пользователем.
Нажмите кнопку «Написать», чтобы поделиться мнением или рассказать о своём проекте.

Написать
{ "author_name": "John Smith", "author_type": "self", "tags": [], "comments": 8, "likes": 0, "favorites": 1, "is_advertisement": false, "subsite_label": "ask", "id": 99318, "is_wide": true, "is_ugc": true, "date": "Thu, 26 Dec 2019 19:11:02 +0300", "is_special": false }
0
{ "id": 99318, "author_id": 382611, "diff_limit": 1000, "urls": {"diff":"\/comments\/99318\/get","add":"\/comments\/99318\/add","edit":"\/comments\/edit","remove":"\/admin\/comments\/remove","pin":"\/admin\/comments\/pin","get4edit":"\/comments\/get4edit","complain":"\/comments\/complain","load_more":"\/comments\/loading\/99318"}, "attach_limit": 2, "max_comment_text_length": 5000, "subsite_id": 199125, "last_count_and_date": null }
8 комментариев
Популярные
По порядку
Написать комментарий...
4

Это значит тонны говнокода без комментариев и, зачастую, без внятных сообщений комитов и ссылок в трекер. Плюс национальная специфика.

Вы часто лазите в исходники фреймфорков?

Ответить
0

Не часто , но ради интереса заглядываю

Ответить
1

У каждого программиста есть свой "почерк". Кто-то привык к определенным конструкциям, логике. Кто-то использует, черт возьми, пробелы вместо табов. И во всем этом "почерке" и чужой логике нужно уметь разбираться

Ответить
1

У каждого свой стиль , это понятно . Одно дело вникнуть в стандарт , другое разбираться в коде. А код бывает разный , и названия переменных и функции иногда забавляют , или когда в коде не комментариев .

Ответить
0

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

 
И в 99 из 100, то что вам встретится, дешевле (для вашего психического здоровья) переписать с нуля

Ответить
0

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

Ответить
0

Я на самом деле не имею претензий к коду, меня удивляет формулировка , но ведь код у всех разный и каждый раз приходится разбираться в новом , по сути , если я работал с кодом одного разработчика , то это считается за опыт , если учесть что на изучение кода нового проекта может уйти неделя . А насчёт говнокода, зависит от обстоятельств . Я за 1000 рублей условных не готов все делать по правилам )).

Ответить
0

ты серьезно)))?! минимум процентов 30 работы это копание чужого года))) это минимум)) или ты считаешь, что в компаниях нужны белоручки, которые пишут только код с нуля))) бывает что 500 строках теряешься в чужом коде)))

Ответить
{ "page_type": "article" }

Прямой эфир

[ { "id": 1, "label": "100%×150_Branding_desktop", "provider": "adfox", "adaptive": [ "desktop" ], "adfox_method": "createAdaptive", "auto_reload": true, "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "bugf", "p2": "ezfl" } } }, { "id": 2, "label": "1200х400", "provider": "adfox", "adaptive": [ "phone" ], "auto_reload": true, "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "bugf", "p2": "ezfn" } } }, { "id": 3, "label": "240х200 _ТГБ_desktop", "provider": "adfox", "adaptive": [ "desktop" ], "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "bugf", "p2": "fizc" } } }, { "id": 4, "label": "Article Branding", "provider": "adfox", "adaptive": [ "desktop" ], "adfox": { "ownerId": 228129, "params": { "p1": "cfovx", "p2": "glug" } } }, { "id": 5, "label": "300x500_desktop", "provider": "adfox", "adaptive": [ "desktop" ], "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "bugf", "p2": "ezfk" } } }, { "id": 6, "label": "1180х250_Interpool_баннер над комментариями_Desktop", "provider": "adfox", "adaptive": [ "desktop" ], "adfox": { "ownerId": 228129, "params": { "pp": "h", "ps": "bugf", "p2": "ffyh" } } }, { "id": 7, "label": "Article Footer 100%_desktop_mobile", "provider": "adfox", "adaptive": [ "desktop", "tablet", "phone" ], "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "bugf", "p2": "fjxb" } } }, { "id": 8, "label": "Fullscreen Desktop", "provider": "adfox", "adaptive": [ "desktop", "tablet" ], "auto_reload": true, "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "bugf", "p2": "fjoh" } } }, { "id": 9, "label": "Fullscreen Mobile", "provider": "adfox", "adaptive": [ "phone" ], "auto_reload": true, "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "bugf", "p2": "fjog" } } }, { "id": 10, "disable": true, "label": "Native Partner Desktop", "provider": "adfox", "adaptive": [ "desktop", "tablet" ], "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "clmf", "p2": "fmyb" } } }, { "id": 11, "disable": true, "label": "Native Partner Mobile", "provider": "adfox", "adaptive": [ "phone" ], "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "clmf", "p2": "fmyc" } } }, { "id": 12, "label": "Кнопка в шапке", "provider": "adfox", "adaptive": [ "desktop" ], "adfox": { "ownerId": 228129, "params": { "p1": "bscsh", "p2": "fdhx" } } }, { "id": 13, "label": "DM InPage Video PartnerCode", "provider": "adfox", "adaptive": [ "desktop", "tablet", "phone" ], "adfox_method": "createAdaptive", "adfox": { "ownerId": 228129, "params": { "pp": "h", "ps": "bugf", "p2": "flvn" } } }, { "id": 14, "label": "Yandex context video banner", "provider": "yandex", "yandex": { "block_id": "VI-223676-0", "render_to": "inpage_VI-223676-0-1104503429", "adfox_url": "//ads.adfox.ru/228129/getCode?pp=h&ps=bugf&p2=fpjw&puid1=&puid2=&puid3=&puid4=&puid8=&puid9=&puid10=&puid21=&puid22=&puid31=&puid32=&puid33=&fmt=1&dl={REFERER}&pr=" } }, { "id": 15, "label": "Баннер в ленте на главной", "provider": "adfox", "adaptive": [ "desktop", "tablet", "phone" ], "adfox": { "ownerId": 228129, "params": { "p1": "byudx", "p2": "ftjf" } } }, { "id": 16, "label": "Кнопка в шапке мобайл", "provider": "adfox", "adaptive": [ "tablet", "phone" ], "adfox": { "ownerId": 228129, "params": { "p1": "byzqf", "p2": "ftwx" } } }, { "id": 17, "label": "Stratum Desktop", "provider": "adfox", "adaptive": [ "desktop" ], "auto_reload": true, "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "bugf", "p2": "fzvb" } } }, { "id": 18, "label": "Stratum Mobile", "provider": "adfox", "adaptive": [ "tablet", "phone" ], "auto_reload": true, "adfox": { "ownerId": 228129, "params": { "pp": "g", "ps": "bugf", "p2": "fzvc" } } }, { "id": 19, "disable": true, "label": "Тизер на главной", "provider": "adfox", "adaptive": [ "desktop", "tablet", "phone" ], "auto_reload": true, "adfox": { "ownerId": 228129, "params": { "p1": "cbltd", "p2": "gazs" } } }, { "id": 20, "label": "Кнопка в сайдбаре", "provider": "adfox", "adaptive": [ "desktop" ], "adfox": { "ownerId": 228129, "params": { "p1": "cgxmr", "p2": "gnwc" } } } ] { "page_type": "default" }