| Autor |
Beitrag |
Xclusive
      
Beiträge: 56
C#
|
Verfasst: Do 19.03.09 21:00
So mein erstes C# Programm ist fertig:
Ich würde mich über eine Bewertung freuen und über verbesserungsvorschläge.
Denkt bitte daran:
Ich bin Anfänger,
Hauptschüler
15 Jahre alt. Seit dem 12. März.
Danke
Ich hoffe ich darf das reinstellen.
mfg David
Download:
Einloggen, um Attachments anzusehen!
Zuletzt bearbeitet von Xclusive am Do 19.03.09 21:15, insgesamt 2-mal bearbeitet
|
|
Christian S.
      
Beiträge: 20451
Erhaltene Danke: 2264
Win 10
C# (VS 2019)
|
Verfasst: Do 19.03.09 21:12
Hallo!
Bitte gib einen direkten Downloadlink an oder hänge das Programm direkt an Deinen Beitrag an. Danke!
Grüße
Christian
P.S.: Wieso ist das Programm so groß? 
_________________ Zwei Worte werden Dir im Leben viele Türen öffnen - "ziehen" und "drücken".
|
|
Xclusive 
      
Beiträge: 56
C#
|
Verfasst: Do 19.03.09 21:16
Ich weis es nicht sorry.
Vielleicht weger dem Hintergrundbild?
Ok, dass mit dem Dateianhang ist geklärt.
|
|
Yogu
      
Beiträge: 2598
Erhaltene Danke: 156
Ubuntu 13.04, Win 7
C# (VS 2013)
|
Verfasst: Do 19.03.09 21:18
Christian S. hat folgendes geschrieben : | P.S.: Wieso ist das Programm so groß?  |
Wieso, 1,93 MB sind doch für eine Anwendung mit einem großen Hintergrundbild noch ok.
Was mir aber auffällt: Das Fenster braucht ewig lange um sich zu zeichnen. Ich denke mal, das liegt genau an dem Hintergrundbild, das, so wie es aussieht, für jede Komponente nochmal gezeichnet werden muss. Einfachster Tipp: Lösche es
Außerdem gibt es eine unbehandelte Ausnahme, wenn nichts eingegeben wurde, und auf "Berechnen" geklickt wird.
Ganz lustig wäre doch auch noch ein Hinweis, ob jetzt bestanden wird, oder nicht. Sonst funktioniert es ganz gut.
Grüße,
Yogu
|
|
Xclusive 
      
Beiträge: 56
C#
|
Verfasst: Do 19.03.09 21:30
Das mit dem Fehler weis ich Danke
Aber keine ahnung wie ich es beheben soll -.-
Das mit dem Hinweis weis ich leider acuh nicht.
Wollte ich machen.
mfg David
|
|
freak4fun
      
Beiträge: 604
Erhaltene Danke: 4
Win 7 Pro
VS 2013 Express, Delphi, C#, PHP, Java
|
Verfasst: Do 19.03.09 21:34
Du solltest etwas mehr auf das Programm Wert legen und nicht sos ehr auf das Äußere. 
_________________ "Ich werde auf GAR KEINEN Fall…!" - "Keks?" - "Okay, ich tu's."
i++; // zaehler i um 1 erhoehen
|
|
Xclusive 
      
Beiträge: 56
C#
|
Verfasst: Do 19.03.09 21:38
Danke für den tip die meiste zeit hab ich fürs Design gebraucht 
|
|
freak4fun
      
Beiträge: 604
Erhaltene Danke: 4
Win 7 Pro
VS 2013 Express, Delphi, C#, PHP, Java
|
Verfasst: Do 19.03.09 22:36
_________________ "Ich werde auf GAR KEINEN Fall…!" - "Keks?" - "Okay, ich tu's."
i++; // zaehler i um 1 erhoehen
|
|
jaenicke
      
Beiträge: 19329
Erhaltene Danke: 1750
W11 x64 (Chrome, Edge)
Delphi 12 Pro, C# (VS 2022), JS/HTML, Java (NB), PHP, Lazarus
|
Verfasst: Do 19.03.09 22:38
Die Oberfläche macht normalerweise den kleineren Teil eines Programms und der Arbeit damit aus.
Bei so kleinen Programmen ist das natürlich etwas anderes.
Was mir auffällt:
Das Hintergrundbild passt zu den Textfarben nicht richtig, denn der Text ist so schlecht lesbar. Vielleicht ginge das besser, wenn das Bild z.B. etwas aufgehellt würde.
Statt Int32.Parse solltest du lieber Int32.TryParse benutzen um einen Fehler dabei abzufangen, wenn keine Zahl eingegeben wurde.
Warum startest du explizit den Internet Explorer? Sowas nervt, lass doch Windows den Standardbrowser des Benutzers verwenden. C#-Quelltext 1:
| Process.Start("http://www.google.de"); |
Warum das Laden so schleppend läuft, weiß ich nicht so genau, aber irgendetwas läuft da schief. Bei der Größe des Bildes könnte es auch einfach nur daran liegen.
freak4fun hat folgendes geschrieben : | Versuch es mal mit:
Delphi-Quelltext 1: 2: 3: 4: 5:
| try ... except ... end; |  |
Es geht um ein .NET Programm in C#. 
|
|
freak4fun
      
