.net - Hjælp til C# projekt. Får objekt-reference fejl - Udvikleren.dk

Tags: .net

<< < 1 2 > >>

Bruger #12411 @ 25.04.08 20:21

Hej
Er gået i gang med et større projekt på skolen. Vi koder i C# og jeg går på 2.sem.
Vi har skulle skabe et "Register" (Ligesom et kasseapparat) som skal kunne klare at ordne et salg og senere hen har vi tilkoblet en kunde med oplysninger til salget. Både varer og kunden er hardcodede ind i starten. Alt dette er gået fint ved at skabe følgende klasser:
Store (Butik)
Register
Customer
CustomerCatalog
Product
ProductCatalog
Sale
SalesLineItem

Register er min controller og alt går igennem den. Da det hele blev harcodede ind kunne jeg godt få lortet til at virke, men nu har vi fået besked på at smide GUI på, hvilket ikke umiddelbart virker vanskeligt. Men det er det åbenbart for mig :/

Jeg skal nu skabe en form som man kan bruge til at oprette varer som så skal kunne gemmes i min arraylist som ligger i ProductCatalog. Vi har fået af vide af læren at da Register er controller skal alt fra GUI og ned til model-niveauet foregå igennem Register-klassen.

Det her er første gang vi for det første skaber så store et projekt selv og første gang vi skal "Add" til et projekt. Jeg tror jeg har gjort det rigtigt, men er dog ikke sikker.
Jeg har skabt 1 get og 1 set-metode i Register-klassen, som henholdsvis bruges til at hente det jeg indtaster om varen i textboxene og føre det ned i Register-klassen, her bliver det smidt over i get-metoden som så bliver kaldt af en anden metode i min ProductCatalog-klasse. Her bliver det så tilføjet min Arraylist til vare (Lidt ligesom et lager).

Kan godt se meningen med alt det her, MEN jeg får følgende fejl:

Error 1 An object reference is required for the nonstatic field, method, or property 'Register.SetCreateProduct(ProductDescription)' (Form2.cs 40)

Error 2 An object reference is required for the nonstatic field, method, or property 'Register.GetCreateProduct()' i linje 67 (Kasseregister2)

ER DER IKKE EN VENLIG SJÆL DER GIDER AT HJÆLPE MIG MED DET HER, HAR SPILDT EN HEL UGE PÅ DET HER ALLEREDE UDEN AT KOMME VIDERE

Kode:

Kode

KasseRegister2.cs

using System;
using System.Collections;

public class CustomerCatalog
{
private ArrayList CustomerList = new ArrayList(); //Skaber en Arraylist til vores kunder

public CustomerCatalog() //Constructor
{
int telf = 0; //HVORFOR KUN 0, OG IKKE HELE TELEFONNUMMERET??????
string knavn = "Casper";
double krabat = 22.5;

Customer cust;

cust = new Customer(telf, knavn, krabat); //Indsætter telfonnr, navn og rabat fra kunden ind i arraylisten
CustomerList.Add(cust);
}

public Customer getCustomer(int tele) //En metode til at hente oplysninger om kunden som skal bruges et andet sted i koden
{
return (Customer)CustomerList[tele]; //Sender hele arraylisten, til dem som skal bruge den
}
}


public class Customer
{
private int telefonnr;
private string kundeNavn;
private double kundeRabat;

public Customer(int telefonnr, string kundeNavn, double kundeRabat) //Constructor
{
this.telefonnr = telefonnr;
this.kundeNavn = kundeNavn;
this.kundeRabat = kundeRabat;
}

public string getCustomerName()
{
return kundeNavn;
}

public double getCustomerRabat()
{
return kundeRabat;
}

public int getCustomerTelefon()
{
return telefonnr;
}
}

