Главная Настройка Mobile Контакты NSFW Каталог Пожертвования Купить пасскод Pics Adult Pics API Архив Реквест доски Каталог стикеров Реклама
Доски


[Ответить в тред] Ответить в тред

Check this out!


[Назад][Обновить тред][Вниз][Каталог] [ Автообновление ] 15 | 1 | 7
Назад Вниз Каталог Обновить

Раскритикуйте Аноним 06/08/17 Вск 01:52:50 1038876  
image.jpg (132Кб, 390x513)
Привет, в общем я студент, начал изучать программирование на этой неделе, недавно прочитал первую книгу C# player guide и сделал первое задание из книги - Caesar Cipher.

Зачем пришёл то в общем - прошу раскритиковать и добавить замечаний, сделал без гугла почти, поэтому реализация скомкано, как придумал. Алсо в TODO секции если кто может помочь с идеями, то прошу помочь. Особенно с русским алфавитом, хотел сделать через преобразование char но одна хитрая буква лежит в совсем другом месте

Ссылка на гитхаб, если лень смотреть код можно просто экзешник запустить из релиза.
https://github.com/TestName2ch/Learning
Аноним 06/08/17 Вск 01:55:26 1038877
>>1038876 (OP)
Никто не станет запускать твой вирус, уходи.
Аноним 06/08/17 Вск 09:02:23 1038953
>>1038876 (OP)
Комментарии на срусском. Обоссал дауна.
Ответы: >>1038974 >>1038982
Аноним 06/08/17 Вск 10:01:03 1038974
>>1038953
> C#. Обоссал дауна.
Пофиксил тебя.
Аноним 06/08/17 Вск 10:14:33 1038982
>>1038953
>Комментарии
Обоссал дауна.
Аноним 06/08/17 Вск 10:18:06 1038985
>>1038982
Уже написал САМОДОКУМЕНТИРУЕМЫЙ КОД ?
Ответы: >>1038992
Аноним 06/08/17 Вск 10:18:47 1038988
>>1038982
Ебать дебил.
Ответы: >>1038992
Аноним 06/08/17 Вск 10:25:36 1038992
>>1038985
>>1038988
А вот и порватки прибежали. Уже все строчки откомментировали, ничего не пропустили? Скобочки тоже не забудьте откоментить.
Аноним 06/08/17 Вск 10:41:24 1038998
>>1038876 (OP)
Ну, в целом для студента сгодится.

Пара мелочей:

1. Сделай кодер независимым от языка, задавай язык оффсетом и размером в кодовой странице
2. fileContent.Length > 250000 Что за глупое ограничение? Сделай кодирование батчами
3. cp-1251 Зачем? Используй utf8, он пересекается с ANSI и покрывает CP-1251.
Ответы: >>1038999
Аноним 06/08/17 Вск 10:52:21 1038999
>>1038998
>ограничение
Долго шифрует
>ср1251
>используй UTF8
Я вроде использовал, он иероглифы записывает
Ответы: >>1039002 >>1039014
Аноним 06/08/17 Вск 10:55:26 1039002
>>1038999
>он иероглифы записывает
Ну так разбирайся в чем дело. Ты не можешь менять кодировку на каждый новый язык ввода.
Аноним 06/08/17 Вск 11:01:15 1039006
>>1038982
Ммм, старый добрый троллинг тупостью.
Аноним 06/08/17 Вск 11:15:47 1039014
>>1038999
> сделал квадратичную сложность
> Долго шифрует
Ответы: >>1039031
Аноним 06/08/17 Вск 11:32:50 1039031
>>1039014
Но она у него не квадратичная.
Ответы: >>1039040 >>1039043
Аноним 06/08/17 Вск 11:43:07 1039040
>>1039031
Я не знаю какая там на самом деле сложность, но оба алфавита прогоняются по несколько раз для каждого символа.
> EnglishLetters.Contains(char.ToLower(letter)
> (EnglishLetters.IndexOf(char.ToLower(letter))

Аноним 06/08/17 Вск 11:51:04 1039043
>>1039031
Ещё строка каждый раз заново создаётся, если стринг работает как в жабе.
> newContent += char

[Назад][Обновить тред][Вверх][Каталог] [Реквест разбана] [Подписаться на тред] [ Автообновление ] 15 | 1 | 7
Назад Вверх Каталог Обновить

Топ тредов
Избранное
Подписывайся на официальный канал Двача в Телеграме и узнавай обо всех новостях и мемах первым! https://tlg.wtf/dvachannel[X]