LINUX.ORG.RU
ФорумTalks

Почему в гугле работают такие говнокодеры?

 


0

2

Субжект. Можно сказать, эта тема впридачу к той, где обсуждают CVE в хроме - >350 критических уязвимостей, найденных за год.

Известно ли вам то, что почти весь гуй в Android написан Java? А известно ли вам то, что только в одном базовом классе для всех виджетов на экране (кнопок, картинок, полей для ввода) - более 18 тысяч строк в одном файле? И такое там повсеместно.

Ппц, как ревью это вообще могло пропустить? Понятно, что в Java нет partial-классов как в шарпе, но разделить функционал на несколько подклассов нельзя было чтоле? Вы че там курите, наркоманы? КАК ЭТО ОБСЛУЖИВАТЬ!???

https://android.googlesource.com/platform/frameworks/base/+/android-4.3_r2.1/core/java/android/view/View.java

В нульпосте линк на View.java из Android 4.3. Для контраста, вот View.java из main бранча:

https://android.googlesource.com/platform/frameworks/base/+/refs/heads/main/core/java/android/view/View.java

Более 30.000 строк кода в ОДНОМ файле.

monobogdan1337
() автор топика
Последнее исправление: monobogdan1337 (всего исправлений: 1)

Как только твои треды форма пропускает? Понятно, что сам без тормозов, но мог бы хоть устать для приличия? Ты чем там упарываешься, что у тебя столько энергии? КАК НАМ ЭТО ТЕРПЕТЬ?!

Горшочек, не вари.

t184256 ★★★★★
()
Ответ на: комментарий от t184256

А ты один день в игнор кидай, в другой выноси - и так чередуй, сам заодно передохнешь))

Мне нравится смотреть на дискуссию после создания топика. Считай это чем-то типа фетиша.

monobogdan1337
() автор топика
Последнее исправление: monobogdan1337 (всего исправлений: 1)

но разделить функционал на несколько подклассов

Сейчас тебе прилетит от математиков.

seiken ★★★★★
()

более 18 тысяч строк в одном файле

Молодцы какие, ещё б не на жаве это было и совсем хорошо. Люблю простые дубовые решения.

yu-boot ★★★★★
()
Ответ на: комментарий от seiken

javadoc, да.

Так если их отключить, все равно будет довольно много.

monobogdan1337
() автор топика
Ответ на: комментарий от monobogdan1337

В трекере ты сейчас вне конкуренции. После того, как SerW выдохся.

А i-rinat последние недели редко бывает на лоре, так что…

greenman ★★★★★
()
Последнее исправление: greenman (всего исправлений: 1)

Ради этого точно стоило разбанивать?

thesis ★★★★★
()
Последнее исправление: thesis (всего исправлений: 1)
Ответ на: комментарий от mydibyje

Остается вопрос, почему этого разбанили и разморозили.

thesis ★★★★★
()
Последнее исправление: thesis (всего исправлений: 1)
Ответ на: комментарий от t184256

У тебя за этот год было шесть выходных. Шесть дней ты не писал ничего не ЛОР. За этот год был как минимум один день, в который ты написал аж 63 сообщения. Возможно, было и больше, я только несколько самых ярких дней проверил. Вообще непросто найти тему, в которой ты бы не отметился.

i-rinat ★★★★★
()
Ответ на: комментарий от monobogdan1337

А для коммьюнити в целом — заморозку и бан, в обоих из которых ты буквально вчера прекрасно себе находился, классно же было.

t184256 ★★★★★
()
Ответ на: комментарий от t184256

Ну, как видишь модераторы не считают нужным меня банить из-за мнения одного-двух мимокроков на форуме))

monobogdan1337
() автор топика

Ну блин вот так. Возможно гайды такие, возможно люди которые писали этот код в душе крестоеды (до сих пор хочу взглянуть в глаза тем кто делал кодогенератор для протобуфа под питон, там нарушены сразу все известные style конвенции), возможно легаси, возможно производительность плюс легаси. У гугла зачастую довольно специфические проблемы, которые толкают людей писать весьма специфический код. Ну и просто говнокодеры везде есть.

upcFrost ★★★★★
()
Ответ на: комментарий от DarkAmateur

Да, жалко. Треть форума — он с бомжедевайсами, хаброкреативом и чистым тупняком, треть — @SerW с его миллионом недоделанных дел, пятнадцатью дисфункциональными бизнеса и в горящем Ейске и и невозможностью купить кресло в результате, ещё четверть — @ConLenov с охами по очередной проприетарной консоли, которую не смог себе позволить и теперь в нарушение здравого смысла влажно прилюдно мечтает эмулировать на какой-нибудь ардуине. Мусор надо чистить.

t184256 ★★★★★
()
Ответ на: комментарий от t184256

Жесть ты негативный. Сам же провоцируешь срач и бардак, вместо того, чтобы закинуть в игнор или просто пройти мимо.

Жизненные топики всегда добавляли ламповости любому форуму. Не все это видят просто.

monobogdan1337
() автор топика
Последнее исправление: monobogdan1337 (всего исправлений: 1)
Ответ на: комментарий от monobogdan1337

Представь, что тебе сосед в десятый раз насрал на лестничную площадку, и на твое праведное возмущение он тебе в ответ:

Жесть ты негативный. Сам же провоцируешь срач и бардак, вместо того, чтобы убрать или просто пройти мимо.

Так понятнее?

t184256 ★★★★★
()
Последнее исправление: t184256 (всего исправлений: 1)
Ответ на: комментарий от i-rinat

Кстати чёт тебя редко видно, чво пилишь новое признавайся.

Признаться можно вот тут Кто что кодит, патчит, чинит, ломает, переводит, иллюстрирует, озвучивает?

