.NET Interview, Code-Struktur und das Design

Ich denke, du hast die Frage nicht verstanden. Der Interviewer fragte die minimale Differenz zwischen "für" und "gegen" Ziele und Ihr Programm berechnet den besten Zieldurchschnitt. Wenn Sie den minimalen Unterschied sehen, dann ist es New York nicht Boston . Lassen Sie mich hier fenix2222 Code aktualisieren.

Nur ein paar Dinge aus einer kursorischen Übersicht:

1. Es gibt zwei Schnittstellen, aber für beide Anwendungen gibt es keinen Wert.
2. Es gibt keinen Grund, eine DataTable in die Problembeschreibung einzufügen.
3. Der Code ist übermäßig komplex.
4. Die Verwendung von IList und IEnumerable in der GetTeams-Methode soll "nur weil" verwendet werden.
5. Die ResultEvaluator-Klasse ist nicht wiederverwendbar, d. h. sobald Sie die Klasse instanziieren, können Sie die csv-Datei nie erneut festlegen. Sie können die gleiche Methode (GetTeamSmallestDifferenceForAgainst) immer wieder aufrufen. keine anderen öffentlichen Eigenschaften sind verfügbar.
6. In der GetDataTable-Methode werden die string [] -Felder in einer Zeile deklariert, und dann wird der Wert in der nächsten Zeile festgelegt.
7. Es gibt weniger als null Gründe, das @ -Symbol für den Parameter 'for' im Konstruktor der Team-Klasse zu verwenden. Benenne einfach das reservierte Wort 'für' in etwas anderes um.
8. Es gibt viele Konstrukte ab 3.5 .NET, mit denen das Problem viel einfacher gelöst werden kann; Das zeigt nur ein mangelndes Verständnis der Sprache.

Von den Blicken scheint es wirklich, dass Sie versucht haben zu zeigen, dass Sie ein bisschen mehr wussten als das, was in der Problembeschreibung gefragt wurde. Aber, wie das Wissen, das Sie wussten, in dieser Übung verwendet wird, ist ziemlich beängstigend, und nicht auf eine gute Art.

In Zukunft würde ich empfehlen, das Problem einfach zu lösen und es nicht zu überdenken. Mach es einfach.