C# Kalenderblatt

[dot]

Was genau ist da das Problem und wo/wie hast du eigentlich BYTE und _ASSERT() definiert?

[Smokee]

dot schrieb:

Was genau ist da das Problem und wo/wie hast du eigentlich BYTE und _ASSERT() definiert?

Ich habe ja gesagt Brett vorm Kopf. Manchmal sind die Fehler so simpel.

Danke

Mfg Smokee

[Smokee]

Smokee schrieb:

Guten Morgen,

danke schön für deine Hilfe.

Habe den Quellcode jetzt noch einmal überarbeitet

short getWochentag(int uTag,int uMonat,int uJahr)
{
// ungült Jan Feb Mrz Apr Mai Jun Jul Aug Sep Okt Nov Dez
BYTE arrMonatsOffset[13] = [b]{ 0, 1, 4, 3, 6, 1, 4, 6, 2, 5, 0, 3, 5};[/b]

BYTE cbJahrhundertOffset = 0;

[b]_ASSERT[/b](uTag &gt 0);
[b]_ASSERT[/b](uTag <= 31);
[b]_ASSERT[/b](uMonat > 0);
[b]_ASSERT[/b](uMonat <= 12);
[b]_ASSERT[/b](uJahr >= 1600);

Bei den Sachen die ich Größer gemacht habe sind die Fehler. Ich hab es schon gegoogelt, aber ich finde nichts wirklich hilfreiches was ich newbie auch verstehe
Hoffe ihr könnt mir wieder einmal helfen. Hab irgendwie nen Brett vorm Kopf xD

Mfg Smokee

Also eigentlich ist das Problem das "_ASSERT" und "BYTE" nicht defeniert wurden was ich aber leider nicht raffe, weil wenn ich es defeniere bzw deklariere. dann kommt genau der selbe fehler wie wenn ich es nicht deklariere. hoffe irgendwer steigt noch hier durch

Mfg Smokee

[dot]

Smokee schrieb:

[...] weil wenn ich es defeniere bzw deklariere. dann kommt genau der selbe fehler wie wenn ich es nicht deklariere.

Verrat uns vielleicht auch mal was genau das für ein Fehler ist

[Smokee]

dot schrieb:

Smokee schrieb:

[...] weil wenn ich es defeniere bzw deklariere. dann kommt genau der selbe fehler wie wenn ich es nicht deklariere.

Verrat uns vielleicht auch mal was genau das für ein Fehler ist

short getWochentag(int uTag,int uMonat,int uJahr)
{ int _ASSERT;
    int Byte;
// ungült Jan Feb Mrz Apr Mai Jun Jul Aug Sep Okt Nov Dez
BYTE arrMonatsOffset[13] = [b]{0, 1, 4, 3, 6, 1, 4, 6, 2, 5, 0, 3, 5}[/b];

//Bei den Zahlen will VS anstatt den Komma, ein Semikolon. Aber wenn ich ein //Semikolon setzte dann sagt er mir trotzdem noch das er eins haben will.

BYTE cbJahrhundertOffset = 0;

[b]_ASSERT[/b](uTag &gt) 0;

//Hierbei sagt er dass die Variable "_ASSERT" wie eine Methode verwendet wird

_ASSERT(uTag [b]<[/b]= [i][b]31)[/b][/i];
_ASSERT(uMonat [b]>[/b] [i][b]0)[/b][/i];
_ASSERT(uMonat [b]<=[/b] [i][b]12)[/b][/i];
_ASSERT(uJahr [b]>=[/b] [i][b]1600)[/b][/i];

//Bei den Zahlen will VS anstatt den Komma, ein Semikolon. Aber wenn ich ein //Semikolon setzte dann sagt er mir trotzdem noch das er eins haben will.

Hoffe das ich es einigermaßen verständlich beschrieben habe.

Mfg Smokee

[dot]

Was genau versprichst du dir davon zwei Variablen mit Namen Byte und _ASSERT zu definieren!?
Dass er sich danach wegen irgendwas aufregt ist auch klar. C# ist eben nicht C++, das sollte wenn dann eher so aussehen:

int[] arrMonatsOffset = new int[]{0, 1, 4, 3, 6, 1, 4, 6, 2, 5, 0, 3, 5};

Und da _ASSERT eine Variable ist kannst du es natürlich nicht wie eine Funktion aufrufen, das macht so alles absolut keinen Sinn, was sollte da deiner Meinung nach passieren!?
Wenn du schon ein assert willst dann verwend doch das im .NET Framework eingebaute und statt diesem BYTE Kram (wie auch immer du drauf kommst) verwend doch einfach int und fertig!? Vielleicht solltest du dir auch einmal das zu Gemüte führen und dir ins Bewusstsein rufen dass C# case sensitive ist.

[Smokee]

So danke für eure Hilfe, wenn ich den Quellcode soweit fertig habe und es läuft kann ich ja auszüge daraus nochmal posten. vielleicht werden dann fragen von neulingen wie mir gleich gelöst ^^.

schönen abend bzw mittag noch.

mfg jan

[Smokee]

Smokee schrieb:

So danke für eure Hilfe, wenn ich den Quellcode soweit fertig habe und es läuft kann ich ja auszüge daraus nochmal posten. vielleicht werden dann fragen von neulingen wie mir gleich gelöst ^^.

schönen abend bzw mittag noch.

mfg jan

hallo zusammen,

ich habe nun endlich geschafft "mein" kalenderblatt zu programmieren und es funtkioniert auch soweit ganz gut. nur mein problem jetzt ist es, das nach erstellen eines kalenderblattes nachgefragt werden soll ob das programm beendet oder ein neues kalenderblatt erstellt werden soll. ich würde das in eine while schleife packen die ungfähr so aussehen würde.

bool exit false

while (exit)
{
     Programmcode vom Kalender
}
Console.Readline();
     if (a=exit)
     exit false;

wäre nun die frage wie genau ich das machen muss um den Kalender immer und immer wieder aufrufen zu können bis ich sage per befehl das er das beenden soll.

hoffe mir können wieder einige helfen.

:xmas1: Fröhliche Weihnachten :xmas1: und einen guten Rutsch schon mal vorweg ^^

Smokee

[EXDW]

Wünsch dir ein C# Buch zu Weihnachten ^^

[Smokee]

David W schrieb:

Wünsch dir ein C# Buch zu Weihnachten ^^

Muss wirklich sagen ein sehr guter Beitrag zum Thema.

Leider ist der Wunschzettel schon abgeschickt. Spass bei seite.

Wäre echt nett wenn ihr mir Lösungsansätze gebt oder mir meine Fehler erklärt.

[Sheldor]

Smokee schrieb:

bool exit false

while (exit)
{
     Programmcode vom Kalender
}
Console.Readline();
     if (a=exit)
     exit false;

Hast du schon gemacht

Gruss Sheldor

[Smokee]

Hallo ihr,

ich habe die "Wiederholungsschleife" eingebaut und schon tritt ein neues Problem auf und zwar mach ich ganz zu anfang diese Eingabe:

Console.WriteLine("Bitte geben Sie das Jahr ein:    ", jahr);
jahr = Convert.ToInt16(Console.ReadLine());

Console.WriteLine("Bitte geben Sie den Monat ein:    ", monat);
monat = Convert.ToInt16(Console.ReadLine());

if (jahr >= 1 && jahr <= 3000 && monat >= 1 && monat <= 12)

Ich habe nun am ende des Programmes gesagt wenn die Eingabe nicht zwischen 1 und 12 ist, dann soll er schreiben das ein falsches Jahr oder Monat eingegeben wurde.

Console.WriteLine("Sie haben ein falsches jahr oder einen falschen Monat eingegeben");

Soweit so gut.

Mein Anliegen ist es wenn ich zum Beispiel als Monat die 13 eingebe und er sagt das es eine falsche Eingabe ist, einfach die Abfrage solange wiederholt bis die Eingabe stimmt.

Frohe Weihnachten wünscht euch Smokee :xmas1:

[EXDW]

Du solltest die Eingabe erst als String holen und dann mit int.TryParse schauen ob es überhaupt eine Zahl ist. Wenn ich "hutzlblutz" eingebe fliegt dir die Applikation um die Ohren.

Zudem sollte in der Eingabeaufforderung bescheid gegeben werden das eine Zahl erwartet wird, könnte ja "January" bzw "Januar" eingeben wollen.

eine "Eingabeschleife" könnte ich mir so vorstellen

//Pseudocode
public int GetMonth()
{
    var isValid = false;
    var result = 0;
    do
    {
        Console.WriteLine("Please give the month number: ");
        var input = Console.ReadLine();
        if (!int.TryParse(input, out result))
        {
            Console.WriteLine(string.Format("\"{0}\" is not a number, please retry", input);
            isValid = false;
        }
        else if (result < 1 || result > 12)
        {
            Console.WriteLine(string.Format("\"{0}\" is not a valid month number, please retry", result);
            isValid = false;
        }
        else
            isValid = true;
    } while (!isValid)
}

Das selbe dann fürs Jahr.
Und da man keine Code Duplikation haben möchte kann man es refactoren:

public int GetNumber(string message, int min, int max)
{
    var isValid = false;
    var result = 0;
    do
    {
        Console.WriteLine(message);
        var input = Console.ReadLine();
        if (!int.TryParse(input, out result))
        {
            Console.WriteLine(string.Format("\"{0}\" is not a number, please retry", input);
            isValid = false;
        }
        else if (result < min || result > max)
        {
            Console.WriteLine(string.Format("\"{0}\" is not in the allowed range, please retry", result);
            isValid = false;
        }
        else
            isValid = true;
    } while (!isValid)
}

schon kannst du es fürs Jahr und für den Monat nehmen

var year = GetNumber("Please give the year: ", 1980, 2040);
var month = GetNumber("Please give the month number: ", 1, 12);

Am ende Bietet sich in der Do-While noch eine "Exit" Funktionalität an oder ein "Maximum Retry".

//Alles hier im Forum getippt, muss nicht korrekt sein oder funktionieren.

//Edit: Blöde fehler im Code

[Smokee]

Hallo zusammen,

hoffe ihr hattet alle schöne Weihnachten.

So ich habe jetzt soweit den Kalender fertig und poste hier jetzt mal den Quellcode, zumindest in teilen. Vielleicht gibts ja noch verbesserungsvorschläge.

Den Quellcode habe ich wieder rausgenommen, da ich nicht möchte das dieser einfach nur kopiert wird. Wer dennoch den Quellcode haben möchte einfach eine PN an mich und ich schicken zu.

Mit freundlichen Grüssen

Smokee

[EXDW]

Hast du meinen Beitrag gelesen? Entschuldige wenn ich es so sagen muss, dein Code ist gruselig und auch sehr gefährlich.

Was mir auffällt beim ersten drüber schauen:
01. Du Kommentierst was man im Code direkt sieht -> Redundant, Schlecht.
02. Kein TryParse sondern ein blankes Convert -> Sehr Schlecht, Gefährlich.
03. Kein Hinweis für den Nutzer das der Monat als Zahl erwartet wird -> Unschön.
04. Keine Aufteilung, alle Funktionalität in der Main Methode -> Unsauber und schwer zu maintainen -> Sehr Schlecht, Gefährlich.
05. Variablen auf Deutsch -> Unschön.
06. Variablen weit außerhalb definiert wo sie verwendet werden und auch noch mehrfache Verwendung -> Unsauber, Gefährlich.
07. Ein unnötiger Scope -> Unsauber.
08. Console.Clear direkt nach WriteLine? -> Sinn?
09. Dein Kommentar "So wie oben geht das noch für alle anderen Wochentage" lässt darauf schließen das du noch Tonnen an doppelten Code hast und zudem die Main Methoden dadurch noch um ein paar Kilometer verlängert hast -> Unsauber, Unschön, Gefährlich

[Smokee]

So um den Code zu vervollständigen hier die Berechnungen für die Wochentage.

Das Gleiche auch hier

Falls hier einer weis wie ich das verkürzen kann, bitte tipps

Und danke für die Tipps die ich schon bekommen habe, werde ich schnellst möglichst umsetzen.

Mit freundlichen Grüssen

Smokee

[fluxy2]

Also,

im Prinzip hat dir David ja schon alles gesagt, dein Problem ist schon längst gelöst. Schmeiss deinen Rotz da weg und mach ausnahmsweise mal copy & paste.

Zu deinem Code kann ich nur sagen: Du hast eine endliche Zahl an Bedingungen, das ist grausam. Um eine Zahlenreihe abzuprüfen, verwende doch bitte den Modulo-Operator. Mit ihm kannst du eine Bedingung vormulieren, mit der schonmal 20000 Zeilen code wegfallen.

Darüber hinaus ist dein Code nicht wartbar. In 2 Wochen weisst du nicht mehr was du gemacht hast. Sagen wir es kommt mal ein 8. Tag hinzu. Dann hast du 500% deiner Entwicklungszeit zu veranschlagen, um diese Änderung da einzubauen. Unter Gesichtspunkten einer Projektplanung ist dieser Code unzumutbar und einfach ein Griff in die Kloschüssel. Sry.

Guck oben im Thread, da steht alles was du wissen musst. Und ansonsten: Überleg dir erstmal ein ordentliches Design.

Grüße Fluxy

[Smokee]

Dank dir. Ich wollte nur veranschaulichen was ich so als erst Entwurf zusammen geschrieben habe.

Naja ich werde mich mal dran setzten und versuchen auf so 100 Zeilen zukommen.

Mfg Smokee

[Smokee]

Hallo,

ich habe nun den Kalender auf 150 zeilen zusammengeschrumpft. Erstmal ^^

Nun zu meinem kleinen ziwschen Problem und zwar muss ich falsche Eingaben wie zum Beispiel Buchstaben oder Sonderzeichen abfangen und den Benutzer darauf hinweisen, danach dann halt nochmal die Abfrage für den Monat und das Jahr. Und das solange bis die Eingaben stimmen, beziehungsweise die Eingabe für den Monat zwischen 1 und 12 liegt und die Eingabe für das Jahr nicht 3000 überschreitet.

also müsste das meiner ansicht so anfangen

while (monat != 1 bis 12)
dann geb aus das eingabe flasche
wenn doch dann weiter

und die nächste schleife oder kann man die ganzen sachen auch in einer großen schleife abfragen??

Hoffe das verstehen alle und für alle anderen schuldigt bitte wenn ich mich nicht klar ausdrücken konnte.

Mit freundlichen Grüssen

Smokee

[O.o]

David W schrieb:

02. Kein TryParse sondern ein blankes Convert -> Sehr Schlecht, Gefährlich.
03. Kein Hinweis für den Nutzer das der Monat als Zahl erwartet wird -> Unschön.