LINUX-ORG-RU ★★★★★
()
Последнее исправление: LINUX-ORG-RU (всего исправлений: 1)
Ответ на: комментарий от t184256

Нет, не понятнее. Форум - это не падик, создание тем, которые интересны одной части сообщества и неинтересны другой - это не тоже самое, что «насрать».

Тебе дали все инструменты для фильтрации интересного только тебе контента и личностей, но ты ими не пользуешься и предпочитаешь прийти и сам лично «насрать». ССЗБ.

monobogdan1337
() автор топика
Ответ на: комментарий от monobogdan1337

Форум — это публичное пространство. То, что создание тем, интересных только двум участникам, запрещено, а только одному — нет — досадное недоразумение, которое тебя никак не оправдывает. В тредах твоих обсуждают твою личность, да и то только потому, что на этом форуме из-за тебя уже и обсудить уже нечего.

t184256 ★★★★★
()
Последнее исправление: t184256 (всего исправлений: 1)
Ответ на: комментарий от t184256

В топиках достаточно и обсуждений по теме. Никто не виноват, что приходят такие как ты и начинают вайпать топики, пытаясь подстроить форум под себя. А потом мы говорим «ууу, лор скатился в говно, новые пользователи нам нинужны!».

monobogdan1337
() автор топика
Ответ на: комментарий от t184256

https://i.imgur.com/Cml1ZDJ.png

Внимательнее в профиль загляни. Давали бы таким, как ты модерку, уже давно лора не существовало бы как форума, или актив был бы нулевой. Я тебе даже пример приведу - gcup.ru

monobogdan1337
() автор топика
Последнее исправление: monobogdan1337 (всего исправлений: 1)

Потому что это не бизнес функционал во внутренней кухне, а фремворк сделанный для пользователей, он написан так чтоб им было удобно и привычно пользоваться, при этом он не должен создавать оверхеда на производительность.
Кстати по этой причине не стоит «учиться правильно программировать» смотря на стандартный Java classpath, там первично не «качество» кода и его поддерживаемость, а контракты и производительность.

P.S. Тема в толкс, так у тебя скор не вырастит

Aber ★★★★★
()
Ответ на: комментарий от Aber

Я специально в толксы и запилил, т.к формально это оффтопик.

С одной стороны это так, но с другой ведь нужно как-то это поддерживать в будущем. С такой «кашей» (сам кодстайл нормальный, но писать больше 1-2к строк в один файл - моветон, потому и в кавычках) это может быть проблематично - в том числе и накосячить в какой-то performance-wise штуке. Интересно глянуть, что там в uikit, но никто ж не даст исходников))

monobogdan1337
() автор топика
Последнее исправление: monobogdan1337 (всего исправлений: 1)
Ответ на: комментарий от monobogdan1337

В топиках достаточно и обсуждений по теме.

В этом — ровно один, где вежливо говорится, что ты — дятел и не разобрался. Это не оправдывает создания топика, скорее наоборот.

Никто не виноват, что приходят такие как ты и начинают вайпать топики

Нет у меня вайпалки, а то я бы порядок навёл.

пытаясь подстроить форум под себя. А потом мы говорим «ууу, лор скатился в говно, новые пользователи нам нинужны!».

Нужны. Обучаемые. Пользующиеся ЛОРом как ЛОРом, а не как микроблогом для регистрации каждого чиха и хабром для написания сомнительных технически статей. Способные понимать что-то, кроме бана и заморозки.

t184256 ★★★★★
()
Ответ на: комментарий от monobogdan1337

Дело в деньгах.
Помню гугл очень долго пыталась добиться плавности UI, чтоб UX был не сильно хуже чем в MacOS. Когда у них что-то получилось к 4.х это было записано в существенные достижения платформы. Это им и приносит профит.

Уж гугл может нанять хороших кодеров, тестеров, и организовать 100% покрытие когда, потому для них хоть 100 LOC в одном файле не проблема.

Aber ★★★★★
()
Последнее исправление: Aber (всего исправлений: 1)

CVE в хроме - >350 критических уязвимостей, найденных за год.

Потому что это не неуловимый Джо, в отличии от.

grem ★★★★★
()
Ответ на: комментарий от monobogdan1337

но писать больше 1-2к строк в один файл - моветон

rtw8822c_table.c - 46301 строк
phy_n.c - 28594 строк
lpfc_sli.c - 22525 строк
Дальше продолжать?

anc ★★★★★
()

На вид там больше половины строк – документация. А сколько % документации в твоём коде?

snizovtsev ★★★★★
()

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

snizovtsev ★★★★★
()
Ответ на: комментарий от seiken

но разделить функционал на несколько подклассов

Сейчас тебе прилетит от математиков.

Я не математик, и на жабе под андроид писал немного и давно. В жабе, насколько помню, нет множественного наследования. И это хорошо, может быть, а то понагородили бы как в плюсах. Зато можно сделать имплементации разных классов сколько угодно.

Ну и чего, что сделали на 18К строк? По максимуму учли для всех View, что они могут делать - хорошо спроектировали. В наследниках меньше писать, причем возможно много раз повторяясь.

bugs-bunny
()

Корпорация бабла, А так, эти господа нанимают малообразованных, но послушных и управляемых обезъянок, которых, главное, любой момент можно без проблем выбросить на улицу, уволив. Сложная оптимизация, математический просчет требует еще не только незаурядную способности ума, но еще определенную долю независимости и свободу программисту в работе, а такое положение невсегда присыпается в их внутреннние представление об работе, повышает управляемые риски в сторону - «бунта», когда корыто или их галера может накрыться медным тазом. А от этого могут постадать интересы 'Уважаемых людей', которые повязаны с google и имеют свои интересы.

nager
()
Закрыто добавление комментариев для недавно зарегистрированных пользователей (со score < 50)