Головнев Максим #251
Open
Головнев Максим #251
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Планирую такую логику: - Текст на вход приходит в Lexer, где он переводится в List<Token> (список слов, пробелов, значащих символов) - List<Token> приходит в Parser, откуда приходит уже выровненный по структуре и иерархии корень дерева типа Node (дерево позволит нам определять вложенные теги) - Затем из этого дерева генерится html в HtmlGenerator
Author
- Добавил проект с тестами и написал простые тесты на MdLexer.cs - Реализовал логику считывания токенов в MdLexer.cs (слова, #, _)
Dimques
suggested changes
Nov 6, 2025
- Создал класс TokenParserTests.cs
- Дописал типы Node в NodeType - Дописал типы Token в TokenType - Убрал поле text из базовой ноды
- Написал базовую реализацию в декларативном стиле и без некоторой логики, поэтому не все тесты проходят
- добавил класс-раширение для List<MdToken> - более менее декомпозировал метод Parse - не работает условие с одинарными тегами внутри двойных/наоборот
- убрал некорректный тест в ListMdTokenTests.cs - сделал логику чтобы работало условие на вложенность Bold в Italic/наоборот - поправил тесты в TokenParserTests.cs
- написал HtmlGenerator.cs
- Пофиксил некорректное экранирование - Косметические правки
- дописал перфоманс тест
- убрал ref из GetLengthChainOfTokenTypeAfter
Dimques
suggested changes
Nov 12, 2025
Dimques
suggested changes
Nov 13, 2025
- правки по ListMdTokenExtension.cs
- переписал перформанс тест, теперь он не флокает - разбил функциональные тесты - пофиксил баг в IsUnderscoreInWordWithNumbers
Dimques
suggested changes
Nov 15, 2025
| { | ||
| var markdown = GenerateRandomMarkdown(length); | ||
| sw.Start(); | ||
| GC.Collect(); |
There was a problem hiding this comment.
Нуууу такое, конечно. А между сборками мусора будет работать, как попало, да?
- дописал тестов на ранее неработающие сценарии - разбил TokenParser.cs на регионы для читаемости
- переписал тесты под нее
Dimques
reviewed
Nov 26, 2025
| [TestCase("# H1\n## H2\n### H3", "<h1>H1</h1><br/><h2>H2</h2><br/><h3>H3</h3>")] | ||
| [TestCase("# _text_ __text__", "<h1><em>text</em> <strong>text</strong></h1>")] | ||
| [TestCase(" ## h1", " ## h1")] | ||
| [TestCase("########### h1", "########### h1")] |
There was a problem hiding this comment.
Здесь лучше было использовать ровно семь гридов, чтобы показать границу числа гридов, начиная с которой другое поведение будет, т.е. подчеркнуть edge-case.
Dimques
reviewed
Nov 26, 2025
| [TestCase("# _text_ __text__", "<h1><em>text</em> <strong>text</strong></h1>")] | ||
| [TestCase(" ## h1", " ## h1")] | ||
| [TestCase("########### h1", "########### h1")] | ||
| [TestCase("#", "<h1></h1>")] // в оригинальном Md также |
There was a problem hiding this comment.
В общем случае это не аргумент - "все совпадения случайны" :)
Dimques
reviewed
Nov 26, 2025
| [Test] | ||
| [TestCase("Text with _underscores_ inside", "Text with <em>underscores</em> inside")] | ||
| [TestCase("__Bold__ _Italic_ __Mixed__", "<strong>Bold</strong> <em>Italic</em> <strong>Mixed</strong>")] | ||
| [TestCase("root1_2_3", "root1_2_3")] |
Dimques
reviewed
Nov 26, 2025
| } | ||
|
|
||
| [Test] | ||
| [TestCase(@"Escaped \_underscore\_", "Escaped _underscore_")] |
There was a problem hiding this comment.
Существует, скажем так, правило хорошего тона - ишуи резолвит тот, кто их завёл.
Author
There was a problem hiding this comment.
Существует, скажем так, правило хорошего тона - ишуи резолвит тот, кто их завёл.
понял, просто удобно, проще понимать что сделал/что нет
Dimques
reviewed
Nov 26, 2025
|
|
||
| [Test] | ||
| [TestCase("TEXT!???", "TEXT!???")] | ||
| [TestCase("_Test_\n## Header ##\n__word__ _word_", |
There was a problem hiding this comment.
\n - эх, проворонил такой повод подушнить :( На будущее - используй лучше Environment.NewLine, в разных системах оно разное значение имеет.
Dimques
reviewed
Nov 26, 2025
| var markdown = GenerateRandomMarkdown(length); | ||
| sw.Start(); | ||
| Render(markdown); | ||
| GC.Collect(); |
There was a problem hiding this comment.
Ну уж если хватило сил все замечания исправить, мог бы и эту ерунду поправить :)
Dimques
reviewed
Nov 26, 2025
| [TestCase(@"\\[text](example.com)", """\<a href="example.com">text</a>""")] | ||
| [TestCase(@"Escaped _underscore\__", "Escaped <em>underscore_</em>")] | ||
| [TestCase(@"[text\\](example.com)", """<a href="example.com">text\</a>""")] | ||
| [TestCase("_underscore _", "_underscore _")] |
There was a problem hiding this comment.
Ну кажется на tab надо ещё внимательно посмотреть
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Планирую такую логику: