spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Dabei ist seine Lösung sehr unschön. Allein das for (int i) statt einem ordentlichen foreach ist wirklich nicht sonderlich gut. Dann noch ein überflüssiges "Contains" dazu...
Mal davon abgesehen, dass ich ja einen kompakten Einzeiler vorgeschlagen habe.

Ich muss Bluecobold zustimmen. Bei diesem, sehr einfachen Problem, sollten höchstens 2 Zeilen reichen.
In meiner Interpretiation deiner Problembeschreibung brauch man zwei Schleifen. Bei dem gemeinten Problem reicht eine Schleife völlig aus. Bluecobolds Lösung ist wohl die beste. Denn übersichtlicher und kompakter Code ist bis zum Stadium, in der man die Performance optimiert, der beste weg.
Und selbst wenn dieses Staduim erreicht wird, wird es sehr unwahrscheinlich, dass gerade diese Stelle dein Programm ausbremst und deshalb optimiert werden muss. Die Regel, dass 20% des Codes in 80% der Zeit ausgeführt wird, gilt in jedem Programm.

Prinzipiell habt Ihr Recht, aber jemand der die Grundlagen nicht beherscht und Wissen will wie etwas funktioniert kann klar eine Kurzschreibweise abtippen und es funzt.

Aber um es zu verstehen halte ich eine möglichst ausführliche Schreibweise immer für Sinnvoller.

Abgesehen davon, bin ich persönlich auch nicht der Freund davon, alles und überall nur Kurzschreibweisen zu verwenden, da dies den Code aus meiner Persönlichen Meinung unleserlicher macht.

Ihr beiden habt Programmieren schon drauf aber Mango scheint gerade zu lernen. Von daher schreibt Ihr es weiterhin in Kurzschreibweise und tetuliert nicht eine Ausführliche Schreibweise als falsch.

Gruß Christian

Zitat von »DeathStorm«

Abgesehen davon, bin ich persönlich auch nicht der Freund davon, alles und überall nur Kurzschreibweisen zu verwenden, da dies den Code aus meiner Persönlichen Meinung unleserlicher macht.

Eigentlich gehört der Code auch in eine Methode.

Schon weiß jeder was der Code macht und man erhält optimale übersicht.

Zitat von »DeathStorm«

Ihr beiden habt Programmieren schon drauf aber Mango scheint gerade zu lernen. Von daher schreibt Ihr es weiterhin in Kurzschreibweise und tetuliert nicht eine Ausführliche Schreibweise als falsch.

Nur weil es eine bessere Variante gibt heißt es nicht, dass die andere falsch ist.

Das wäre die Standardversion, die jeder Programmierer selbstständig implementieren können muss. Sie ist nicht falsch.
BlueCobolds Lösung ist aber trotzdem besser weil die Quellcodedatei dadurch kürzer und übersichtlicher wird.

Genau das versuche ich zu sagen.

Ob der Code Lesbarer oder "besser" ist, ist eine persönliche Entscheidung.

Ob ich irgendwo eine zwei oder 5 Zeilen mehr stehen habe ist doch völlig wurscht. Solange ich und ggf. mein Team mit Code gut umgehen können ist doch alles in Ordnung.

Ich muss nicht zwingend versuchen immer alles auf eine Minimalform zu bringen.

gruß Christian

Ich für meinen Teil würde aber ebenfalls die von BlueCobold vorgeschlagene Variante bevorzugen.
Ob die paar Zeilen mehr oder weniger einen Unterschied machen, kommt aber auch noch auf andere Dinge an: hat man es in eine eigene Methode ausgelagert? Geht man ständig so vor, ausführlicheren Code zu schreiben? Gibt es einen Performanceunterschied? ...

Ansonsten, wenn man es bereits in eine separate Methode auslagert, dann kann man auch gleich eine Erweiterungsmethode daraus machen:
Der Aufruf wäre dann nur noch stringList.CountString("abc").
[Nachtrag: wenn man es ganz allgemein halten will, kann man natürlich auch eine generische Erweiterungsmethode CountElement implementieren, damit man diese für jede Liste aufrufen kann...]

@Nachoman:
du kannst auch die Code-Tags für C#-Code ("cs") verwenden.

Das ist ja das was ich sage.

In den meisten Fällen macht es Performance technisch keinen soo großen Unterschied ob ausführlich oder Kurzform.

Ja das für wiederkehrende Aktionen eine Funktion geschrieben wird ist auch jedem iwann klar.

Doch zum lernen ist es erstmal einfache nen ausführlichen Code zu lesen und zu schreiben und dann den Aha Effekt zu haben das es auch kürzer geht.

Ansonsten ist es einfach Geschmackssache. Da weder das eine noch das andere Richtig oder Falsch ist.

Und ich denke das Thema ist soweit erschöpft und wir Diskutieren hier Dinge die eingentlich keinen Interessiert und im schlimmsten Fall werden Leute wie ggf. Mango dadurch nur mehr verwirrt als das seine Fragen beantwortet werden.

gruß Christian

Zitat von »DeathStorm«

Ob der Code Lesbarer oder "besser" ist, ist eine persönliche Entscheidung.

Nein, falsch. Eine Zeile ist immer besser als 20.

@BlueCobold:

NOPE, da widerspreche ich.

Vor allem da die Definition hier von "Besser" noch fehlen würde um eine fundierte Diskussion führen zu können.

Zitat von »DeathStorm«

@BlueCobold:

NOPE, da widerspreche ich.

Vor allem da die Definition hier von "Besser" noch fehlen würde um eine fundierte Diskussion führen zu können.

Ich nicht.
Eins der von mir aufgezählten Punkte ist, ob man grundsätzlich längere Schreibweisen verwendet oder ob es nur in diesem einen Fall gemacht wurde. Verwendet man aus Prinzip längere Codeabschnitte, die sich auch mit Einzeilern und vorgefertigten Methoden lösen lassen, kann es schnell passieren, dass eine eigentlich 20 Zeilen lange Methode doch eher 100 Zeilen lang wird.
Mal abgesehen davon, dass Einzeiler grundsätzlich schneller zu lesen sind, als 5-zeilige Umsetzungen.

Zitat von »DeathStorm«

Und ich denke das Thema ist soweit erschöpft und wir Diskutieren hier Dinge die eingentlich keinen Interessiert und im schlimmsten Fall werden Leute wie ggf. Mango dadurch nur mehr verwirrt als das seine Fragen beantwortet werden.

Solange in den Beiträgen nichts weiter zur Debatte beigetragen wird und stattdessen nur die eigene Meinung nochmal wiederholt wird, kann ich dir nur Recht geben. Aber interessant ist es, dass du darauf hinweist, nachdem ich in meinem Beitrag noch die Erweiterungsmethoden erwähnt habe.