public class ProductCatalog
{
private ArrayList descriptions = new ArrayList();

public ProductCatalog() //Constructor
{

}

public void CreateProduct()
{
ProductDescription Ting = Register.GetCreateProduct();
descriptions.Add(Ting);
}

public ProductDescription getProductDescription(int id) //En metode til at hente produktoplysninger til et andet sted
{
return (ProductDescription)descriptions[id];
}
}
public class Register
{
private ProductCatalog catalog;
private CustomerCatalog custCatalog;
private Sale currentSale;
private static Register instance = null; //Singleton
private ProductDescription Ting2 = null;


private Register(ProductCatalog catalog, CustomerCatalog custCatalog) //Constructor
{
this.catalog = catalog;
this.custCatalog = custCatalog;
}

public static Register GetInstance(ProductCatalog catalog, CustomerCatalog custCatalog) //Singleton
{
if (instance == null)
{
instance = new Register(catalog, custCatalog);
}
return instance;
}
public void SetCreateProduct(ProductDescription Ting)
{
Ting2 = Ting;
}

public ProductDescription GetCreateProduct()
{
return (ProductDescription)Ting2;
}

public void endSale() //Afslut salg
{
currentSale.becomeComplete();//Bruger metoden becomeComplete() til at finde ud af om salget er afsluttet
}

public void enterItem(int id, int quantity) //Indtastning af vare + antal
{
ProductDescription desc = catalog.getProductDescription(id); //henter produkt beskrivelsen af varene og smider dem over i desc
currentSale.makeLineItem(desc, quantity); //Smider dem ind i et Arraylist med navnet LineItems, som ligger i Sale
}

public void enterCust(int tele) //Indtastning af eksisterende kunde til en regning
{
Customer cust = custCatalog.getCustomer(tele);
currentSale.AddCustToSale(cust);
}

public void makeNewSale() //Skaber et nyt salg
{
currentSale = new Sale();
}

public void makePayment(double cashTendered)//Finder ud af hvad man skal betale
{
currentSale.makePayment(cashTendered);//Henter fra salg (currentSale)
}

public double getSaleTotal() //Henter den samlede pris for alle varene i getTotal som ligger i Sale klassen
{
double total = currentSale.getTotal();//Tager værdien fra getTotal i Sale-klassen og overfører det i en double total
return total; // returnere den samlede pris, TIL Larman klassen
}

public double getMoms() //Finder momsen af de varer man køber
{
double moms = currentSale.Moms(); //Henter oplysninger fra Moms-metoden i Sale-klassen
return moms;
}

public void Print()
{
currentSale.Print(); //Henter fra print metoden i Sale-klassen
}

}

public class ProductDescription
{
private int id;
private double prize;
private string description;

public ProductDescription(int id, double prize, string description) //Constructor
{
this.id = id;
this.prize = prize;
this.description = description;
}

public int getItemID() //En metode til at hente produkt id'et
{
return id;
}

public double getPrize()//En metode til at hente prisen på varen
{
return prize;
}

public string getDescription() //En metode til at hente produkt beskrivelse af varen
{
return description;
}
}

