Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Код просто убогий #31

Closed
ghost opened this issue Sep 24, 2019 · 167 comments
Closed

Код просто убогий #31

ghost opened this issue Sep 24, 2019 · 167 comments

Comments

@ghost
Copy link

ghost commented Sep 24, 2019

Как так можно писать, вы хоть знаете что такое ООП или оптимизация, этот код читать невозможно, это просто пиз.. мусор.

@NotStatilko
Copy link

Зато работает и Telegram

@Kwaskoff
Copy link

Зато работает и Telegram

этот аргумент используется в энтерпрайзе с явой =)

@Piterden
Copy link

Piterden commented Sep 25, 2019

@Kwaskoff вы хотите сказать, что был использован использованный аргумент?

@NotStatilko а пруфы можно?

@begetan
Copy link

begetan commented Sep 25, 2019

Как так можно писать, вы хоть знаете что такое ООП или оптимизация, этот код читать невозможно, это просто пиз.. мусор.

Это не люди такие, это C++ такой.

@NotStatilko
Copy link

@Kwaskoff вы хотите сказать, что был использован использованный аргумент?

@NotStatilko а пруфы можно?

Пруфы чего? Ноду запусти у себя

@nikitasius
Copy link

@Kwaskoff не нужно Java очернять 😄

@talentlessguy
Copy link

Можно примеры плохого кода в тоне? Я не сишник но на всякий случай

@NotStatilko
Copy link

Можно примеры плохого кода в тоне? Я не сишник но на всякий случай

Николай Валерьевич Дуров, перелогиньтесь

@talentlessguy
Copy link

Можно примеры плохого кода в тоне? Я не сишник но на всякий случай

Николай Валерьевич Дуров, перелогиньтесь

кек

мне для спора нужно показать что даже у телеги плохой код может быть

@Piterden
Copy link

Piterden commented Sep 26, 2019

Однобуквенные переменные

Такие места, я бы делал через динамическое название вызываемого метода, как на 303 строке и подобных ей. Ну или хотябы через switch statement

Как здесь - я бы работал в цикле, подставляя набор параметров

@Piterden
Copy link

@Kwaskoff вы хотите сказать, что был использован использованный аргумент?
@NotStatilko а пруфы можно?

Пруфы чего? Ноду запусти у себя

Я не о работоспособности. Но вроде есть...

@ghost
Copy link
Author

ghost commented Sep 27, 2019

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

История из жизни программистов Дурова

Парни смотрите у меня новый рекорд по зп, я за сегодня напечатал 20 страниц кода.
Красава но это можно было сделать используя 5 строк кода.
Не не гони так бабок мало дадут, а люди все равно схавают :)

Заходит ботаник и спрашивает парни как назвать переменную, а ему отвечают назови S, просто S? Да просто S это будет пушка, реал топ спасибо друзья вы лучшие, так и сделаю

Интересно когда вы будете оптимизировать код и введете в поиски S сколько букв S найдет редактор :)

@NotStatilko
Copy link

Обфускация же. Так бы Facebook со своей хренью хуLibra всё списали.

Я надеюсь что это было сделано специально

Честно

@ghost
Copy link
Author

ghost commented Sep 27, 2019

Возможно это обфускация но зачем?
В чем смысл тогда открытого исходного кода в github, если его невозможно читать. Программисты не смогут помочь улучшать код, пока не увидят нормальный оригинальный код.
А разбираться в этом, спасибо конечно но я откажусь, вы бы еще ассемблерный код скинули.

@NotStatilko
Copy link

Возможно это обфускация но зачем?
В чем смысл тогда открытого исходного кода в github, если его невозможно читать. Программисты не смогут помочь улучшать код, пока не увидят нормальный оригинальный код.
А разбираться в этом, спасибо конечно но я откажусь,

Возможно это обфускация но зачем?
В чем смысл тогда открытого исходного кода в github, если его невозможно читать. Программисты не смогут помочь улучшать код, пока не увидят нормальный оригинальный код.
А разбираться в этом, спасибо конечно но я откажусь, вы бы еще ассемблерный код скинули.

Ну и зачем ты им подсказал?

@melekhine
Copy link

Да нормальный код, он в первую очередь должен работать. Вы же не знаете в каких условиях он появился. Причешут со временем. А качество кода для MVP вредно (тк нужно тратить больше времени)

@serderovsh
Copy link

serderovsh commented Sep 27, 2019

