1. C++ / Говнокод #26628

    +1

    1. 001
    2. 002
    3. 003
    4. 004
    5. 005
    6. 006
    7. 007
    8. 008
    9. 009
    10. 010
    11. 011
    12. 012
    13. 013
    14. 014
    15. 015
    16. 016
    17. 017
    18. 018
    19. 019
    20. 020
    21. 021
    22. 022
    23. 023
    24. 024
    25. 025
    26. 026
    27. 027
    28. 028
    29. 029
    30. 030
    31. 031
    32. 032
    33. 033
    34. 034
    35. 035
    36. 036
    37. 037
    38. 038
    39. 039
    40. 040
    41. 041
    42. 042
    43. 043
    44. 044
    45. 045
    46. 046
    47. 047
    48. 048
    49. 049
    50. 050
    51. 051
    52. 052
    53. 053
    54. 054
    55. 055
    56. 056
    57. 057
    58. 058
    59. 059
    60. 060
    61. 061
    62. 062
    63. 063
    64. 064
    65. 065
    66. 066
    67. 067
    68. 068
    69. 069
    70. 070
    71. 071
    72. 072
    73. 073
    74. 074
    75. 075
    76. 076
    77. 077
    78. 078
    79. 079
    80. 080
    81. 081
    82. 082
    83. 083
    84. 084
    85. 085
    86. 086
    87. 087
    88. 088
    89. 089
    90. 090
    91. 091
    92. 092
    93. 093
    94. 094
    95. 095
    96. 096
    97. 097
    98. 098
    99. 099
    100. 100
    template<typename CharType>
    class Formatter
    {
    private:
    	static int _ToString(Int32 Value){
    		return CString<CharType>::Snprintf(
    			(CharType*)GSupportToStringBuffer,
    			TO_STRING_BUFFER_SIZE,
    			"%" PRId32,
    			Value);
    	}
    	static int _ToString(float Value){
    		return CString<CharType>::Snprintf(
    			(CharType*)GSupportToStringBuffer,
    			TO_STRING_BUFFER_SIZE,
    			"%f",
    			Value);
    	}
    	template<typename First, typename ... Args>
    	static void _ConvertArgs(Array<GenericString<CharType>>& _ArgBuffer, const First& _First, const Args& ... _Args)	{
    		_ArgBuffer.Add(ToString(_First));
    		_ConvertArgs(_ArgBuffer, _Args ...);
    	}
    	template<typename First>
    	static void _ConvertArgs(Array<GenericString<CharType>>& _ArgBuffer, const First& _First)	{
    		_ArgBuffer.Add(ToString(_First));
    	}
    	static bool _ScanPlaceholder(const CharType* Format, size_t Index, size_t Length, size_t& Placeholder, size_t& LastIndex)	{
    		size_t i = Index;
    		size_t j = 0;
    		while (i < Length && Format[i] != FormatterPlaceholder<CharType>::End)
    		{
    			if (!Char<CharType>::IsDigit(Format[i]))
    			{
    				return false;
    			}
    			else
    			{
    				GSupportToStringBuffer[j] = Format[i];
    				j++;
    			}
    			i++;
    		}
    		if (i == Length)
    			return false;
    		
    		GSupportToStringBuffer[j] = 0;
    #if defined(64BIT)
    		Placeholder = CString<CharType>::Atoi64((const CharType*)GSupportToStringBuffer);
    #else
    		Placeholder = CString<CharType>::Atoi((const CharType*)GSupportToStringBuffer);
    #endif
    		LastIndex = i;
    		return true;
    	}
    public:
    	template<typename T>
    	static GenericString<CharType> ToString(const T& Value)	{
    		int Length = Formatter<CharType>::_ToString(Value);
    		return GenericString<CharType>((char*)GSupportToStringBuffer, Length);
    	}
    	template<typename ... Args>
    	static GenericString<CharType> Format(const CharType* Format, const Args& ... _Args)	{
    		Array<GenericString<CharType>> _FormatedArgs;
    		_FormatedArgs.Reserve(sizeof...(Args));
    		Formatter<CharType>::_ConvertArgs(_FormatedArgs, _Args ...);
    		
    		const size_t _Length = CString<CharType>::Strlen(Format);
    		size_t Index = 0;
    		for (size_t i = 0; i < _Length; i++)
    		{
    			if (Format[i] == FormatterPlaceholder<CharType>::Begin)
    			{
    				size_t Placeholder = 0;
    				size_t LastIndex = 0;
    				if (_ScanPlaceholder(Format, i + 1, _Length, Placeholder, LastIndex) && Placeholder < sizeof...(Args))
    				{
    					Memory::Copy(GSupportFormatBuffer + Index,	_FormatedArgs[Placeholder].Data(),	_FormatedArgs[Placeholder].Length() * sizeof(CharType));
    					Index += _FormatedArgs[Placeholder].Length();
    					i = LastIndex;
    				}
    			}
    			else
    			{
    				GSupportFormatBuffer[Index] = Format[i];
    				Index++;
    			}
    		}
    		GSupportFormatBuffer[Index] = 0;
    		return GenericString<CharType>((const CharType*)GSupportFormatBuffer);
    	}
    };
    template<typename T>
    forceinline String ToString(const T& Value){
    	return Formatter<char>::ToString<T>(Value);
    }
    template<typename ... Args>
    forceinline String Format(const char* Format, const Args& ... _Args){
    	return Formatter<char>::Format(Format, _Args ...);
    }

    Три года назад писал printf аля Console.WriteLine в C#. Тут порезал до ста строк. https://pastebin.com/8BCLuBEm

    Запостил: Avery, 01 Мая 2020

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

    • у меня было проще

      template <class Arg1>
          inline void log_req(Arg1 arg1)
          {
              if constexpr (std::is_enum_v<Arg1>)
              {
                  std::cout << static_cast<int>(arg1);
              }
              else
              {
                  std::cout << arg1;
              }
          }
      
          template <class Arg1, class... Args>
          inline void log_req(Arg1 arg1, Args... args)
          {
              std::cout << arg1;
              log_req(args...);
          }
      
          template <class... Args>
          void log(Args... args)
          {
              log_req(args...);
              std::cout << std::endl;
          }
      Ответить
      • Копирование аргументов на каждый вызов — не самое лучшее поведение.
        Ответить
        • там нет копирования аргументов - это все инлайн .... инлайн :)
          Ответить
          • Увы, инлайн не спасает. Хотя бы потому, что с новых версий крестов ключевое слово «inline» не имеет к, собственно, инлайну, никакого отношения.
            https://wandbox.org/permlink/vEXXheiWMqkRU2Q7
            Ответить
            • Есть же какие-то нестандартные питушни, чтобы форсить инлайн.
              Ответить
              • Да, конпеляторозависимые, вроде «__forceinline» в «MSVC» и «__attribute__((always_inline))» в «gcc». Они, разумеется, в данном случае тоже не помогут, потому что при педераче аргументов по значению конпелятор обязан их педерать, внезапно, по значению — то есть скопировать.
                Ответить
                • А если речь не про кресты, а про няшную?

                  Копирование говна у няшной вроде бы не имеет спецэффектов, так что можно скопировать , а можно заинлайнить. А у крестов же вроде не так
                  Ответить
                  • Надо Стандарт няшной читать, но на первый взгляд вроде нормально работает.
                    https://gcc.godbolt.org/z/f1vn2h — конпелятор настолько умный, что даже не выделяет память под структурку, а сразу передаёт в log() нужные значения. Какой багор )))
                    Ответить
            • inline вроде бы всегда было добрым советом, примерно как register, нет?
              Ответить
              • Раньше — да, а теперь это просто указание на то, что функция может встретиться в нескольких transltion unit-ах.
                https://en.cppreference.com/w/cpp/language/inline
                Ответить
                • показать все, что скрытоvanished
                  Ответить
                  • > хм, а что будет, если я не поставлю inline?
                    ODR нарушица.
                    // test_1.h
                    #pragma once
                    
                    int sum(int a, int b)
                    {
                        return a + b;
                    }
                    
                    // test_1.cpp
                    #include <iostream>
                    #include "test_1.h"
                    
                    int main()
                    {
                        std::cout << sum(40, 2) << std::endl;
                    
                        return 0;
                    }
                    
                    // test_2.cpp
                    #include <iostream>
                    #include "test_1.h"
                    
                    void some_func(int a, int b)
                    {
                        std::cout << sum(a, b) << std::endl;
                    }
                    
                    // 1>test_2.obj : error LNK2005: "int __cdecl sum(int,int)" (?sum@@YAHHH@Z) already defined in test_1.obj
                    // fatal error LNK1169: one or more multiply defined symbols found


                    UPD: Джвух заголовочных файлов не надо, одного достаточно.
                    Ответить
                    • показать все, что скрытоvanished
                      Ответить
                      • Да.

                        > мне казалось, что ODR не нарушается, если код один-в-один одинаковый везде, нет?
                        Так будет с «inline» (причём если код будет не один-в-один, то возникнет что? Пра-а-авильно, UB!), а без него не сработает.
                        «static» поможет, но объявленные таким образом функции будут разными в каждом translation unit-е. А вот с «inline» они будут одними и теми же:

                        // test_1.h
                        #pragma once
                        
                        inline int sum(int a, int b)
                        {
                            static int call_count = 0;
                            call_count++;
                            std::cout << "Sum, call_count: " << call_count << std::endl;
                            return a + b;
                        }
                        
                        // test_1.cpp
                        #include <iostream>
                        #include "test_1.h"
                        
                        void some_func(int a, int b);
                        int main()
                        {
                            some_func(80, 4);
                            std::cout << sum(40, 2) << std::endl;
                        
                            return 0;
                        }
                        
                        // test_2.cpp
                        #include <iostream>
                        #include "test_1.h"
                        
                        void some_func(int a, int b)
                        {
                            std::cout << sum(a, b) << std::endl;
                        }


                        С «static sum» вывод будет таким:
                        Sum, call_count: 1
                        84
                        Sum, call_count: 1
                        42

                        С «inline sum»:
                        Sum, call_count: 1
                        84
                        Sum, call_count: 2
                        42


                        UPD: ну и помечать static-ом функции, определяемые в заголовочном файле, всё же несколько странно (это же и есть экспортируемый интерфейс).
                        Ответить
                        • >функции, определяемые в заголовочном файле
                          А что странного? ну будет в каждом .c файле своя функция

                          А вообще крестопроблемы же, не?

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

                          В общем стало понятнее, спасибо
                          Ответить
                          • > А что странного?
                            Ну просто static — это же «внутренняя» функция, и помещать её в публичный интерфейс кажется несколько нелогичным. Но да, это чисто философия, поэтому ничего такого.

                            > А вообще крестопроблемы же, не?
                            Да не, в няшной тоже иногда так хочется. Особенно приятно объявить какую-нибудь инлайн-переменную (так в новых крестах тоже можно), и не ебаться с решением, куда вставлять её определение.

                            Пожалуйста.
                            Ответить
                            • показать все, что скрытоvanished
                              Ответить
                              • Да, переменную.

                                // header.h
                                #pragma once
                                #include <iostream>
                                
                                inline int inline_variable;
                                
                                void test_1();
                                void test_2();
                                
                                
                                // test_2.cpp
                                #include "header.h"
                                
                                void test_2()
                                {
                                    std::cout << "inline_variable = " << inline_variable << std::endl;
                                }
                                
                                
                                // test_1.cpp
                                #include "header.h"
                                
                                void test_1()
                                {
                                    std::cout << "inline_variable = " << inline_variable << std::endl;
                                }
                                
                                int main()
                                {
                                    inline_variable = 1;
                                    test_1();
                                    inline_variable = 2;
                                    test_2();
                                
                                    return 0;
                                }
                                // Вывод:
                                // inline_variable = 1
                                // inline_variable = 2
                                Ответить
                                • я понял, хотя красота решения так себе.
                                  Где-то в глобальном скопе лежит волатльная хуйня, и ну её нахуй кмк
                                  Ответить
            • specially for you with love :)

              template <class Arg1>
              constexpr void log_req(Arg1&& arg1)
              {
                  if constexpr (std::is_enum_v<Arg1>)
                  {
                      std::cout << static_cast<int>(arg1);
                  }
                  else
                  {
                      std::cout << arg1;
                  }
              }
               
              template <class Arg1, class... Args>
              constexpr void log_req(Arg1&& arg1, Args&&... args)
              {
                  std::cout << arg1;
                  log_req(args...);
              }
               
              template <class... Args>
              void log(Args&&... args)
              {
                  log_req(args...);
                  std::cout << std::endl;
              }
              Ответить
              • Универсальные ссылки в данном случае всё же излишни, достаточно старого доброго «const Arg1 & arg1».
                Ответить
            • а вообще ты мне жизнь сломал... ведь я свято верил что инлайн инлайнит ....
              Ответить
              • особенно когда ты взял на функцию поинтер
                Ответить
              • доверяй, но проверяй... выхлоп на асме
                Ответить
              • Трудна жизнь крестовика…
                Ответить
              • И при этом он инлайнит то что не помечено инлайном.
                Ответить
                • Или не инлайнит — тут уж как повезёт!
                  Ответить
                  • Он может одновременно инлайнить и не инлайнить: инлайнить в конкретном вызове, чтобы оптимизировать скорость выполнения, и оставить незаинлайненную версию, потому что ты взял указатель.
                    Ответить
    • а можно эту херню тупо "регексом" забадяжить :)?
      Ответить

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