C# - Problemi i rješenja

imam kao console aplikaciju jedan kalkulator, ali imam problem. ako se ne upiše broj program se ruši, dakle ako je tekst izbaci neki error i program neće dalje. kako to srediti?

A da koristi int.TryParse. I try catch ne bih preporučio za kontrolu logike i granjanja.

Ovaj kod je loše dizajniran i problematičan:

Što ako je broj jedan od ovih tipova sbyte, byte, char, short, ushort ,uint, ulong?

Što ako je Double ima vrjednost NaN ili NegativeInfinity?

Double ima TryParse od ako se ne varam 1.0 .NET-a; nema potrebe koristiti catch 

Što ako je input string 1.0M ili 1d  dali je to ispravan input?

Dali je ispravan unos 123,123 ili 123.123

Dali je  12e3 ispravan input?

Mislim da je ovaj  kod dosta sigurniji i bolji ako pišemo vlastitu funkciju

public static class Extensions
{
    public static bool IsNumeric(this string s)
    {
        foreach (char c in s)
        {
            if (!char.IsDigit(c) && c != '.')
            {
                return false;
            }
        }

        return true;
    }
}

Jel netko zna dobru literaturu na internetu za naučit više o formama?

Zar vi ljudi uopce ne validirate unose prije nego ih pocmete koristiti u algoritmima? Vi samo procitate unos, opalite, pa di pukne pukne...

..hehehe..zato ljudi dođu ovdje pitati da im kažemo ... 

Da, zbilja je lakse throwat exception nego provjerit jel korisnik unio podatak u formatu koji je potreban, te u slucaju da nije obavijestiti korisnika... sta ja znam... zacrvenjivanjem input polja? Da :)

Ono sto hocu reci je da se ucite nekom redu iz pocetka. Na programima od 10-ak linija koda. Jer se losih navika tesko rijesiti. Validacija inputa je must. Prije nego dodje do business layera.

http://msdn.microsoft.com/en-us/library/xhbhezf4.aspx

http://msdn.microsoft.com/en-us/library/z5z9kes2%28v=vs.71%29.aspx

Visual C#

Imam mali problem. Ovo ispod je jednostavna login forma koja bi trebala iz lokalne sqlite3 baze pokupiti korisnike.

Problem je u tome što login pušta samo zadnjeg korisnika u bazi. Već 3 sata piljim u ovo, isprobao sam x različitih načina i ovaj mi se čini "najčišći" ali nikako skužit zašto pušta samo zadnjeg iz popisa.

Evo koda. Ovaj button1 click je problematičan.

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Data.SQLite;

namespace SimpleGym
{
    public partial class Form1 : Form
    {
        string username;
        string user;
        string password;
        int pass = 0;

        List<string> usernames = new List<string>();
        List<string> passwords = new List<string>();

        int ucount;

        DataTable t_users = new DataTable();

        public Form1()
        {
            InitializeComponent();

            string connString = "Data Source=C:\\Documents and Settings\\User\\SimpleGym\\main.db;Version=3;"; // conn string

            SQLiteConnection conn = new SQLiteConnection(connString);

            string getUsers = "SELECT * from users";

            SQLiteDataAdapter usersAdapter = new SQLiteDataAdapter(getUsers, conn);

            DataSet users = new DataSet();

            usersAdapter.Fill(users, "users");

            t_users = users.Tables["Users"];

            ucount = t_users.Rows.Count;

            for (int i = 0; i < ucount; i++ )
            {
                username = t_users.Rows[i]["user_name"].ToString();
                usernames.Add(username);

                password = t_users.Rows[i]["user_password"].ToString();
                passwords.Add(password);
            }
        }

        private void Form1_Load(object sender, EventArgs e)
        {
        }

        private void button1_Click(object sender, EventArgs e)
        {
            user = textBox1.Text;
            password = textBox2.Text;

            for (int i = 0; i < ucount; i++)
            {
                string usr = t_users.Rows[i]["user_name"].ToString();
                string pw = t_users.Rows[i]["user_password"].ToString();

                label3.Text = pass.ToString();

                if (usr == textBox1.Text && pw == textBox2.Text)
                {
                    pass = 1;
                }
                else
                {
                    pass = 2;
                }
            }
            if (pass == 1)
                {
                    MessageBox.Show("Login Successfull!", "Success", MessageBoxButtons.OK, MessageBoxIcon.Information);
                }
            else if (pass == 2)
                {
                    MessageBox.Show("Wrong username or password!", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
                }
        }
    }
}

Uvjeren sam da je neka glupa greška u ovim IFovima na kraju. Svi korisnici osim posljednjeg dobiju "Wrong username or password" iako su im podaci ispravni.

Ako netko ima ideju za elegantnije rješenje, nek slobodno kaže.

[edit] - ova forma će loadati i postavke, liste usera i još dosta toga tako da se nemojte previše čudit

[edit 2] - problem riješen. Kao što rekoh, glup problem među IFovima. Falio je jedan break;

Ja bih samo htio još jednom zahvaliti na preporuci knjiga i vidim da nisam fulao prelazak na C# jer čitam Wroxovu knjigu, došao sam do enumova, structova, arraya i ful je zanimljivo :)

Nisam baš volio programiranje, uvijek je bilo nekako šturo, ali sad izgleda puno bolje :)

  e, nezz :D Ja sebe valjda smatram total retardom, ali sam imao neko znanje iz consolnih appova u C++-u (za natjecanja)... i onda mi je ova knjiga za sada super legla... sve fino objasni, nema previše stručnih riječi koje su nerazumljive, puno generalnih primjera i try it out primjera...

Sviđa mi se, sve u svemu :)

skinuo sam, pa kaj bude, bude :D

inače sam pratio jednu knjigu, isto za početnike, ali je ipak nekako prebrz ritam. ovih 1000 stranica je već bolje :D

10 stranica na dan, naučim da 100 dana :D (kako sam dobar u matematici :) )

sad nemam previše vremena pa ću vrtiti 10 str/dan.

za godinu dana java (minecraft modding :D)

Moglo se i tako ali ovo mi je usput poslužilo da popuni jednu klasu koje tu nema.

Bilo je tuda svakakvog kemijanja, taj login moram popravit. Hvala svejedno