Moin moin.
Als abschließendes Projekt in meinem Kurs "Grundlagen Hochsprachenprogrammierung" (Teil meines Maschinenbaustudiums) sollen wir ein Programm schreiben. Vorgegeben ist uns dabei nur, was das Programm können soll.
Ich bin damit jetzt auch fürs erste fertig. Mein Programm erfüllt alle gestellten Anforderungen und funktioniert einwandfrei.

Ich würde mich nun freuen, wenn ihr einen Blick über meinen Quellcode werfen würdet und mir sagt, was ihr davon haltet. Bedenkt dabei aber bitte, ich habe jediglich Grundkenntnisse mit wenig Erfahrung und kann mit dem Programmierstil von Profis natürlich nicht mithalten. Wäre also nett wenn ihr die Messlatte nicht auf euer Niveau legt, sondern entsprechend tiefer ;). Ich möchte damit erreichen, meinen Quellcode noch etwas abzurunden, was den Stil angeht.
Vielen Dank schonmal

Zweck des Programms:
Der User soll sich einen Balken definieren können, der mit beliebig vielen Kräften belastet werden kann, welche der User selber eingibt.
Zum Schluss sollen die eingegeben Werte ausgegeben werden, wobei die Kräfte nach ihrer Position auf dem Balken sortiert werden sollen

Hallo ElBarto,

YESS!! Im nächsten Herbstsemester darf auch ich mein Maschinenbaustudium beginnen! Puh. Ich freu mich schon voll. Ich meine: ausgebildeter Konstrukteur zu sein ist ja auch schön, aber warum dabei bleiben?

Zum Code:

• Verwende doch eine List(Of Double) statt Arrays.
  
• Ersetze bei der Linie den String folgendermassen: Space(50).Replace(" "c, "_"c)
  
• Ich hätte jetzt eine Struktur für den belastenten Balken angelegt, die die Rechnung automatisiert. So wird der "Haupt-Code" kürzer, leserlicher und verständlicher.

Die Logik habe ich jetzt nicht angeschaut, weil ich zu müde bin(Scheiss "Wirtschaftswoche" ).

Konstruktive Grüsse

Higlav

code ist grauenhaft, aber für jmd, der annähernd noch nie, und auch nie wieder programmiert, völlig ok.

Console selbst ist grauenhaft.

Allenfalls die Array.Sort()-Methode zu nutzen wäre eine wirkliche Verbesserung.
Weil einen sortier-Algo selbst zu schreiben ist zwar kunstvoll und zeugt von Talent , ist aber alles annere als "Hochsprachen-Programmierung", und daher eigentlich total Fail .
Aber dazu müsstest du dich schon viel zu sehr in die Materie einarbeiten, und würdest wahrscheinlich auch deinen Lehrkörper überfordern (was ich immer und immer wieder mitkriege: die Lehrkörper solcher Kurse haben normalerweise selbst keine Ahnung von .Net).

@Higlav
- List(Of Double) sagt mir leider überhaupt nichts.
- Was meinst du mit Struktur anlegen und Berechnung? Weil gerechnet wird noch garnichts und ist auch nicht gefordert. Das wäre die Fortführung des Grundkurses. Aber für uns gibt es garkeine Fortführung
Oder meinst du eine weitere Aufgliederung in Unterprogramme? Wie man sieht, hab ich da schon ein bischen was gemacht, da wir das auch in der Vorlesung angeschnitten hatten. Für die Aufgabe ist das aber auch nicht gefordert.

@ErfinderDesRades
Ich habs nicht anders erwartet^^ Grauenhaft ist bestimmt die beste Beschreibung aus Sicht eines geübten Programmierers.
An unserem Proff liegts aber definitiv nicht. Der ist äußerst kompetent. Ich weiß zwar nicht, wie tief sein Kenntnisse in VB gehen. Aber zumindest über Grund- und Erweitertekenntnisse ist der weit hinaus.

Array.Sort()? Ich ahne schon was dahinter steht. Aber das geht über das hinaus, was gefordert wird.