@EosBot, если тебе непонятно, возьми да и перепиши. Сидит и ноет, Код написан нормально, это просто обфускация. Как выйдет офф. релиз думаю будет весь исходник.

@Piterden
Copy link

Да нормальный код, он в первую очередь должен работать. Вы же не знаете в каких условиях он появился. Причешут со временем. А качество кода для MVP вредно (тк нужно тратить больше времени)

Ну хз. Про свитч я узнал почему, но вот это и это...

А в каком виде доки видел?

@ghost
Copy link
Author

ghost commented Sep 28, 2019

@serderovsh Во первых это неуважения к сообществу github скидывать код пройденный через обфускацию. Во вторых это неуважения к программистам, для тех кто собирается участвовать в конкурсе. А так же у них меньше шансов чем у команды Дурого победить в конкурсе. Команда видит оригинальный код, им проще фиксить баги они ка краз бабки и получат.
А те кто участвую нефига не понимают, код написан машиной, как можно предложить исправления.
А теперь о главном, как можно создавать конкрус без обратной связи, даже на фрилансе есть человек, который отвечает, хорошая работа или плохая, то или не то.
Здесь ничего нету, не обратной связи не уважения.

Как трудно нанять 3 сапортов для конкруса, ведь они дахера денег стоят.
Это кстати уже помощь проекту от меня, информация стоит денег если что.
Если вы собираетесь добавить сапортов к концу конкруса и сказать что ваша команда победила а мы нет, пусть будет так. Только не нужно тратить время хороших программистов и давать им надежду победить. Просто бесит уже такое неуважения. Я участвовал на конкурсе EOS там такая же фигня была, помог исправить пару багов нехера не получил не копейки.

@ghost
Copy link
Author

ghost commented Sep 28, 2019

Тем более я бы с радостью поучаствовал, у меня большой опыт с крестами но я не вижу выгоды, что бы потратить свое время на этот проект.
Бесплатно не работаю.

@melekhine
Copy link

melekhine commented Sep 28, 2019

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

Так это вообще не очень и исходники, это же FunC, к нему даже документации еще нет. Немного терпения.

@fskuratov
Copy link

@serderovsh Во первых это неуважения к сообществу github скидывать код пройденный через обфускацию. Во вторых это неуважения к программистам, для тех кто собирается участвовать в конкурсе. А так же у них меньше шансов чем у команды Дурого победить в конкурсе. Команда видит оригинальный код, им проще фиксить баги они ка краз бабки и получат.
А те кто участвую нефига не понимают, код написан машиной, как можно предложить исправления.
А теперь о главном, как можно создавать конкрус без обратной связи, даже на фрилансе есть человек, который отвечает, хорошая работа или плохая, то или не то.
Здесь ничего нету, не обратной связи не уважения.

Как трудно нанять 3 сапортов для конкруса, ведь они дахера денег стоят.
Это кстати уже помощь проекту от меня, информация стоит денег если что.
Если вы собираетесь добавить сапортов к концу конкруса и сказать что ваша команда победила а мы нет, пусть будет так. Только не нужно тратить время хороших программистов и давать им надежду победить. Просто бесит уже такое неуважения. Я участвовал на конкурсе EOS там такая же фигня была, помог исправить пару багов нехера не получил не копейки.

Конкурс саппортится сообществом.
Чаты @tondev_ru и @tondev_en, на все вопросы ответят там.

@serderovsh
Copy link

serderovsh commented Sep 28, 2019

@EosBot За баги тебе никто и не должен платить, это помощь со стороны сообщества. Каждый день тысячи разработчиков помогают в развитии tron, eos, eth- и никто за это деньги им не платит.

Если платили за каждый найденный баг, все миллионерами стали бы

@ghost
Copy link
Author

ghost commented Sep 28, 2019

@serderovsh тогда смысла не вижу тут время тратить, пойду пилить свою программу.

@melekhine
Copy link

I suggest this ticket to removal.

@Piterden
Copy link

Piterden commented Sep 28, 2019

@awsom82 давайте переименуем в необычный

I propose to keep this ticket.

@onemanstartup
Copy link

@serderovsh Проект собравший 2 миллиарда баксов не будут платить за каждый найденный баг? В проекте в котором баги это денежные потери? OpenSource лол, ну запусти свой блокчейн, назови его TonClassic и посмотрим будет ли он успешен.

@serderovsh
Copy link

serderovsh commented Sep 28, 2019