Beiträge: 604
Erhaltene Danke: 4
Win 7 Pro
VS 2013 Express, Delphi, C#, PHP, Java
|
Verfasst: Do 19.03.09 22:49
_________________ "Ich werde auf GAR KEINEN Fall…!" - "Keks?" - "Okay, ich tu's."
i++; // zaehler i um 1 erhoehen
|
|
Xclusive 
      
Beiträge: 56
C#
|
Verfasst: Fr 20.03.09 07:47
|
|
Xclusive 
      
Beiträge: 56
C#
|
Verfasst: Fr 20.03.09 12:58
Entschuldigung für die doppelte Antwort.
Das ist ziemlich unübersichtlich geworden.
Mein Code:
1: 2: 3: 4: 5: 6: 7: 8: 9: 10: 11: 12: 13: 14: 15: 16: 17: 18: 19: 20: 21: 22: 23: 24: 25: 26: 27: 28: 29: 30: 31: 32: 33: 34: 35: 36: 37: 38: 39: 40: 41: 42: 43: 44: 45: 46: 47: 48: 49: 50: 51: 52: 53:
| using System; using System.Collections.Generic; using System.ComponentModel; using System.Data; using System.Drawing; using System.Text; using System.Windows.Forms;
namespace WindowsFormsApplication1 { public partial class Form1 : Form { public Form1() { InitializeComponent(); }
private void button1_Click(object sender, EventArgs e) { textBox13.Text = ((int.Parse(textBox1.Text) * 2 + int.Parse(textBox2.Text) * 2 + int.Parse(textBox3.Text) + int.Parse(textBox7.Text) * 2 + int.Parse(textBox8.Text) * 2 + int.Parse(textBox9.Text) + int.Parse(textBox4.Text) * 2 + int.Parse(textBox5.Text) + int.Parse(textBox6.Text) + int.Parse(textBox11.Text) + int.Parse(textBox12.Text) + int.Parse(textBox10.Text) * 2) / 18.0).ToString();
}
private void linkLabel1_LinkClicked(object sender, LinkLabelLinkClickedEventArgs e) { string goTo = "proxy.kartoffel-hack.com/index.php?q=uggc%3A%2F%2Fjjj.grnz-hyz.qr%2F";
System.Diagnostics.Process.Start("iexplore.exe", goTo); }
private void linkLabel2_LinkClicked(object sender, LinkLabelLinkClickedEventArgs e) { string goTo = "www.pennergame.de";
System.Diagnostics.Process.Start("iexplore.exe", goTo); }
private void linkLabel3_LinkClicked(object sender, LinkLabelLinkClickedEventArgs e) { string goTo = "www.hsweis.schule.neu-ulm.de";
System.Diagnostics.Process.Start("iexplore.exe", goTo); } |
|
|
jaenicke
      
Beiträge: 19329
Erhaltene Danke: 1750
W11 x64 (Chrome, Edge)
Delphi 12 Pro, C# (VS 2022), JS/HTML, Java (NB), PHP, Lazarus
|
Verfasst: Fr 20.03.09 13:04
Dass der Quelltext so aussieht, war mir schon klar. .NET Programme lassen sich sehr gut disassemblieren.
Bau als erstes einmal TryParse statt Parse ein. Dann kannst du die Prüfung in welchem Bereich die Eingaben sind auch gleich einbauen.
|
|
Xclusive 
      
Beiträge: 56
C#
|
Verfasst: Fr 20.03.09 15:31
|
|
jaenicke
      
Beiträge: 19329
Erhaltene Danke: 1750
W11 x64 (Chrome, Edge)
Delphi 12 Pro, C# (VS 2022), JS/HTML, Java (NB), PHP, Lazarus
|
Verfasst: Fr 20.03.09 15:35
|
|
Xclusive 
      
Beiträge: 56
C#
|
Verfasst: Fr 20.03.09 15:41
Es geht irgentwie nicht  *verzweifelt*
Kann jemand schnell über TeamViewer schauen?
Bitte,
Hcoke schon ewig drann -.-
Währe demjenigen sehr Dankbar.
|
|
jaenicke
      
Beiträge: 19329
Erhaltene Danke: 1750
W11 x64 (Chrome, Edge)
Delphi 12 Pro, C# (VS 2022), JS/HTML, Java (NB), PHP, Lazarus
|
Verfasst: Fr 20.03.09 15:44
Xclusive hat folgendes geschrieben : | Es geht irgentwie nicht *verzweifelt* |
Was heißt das? Ich habe doch ein konkretes Beispiel gepostet.
Wie hast du das denn jetzt versucht?
|
|
Xclusive 
      
Beiträge: 56
C#
|
Verfasst: Fr 20.03.09 15:51
Einloggen, um Attachments anzusehen!
|
|
Xclusive 
      
Beiträge: 56
C#
|
Verfasst: Fr 20.03.09 16:59
|
|
Jakob_Ullmann
      
Beiträge: 1747
Erhaltene Danke: 15
Win 7, *Ubuntu GNU/Linux*
*Anjuta* (C, C++, Python), Geany (Vala), Lazarus (Pascal), Eclipse (Java)
|
Verfasst: Fr 20.03.09 17:17
Also es geht darum, dass Fehler behandelt werden. Du führst nachfolgenden Code immer nur aus, wenn es keine Exception gab, ansonsten wird halt gar nichts gemacht / du teilst dem Benutzer seinen Fehler mit.
|
|