ElBarto schrieb:

An unserem Proff liegts aber definitiv nicht. Der ist äußerst kompetent.

Wie solltest du das beurteilen können?
Imo ists von ihm schomal Fail, dass er euch mit VB.Net anfangen lässt, und also mit Option Strict Off.
C# wäre hier viel sauberer, denn da gibts die Strict-Off-Peinlichkeit garnet.

Ich merk schon wieder, die Diskussion läuft am Thema vorbei. Aber nagut:
Da ich nicht auf den Kopf gefallen bin und nicht naiv durch leben schreite, sehe ich mich druchaus in der lange die Kompetenz von jemandem beurteilen zu können auch ohne selber über dessen Kompetenz zu verfügen. Wenn man eine zeitlang mit jemandem arbeitet merkt man einfahc ob der sein Fach versteht oder nicht. Also glaubt es mir oder lasst es bleiben: Er ist kompetent.

Warum Option Strict OFF habe ich in diesem Forum schonmal erläutert. Für uns ist es OK.

Warum VB-Net? 90% seiner bisherigen Maschinenbaustudenten haben später in der Praxis eher mit VB-Net als anderen Sprachen zu tun (wenn sie überhaupt mit Programmieren konfrontiert werden). Daher hat er sich vor Jahren dafür entschieden Maschinenbauer nicht weiter in C# zu unterrichten.
Seine Elektrotechniker unterrichtet er weiterhin in C# oder C++ (keine Ahnung welches er erwähnt hatte), da es der Branche entspricht.

Wenn noch wer gerne wa zum eigentlichen Topic sagen möchte, nur zu.
Ich hab das Thema aber als erledigt markiert, da mir die Aussage "code ist grauenhaft, aber für jmd, der annähernd noch nie, und auch nie wieder programmiert, völlig ok." als Bestätigung ausreicht.

Du vergleichst hier einen String mit einem Integer, daß kannn man so nicht stehen lassen. Du solltest versuchen strEingabe vor deinem Vergleich in einen Integer zu konvertieren.

Stehen lassen kann man es schon. Das Programm ist so geschrieben, dass an der Stelle in strEingabe nur ein nummerischer Wert stehen kann und dieser dann auch mit 0 verglichen werden kann. Nicht sehr elegant, aber ich wusste mir nicht anders zu helfen.

Mir fällt gerade ein, ich könnte noch eine zusätzliche Variable einführen:



Das würde an der Funktionsweise des Programms nichts ändern. Aber ob es eleganter ist, weiß ich auch nicht.

Kannst du mir die Funktionsweise deines Vorschlages erklären? Int.TryParse ist mir nicht bekannt, also was soll dieser Befehl bewirken?

ElBarto schrieb:

Der User soll sich einen Balken definieren können

Klingt nach Interaktion zwischen Programm und User.
Dafür ist eine Console-Anwendung der denkbar schlechteste Rahmen.
Mach eine ereignisgesteuerte WinForm-Anwendung:
n TextBoxen oder NumericUpDowns für die Eingaben und einen Start-Button zur Berechnung sowie Controls für das Resultat.
Falls (Konjunktiv) jetzt Deine Antwort sein sollte:

Die Console ist uns vorgegeben

dann sage Deinem Prof, er möge bitte mal seine Augen öffnern und sich die Realität anschauen, in der Fahrschule hat man die Kurse für Holzrollerfahrer auch schon abgeschafft.

Es stand uns frei, dass als Consolen-Anwendung oder WPF-Anwendung zu machen. Da ich es in WPF aber schwieriger hab ichs so gemacht.

Meinem Proff werde ich ganz sicher nicht sagen, dass er seine Prüfung ändern soll, weil wir was machen, was eigentlich nicht praxistauglich ist. Denn so wie wir es machen ist es trotzdem in Ordnung!
Die meisten hier im Forum sind bestimmt clevere Programmierer und ihr habt mit euren Aussagen ja auch recht. Nur vergesst ihr allzu oft die Hintergründe zu beachten und kritisiert einfach drauf los.

