[C#] Klassenbeispiel

[Snees]

Hallo, ich beschäftige mich momentan mit C# und habe eine kleine Klasse geschrieben, um die Arbeit damit zu veranschaulichen.

Eventuell kann es ja jemand von euch gebrauchen.

Downloader.cs

/* 
 * Eine einfache Klasse
 * am Beispiel eines Webclienten mit der DownloadString() Methode
 * Autor: Nico @ U-Hacks.net
*/

using System.Net;

namespace ConsoleApplication1
{
    class Downloader
    {
        /// <summary>
        /// Quelltext wird in diesem String gespeichert
        /// </summary>
        public string Quelltext { get; set; }

        /// <summary>
        /// Downloader
        /// </summary>
        public Downloader()
        {
        }

        /// <summary>
        /// Quelltext herunterladen und in einer Variablen speichern
        /// </summary>
        /// <param name="_url">Die URL der Website</param>
        /// <returns></returns>
        public string get_quelltext(string _url)
        {
            WebClient client = new WebClient();
            try
            {
                Quelltext = client.DownloadString(_url);
            }
            catch
            {
                Quelltext = "Es ist ein Fehler aufgetreten. Überprüfen Sie Ihre Eingabe.";
            }
            return Quelltext;
        }
    }
}

Program.cs

/* 
 * Eine einfache Klasse
 * am Beispiel eines Webclienten mit der DownloadString() Methode
 * Autor: Nico @ U-Hacks.net
*/

using System;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            Downloader DL = new Downloader();

            Console.WriteLine("Bitte die URL eingeben. (z.B. http://google.de)");
            string input = Console.ReadLine();

            string quelltext = DL.get_quelltext(input);

            /* Alternativ kann man den Quelltext auch z.B. in einer Windows Forms Anwendung
             * in einer RichTextBox oder MessageBox oder wie auch immer anzeigen lassen */
            Console.WriteLine(quelltext);
            Console.ReadKey();
        }
    }
}

Der Code dürfte sich von selber erklären, falls nicht, einfach fragen :}

[Brainy]

Warum hast du den Quelltext in einer globalen Variable, während du eine Funktion hast, die dir den Quelltext sofort zurückgibt (String)?

MfG

[Snees]

Ist mir auch aufgefallen, das liegt daran, dass ich die Klasse vorher etwas anders gestaltet habe, sie dann aber umgeschrieben habe

[patlux]

Laut den Microsoft-Richtlinien sollte der Anfangsbuchstabe bei einer Methode immer Groß geschrieben werden. Nennt man auch Pascal-Schreibweise.
Auch sollte man - meiner Meinung nach - Unterstriche "_" in Methodennamen vermeiden.

In deinem Beispiel sollte "get_quelltext" also "GetQuelltext" heißen.

Siehe

• Richtlinien für die Verwendung von Methoden
• Richtlinien für die Benennung von Methoden
• Formate für die Großschreibung

Sind aber nur Kleinigkeiten.
In Java würde es passen.

[Mentos]

Hm. Ich würde es nicht GetQuelltext nennen sondern GetSource.

[Brainy]

Und ich würde nicht jedes mal ein neues WebClient Objekt instanzieren, sondern nur 1x im Konstruktor, Schönheitssache.

MfG

[Mr. White]

In Java würde es passen.

Nee.
In Java sollten Variablen und Methoden mit einem Kleinbuchstaben beginnen, in "camelCase" Schreibweise, wofür du bereits ein Beispiel geliefert hast. getSource().
Für die öffnende geschweifte Klammer { verschwendet man auch keine ganze Zeile.

try
{
  Quelltext = client.DownloadString(_url);
}
catch
{
  Quelltext = "Es ist ein Fehler aufgetreten. Überprüfen Sie Ihre Eingabe.";
}

Ich würde hier catch nicht argumentslos verwenden, sondern gezielt auf mögliche Exceptions eingehen.

/// <summary>
/// Downloader
/// </summary>
public Downloader()
{
}

Da hier nichts passiert, kann der Standardkonstruktor ganz weggelassen werden. Der Compiler erzeugt sowieso einen.

Was ich auch nicht schön finde, dass es quasi zwei Getter für Quelltext gibt. Ich würde entweder auf die Variable verzichten, oder die Methode get_quelltext() umschreiben, dass sie die Variable nur initialisiert aber nichts zurückgibt. Bei ersterem würde die Variable quelltext in der Main-Methode auch Sinn ergeben. Die, bzw. Quelltext in Downloader ist überflüssig.

Edit: Sorry, der letzte Teil wurde bereits erwähnt, habe es nicht gelesen.

[Ta1lor]

Ich verstehe nicht ganz wieso du die Quelltext Variable public deklariert hast, wo ist da der Sinn? Oder macht man das in C# so?

[Mr. White]

Ich verstehe nicht ganz wieso du die Quelltext Variable public deklariert hast, wo ist da der Sinn? Oder macht man das in C# so?

Er war tippfaul. Das geht auf die Getter und Setter zurück.
Die Variable hätte als private deklariert werden können und dann irgendwo öffentliche Getter und Setter definieren.
Das was er gemacht hat, ist die Kurzschreibweise.