Guck dir mal die Definition von deiner Node- und der Map-Klasse an. Denn Fehler 1 besagt ja, dass die wohl auf private oder internal oder sowas stehen.

Aus Konstanten Properties wegen den Fehlern oben wird wohl kaum die richtige Lösung sein. Vielmehr Konstanten als public deklarieren?

Also suchst du eher was wie Enumerationen (in Bezug auf deine Color Aussage)? msdn.microsoft.com/de-de/library/6e3t9w21%28v=vs.90%29.aspx
Nebenbei, new HeuristicFunction((start, end) => Math.Max(Math.Abs(start.X - end.X), Math.Abs(start.Y - end.Y)), false); sieht mir nicht wirklich Konstant aus

ich tät sehr empfehlen, dieses ganze schwachsinnige Kommentar-Gelaber erstmal zu entfernen.

Und dann Kommentare hinmachen, die wirdklich erklären, was man aussm Code nicht direkt versteht.
Etwa: Was soll die Klasse HeuristicFunctions?

sowas wäre glaub interessanter, als zu erfahren, dass Euclean eine Eucledean-Heuristik darstellt (letzteres ist nämlich eine Tautologie, und somit keine Erklärung, sondern eine Verarschung)

Von den ganzen 287 Zeilen blieben vlt. 50 übrig, und das Bandwurm-Problem wäre gelöst.
Dann kannman noch die ganzen Überladungen entfernen, und sich auf erstmal eine Methode konzentrieren, die auch mal was macht, weil dassis grad was mir fehlt in dem Snippet: Dass iwo mal eine Methode mal wirklich was macht.
Zur Funktion kannman auch deshalb nix sagn, weil die Map-Klasse uns unbekannt ist.

Wenn alles eh klar ist, kannste auf Kommentation auch komplett verzichten. Bzw, es ist erforderlich, dann auf Kommentation zu verzichten, denn was selbsterklärender Code selbst erklärt, kann durch Kommentation nur verwässert oder gar verfälscht werden.
Aber wenn dir bereits klar ist, wie die Heuristics-Klasse zu verwenden ist, und da du das auch bei deinen Lesern vorraussetzst - dann kein Kommentar.
Ich persönlich hätte eine Erklärung wie "Die Distanz einer Zelle zum Ziel kann auf verschiedene Weisen berechnet werden (etwa Luftlinie, Anzahl der Schritte, weitere) Die hier als Delegat niedergelegten Bewertungs-Funktionen werden dem Lösungs-Algorithmus übergeben und führen zu jeweils unterschiedlichen Ergebniswegen."

Das wäre eine Erklärung, die man imo nicht so ohne weiteres dem Code ablesen kann, und solch Kommentar hätte dann eine Berechtigung. Bzw. er ist erforderlich (im Gegensatz zu allem anneren, was da derzeit hingeschrieben ist).
Also wäre dieser Kommentar der einzige im Code, dann wäre dein Code vielfach leserlicher: kürzer, selbsterklärend, und der einzige Kommentar wäre wesentlich, und fände dann auch die nötige Beachtung.

---

Also gut, du hast nun die grob Struktur. Dann solltest du wohl jetzt die Map ausprogrammieren - was hast du da bisher?

...ist mir klar, dass das besser verständlich ist...

besser verständlcih als was?
bisher enthält deine Kommentation doch keinerlei Erklärungen - das sind doch alles nur Tautologien.

Wo das hingehört, ist mir egal. Du hast beim Coden jede Freiheit der Kommetation.
Ich würde das als Header in die Datei setzen, wo die Heuristiken drinne sind.
vielleicht auch als Summary der Heuristics-Klasse, vlt. verkürzt: "Enthält Delegaten, die verschiedene Strategien zur Distanz-Berechnung zweier Zellen anwenden. Einer davon wird dem A*-Algorithmus mitgegeben und beeinflusst, was als kürzester Weg betrachtetet wird."

---

Jut - haste ein Map-Konzept - im wesentliche ein JaggedArray von Nodes - fehlt noch das Node-Konzept. Als Node täte m.E. ein einfacher Enum reichen, der verschiedene Status' der Zelle aussagt, obs ein Hindernis ist oder frei, ob schon besucht und dergleichen.
Ich würde auch nicht von Nodes reden, sondern von Zellen. Nodes sind Objekte der Graphen-Theorie, und werden explizit miteinander verknüpft.
Zellen sind einfach Elemente in einer Matrix, und ihre Nachbarn ergeben sich direkt aus der Zell-Position. Also aus etwas, was streng genommen in der Zelle garnet enthalten ist.
Der Algo würde dann also nicht mit Nodes oder Zellen arbeiten, sondern mit Positionen, mit Points.

---

Edit:
Was bedeutet eiglich die TieBreak-Property der Heuristics-Klasse? (Siehste, solch Fragen tauchen erst auf, wennich mir das mal rauskopiere, und die Laberkommentare rausschmeisse)

wer ist H und was für eine Konstante?

dann kann das weg, und du gibst die Konstante halt immer an. Gibst du 1 an, wird halt mit 1 multipliziert.

Und wenn das weg kann, kannste die Salami wesentlich vereinfachen:

wie gesagt: TieBreak kann weg, und die ganze Klasse kann aufgelöst werden, und als Statische Klasse designed - also nurnoch ein Sammelsurium von
Func(Point, Point, Double)-Delegaten.
Es ist nicht mehr als ein Sammelsurium solcher Delegaten, und daher solls auch nicht so tun als obs ein Objekt wäre, was man instanzieren müsste.

Von meinen Vorschlägen scheinst du nur die Kommentation übernommen zu haben - nicht die Vereinfachung des Codes.

Die eigentlich sinnlose HeuristicFunction-klasse erschwert enorm das Verständnis, weil sinnlosigkeiten sind am schwierigsten zu verstehen von allem (nämlich garnet), und dass etwas einfach sinnlos ist, darauf kommt man meist als letztes.

Nein, die Klasse ist von hinten durch die Brust ins Auge.
Stell dir die Benutzung vor:

Wassn Crap!


Einfach anweisen, was zu tun ist, ohne ein sinnloses Heuristic-Objekt drumrum.

Und derselbe Quatsch dann nochmal im Algo, wo die Benutzung eines Heuristic-Objects allein darin bestünde, dieFunction-Property da rauszuholen und zu nutzen.
Also da kann man die Function auch direkt übergeben, dann muss der Algo sie nicht erst aus diesem Quatsch-Objekt herausholen.