Die Tatsache, dass du ein Konsolenprogramm erstellst, ist erstmal total okay, lass dir da nichts einreden.
Nur weil gerade hier im Forum das Aussehen von Programmen häufig wichtiger erscheint als deren Funktion oder die Lösungsansätze, die dahinter stecken, heißt das nicht, dass das bessere Programm immer das mit dem besseren (G)UI ist.

Auch werden hier im Forum häufig Programme erstellt und gezeigt, die von einer breiten Masse verwendet werden (sollen), sprich Editoren, Browser, YouTube-Downloader etc. Solche Programme müssen leicht und intuitiv bedienbar sein und somit eine gute GUI besitzen, der Code dahinter kommt leider häufig zu kurz oder wird zumindest nicht so beachtet. Als Maschinenbaustudent (bin ich quasi auch), wirst du solche Programme nicht schreiben müssen, häufig geht es um Programme, die Berechnungen durchführen oder auch nur bestimmte Forschungsergebnisse filtern. Solche Programme brauchen keine grafische Oberfläche, sondern nur eine möglichst effiziente und fehlerresistente Vorgehensweise.

In diesem Sinne solltest du dir die restlichen Kritiken schon eher zu Herzen nehmen. "Option Strict Off" scheint zwar erstmal bequem, wird aber ganz schnell zu einer Fehlerquelle, die nicht nur weitreichende, sondern vor allem auch schwer zu findene Fehler erzeugt.

Ein paar Tipps von mir, die hoffentlich recht einfach zu verstehen sind:

- Trenne das Frontend vom Backend, sprich: Richte einen Teil deines Codes darauf ein, die benötigten Daten vom User einzulesen und richte eine feste Schnittstelle (möglicherweise einen Datentyp) dafür ein, diese Daten an den Teil des Programmes weiterzugeben, der letztendlich die Berechnung durchführt. Dieser Teil sollte dann auch nichts mit dem Benutzer zu tun habe (also kein Console.Read/Write).

- Englische Variablennamen: Vllt. ist das nur ne persönliche Sache, aber wenn ich nen fremden Code mit deutschen Variablen- oder Funktionsnamen sehe, kriege ich erstmal das kalte Ko****. Die meisten Programmiersprachen sind auf Englisch gehalten (if, then, else, do, switch ....), dazu kommt noch das englische .NET-Framework (WriteLine etc.), da machen sich deutsche Worte irgendwie schlecht.

- Wenn du .NET lernst, dann lerne auch .NET (geht vermutlich mehr an deinen Prof/Dozenten). Dinge wie Array.Sort() oder List(Of T) sind elementare Funktionen von .NET und sollten daher auch mit .NET beigebracht werden, ansonsten ist Visual Basic nämlich nur eine besondere Sytax, die leider im Vergleich zu der von C# oder C++ nicht besonders mächtig ist, in dem Sinne, dass du damit später (möglicherweise im Berufsleben) nicht viel anfangen kannst. Kennst du (große) Teile des .NET-Frameworks, kannst du auch problemlos zu C# oder auch C++ (etwas schwieriger) switchen und direkt richtig anfangen.

Was mir daneben sehr positiv aufgefallen ist, sind die Kommentare. Schreibt man selber ein Programm, kommt einem Vieles absolut logisch vor, jemand anderem, der sich den Code anschaut, jedoch nicht. Die Kommentare dürfen hier im deutschsprachigen Raum auch gerne auf Deutsch gehalten werden, gerade an der Uni hat man jedoch auch häufig Kontakt zu ausländischen Studenten oder Professoren. Daher immer die Frage: Wer muss mein Programm noch lesen oder bearbeiten?
-> Also immer schön weiter kommentieren, gerne noch ein bisschen mehr

ElBarto schrieb:

Nur vergesst ihr allzu oft die Hintergründe zu beachten

Das vefrgessen wir nicht, die Thread-Ersteller erachten es nicht als notwendig, solch "Zwangsvorgaben" im Eröffnungspost zu erwähnen.