c# class

ti pa tole najbrž nimaš v istem namespacu ?

Class_delo_z_njimi.Class3.prikaži() mislim, da bi tako moralo delovati...

a, itak, hvala noraguta.
private void button9_Click(object sender, EventArgs e)
{
Class_delo_z_njimi.Class3 klas3 = new Class_delo_z_njimi.Class3();
klas3.prikaži();
}

še nekej: mam en mini paint program (rasanje črt, krogov,...). ob kliku na button1 rišeš črto, z buttnom2 rišeš krog, itd. a ma kdo kak primer kode, da:
- ne rabm za vsako klikat na button1 da bi risou črto
- ko kliknem na button2 - da prekine button1

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.IO;

namespace Class_delo_z_njimi
{
class Class5
{
public void izračunaj()
{
Form1 frm1 = new Form1();
int a = System.Int32.Parse(frm1.textBox1.Text);
int b = System.Int32.Parse(frm1.textBox2.Text);

int c;
c = a + b;
MessageBox.Show("rezultat je:" + c.ToString());
}
}
}

v textbox1 in 2 vstavim cifre, a mi vrže error v vrstici int a = ..., error je: Input string was not in a correct format. sm kej spregledou?

mi lahk pomagaš pr zadnem postu?

v vrstici:
int a = System.Int32.Parse(frm1.textBox1.Text);
mi prkaše error:
Input string was not in a correct format.
pa sej pišem cifre v textbox1 in 2.

do zdej mam:
v class5.cs je koda:
public static void izračunaj4()
{
Form1 frm1 = new Form1();
int a = System.Int32.Parse(frm1.textBox1.Text);
int b = System.Int32.Parse(frm1.textBox2.Text);
int c;
c = a + b;
MessageBox.Show("rezultat je:" + c.ToString());
}

v form1.cs je koda:
private void button13_Click(object sender, EventArgs e)
{
Class5.izračunaj4();
}

Malo imaš to res nelogično, poskusi tako

public static void izračunaj4(int a, int b)
{
int c = 0;

c = a + b;
MessageBox.Show("rezultat je:" + c.ToString());
}

private void button13_Click(object sender, EventArgs e)
{
int a1 = System.Int32.Parse(textBox1.Text);
int b1 = System.Int32.Parse(textBox2.Text);

Class5.izračunaj4(a1, b1);
}

No ampak če pravilno zastopim, ti podatke vnašaš/bereš samo preko forme.

To da se ti potem vrednosti shranjujejo v classe pa kličejo funkcije narediš vse na formi(ki je dejansko tvoj Main), tako da uporabiš ali objekt tistega razreda ali pa static metodo. Vse kličeš preko forme, potem se ti preko objektov/static metod, polni ali prebere v classu.

Nisem nek strokovnjak, tako da upam da ti bom prov napisal, če ne me naj pa kdo popravi.

Form1 frm1 = new Form1(); ti ustvari novo instanco razreda, on ne ve kaj se skriva v tvojem textboxu na formi.
In s tem ko narediš frm1.textBox1.Text, je vrednost tega verjetno "", torej prazno polje. Ko želi to pretvorit v int se mu pa zatakne.

Sedaj kaj lahko narediš. Najbolj pametno bi bilo še malo preštudirat objekte.
Druga rešitev, če želiš tako delati je da nabrišeš notri en konstruktor(sedaj ker ga nimaš ti vzame privzetega) in mu potem ko kličeš nov objekt že tam nafilaš vrednosti.

Bolša rešitev se skriva že par objav nazaj... Torej uporabi funkcijo tako kot sem ti zgoraj napisal, da je funkcija samo funkcija, brez da vsakič naredi nov objekt. Vrednosti prebereš pa že v razredu forma, ob klicanju funkcije(torej ob kliku na gumb).

no ja static metod se izogibaj , iz principa.
tvoj problem je na na obstojeci formi ustvariš še eno formo

in ne berez iz this.tb1.text .... amamk iz frm1.tb1.text , ampak ti si to formo zgol ustvaril niti jo nisi prikazal. + tega formo ustvariš potem pa že prirediš vrednosti spremenljivkam(ja vrednost je null in to ne gre v int).

Prikaži formo uporabniku, da vnese podakte...

public static void izračunaj4()
{
    Form1 frm1 = new Form1();
    frm1.ShowDialog(); // <- to si pozabil

    // nekje še dodatno preveri smiselnosti vnešenih podatkov
    int a = System.Int32.Parse(frm1.textBox1.Text);
    int b = System.Int32.Parse(frm1.textBox2.Text);
    int c;
    c = a + b;
    MessageBox.Show("rezultat je:" + c.ToString());
}

Če te prav razumem bi rad imel razred z medoto izračunaj. Ob klicu medotde izračunaj bi se uporabniku prikazalo okno v katerega bi vnesel podatke. Po tem ko uporabnik vnese podatke pa bi jih obdelal. Je to to ali ne?

Če ja, potem je zgornji primer v redu! Predpostavljam, da imaš na Form1 dva gumba - [v redu] in [prekliči]. Za oba gumba nastaviš lastnost DialogResult - gumb [v redu] DialogResult.OK, gumb [prekliči] pa DialogResult.Cancel.
Podatke NE obdeluješ na Form1 ampak za klicem frm1.ShowDialog() metode. S pomočjo vrednosti, ki ti jo vrne medota ShowDialog() pa preveriš kateri gumb je uporabnik pritisnil:

if (frm1.ShowDialog() == DialogResult.OK)
{
    // zračunaj oz. obdelaj podatke
}

Čak mal, počasi... ti malo mešaš zadeve.
Imaš osnvoni (glavni) okno, to je Form1. Imaš pa tudi Form5, ki ga odpreš iz Form1, je tako?
V Form1 imaš 2 textBoxa, v katera vneseš števila, a ne?
In v Form5 imaš metodo za izračun. Seštevek iz textBoxov iz Form1, bi rad prikazal v Form5, je tako?

samo malo ti naredim domačo nalogo.

Preverjanje decimal števila se lahko izvaja tudi na sledeči način, ki je v bistvu "še boljši" od tryParse() metode:

Regex jeStevilo = new Regex(@"(^[-+]?\d+(,?\d*)*\.?\d*([Ee][-+]\d*)?$)|(^[-+]?\d?(,?\d*)*\.\d+([Ee][-+]\d*)?$)");
Match SteviloPravo = jeStevilo.Match(strEnoStevilo);
if (!SteviloPravo.Success)
{
   MessageBox.Show("Prosim v vnosna polja vpišite samo števila");
   return;
}

.. just for info
Mitja

Ti imaš malo pomešane pojme, ampak vseeno ... kvečjemu naredi tako koj je spodaj:
Form1:

        private void button4_Click(object sender, EventArgs e)
        {
            string strStevilo1 = textBox3.Text.Trim();
            string strStevilo2 = textBox4.Text.Trim();
            if (strStevilo1 != String.Empty && strStevilo2 != String.Empty)
                Class1.Izracunaj(strStevilo1, strStevilo2);
            else
                MessageBox.Show("Za izračun prosim vpišite števila v oba vnosna polja.");
        } 

Class5:

        public static void Izracunaj(string strStevilo1, string strStevilo2)
        {
            Form1 form1 = new Form1();
            
            //PREVERJANJE ŠTEVIL, ČE SO LE TA TAPRAVA
            bool boolPravoStevilo = false;
            decimal decStevilo = 0;
            string[] arrayVsaStevila = new string[] { strStevilo1, strStevilo2 };

            foreach (string strEnoStevilo in arrayVsaStevila)
            {
                Regex jeStevilo = new Regex(@"(^[-+]?\d+(,?\d*)*\.?\d*([Ee][-+]\d*)?$)|(^[-+]?\d?(,?\d*)*\.\d+([Ee][-+]\d*)?$)");
                Match SteviloPravo = jeStevilo.Match(strEnoStevilo);
                if (!SteviloPravo.Success)
                {
                    System.Windows.Forms.MessageBox.Show("Prosim v vnosna polja vpišite samo števila");
                    return;
                }

                boolPravoStevilo = decimal.TryParse(strEnoStevilo, out decStevilo);
                if (!boolPravoStevilo)
                {
                    System.Windows.Forms.MessageBox.Show("Prosim v vnosna polja vpišite samo števila");
                    break;
                }
            }
            if (boolPravoStevilo)
            {
                //PO PREVERJANJU ŠTEVIL SE LE TA SPREMENI V DECIMAL:
                decimal decStevilo1 = Convert.ToDecimal(strStevilo1);
                decimal decStevilo2 = Convert.ToDecimal(strStevilo2);
                
                decimal decVsota = decStevilo1 + decStevilo2;
                System.Windows.Forms.MessageBox.Show("Rezultat je: " + decVsota);
            }            
        }

To kar ti delaš se meni osebno zdi brez potrebe.
Če že želiš vedeti, zakaj ne vidiš textBoxov, ko vpišeš:
Form1 frm1 = new Form1();
int a = System.Int32.Parse(frm1. ... ->
textBox1 moraš nastaviti na PUBLIC, da bo viden iz drugih classov, to narediš na sledeči način:
Greš v Form1.Designer.cs in na koncu dol imaš listo vseh kontrol, ena izmed njih sta textBox1, in textBox2. Spremeni iz private v public:
public System.Windows.Forms.TextBox textBox1;
public System.Windows.Forms.TextBox textBox2;
Sedaj lahko narediš tvojo finto (textBox bo viden iz Class5):
Form1 frm1 = new Form1();
int a = System.Int32.Parse(frm1.textBox1);
OPOMBA: controle ni pametno imeti nastavljene kot public.

Imam pa pripombe:
Zakaj bi iz Class5 dostopal v Form1? To je čisto nepotrebno. Ti imaš na Classu5 metodo za izračun, ZATO SE PARAMETER (VREDNOSTI) PRENESE iz metode v Form1 v Class5 - as simple as that. Poglej mojo zgornjo kodo.

Če pa delaš kakšen drug primer (ne tega tako simple z c= a+b), pa ga prosim pokaži, da bom videl zakaj želiš dostopati iz classa5 v form1, ok?
lp,
Mitja

Popravljena koda: spremenjeni parametri (decimal, string)

Nevem, ampak jaz sem enkrat to že delal in je stvar delala.
Ti pa to odsvetujem, kot so do meni.
Zato ti predlagam, da narediš tako kot sem jaz naredil na zgornji kodi.
Parametre pošlješ iz Form1 v Class5 na 2. načina:
1. ali tako kot kot sem jih jaz (skupaj z metodo)
2 ali pa preko get, set metode
get, set kako:
V classu5 narediš:

public static decimal PrvoStevilo { get; private set; }
public static decimal DrugoStevilo { get; private set; }

Nato jih v metodi Izracunaj samo pokličeš.

Na Formi1 v metodi od kjer pošiljaš podatke na class5:

Class5.PrvoStevilo = decPrvoStevilo; //decPrvoStevilo je decimal number od vpisanega stringa
Class5.DrugoStevilo = decDrugoStevilo;

Ericssony je 30. jun 2010 ob 07:33 izjavil:

Z "Bončem" sva ti lepo dopovedovala kako in kaj in podala možne rešitve ampak ti hočeš na vsak način z glavo skozi zid...

Ma pust ga, je brez veze. Sej ko bo pisal kakšno večjo (daljšo) kodo tam čez par 10.000 vrstic, bo že pogruntal.
Mene je to že izučilo, pa se z programiranjem ukvarjam dobro leto dni.
ajde