Придумал прогу, а она работает с первого раза КАЙФ

avatar Kujivunia

1324

35

Kujivunia

Пользователь

Регистрация: 10.01.2015

Сообщения: 5526

Рейтинг: 1004

Kujivunia

Регистрация: 10.01.2015

Сообщения: 5526

Рейтинг: 1004

1ncr3dible сказал(а):

скриншот не грузит.

интересные вариации авторизации текста, было бы интересно посмотреть на код, если получится у тебя реализовать

Нажмите, чтобы раскрыть...

Пока код очень скучный (чистая статистика), но через годик, к диплому, я хочу привнести методы из ИИ: кластеризация, нейронки. Также добавить NLP (https://habr.com/ru/company/Voximplant/blog/446738/) для анализа не буковок, а уже лингвистических структур. 

PlayingDota

Пользователь

Регистрация: 14.09.2020

Сообщения: 119

Рейтинг: 154

PlayingDota

Регистрация: 14.09.2020

Сообщения: 119

Рейтинг: 154

img

Почитай про паттерны, если такие проблемы возникают, с такой простой задачей PepeWh.gif?1610052035

 

В твоём случае, почитай про Декораторы, Шаблонный Метод и Фасад. Не обязательно все три использовать, но даже один из них улучшит твой и добавит ясности в структуру кода ZeroTwoJump.gif?1615670500 (не будет проблем с добавлением новых алгоритмов, не надо будет 4 раза вызывать один и тот же набор алгоритмов и т.д.)

Kujivunia

Пользователь

Регистрация: 10.01.2015

Сообщения: 5526

Рейтинг: 1004

Kujivunia

Регистрация: 10.01.2015

Сообщения: 5526

Рейтинг: 1004

1ncr3dible сказал(а):

скриншот не грузит.

интересные вариации авторизации текста, было бы интересно посмотреть на код, если получится у тебя реализовать

Нажмите, чтобы раскрыть...

А пока весь код который есть тут: 

1. Старый, который я решил радикально отрефакторить, сменив даже язык (PascalABC.NET -> C#) https://gitlab.com/Roman.Svetlov/text-author-identification-library

2. Новый (ещё не написан и я не разобрался в гитхабе пока, поэтому ветки и их содержимое не соответствует не то что стандартам, но даже моим хотелкам.) https://github.com/Kujivunia/Text-Authorizer

(скриншот кстати должен появиться сейчас, если нет, то виноват ВК)

PlayingDota сказал(а):

(не будет проблем с добавлением новых алгоритмов, 

Нажмите, чтобы раскрыть...

А у меня теперь есть 0_0

Цитата:

не надо будет 4 раза вызывать один и тот же набор алгоритмов и т.д.)

Нажмите, чтобы раскрыть...

А у меня надо? 0_0

 

падажжи, что я проглядел? А то я уже начинаю переносить старый алгоритм потихоньку, не хочу внезапно обнаружить фатальную ошибку за сутки до конференции

PlayingDota

Пользователь

Регистрация: 14.09.2020

Сообщения: 119

Рейтинг: 154

PlayingDota

Регистрация: 14.09.2020

Сообщения: 119

Рейтинг: 154

img
Kujivunia сказал(а):

падажжи, что я проглядел? А то я уже начинаю переносить старый алгоритм потихоньку, не хочу внезапно обнаружить фатальную ошибку за сутки до конференции

Нажмите, чтобы раскрыть...

 

Фатальных ошибок нет, раз программа работает. Но повторение 8 строчек кода (на 4 и 16 строке) выглядит некрасиво

1ncr3dible

Пользователь

Регистрация: 04.10.2015

Сообщения: 2820

Рейтинг: 1620

1ncr3dible

Регистрация: 04.10.2015

Сообщения: 2820

Рейтинг: 1620

Kujivunia сказал(а):

2. Новый (ещё не написан и я не разобрался в гитхабе пока, поэтому ветки и их содержимое не соответствует не то что стандартам, но даже моим хотелкам.) https://github.com/Kujivunia/Text-Authorizer

(скриншот кстати должен появиться сейчас, если нет, то виноват ВК)

Нажмите, чтобы раскрыть...

было бы неплохо раскинуть файлы по пекеджам, а то сейчас каша 

Kujivunia

Пользователь

Регистрация: 10.01.2015

Сообщения: 5526

Рейтинг: 1004

Kujivunia

Регистрация: 10.01.2015

Сообщения: 5526

Рейтинг: 1004

PlayingDota сказал(а):

 

Фатальных ошибок нет, раз программа работает. Но повторение 8 строчек кода (на 4 и 16 строке) выглядит некрасиво

Нажмите, чтобы раскрыть...

А, ну это я написал просто протестить, работает ли алгоритм. 

 

Например, вполне достаточно было бы написать вместо этого

NormalTEST nt2 = new NormalTEST();

fa2.EnqueueStep(nt2);

это

fa2.EnqueueStep(new NormalTEST());

 

В реальной проге скорее всего будет какой-нибудь цикл, читающий  книги, упаковывающий и обрабатывающий их в духе

 

books = ReadAllBooks(path);

 

fa.EnqueueStep(new First());

fa.EnqueueStep(new Second());

...

fa.EnqueueStep(new Last());

 

foreach book in books{

  fa.Execute(book);

}

 

1ncr3dible сказал(а):

было бы неплохо раскинуть файлы по пекеджам, а то сейчас каша 

Нажмите, чтобы раскрыть...

Знать бы ещё, что это такое... 

7even7een

Пользователь

Регистрация: 23.04.2021

Сообщения: 979

Рейтинг: -52

Нарушения: 110

7even7een

Регистрация: 23.04.2021

Сообщения: 979

Рейтинг: -52

Нарушения: 110

DarthGandalf сказал(а):

а ты тогда кто? Если раз уж я никто? С -62 зачем ты пишешь клоун?

Нажмите, чтобы раскрыть...

Ты какую-то хрень сморозил, зачем умничать?)

Знаешь как это выглядит? "За выбор героя лайк, очень круто что ты понимаешь как добивать крипов".

 

То-есть твоя "блок-схема" = uml диаграмма.

А твоя табуляция = не понимание того, что на скриншоте клиентский код в мейн методе и твой упрек выглядит супер смешно))

