Привет, в общем я студент, начал изучать программирование на этой неделе, недавно прочитал первую книгу C# player guide и сделал первое задание из книги - Caesar Cipher. Зачем пришёл то в общем - прошу раскритиковать и добавить замечаний, сделал без гугла почти, поэтому реализация скомкано, как придумал. Алсо в TODO секции если кто может помочь с идеями, то прошу помочь. Особенно с русским алфавитом, хотел сделать через преобразование char но одна хитрая буква лежит в совсем другом местеСсылка на гитхаб, если лень смотреть код можно просто экзешник запустить из релиза.https://github.com/TestName2ch/Learning
>>1038876 (OP)Никто не станет запускать твой вирус, уходи.
>>1038876 (OP)Комментарии на срусском. Обоссал дауна.
>>1038953> C#. Обоссал дауна.Пофиксил тебя.
>>1038953>КомментарииОбоссал дауна.
>>1038982Уже написал САМОДОКУМЕНТИРУЕМЫЙ КОД ?
>>1038982Ебать дебил.
>>1038985>>1038988А вот и порватки прибежали. Уже все строчки откомментировали, ничего не пропустили? Скобочки тоже не забудьте откоментить.
>>1038876 (OP)Ну, в целом для студента сгодится.Пара мелочей: 1. Сделай кодер независимым от языка, задавай язык оффсетом и размером в кодовой странице2. fileContent.Length > 250000 Что за глупое ограничение? Сделай кодирование батчами3. cp-1251 Зачем? Используй utf8, он пересекается с ANSI и покрывает CP-1251.
>>1038998>ограничениеДолго шифрует>ср1251>используй UTF8Я вроде использовал, он иероглифы записывает
>>1038999>он иероглифы записываетНу так разбирайся в чем дело. Ты не можешь менять кодировку на каждый новый язык ввода.
>>1038982Ммм, старый добрый троллинг тупостью.
>>1038999> сделал квадратичную сложность> Долго шифрует
>>1039014Но она у него не квадратичная.
>>1039031Я не знаю какая там на самом деле сложность, но оба алфавита прогоняются по несколько раз для каждого символа.> EnglishLetters.Contains(char.ToLower(letter)> (EnglishLetters.IndexOf(char.ToLower(letter))
>>1039031Ещё строка каждый раз заново создаётся, если стринг работает как в жабе.> newContent += char