Говнокод #27104 — PHP — Говнокод.ру

PHP / Говнокод #27104

0
1. 01
2. 02
3. 03
4. 04
5. 05
6. 06
7. 07
8. 08
9. 09
10. 10
11. 11
12. 12
13. 13
14. 14
15. 15
16. 16
17. 17
18. 18
19. 19
20. 20
21. 21
22. 22
23. 23
24. 24
25. 25
26. 26
27. 27
28. 28
29. 29
30. 30
31. 31
32. 32
33. 33
34. 34
```
https://habr.com/ru/company/vk/blog/527420/

ВКонтакте снова выкладывает KPHP 

....

Сгенерировать С++ из PHP — этого мало. Собственно говоря, это самое простое.

Во-первых, в PHP мы используем кучу функций стандартной библиотеки: header(),
 mb_strlen(), curl_init(), array_merge(). Их тысячи — и все должны быть
реализованы внутри KPHP с учётом типизации и работать так же, как в PHP.
Реализация всего PHP stdlib (а также KPHP-дополнений), всех PHP-типов с
операциями и допущениями — это называется runtime, вон там квадратик сверху.

Во-вторых, PHP-сайт — это веб-сервер. Следовательно, и в KPHP должна быть вся
серверная часть, чтобы можно было в том же nginx подменить PHP-шный upstream
на KPHP-шный — и всё продолжало работать так же. KPHP поднимает свой веб-сервер,
оркестрирует процессы, заполняет суперглобалы и переинициализирует состояние,
как и PHP… Это тоже хардкорная часть — называется server, квадратик снизу.

И только имея результирующий код C++, написанные runtime и server, всё это можно
объединить и отдать на откуп плюсовым компиляторам. Мы используем g++ — там в
диаграмме есть квадратик g++. Но не совсем так: у vk.com настолько огромная
кодовая база, что этот компилятор не справляется, и поэтому мы применяем патченный
distcc для параллельной компиляции на множестве агентов. В итоге всё линкуется в
один огромный бинарник (это весь vk.com), он раскидывается на кучу бэкендов и
синхронно перезапускается. Каждая копия запускает мастер-процесс, который порождает
группу однопоточных воркеров. Вот они на самом деле и исполняют исходный PHP-код.

Многие технические проблемы остаются за кадром — их не опишешь в статье на Хабре.
Чего стоит один только сбор трейсов при ошибках: ведь в С++ не получить
человекочитаемый стек, а хочется разработчику вообще его на PHP-код намаппить.
Гигантское количество внутренних нюансов, множество подпорок и легаси — но в итоге
продукт хорошо работает и развивается.
```
PHP и C++... два говна нашли друг друга

Запостил: j123123, 11 Ноября 2020

Tweet
Комментарии (46) RSS
- j123123 11.11.2020 22:11 # +2
  https://github.com/VKCOM/kphp/blob/master/runtime/datetime.cpp#L42
  
  static inline int32_t is_leap(int32_t year) { return ((year % 4 == 0) ^ (year % 100 == 0) ^ (year % 400 == 0)); }
  
  Надеюсь, это говно не понадобится человечеству в 2100 году. Тут еще операция xor, какая оптимизация )))
  Ответить
  - Kypumca_rpuJlb 11.11.2020 23:18 # 0
    
    Какой багор )))
    Это же надо было так обосраться )))
    Ответить
    - defecate-plusplus 11.11.2020 23:20 # 0
      
      а что не так?
      Ответить
      
      Kypumca_rpuJlb 11.11.2020 23:21 # 0
      
      Для 2100 это вернёт 1, а должно 0.
      Ответить
      
      defecate-plusplus 11.11.2020 23:23 # 0
      
      почему 1?
      https://ideone.com/Imj6zB
      Ответить
      
      guest6 11.11.2020 23:32 # 0
      
      А да, это я лопухнуться, извините.
      Ответить