pochemyzamenya

Пользователь

Регистрация: 25.02.2016

Сообщения: 4459

Рейтинг: 4062

pochemyzamenya

Регистрация: 25.02.2016

Сообщения: 4459

Рейтинг: 4062

Kujivunia сказал(а):

говнокодом. 

Нажмите, чтобы раскрыть...

Говнокод это про другое, а у тебя проблемы с проектированием

Уже через неделю ты сам ничего не будешь понимать с такой структурой, важно что бы любой вкатывальщик с дота2ру зашел на твой гитхаб и сразу понял что к чему)

AMDkrolyan

Пользователь

Регистрация: 21.04.2013

Сообщения: 7662

Рейтинг: 2908

Нарушения: 10

AMDkrolyan

Регистрация: 21.04.2013

Сообщения: 7662

Рейтинг: 2908

Нарушения: 10

Kujivunia сказал(а):

Знать бы ещё, что это такое... 

Нажмите, чтобы раскрыть...

 

Раскидать по отдельным директориям (папкам)

 

 

dondosai04

Пользователь

Регистрация: 29.08.2019

Сообщения: 426

Рейтинг: 140

dondosai04

Регистрация: 29.08.2019

Сообщения: 426

Рейтинг: 140

если я правильно понял то тебе нужен паттерн стратегия и тогда хоть 100 алгоритмов делай https://refactoring.guru/design-patterns/strategy

кекнул конечно с того что тебе тут насоветовали Haha.gif

- Shtirlitz -

Пользователь

Регистрация: 21.03.2019

Сообщения: 1005

Рейтинг: 269

- Shtirlitz -

Регистрация: 21.03.2019

Сообщения: 1005

Рейтинг: 269

img
DarthGandalf сказал(а):

Читать  ... ( я про код), табуляция не? 

Нажмите, чтобы раскрыть...

Это у ТС-а клиентский код, написанный последним. Наверное, не хватило терпения подчистить, хотелось быстрее показать. 

 

Хороший тред!