Брозовский Максим #234
Брозовский Максим #234
Conversation
cs/Markdown/IConverter.cs
Outdated
|
|
||
| namespace Markdown; | ||
|
|
||
| public interface IConverter |
cs/Markdown/Tags/ITag.cs
Outdated
| bool IsOpenedCorrectly((char left, char right) contextChars); | ||
| bool IsClosedCorrectly((char left, char right) contextChars); |
There was a problem hiding this comment.
Выдели мб отдельную модельку под это? Таплы передавать публично плохая практика. У тебя уже два места, где одна и та же модель используется. При изменении одно из этих мест надо отдельно помнить и о другом
cs/Markdown/Tags/PairTag.cs
Outdated
| @@ -0,0 +1,12 @@ | |||
| namespace Markdown.Tags; | |||
|
|
|||
| public abstract class PairTag : ITag | |||
There was a problem hiding this comment.
Чет не отразил чем отличается от одинарного
There was a problem hiding this comment.
И как определяем закрываюищий тег?
There was a problem hiding this comment.
для MdTag добавил, а HtmlTag там симметричный, т.е. <HtmlTag></HtmlTag>
There was a problem hiding this comment.
Чет не отразил чем отличается от одинарного
Главное отличие в том, что у одинарного не будет закрвающего MdTag и он действует на всю строчку, а для HtmlTag мы делаем как в коментарии выше - симметрично
There was a problem hiding this comment.
Но там тоже могут быть особые случаи типа <img> тега....
There was a problem hiding this comment.
но мне кажется это уже отвественность IMdConverter, как он будет эти токены укладывать в строку
cs/Markdown/Tokens/TagToken.cs
Outdated
|
|
||
| namespace Markdown.Tokens; | ||
|
|
||
| public class TagToken(ITag tag, char left, char right) : IToken |
There was a problem hiding this comment.
А почему char? Например, __ - это два подчеркивания.
cs/Markdown/Tags/ItalicTag.cs
Outdated
| @@ -0,0 +1,9 @@ | |||
| namespace Markdown.Tags; | |||
|
|
|||
| public class ItalicTag : PairTag, ITag | |||
There was a problem hiding this comment.
А зачем интерфейс еще раз указывать?
No description provided.