@awsom82 - I agree, a meaningless topic!
@onemanstartup - нет, баги всегда были и будут. Вознаграждение зависит от самого уровня бага, в 3-5% случаях наносят удар по средствам клиентов.

@NotStatilko
Copy link

Да успокойтесь уже, дебилы. Пошутили и хватит. Зачем срач устраивать

@Piterden
Copy link

Займитесь лучше делом, господа...

@okgolove
Copy link

Российское сообщество даже на Github показывает всю свою дружелюбность и вежливость.

@frolovdev
Copy link

Даже Павел Дуров после sec не подозревал на что способны русские программисты ... Смотреть в продолжении

@anonymous2ch
Copy link

Тем временем, безталанные цуркерберги переименовали и запустили свою крипту: https://novi.com/

@j123123
Copy link

j123123 commented May 26, 2020

DPFvZuqU8AAeU0C

@anonymous2ch
Copy link

DPFvZuqU8AAeU0C

А что, какие-то другие БД с историей правок завезли уже? Или они по бинарным логам предлагают откатывать?

@j123123
Copy link

j123123 commented May 26, 2020

какие-то другие БД с историей правок

git

@anonymous2ch
Copy link

какие-то другие БД с историей правок

git

И правда

@j123123
Copy link

j123123 commented Jun 13, 2020

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

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

@anonymous2ch
Copy link

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

@j123123
Copy link

j123123 commented Jun 25, 2020

Там кстати из телеграма утечка персональных данных, если кто не в курсе. Это всё, что вам надо знать о приватности Телеграма и о компетентности "специалистов", которые его разрабатывали.
govnogram
https://www.youtube.com/watch?v=V1noby9YOq8

@AndreyKostromskoy
Copy link

AndreyKostromskoy commented Jun 28, 2020

@fisher @j123123 Ладно, про std::mutex::lock я услышал. Что ты ещё скажешь? 1-2 оплошности на >400k строк кода вполне нормальное явление. И да, делай скидку на то что Николай в первую очередь математик, а не программист.

@ghost
Copy link
Author

ghost commented Jul 4, 2020

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

@j123123
Copy link

j123123 commented Jul 5, 2020

Нет, код говно. Но на самом-то деле на код более-менее насрать. Тут вопрос в другом - кому оно нахрен надо? Этих криптовалют и так как грязи. Чем это отличается от Ethereum?

@AndreyKostromskoy
Copy link

AndreyKostromskoy commented Jul 5, 2020

@nullptr2
Прочитай WhitePaper

@AndreyKostromskoy
Copy link

AndreyKostromskoy commented Jul 5, 2020

@j123123 На данный момент в репозитории примерно 450k кода. Приведи хотя бы 5-10 отрывков объёмом по тысяче строк говнистого, по твоему мнению, кода. Либо признай, что ты недостаточно компетентен в блокчейне и смежных с ним технологиях

@j123123
Copy link

j123123 commented Jul 5, 2020

@DreamyDevil

Приведи хотя бы 5-10 отрывков

Мне лень бесплатно копаться в этом говне, хотя вот сегодня я как раз ради любопытства решил почитать немного этого говнокода, вот: https://govnokod.ru/26767#comment557495 можно ознакомиться

@AndreyKostromskoy
Copy link

@j123123
А теперь смотри сюда, мой дорогой друг. Во первых - единственный реальный пример плохого кода это Мьютекс. Пример с RLDP-HTTP-proxy для меня не выглядит ужасным. Насчёт github.com/vk-com/kphp-kdb - ты знаешь историю ВК? Сначала в 2006-2007 году сайт был стандартным: пользователь приходил на NGINX/Apache, шёл на PHP, а PHP шёл в MySQL за данными + кешируя в Мемкэше, если запросов к базе очень много. Но в середине 2008 у проекта был взрывной рост популярности, а шардироваться уже было некуда. Тогда Павел пригласил брата Доктора Николая Дурова и других олимпиадников, чтобы они помогли. А какие варианты NoSQL были? Реддиса тогда не было, Кассандры тоже не было. Был Хадуп, но он написан на Java(имхо, я бы тоже не стал бы использовать). Что касается говнистости репозитория. Да код плох и трудно читаем, но по сути это НЕ ОПЕНСОРС. Его не планировали ВЫКЛАДЫВАТЬ. По факту код выложен, но его забросили, и сообщество в него не контрибьютит. Это сделали лишь для того, чтобы Телеграм смог безболезненно забрать код со стороны ВКонтакте. Так что по факту это опенсорс, а на деле нет - просто груда лежачего кода без поддержки. Во-вторых Вконтакте и сейчас работает на KPHP и движках, пришли новые люди, чемпионы ICPC - Дмитрий Егоров, Борис Минаев, Павел Кунявский - они понимают код своих предшественников. Значит КОД можно понять, просто нужны навыки и желание. Также и с TON. Есть Free TON, который разрабатывается на основе этого репозитория, значит специалисты в области Блокчейна могут читать код и могут его развивать. А тебе всё лень. Не было бы лень, выучил бы как устроены Блокчейны, поднапрягся и понял бы весь код. Единственнное, что меня смущает это язык Fift он выглядет как помесь Ассемблера и Форта. Ну уже есть транслятор, которые опять таки написаны сообществом Free TON - значит у них есть желание и возможность разбираться в этом репозитории.

@anonymous2ch
Copy link

@DreamyDevil В точку. Добавлю что если кого-то не устраивает "чистота" дуровского (или любого другого) кода, то, вероятнее всего, свой код на <пиздатый язык программирования> с нуля этот человек написать зафейлил, а когда и смысл чужого не получается уловить, то в голове юного криптомагната происходит когнитивный диссонанс, выходом из которого и является обвинение окружающих в недостаточном использовании линтеров....

@j123123
Copy link

j123123 commented Jul 6, 2020

@DreamyDevil

Во первых - единственный реальный пример плохого кода это Мьютекс. Пример с RLDP-HTTP-proxy для меня не выглядит ужасным

Меня не волнует то, что там для тебя выглядит или не выглядит ужасным. Мое мнение - этот код говно. Не считаешь его говном, придерживаешься другого мнения - твои проблемы.
Если хочешь - можешь нанять группу независимых экспертов, которые будут оценивать говнистость кода TON по десятибальной шкале, или еще как-нибудь.

Что касается говнистости репозитория. Да код плох и трудно читаем, но по сути это НЕ ОПЕНСОРС. Его не планировали ВЫКЛАДЫВАТЬ.

Т.е. если выкладывать не планируется, можно и говнокодить? Ну ок, буду знать.

Вконтакте и сейчас работает на KPHP и движках, пришли новые люди, чемпионы ICPC - Дмитрий Егоров, Борис Минаев, Павел Кунявский - они понимают код своих предшественников

Мне откровенно насрать на все эти лычки и звания. Пусть показывают свою крутизну через качество своего кода.

@AndreyKostromskoy
Copy link

AndreyKostromskoy commented Jul 6, 2020

@j123123 https://govnokod.ru/25999#comment510263 Ты даже меня цитировать умудряешься. Какой ты интересный человек

Меня не волнует то, что там для тебя выглядит или не выглядит ужасным

Это я могу сказать и о тебе. Меня не интересует что для тебя выглядит или не выглядит ужасным . Группу экспертов, я бы посоветовал тебе нанять, ведь ИМЕННО ТЫ хочешь доказать говнистость кода, а не я. Мне доказывать нечего - я не утверждаю, что этот код говнистый, либо кристально чистый. Кристально чистого кода не существует, но то как ты чмыришь код и команду Доктора Дурова наводит на мысли, а не завидуешь ли ты ему?

Т.е. если выкладывать не планируется, можно и говнокодить? Ну ок, буду знать.

Знаешь, мои проекты я тоже пишу КАК МНЕ удобно, ведь я не планирую их выкладывать. И поддерживать их буду только я и мой друг, с которым мы вместе их писали. Так что да, если человек работает маленькой группой, и если он не планирует выкладывать в опенсорс, то он может писать КАК ЕМУ(и его группе) УДОБНО ЧИТАТЬ КОД.

Мне откровенно насрать на все эти лычки и звания. Пусть показывают свою крутизну через качество своего кода

А если бы тебе было не насрать, то ты бы знал, что ICPC самая престижная олимпиада в мире, и все топовые компании типа Google, Apple, Microsoft активно начинают охотиться за финалистами ICPC. И их не пугает, что человек будет писать "олимпиадный" код, потому что они знают, что человек доказавший своё МИРОВОЕ превосходство в программировании будет очень полезен для команды

@j123123
Copy link

j123123 commented Jul 6, 2020

ведь ИМЕННО ТЫ хочешь доказать говнистость кода

Нет. Это ты у меня спрашивал, чтобы я тебе какие-то примеры приводил. Ничего доказывать я не хочу. Можешь верить в его неговнистость.