- j123123 11.11.2020 22:14 # +1
  https://github.com/VKCOM/kphp/blob/master/runtime/datetime.cpp#L682
  
  if (s[0] == '-' || (s[0] == '+' || ('0' <= s[0] && s[0] <= '9'))) { int cnt; pos = -1; if (sscanf(s.c_str(), "%d%20s %n", &cnt, str, &pos) == 2 && pos != -1) { bool error = false; switch (str[0]) { case 'd': if (str[1] == 'a' && str[2] == 'y' && ((str[3] == 's' && str[4] == 0) || str[3] == 0)) { t.tm_mday += cnt; break; } error = true; break; case 'h': if (str[1] == 'o' && str[2] == 'u' && str[3] == 'r' && ((str[4] == 's' && str[5] == 0) || str[4] == 0)) { t.tm_hour += cnt; break; } error = true; break; case 'm': if (str[1] == 'o' && str[2] == 'n' && str[3] == 't' && str[4] == 'h' && ((str[5] == 's' && str[6] == 0) || str[5] == 0)) { t.tm_mon += cnt; break; } if (str[1] == 'i' && str[2] == 'n' && str[3] == 'u' && str[4] == 't' && str[5] == 'e' && ((str[6] == 's' && str[7] == 0) || str[6] == 0)) { t.tm_min += cnt; break; } error = true; break; case 'w': if (str[1] == 'e' && str[2] == 'e' && str[3] == 'k' && ((str[4] == 's' && str[5] == 0) || str[4] == 0)) { t.tm_mday += cnt * 7; if (!strncmp(s.c_str() + pos, "1 day", 5)) { t.tm_mday += (s[0] == '+' ? 1 : -1); pos += 5; while (s[pos] == ' ') { pos++; } } break; } error = true; break;
  
  ухх бля, вот сразу видно, чувствуется рука вконтактовского олимпиадника
  Ответить
  - rotoeb 12.11.2020 13:02 # 0
    
    Ну напиши сам эту функцию и выложи её сюда. Посмотрим, потечём.
    Ответить
- MAPTbIwKA 11.11.2020 22:16 # +4
  
  > if (str[1] == 'o' && str[2] == 'n' && str[3] == 't' && str[4] == 'h' && ((str[5] == 's' && str[6] == 0) || str[5] == 0))
  
  никогда не берите на работу олимпиадников и пыхеров
  
  Кстати, надо бы Ротоёбу присоветовать ВК в качестве работы: он примерно в таком же стиле пишет, правда на пхп, но там и пхп такой же ведь
  Ответить
  - defecate-plusplus 11.11.2020 23:30 # 0
    
    эт че получается, вконтакшле не на пхп написан?
    Ответить
    - MAPTbIwKA 11.11.2020 23:31 # 0
      
      там бекенд с алгоритмами на сишке, веб морда на пыхе, а потом они стали пых в сишку компилировать (по аналогии с фейсбуком)
      Ответить
      
      defecate-plusplus 11.11.2020 23:36 # 0
      
      какой багор
      впрочем бекенд на сишке я писал ещё в 2004
      и че т больше не хочу повторять этот опыт
      Ответить
      
      guest6 11.11.2020 23:39 # 0
      
      Ого. Ты давно вуз окончил?
      Ответить
      
      defecate-plusplus 11.11.2020 23:45 # 0
      
      как будто вчера
      Ответить
    - j123123 11.11.2020 23:31 # 0
      
      Сам вконтакт написан-таки не совсем на пхп, а на особом подмножестве пхп, который через вот эту ебаную срань как-то компилируется в C++ и запускается. Подробности см. на хабре.
      Ответить
- j123123 11.11.2020 23:27 # +1
  https://github.com/VKCOM/kphp/blob/04819c075f5599582f4f8780775f26a6377ca05d/runtime/string_functions.cpp#L375
  
  string f$html_entity_decode(const string &str, int64_t flags, const string &encoding) { if (flags >= 3) { php_critical_error ("unsupported parameter flags = %ld in function html_entity_decode", flags); } bool utf8 = memchr(encoding.c_str(), '8', encoding.size()) != nullptr; if (!utf8 && strstr(encoding.c_str(), "1251") == nullptr) { php_critical_error ("unsupported encoding \"%s\" in function html_entity_decode", encoding.c_str()); return str; } int len = str.size(); string res(len * 7 / 4 + 4, false); char *p = &res[0]; for (int i = 0; i < len; i++) { if (str[i] == '&') { int j = i + 1; while (j < len && str[j] != ';') { j++; } if (j < len) { if ((flags & ENT_QUOTES) && j == i + 5) { if (str[i + 1] == '#' && str[i + 2] == '0' && str[i + 3] == '3' && str[i + 4] == '9') { i += 5; *p++ = '\''; continue; } } if (!(flags & ENT_NOQUOTES) && j == i + 5) { if (str[i + 1] == 'q' && str[i + 2] == 'u' && str[i + 3] == 'o' && str[i + 4] == 't') { i += 5; *p++ = '\"'; continue; } } int l = 0, r = entities_size; while (l + 1 < r) { int m = (l + r) >> 1; if (strncmp(str.c_str() + i + 1, ent_to_num_s[m], j - i - 1) < 0) { r = m; } else { l = m; } } if (strncmp(str.c_str() + i + 1, ent_to_num_s[l], j - i - 1) == 0) { int num = ent_to_num_i[l]; i = j; if (utf8) { if (num < 128) { *p++ = (char)num; } else if (num < 0x800) { *p++ = (char)(0xc0 + (num >> 6)); *p++ = (char)(0x80 + (num & 63)); ...
  
  Не, ну что это за пиздец?
  Ответить
  - MAPTbIwKA 11.11.2020 23:33 # 0
    
    почти что число кармака по крутости, лол
    
    Причм мне нравится, как ловко смешана весьма "высокоуровневая" крестушня типа std::string и
    *p++ = (char)(0xc0 + (num >> 6));
    *p++ = (char)(0x80 + (num & 63));
    Ответить
    - bormand 11.11.2020 23:34 # +1
      
      > utf8
      
      Настоящий олимпиадник инлайнит функции сам, не доверяя это конпелятору.
      Ответить
  - bormand 11.11.2020 23:36 # 0
    
    Блядь, они серьёзно детектят utf8 по наличию цифры 8 в названии кодировки?
    
    За каким хуем это вообще строка, если там джва допустимых значения. А ещё этот код вызывается на каждую энтитю, заебись для пирфоманса.
    Ответить
    - bormand 11.11.2020 23:44 # 0
      
      З.Ы. А нет, не на каждую. Ну про 8 всё равно пиздец.
      Ответить
      
      guest6 11.11.2020 23:48 # 0
      
      Ну там заебешься все кобенации перебирать:
      UTF8
      utf8
      UTF-8
      utf-8
      Utf-8
      и так далее
      
      Проще на восьмерку проверить.
      Ответить
      
      bormand 11.11.2020 23:51 # +3
      
      Ну тогда так: int is_utf8 = enc[enc.size() - 1] == '8'. И быстрее и точнее чем их говно.
      Ответить
      
      guest6 11.11.2020 23:54 # +2
      
      Это уже на сениора тянет.
      Ответить
      
      bormand 11.11.2020 23:54 # 0
      
      Оно на пустой строке правда крашится, ну да ладно.
      Ответить
      
      Kypumca_rpuJlb 11.11.2020 23:56 # 0
      
      Не крашнется. Передавать пустую строку ведь никто не будет...
      Ответить
      
      j123123 12.11.2020 00:03 # 0
      
      Можно даже так: int is_utf8 = enc[4] == '8'
      
      Если судить из документации соотв. функции в "PHP", для UTF-8 кодировки корректно только "UTF-8" в качестве параметра
      А вот у "cp1251" есть псевдонимы "Windows-1251", "win-1251", "1251". Т.е. достаточно просто проверять четвертый символ на восьмерку
      Ответить
      
      guest6 12.11.2020 00:05 # 0
      
      Длину ещё нужно проверить.
      Ответить
      
      j123123 12.11.2020 00:09 # 0
      
      Тогда можно и на основе длины решить. Если длина равна 5 то тогда это "UTF-8", иначе это "cp1251". Ни один из синонимов "cp1251" не имеет длину 5
      Ответить
      
      bormand 12.11.2020 00:05 # 0
      
      Ну да, у koi8-r и cp866 восьмёрки в другой позиции. А остальные кодировки в русском проекте маловероятны.
      Ответить
      
      bormand 12.11.2020 00:10 # 0
      
      switch (strchr(s, '8') - s) { case 2: // cp866 case 3: // koi8-r case 4: // utf-8 default: // cp1251 }
      Ответить
    - j123123 11.11.2020 23:45 # 0
      
      > Блядь, они серьёзно детектят utf8 по наличию цифры 8 в названии кодировки?
      
      Ну видимо их аналог пхп-шной функции html_entity_decode https://www.php.net/manual/ru/function.html-entity-decode.php поддерживает только "UTF-8" и "CP1251". Большего им не надо.
      Ответить
      
      bormand 11.11.2020 23:49 # 0
      
      Ну и нахуй тогда проверять на 1251 в следующей строке? И так же понятно, что раз восьмёрки нет - то 1251. А на koi8-r и cp866 всё равно глючит и не возвращает ошибку.
      Ответить
      
      j123123 11.11.2020 23:50 # +1
      
      Отличная оптимизация. Ждем вас в команду вконтакта!
      Ответить
      
      bormand 11.11.2020 23:52 # 0
      
      Я там выше оптимизнул лучше. Последний символ на 8 проверить. Такое вроде только в утф.
      Ответить
  - Kypumca_rpuJlb 11.11.2020 23:52 # 0
    
    Ого, даже бинарный поиск есть встроенный.
    Ответить
- Fike 11.11.2020 23:39 # +1
  
  > ВКонтакте снова выкладывает KPHP
  
  обиделись что в прошлый раз никто не заметил?
  
  > KPHP берёт PHP-код и превращает его в С++
  
  ребят, тайпскрипт не так работает
  Ответить
  - j123123 12.11.2020 09:34 # 0
    
    > обиделись что в прошлый раз никто не заметил?
    
    Почему никто? Я заметил. На говнокод даже отправлял куски кода оттуда
    Ответить
    - Fike 12.11.2020 11:10 # 0
      
      Обиделись что заметили
      Ответить
      
      bormand 12.11.2020 11:32 # 0
      
      Да оно просто нинужно за пределами ВК.
      
      Судя по статье, 99% существующего кода там тупо не взлетит т.к. все динамические фишки не поддерживаются. Не заюзать привычные экстеншены для СУБД, не запустить привычные фреймворки и ORM'ы.
      
      Т.е. нельзя взять и портировать свои PHP'шные наработки на K-PHP. Код нужно снуля разрабатывать специально под эту платформу.
      
      З.Ы. Хотя код Ротоёба, наверное, взлетит. Если поддержку mysql самостоятельно привернуть.
      Ответить
      
      j123123 12.11.2020 12:22 # 0
      
      > привычные экстеншены для СУБД
      
      mysql_real_escape_string()
      Ответить
- bormand 12.11.2020 00:07 # +2
  
  > мы тестируем на PHP а KPHP - для продакшена
  
  Заебись наверное отлаживать ситуации, когда в обычном пхп всё работает а на кпхп крашится.
  Ответить
  - guest6 12.11.2020 00:12 # 0
    
    Это уже забота отдела разработки кпхп.
    Ответить
    - defecate-plusplus 12.11.2020 00:13 # +1
      
      нахуй вообще это корейское говно, где импортозамещение??
      Ответить
      
      TOPT 12.11.2020 02:29 # 0
      
      Смысле нахуй. У них лапша топовая
      Ответить
  - Fike 12.11.2020 02:33 # 0
    
    это ж какой багор, когда твой код даже пхп нормально выполняет, а твоя поделка не может )))
    Ответить
- w-495 25.08.2021 18:34 # 0
  
  vanished
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только во вторник, пятницу или субботу
Где здесь C++, guest?!

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

PHP / Говнокод #27104

Комментарии (46) RSS

Добавить комментарий