public class Sale
{
private ArrayList lineItems = new ArrayList();
private Customer thisCust;
//private DateTime date = new DateTime();
private bool isComplete = false;
private double payment = 0;

public Sale() //Constructor
{

}

public double getBalance()//En metode til at hente balancen efter det indbetalte og det købte
{
return payment - getTotal(); //Her minuser man payment(betaling) med metoden getTotal
}

public void becomeComplete() //En metode som spørger om salget er færdigt
{
isComplete = true; //Sætter isComplete til at være sand
}

public bool IsComplete()//En bool metode som kan vise om salget er færdigt eller ej
{
return isComplete;
}

public void makeLineItem(ProductDescription desc, int quantity) //En metode som indsætter de ønskede varer + antal i arraylisten lineItems, skal indeholde de ønskede varer
{
lineItems.Add(new SalesLineItem(desc, quantity));
}

public void AddCustToSale(Customer cust) //Indsætter den ønskede kunde i en variabel som kan bruges her i klassen
{
thisCust = cust;
}

public double getTotal() //Lægger prisen sammen på de ønskede varer i vores arraylist
{
double total = 0;
double subtotal = 0;

foreach (SalesLineItem lineItem in lineItems)
{
subtotal = lineItem.getSubtotal();
total += subtotal;
}
return total;
}

public double Moms() //Finder momsen af det samlede køb
{
double moms = getTotal() * 0.25;
return moms;
}

public void makePayment(double cashTendered) //Sætter det input(penge) kunden betaler til at være = payment
{
double payment = cashTendered;
}

public void Print()
{
//Indhentning
double ddMoms = Moms(); //Henter momsen på beløbet
string ssCust = thisCust.getCustomerName(); //Henter kundens navn
double ddCust = thisCust.getCustomerRabat(); //Henter kundens rabat
int iiCust = thisCust.getCustomerTelefon(); //Henter kundens telefonnummer
double total = getTotal(); //Beregn totalen


//Udskrift
Console.Clear(); //Rengøre konsollen
Console.WriteLine("\\n Kunde: " + ssCust.ToString().PadRight(11) + "Kundenr: " + iiCust.ToString().PadLeft(5));
Console.WriteLine("\\n----------------------------------------------\\n");
foreach (SalesLineItem s in lineItems) //Printer produktbeskrivelsen ud og prisen på de enkelte produktenheder
{
Console.WriteLine(" " + s.getQuantity().ToString().PadRight(3) + " " + s.getProductDescription().ToString().PadRight(18) + " " + s.getSubtotal().ToString().PadLeft(8) + " kr");
}
Console.WriteLine("\\n-----------------------------------------------\\n");
Console.WriteLine("\\n Total: {0} kr", total.ToString().PadLeft(25));
Console.WriteLine("\\n Moms: {0} kr", ddMoms.ToString().PadLeft(26));
Console.WriteLine("\\n Rabat: {0}% ", ddCust.ToString().PadLeft(27));
Console.WriteLine("\\n----------------------------------------------");
Console.WriteLine("\\n Total inkl. rabat: {0} kr", (total - (total * (ddCust / 100))).ToString().PadLeft(13));
Console.WriteLine("______________________________________________");
Console.WriteLine("----------------------------------------------");

}
}

public class SalesLineItem
{
private int quantity;
private ProductDescription description;

public SalesLineItem(ProductDescription description, int quantity) //Constructor
{
this.description = description;
this.quantity = quantity;
}

public int getQuantity() //Metode til at hente antallet af vare
{
return quantity;
}

public string getProductDescription() //Metode til at hente produktbeskrivelsen af et given produkt
{
return description.getDescription();
}

public double getSubtotal()//Finder prisen for den enkelte vare X antal
{
return description.getPrize() * quantity; //Henter oplysninger fra metoden getPrize som ligger i Register-klassen
}
}

public class Store //HVAD GØR DEN HER?
{
private ProductCatalog catalog;
private Register register;
private CustomerCatalog Ccatalog;
private static Store instance = null; //Singleton

private Store()
{
catalog = new ProductCatalog();
Ccatalog = new CustomerCatalog();
register = Register.GetInstance(catalog, Ccatalog); //Singleton
}
public static Store GetInstance()
{
if (instance == null)
{
instance = new Store();
return instance;
}
else
return instance;
}

public Register getRegister()
{
return register;
}

}







Form2.cs

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

namespace WindowsApplication7
{
public partial class Form2 : Form
{
public Form2()
{
InitializeComponent();
}

private void Form2_Load(object sender, EventArgs e)
{
Store s = Store.GetInstance(); //Opret et Store - objekt
Register r = s.getRegister();

}

private void button4_Click(object sender, EventArgs e)
{
Close();
}

private void button1_Click(object sender, EventArgs e)
{
int id = Convert.ToInt32(textBox1.Text);
string beskrivelse = textBox2.Text;
double price = Convert.ToDouble(textBox3.Text);


ProductDescription desc;
desc = new ProductDescription(id, price, beskrivelse);

Register.SetCreateProduct(desc);


MessageBox.Show("Product id: " + textBox1.Text + "\\nDescription: " + textBox2.Text + "\\nProduct price: " + textBox3.Text + "\\nEr nu oprettet i varekartotekket");
}
}
}

Jeg har tilknyttet hele projekt som fil.
http://www.sendspace.com/file/892yfp

Indlæg senest redigeret d. 25.04.2008 20:49 af Bruger #12411

13 svar postet i denne tråd vises herunder
2 indlæg har modtaget i alt 4 karma

