1. C# / Говнокод #17209

    +100

    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
    class Book
    {
        public void BookName(ref string file, ref int Index)
        {
            string[] ReadFile = File.ReadAllLines("NameFile.txt", Encoding.Default);
            file = ReadFile[Index];
        }

        public void BookAuthor(ref string file, ref int Index)
        {
            string[] ReadFile = File.ReadAllLines("AuthorFile.txt", Encoding.Default);
            file = ReadFile[Index];
        }

        public void BookDescription(ref string file, ref int Index)
        {
            string[] ReadFile = File.ReadAllLines("DescriptionFile.txt", Encoding.Default);
            file = ReadFile[Index];
        }

        public void BookTags(ref string file, ref int Index)
        {
            string[] ReadFile = File.ReadAllLines("TagFile.txt", Encoding.Default);
            file = ReadFile[Index];
        }
    }

    Запостил: LightningAtom, 28 Ноября 2014

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

    • ava guest 28.11.2014 12:13 # +1

      Говно какое-то
      Ответить

    • ava pushistayapodmyshka 28.11.2014 14:09 # +1

      Допустим, с файлом всё понятно. Но индекс-то зачем рефить? Жаль, автор не оставил никаких комментов к своему детищу.
      Ответить

      • ava bormand 28.11.2014 14:17 # +1

        На всякий случай. Вдруг ему захочется поменять значение...
        Ответить

      • ava guest 28.11.2014 14:26 # +1

        Рефы задумывались для вызова сишколиб, за другое использование нужно убивать. Кстати, какой-нибудь решарпер на это не матерится?
        Ответить

        • ava hack2root 28.11.2014 18:43 # 0

          не а
          Ответить

          • ava guest 28.11.2014 18:53 # 0

            Почему так? В додиезе овердохуища фич с применением в одном конкретном случае, в остальный за их использование нужно убивать нахуй сразу.
            Ответить

            • ava Анонимус 28.11.2014 21:12 # +1

              Потому что он наследник C++ в этом смысле.

              Вот в Java фич сделали поминимуму, за что теперь их все ругают.
              Ответить

              • ava guest 29.11.2014 01:13 # 0

                ++Си имеет много легаси говна, включая, например, > > в шаблонах, т.к. >> - оператор сдвига. В сишарп такую хуйню изначально не включали. Но речь не об этом шла, а о варнингах на неоправданное использование фич, не вижу тут связи. Кстати, не в курсе, как там в ++c с качеством статического анализа.
                Ответить

                • ava Анонимус 30.11.2014 21:24 # 0

                  >>статического
                  PVS-Studio
                  Ответить

                  • ava guest 01.12.2014 17:40 # 0

                    >как там с качеством
                    Ответить

                    • ava Lokich 01.12.2014 22:37 # 0

                      я думаю, что очень хорошо
                      http://www.viva64.com/en/n/0090/
                      https://twitter.com/ID_AA_Carmack/status/258940507869310976
                      Ответить

                      • ava guest 01.12.2014 23:54 # 0

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

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

    • ava hack2root 28.11.2014 18:26 # 0

      Нехватает внутреннего кеша данных из файлов, записанных в Hashtable по ключу - имени файла, и проверке промахивания мимо кеша перед считываением. Лучше всего реализовать собственную версию Hashtable, с загрузкой из файла блоками по 4K байт, хранящихся в WeakReference ссылках, чтобы в случае, если файлы большие - кешировать не весь файл, а только его часть. Идеальным вариантом была бы предусмотренная возможность создания кеша второго уровня с обычными сссылками внутри этого Hashtable, чтобы помнить последние 4 блока памяти, в которые попадал индекс при вызове методов. Ну как мой вариане говнокода? Перфектный говнокодище? :)
      Ответить

      • ava bormand 28.11.2014 18:29 # +1

        СУБД на вас нет :)
        Ответить

        • ava hack2root 28.11.2014 18:41 # 0

          А это вариааааааааааааааант. Точно, в следующей итерации создадим публичные и внутренние интерфейсы для прокси, полностью вынесем весь код из этих двух Hashtable в фабрики, и сделаем конструктор фабрики, который завернем в internal proxy который будет переключаться в БД или использовать внутренние in-memory реализации в зависимости от доступности БД или флага отладчика DEBUG (исключающее или)
          Ответить

    • ava koodeer 28.11.2014 18:36 # +3

      Это исходники MS Sql Server?
      Ответить

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