[C#] Opinione su codice che ho scritto

[dwb]

Sera a tutti forumisti!Sto studiano C# per lo sviluppo software su Windows e, dopo aver studuato l'inizio dei metodi, ho buttato giù questo codice con le prime conoscenze acquisite(vengo dal mondo dello sviluppo web sempre con tecnologia Microsoft):

namespace DanyTest
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void btnCalcola_Click(object sender, EventArgs e)
        {
            
            

                int val1 = Int32.Parse(txtSx.Text);
                int val2 = Int32.Parse(txtDx.Text);
                int risultato = calcoloValori(val1, val2);
                txtResult.Text = risultato.ToString();
         

            
        }

        private int calcoloValori(int val1, int val2)
        {
            return val1 + val2;
        }
    }
}

Ho volutamente omesso il gestore di eccezione per semplicità, consapevole però che non è certo una best pratice ma, trattandosi di un esercizio "primordiale" ho pensato di bypassare. Cosa pensate del codice che ho scritto?A livello di stile e lunghezza, ho sbagliato qualcosa?Si può migliorare qualcosina?

P.S. Si tratta di un'applicazione Windows Form con un bottone e tre textbox...

[alka]

La semplicità del codice che hai scritto impedisce di fare qualsiasi considerazione a riguardo.

In sostanza, hai preso due valori dalle caselle di testo, ne fai la somma e la rivisualizzi: quale tipo di miglioria si potrebbe mai fare a questa implementazione? Sono 4 righe...

[dwb]

Lo so alka!
Era per capire se avevo sbagliato qualcosa, anche a livello semantico, e pur trattandosi di quattro semplici righe!

[pietro09]

Originariamente inviato da dwb
Lo so alka!
Era per capire se avevo sbagliato qualcosa, anche a livello semantico, e pur trattandosi di quattro semplici righe!

sono solo quattro righe ma si può migliorare.

Manca il gestore degli errori.
Se digiti dei caratteri al posto dei numeri, non dovrebbe andare in errore?

Poi, hai previsto che il textbox accetti solo numeri intercettando i caratteri digitati?




ps. per la semantica, ti riferisci alla semantica diacronica e semantica sincronica? (scherzo) ciao

[dwb]

Originariamente inviato da pietro09
sono solo quattro righe ma si può migliorare.

Manca il gestore degli errori.
Se digiti dei caratteri al posto dei numeri, non dovrebbe andare in errore?

Poi, hai previsto che il textbox accetti solo numeri intercettando i caratteri digitati?




ps. per la semantica, ti riferisci alla semantica diacronica e semantica sincronica? (scherzo) ciao

Ho scritto di aver volutamente omesso il gestore delle eccezioni caro peter!

[cyanuro]

Se non hai già controllato l'input per gli errori prima della conversione puoi utilizzare (secondo me è comunque meglio) integer.tryparse

[pietro09]

Originariamente inviato da dwb
Ho scritto di aver volutamente omesso il gestore delle eccezioni caro peter!

Che tradotto vuol dire che debbo cambiarmi gli occhiali dato che non l'ho visto ma c'è bello e grande

Si, avrei usato pure io il tryparse.

comunque, volutamente o no, hai sbagliato ad omettere il gestore degli errori, assolutamente necessario in questo tipo di operazioni

ciao

[dwb]

Ti regalo io gli occhiali...
Ciau

[pietro09]

Originariamente inviato da dwb
Ti regalo io gli occhiali...
Ciau

mandaci un po' di codice, così discutiamo meglio

[alka]

Per evitare di andare off topic, suggerisco di aprire discussioni con domande più specifiche in merito a curiosità o dubbi rispetto al proprio codice, senza lasciare troppa libertà di interpretazione, altrimenti sono infinite le possibili migliorie o alternative che si possono suggerire, e personalmente vorrei evitare "guerre di religione" per finezze minori o maggiori su un codice di quattro righe che somma due valori.