[Review] - Mein erstes C# Programm

Hallo ich bin neu Hier im Forum und auch neu in der Programmier Sprache C#!

Hier ist mein erstes Programm...Ich bin für Verbesserungs Vorschläge offen 🙂

/*
 * Written by 0xDav
 * Created at 17th July 2015
 * Open Source
 */



using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace Cleverbots_kleiner_Bruder
{
    class Program
    {
        static void Main(string[] args)
        {
            System.Console.Write("Hallo\n" + "Mein Name ist Clever Junior\n" + "Und Wie ist dein Name?\n");
            string Name = System.Console.ReadLine();
            System.Console.Write(Name + " das ist aber ein Schöner Name\n");
            System.Console.Write("Und Wie Alt bist du?\n");
            string Alter = System.Console.ReadLine();
            System.Console.Write("Cool dass du " + Alter + " Jahre Alt bist!\nIch bin auch\n");
            System.Console.Write("Guckst du oft Youtube? Ja oder Nein\n");
            string Youtube = System.Console.ReadLine();
            if (Youtube == "Ja")
                System.Console.Write("Cool ich schaue immer die Cleverbot Videos\n");
            else if (Youtube == "Nein")
                System.Console.WriteLine("Ja ich schaue auch keine Youtube Videos\n");
            else
                System.Console.Write("OK!\n");
            System.Console.Write("Wie Gehts?\n Gut Oder Schlecht?\n");
            string feeling = System.Console.ReadLine();
            if (feeling == "Gut")
                System.Console.Write("Das ist schön!\n Mir auch!\n");
            else if (feeling == "Schlecht")
                System.Console.Write("Das ist Schade!\n");
            else
                System.Console.Write("OK!\n");


            System.Console.ReadKey();

        }
    }
}

Ich freue mich über Jede Antwort!

Hi dariDave,

dein Programm ist ja jetzt nicht wirklich lang. Da gibt es nicht wirklich viel zu sagen.

Die Variablen namen sind OK. Für YouToub (das es Ja oder Nein, auch wenn es jetzt ein string ist) enthalten sollt. Hätte ich vieleicht Is als Präfix gewählt (vielleicht noch Many oder so was). (Bei boolen Variablen bietet sich sowas Grundlegend an).

Bei der Formatierung, solltest du vileicht auch Innerhalb einer Methode mal eine Leere Zeile einfügen um die Logischen Blöcke zu trennen.

Eingabe Validierungen und OOP Konzepte fehlen natürlich ganz. Aber für eine erstes Programm ist es für mich Vollkommen OK.

Weiter so und noch viel Spaß beim Programmieren.

MFG
Björn

Hallo dariDave,

dadurch daß du im Kopf deiner Datei "using System" stehen hast, kannst du auf "System." in den Methoden verzichten:

Console.Write("OK!\n");

Und diese Anweisung kannst du noch verfeinern, indem du

Console.WriteLine("OK!");

schreibst, so daß du nicht jedesmal '\n' schreiben mußt.
Dadurch wirkt der ganze Code dann aufgeräumter und besser lesbar.

Ist vielleicht etwas spät, aber vielleicht hilft es ja anderen Lesern:

Du kreierst für jede Eingabe eine neue string-Variable. Das scheint mir unnötig aufwändig.
Ich würde daher zwei Dinge ändern:

1. Variablen am Anfang der Main-Methode deklarieren (oder aber ausserhalb der Methode als globale Variablen.
2. Nur EINE Variable, der nach jeder Eingabe ein neuer Wert zugewiesen wird. Spart ne Menge Code und ist viel übersichtlicher 😉

Hi NiggiBS,

in C# programmiert man nicht wie in C, daher sind deine beiden Tipps hier nicht so sinnvoll.

Hi NiggiBS,

in C# programmiert man nicht wie in C, daher sind deine beiden Tipps hier nicht so sinnvoll.

Ich habe noch nie in C programmiert. Also sind meine Tipps doch sinnvoll.

Hallo NiggiBS,

Also sind meine Tipps doch sinnvoll. ((Bitte einen anderen "Ton")

NiggiBS, auch von meiner Seite die Aufforderung zu einem anderen Ton.
Das hier ist kein Platz um andere zu beleidigen. Dahingehend eine Verwarnung von meiner Seite.
Du bist hier nicht anonym unterwegs - Beleidigungen gehen i.d.R. nach hinten los.
Über Google findet man viele Profile, die offensichtlich Dir gehören.

Mit dem Risiko, dass auch Du NiggiBS mich als amateurhaft beleidigen wirst:
fachlich sind Deine Tipps und weit weg von empfehlenswert.
Zudem ist es i.d.R. unnötig auf einen 9 Monate alten Beitrag zu antworten. Der Quellcode wird sehr wahrscheinlich mittlerweile anders aussehen und damit wird die Antwort irrelevant.

Da ich nicht damit rechne, dass sich eure Auseinandersetzung (felix, NiggiBS) und vor allem die Tonlage bessert ist der Thread auch nach Beschwerden/Hinweisen anderer Forenteilnehmer zu.
Den Spam hab ich entfernt.