Sorter efter stemmer Sorter efter dato

Bruger #6559 @ 25.04.08 20:37

843

Start med at kigge på de fejl du får smidt i hovedet.

Error 1:

Error 1 An object reference is required for the nonstatic field, method, or property 'Register.SetCreateProduct(ProductDescription)' (Form2.cs 40)

An object reference is required for the nonstatic field...

Det vil sige du skal have en reference eller oprettet en ny instans. f.eks

Kode

Register nyRegister;
nyRegister.SetCreateProduct(ProductionDescription);

Error 2:

Error 2 An object reference is required for the nonstatic field, method, or property 'Register.GetCreateProduct()' i linje 67 (Kasseregister2)

Igen 'An object reference is required...', så her skal du igen have oprettet en reference.

Håber det er noget du kan bruge.

EDIT:

Så i stedet for dette:

Kode

private void button1_Click(object sender, EventArgs e)
{
int id = Convert.ToInt32(textBox1.Text);
string beskrivelse = textBox2.Text;
double price = Convert.ToDouble(textBox3.Text);


ProductDescription desc;
desc = new ProductDescription(id, price, beskrivelse);

Register.SetCreateProduct(desc);


MessageBox.Show("Product id: " + textBox1.Text + "\\nDescription: " + textBox2.Text + "\\nProduct price: " + textBox3.Text + "\\nEr nu oprettet i varekartotekket");
}

vil jeg tro du skal have dette:

Kode

private void button1_Click(object sender, EventArgs e)
{
int id = Convert.ToInt32(textBox1.Text);
string beskrivelse = textBox2.Text;
double price = Convert.ToDouble(textBox3.Text);


ProductDescription desc;
Register nyRegister;
desc = new ProductDescription(id, price, beskrivelse);

nyRegister.SetCreateProduct(desc);


MessageBox.Show("Product id: " + textBox1.Text + "\\nDescription: " + textBox2.Text + "\\nProduct price: " + textBox3.Text + "\\nEr nu oprettet i varekartotekket");
}

Skal lige siges, at jeg har ikke testet det.

Ps Læg venligst din kode i en kodeboks næste gang, så er det nemmere at overskue koden.

Indlæg senest redigeret d. 25.04.2008 20:41 af Bruger #6559

Bruger #13106 @ 26.04.08 09:09

Det ligner da meget et eksempel fra min gamle ven Craig Larman, jeg gad desværre bare aldrig læse i Applying UML and patterns!!!

Når du skal bruge Register som controller skal du have mulighed for at få fat i den. Hvis du kun ønsker at have een instans af den er det rigtigt, når du bruger singleton.

I dit projekt er der to muligheder for at få fat i en Register:
Store.GetInstance().getRegister()

Register.GetInstance(ProductCatalog catalog, CustomerCatalog custCatalog)

Når du bruger den øverste har du et problem, forbi den instans, du får retur vil være null. Du bliver nødt til at lave en instans af Register i Store før du returnerer den.

Den anden mulighed giver også problemer, da GetInstance methoden kræver parametre. Nu skal du ned i din model og lave instanser før du kan få fat i en instans af din controller, som er indgang til modellen. Håber du kan se problemet.

Jeg var lige inde og finde bogen i mine gemmer. Craig og jeg ser ud til at være enige om at du skal bruge Store til at lave en instans af Register. Det er stores ansvar at lave instanser af de objekter, som Register har brug for, for at kunne fungere.

Håber svaret leder dig i den rigtige retning.

Bruger #12411 @ 25.04.08 20:48

Hm den æder det ikke rigtigt.
Får følgende fejl. Har stiret mig helt blind på det.

Error 1 Use of unassigned local variable 'nyRegister' \\Form2.cs l. 42, 9

Error 2 An object reference is required for the nonstatic field, method, or property 'Register.GetCreateProduct()'\\KasseRegister2.cs l. 67, 35
Sry for måden jeg har sat det op, skal huske det til en anden gang, tak for alt den hjælp du kan tilbyde

Bruger #6559 @ 25.04.08 20:51

843

Sry en lille rettelse, det skal være

Register nyRegister = new Register();