Знаешь, мои проекты я тоже пишу КАК МНЕ удобно, ведь я не планирую их выкладывать.

Молодец, возьми с полки пирожок.

А если бы тебе было не насрать, то ты бы знал, что ICPC самая престижная олимпиада в мире, и все топовые компании типа Google, Apple, Microsoft активно начинают охотиться за финалистами ICPC.

Меняй методичку. Надоело уже слушать этот бред про мегагениальных олимпиадников, которые все такие крутые.
Talk is cheap. Show me the code.

@j123123
Copy link

j123123 commented Aug 5, 2020

https://www.ixbt.com/news/2020/08/05/telegram-mail-ru.html - там вот кстати пишут, что телегу в мылосру продать хотят. Так что готовьтесь
усманов-мем

@isgulkov
Copy link

isgulkov commented Aug 5, 2020

@j123123

Мне лень бесплатно копаться в этом говне, хотя вот сегодня я как раз ради любопытства решил почитать немного этого говнокода, вот: https://govnokod.ru/26767#comment557495 можно ознакомиться

Позволю себе заметить, что если речь о неуклюжей работе со строками (substr, find...), то это говнистость не столько кода, сколько всей "экосистемы" C++. Какие есть альтернативы этой реализации? Их две:

  • Невероятно, чудовищно медленный <regex> из стандартной библиотеки
  • Быстрая, но хуй знает кем на коленке написанная либа, которую надо будет втягивать с гитхаба сабмодулем и ставить свечку, чтоб ее потом кто-нибудь поддерживал

То есть, загадку про два стула автор этого кода отгадал верно, а диванный код-ревьюер-приебщик опять бесплатно соснул.

@j123123
Copy link

j123123 commented Aug 6, 2020

@isgulkov

Быстрая, но хуй знает кем на коленке написанная либа, которую надо будет втягивать с гитхаба сабмодулем и ставить свечку, чтоб ее потом кто-нибудь поддерживал

Ну так пусть бы налабали свою библиотеку, они ж там любят свои наколенные велосипеды костылить.
Хотя может оно и к лучшему, они отошли от стиля kphp, тогда они писали какую-то нечитаемую поебень на сишечке https://www.govnokod.ru/23348

@AndreyKostromskoy
Copy link

AndreyKostromskoy commented Aug 9, 2020

@isgulkov @j123123
https://www.govnokod.ru/23348 Вспомни школу. Ты когда решаешь уравнение на уроке алгебры результат ты пишешь так: iks=5, igrek=4, pervaya_peremennaya = 8, vtoraya_peremennaya = 10? Или так: x=5, y=4 , a=8, b=10? Скорее всего второй вариант.

@j123123
Copy link

j123123 commented Aug 9, 2020

Когда я пишу код, который потом надо будет кому-то читать, понимать и править, я буду стараться давать переменным осмысленные названия.

@j123123
Copy link

j123123 commented Jan 14, 2021

Кстати вот вести с полей:
https://techcrunch.com/2021/01/13/telegram-channels-banned-violent-threats-capitol/

Telegram confirmed to TechCrunch that it has removed dozens of channels over the course of the last day, citing concerns that the accounts were inciting violence.

Как говорится, enjoy your централизованная_параша. Cамолетики за телеграм только бросать не забывайте

@melekhine
Copy link

melekhine commented Jan 14, 2021

Когда я пишу код, который потом надо будет кому-то читать, понимать и править, я буду стараться давать переменным осмысленные названия.

да я даже для себя когда пишу так делаю, иначе уже через полгода придется тратить время

Telegram confirmed to TechCrunch that it has removed dozens of channels over the course of the last day, citing concerns that the accounts were inciting violence.

Как говорится, enjoy your централизованная_параша. Cамолетики за телеграм только бросать не забывайте

Thanks for sharing!

@j123123
Copy link

j123123 commented Sep 18, 2021

E_hI-vKWYAEkCtB
Свободный мессенжер.

@quenbyako
Copy link

Каждый раз, когда у меня горит пердак от убогого telegram api, и такой же убогой реализации ton и tl-b (До сих пор! В 2024, Карл!), я прихожу на этот ишью, и хихикаю с комментов. Это придает мне сил заниматься убогим телеграмом дальше.

Спасибо тебе, (уже) удаленный ОП, искренне тебя люблю за то, что ты создал буквально один из самых легендарных ишью на гитхабе ❤️🧡💛💚💙💜

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests