Matrix Calculator done - kindof #32
Conversation
There was a problem hiding this comment.
Szép munka. A program.cs-en lehetne egy picit szépíteni, illetve .NET 8 óta lehet Regex-et kód generáltatni: https://learn.microsoft.com/en-us/dotnet/standard/base-types/regular-expression-source-generators
|
|
||
| namespace GTQPL7.Classes; | ||
|
|
||
| public record Matrix |
There was a problem hiding this comment.
Mivel implementál operátorokat, érdemes lehetne rátenni a Generic Math Interfészekből a megfelelőket
| @@ -0,0 +1,6 @@ | |||
| namespace GTQPL7.Exceptions; | |||
|
|
|||
| public class RpnEvaluatorException : Exception | |||
There was a problem hiding this comment.
Exception esetén ajánlás, hogy ugyan annyi konstruktora legyen, mint a base osztálynak. VS-ben a CTRL + . menüből ki lehet őket generálni.
| @@ -0,0 +1,6 @@ | |||
| namespace GTQPL7.Exceptions; | |||
|
|
|||
| public class SymbolSorterException : Exception | |||
There was a problem hiding this comment.
Exception esetén ajánlás, hogy ugyan annyi konstruktora legyen, mint a base osztálynak. VS-ben a CTRL + . menüből ki lehet őket generálni.
| @@ -0,0 +1,6 @@ | |||
| namespace GTQPL7.Utils.TokenConverter; | |||
|
|
|||
| public class TokenConverterException : Exception | |||
There was a problem hiding this comment.
Exception esetén ajánlás, hogy ugyan annyi konstruktora legyen, mint a base osztálynak. VS-ben a CTRL + . menüből ki lehet őket generálni.
|
|
||
| public class TokenizerException : Exception | ||
| { | ||
| public TokenizerException(string message) : base(message) { } |
There was a problem hiding this comment.
Exception esetén ajánlás, hogy ugyan annyi konstruktora legyen, mint a base osztálynak. VS-ben a CTRL + . menüből ki lehet őket generálni.
|
|
||
| public class Tokenizer : ITokenizer | ||
| { | ||
| private const string IntegerRegex = "0|[1-9][0-9]*"; |
There was a problem hiding this comment.
Ez greedy match-el a * miatt. Érdemes lenne leclampelni egy maximum hosszra.
| [TestFixture] | ||
| public class MatrixTests | ||
| { | ||
|
|
| @@ -1,38 +1,15 @@ | |||
| # SZTE egyetemi kurzus, 2024 Őszi félév házi feleadat repó | |||
| # Matrix Calculator | |||
No description provided.