Jeg ved godt den vil lave en ny instans af din klasse, men jeg tror desværre det er nødvendigt.

Bruger #12411 @ 25.04.08 20:56

Det her er meget nyt for mig, men er det ikke netop det som min singleton i Store-klassen og singleton i Register-klassen er til for at undgå?

Bruger #6559 @ 25.04.08 21:07

843

Jo, nu har jeg ikke fuld oversigt over din kode, fordi jeg ikke har en editor inde pt, men ellers kan du ændre din funktion SetCreateProduct til at være static, men det kan give nogle komplektioner med resten af din kode.

Du kan evt se om du kan nå din Register r så du skriver

r.setCreateProduct, da den er sat, men jeg ved det ikke lige.

Bruger #12411 @ 25.04.08 23:13

Hm tja det vil sgu ikke rigtig virke med mindre jeg skriver Store s = Store.GetInstance();
Register r = s.getRegister();
Igen og igen der hvor jeg bruger r. som i r.setCreateProduct og det kan ikke være rigtigt.

Bruger #6559 @ 25.04.08 23:55

843

Okay, jeg har lige læst fundet lidt mere om singleton. Prøv med følgende:

Register.instance.SetCreateProduct(desc);

Ellers så er jeg ved at løbe tør for løsninger.

Bruger #12411 @ 26.04.08 10:53

Det er helt rigtigt at det er projektet fra Larmans bog, bare skrevet i C# istedet. Har desværre ikke rigtig tid i dag til at få kigget på det, men vil meget gerne have jeres hjælp/ekspertise igen i morgen når jeg kaster mig over det.

Bruger #12411 @ 28.04.08 17:23

Hm har nu siddet i et par dage endnu engang og kæmpet med det her l***, med lidt hjælp fra en anden side fik jeg også mit program til at køre. jeg kan indtaste en ny vare ude i min UI og så troede jeg at jeg fik sendt den ned i min Register klasse og videre til min ArrayList i min ProductCatalog-klasse, men ak nej. Jeg tror ikke den kommer frem. For hvis jeg prøver at få vist indholdet af min ArrayList i selv samme form ved hjælp af en opdater knap og en listbox, så dukker der ingen ting op. Jeg har prøvet at hardcoded en vare ind og det kan sagtens lykkedes at få den op i listboxen, ergo er det bindeledet fra jeg skriver den ind i UI og til den rammer ProductCatalog der er noget galt med, MEN HVAD?????

Form2 UI:

Kode

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

namespace WindowsApplication8
{
    public partial class Form2 : Form
    {
        public Form2()
        {
            InitializeComponent();
        }

        private void button4_Click(object sender, EventArgs e)
        {
            Close();
        }

        private void Form2_Load(object sender, EventArgs e)
        {
            Store s = Store.GetInstance();
            Register r = s.getRegister();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            int id = Convert.ToInt32(textBox1.Text);
            string beskrivelse = textBox2.Text;
            double price = Convert.ToDouble(textBox3.Text);


            ProductDescription desc;
            desc = new ProductDescription(id, price, beskrivelse);
            Register r = Register.GetInstance();//Kalder min singleton metode i Register og spørger efter om der er oprettet et register, 
            //hvis ikke så opretter den en ny
            r.SetCreateProduct(desc); //Her bliver den indtastede varer smidt til metoden i Register


            MessageBox.Show("Product id: " + textBox1.Text + "\\nDescription: " + textBox2.Text + "\\nProduct price: " + textBox3.Text + "\\nEr nu oprettet i varekartotekket");

            
        }

        private void button5_Click(object sender, EventArgs e)
        {
            Register r = Register.GetInstance();
            ArrayList array = r.hentProdukt();

            for (int i = 0; i < array.Count; i++)
            {
                listBox1.Items.Add(((ProductDescription)array[i]).getDescription());

            }
        }
    }
}

Kode

using System;
using System.Collections.Generic;
using System.Text;
using System.Collections;

namespace WindowsApplication8
{
    class Register
    {
        private ProductCatalog catalog;
        private CustomerCatalog custCatalog;
        private Sale currentSale;
        private static Register instance = null; //Singleton
        private ProductDescription varer2;

