Answer by Saul Marquez for Matrix class in C#
New line character differs depending on system. Can't rely on \n. Let me introduce you to your new friend, Environment.NewLine Concatenating strings in a loop is very inefficient. It's recommended to...
View ArticleAnswer by harold for Matrix class in C#
A bug: Transpose does data[i, j] = data[j, i];, but it returns a new matrix, it's not supposed to change the matrix in-place, and that doesn't actually work (going out of range for a non-square matrix...
View ArticleAnswer by Mathieu Guindon for Matrix class in C#
I like that your braces and indentation is beautifully consistent. But, this is C#, and your Java is showing ;-) - Kudos for the mostly-consistent PascalCase type and member names, but it's the K&R...
View ArticleMatrix class in C#
I've been learning C# during my free time in the past months; before that, I was mostly writing Java, so the transition hasn't been too hard, but I've never had my code reviewed or read by someone...
View ArticleAnswer by Nat for Matrix class in C#
Next step: Make it Expression<>-oriented.Making your own math tools is tons of fun and practical! Since you're asking about potential improvements, my suggestion is to move toward...
View Article