        private Register() //Constructor
        {
            this.catalog = new ProductCatalog();
            this.custCatalog = new CustomerCatalog();
        }

        public ArrayList hentProdukt()
        {
            return catalog.hentProduktet();
        }

        public static Register GetInstance() //Singleton
        {
            if (instance == null) // Vi siger her at hvis instance er lig med null (altså at der ingen register er
            //så skal den sætte instance til at være lig med nyt register
            {
                instance = new Register();
                return instance;
            }
            
           else
                return instance;
        }

        public void SetCreateProduct(ProductDescription varer)//Her placerer jeg varen som kommer fra Form2.
        {
            varer2 = varer;
        }

        public ProductDescription GetCreateProduct()//Her kan varen fra Form2 --> SetCreateProduct, hentes
        {
            return (ProductDescription)varer2;
        }

        public void endSale() //Afslut salg
        {
            currentSale.becomeComplete();//Bruger metoden becomeComplete() til at finde ud af om salget er afsluttet
        }

        public void enterItem(int id, int quantity) //Indtastning af vare + antal
        {
            ProductDescription desc = catalog.getProductDescription(id); //henter produkt beskrivelsen af varene og smider dem over i desc
            currentSale.makeLineItem(desc, quantity); //Smider dem ind i et Arraylist med navnet LineItems, som ligger i Sale
        }

        public void enterCust(int tele) //Indtastning af eksisterende kunde til en regning
        {
            Customer cust = custCatalog.getCustomer(tele);
            currentSale.AddCustToSale(cust);
        }

        public void makeNewSale() //Skaber et nyt salg
        {
            currentSale = new Sale();
        }

        public void makePayment(double cashTendered)//Finder ud af hvad man skal betale
        {
            currentSale.makePayment(cashTendered);//Henter fra salg (currentSale)
        }

        public double getSaleTotal() //Henter den samlede pris for alle varene i getTotal som ligger i Sale klassen
        {
            double total = currentSale.getTotal();//Tager værdien fra getTotal i Sale-klassen og overfører det i en double total
            return total; // returnere den samlede pris, TIL Larman klassen
        }

        public double getMoms() //Finder momsen af de varer man køber
        {
            double moms = currentSale.Moms(); //Henter oplysninger fra Moms-metoden i Sale-klassen
            return moms;
        }

        public void Print()
        {
            currentSale.Print(); //Henter fra print metoden i Sale-klassen
        }
    }
}

ProductCatalog-klassen:

Kode

using System;
using System.Collections.Generic;
using System.Text;
using System.Collections;

namespace WindowsApplication8
{
    class ProductCatalog
    {
        private ArrayList descriptions = new ArrayList();
        
        public ProductCatalog() //Constructor
        {
           
        }
        
        public ArrayList hentProduktet() //PROBLEM
        {
            return descriptions;
        }

        public void CreateProduct()
        {
            Register r = Register.GetInstance();

            ProductDescription varer = r.GetCreateProduct();
            descriptions.Add(varer);
        }

        public ProductDescription getProductDescription(int id) //En metode til at hente produktoplysninger til et andet sted
        {
            return (ProductDescription)descriptions[id];
        }
    }
}

Store-klassen:

Kode

using System;
using System.Collections.Generic;
using System.Text;

namespace WindowsApplication8
{
    class Store
    {
        private ProductCatalog catalog;
        private Register register;
        private CustomerCatalog Ccatalog;
        private static Store instance = null; //Singleton

        private Store()
        {
            catalog = new ProductCatalog();
            Ccatalog = new CustomerCatalog();
            register = Register.GetInstance(); //Singleton
        }
        public static Store GetInstance()
        {
            if (instance == null)
            {
                instance = new Store();
            }
            return instance;
        }

        public Register getRegister()
        {
            return register;
        }
    }
}

Hvis jeg prøver at gør lidt som du siger Hans Jørgen og lader Store være den som opretter en ny Register instans og ikke Register r = r.GetInstance(), så får jeg fejl meddelser

<< < 1 2 > >>

Hjælp til C# projekt. Får objekt-reference fejl

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler