Topics

C#•2y ago

❔ OOP Product Inventory Project

Hi I'm making a product inventory project to get better at OOP, and I have another class called Inventory where I have a method that calculates the value of the entire inventory. I'm trying to figure out how I would go about doing this, and so far my logic is whenever a new product is initialized, I want to add the quantity of that product and price of the product to a list so I can access that list in the Inventory class and then go about calculating the sum. My question is: for the constructor, when I initialize a new product in my Program.cs file, will it add the desired information to the list? I'm not sure if I'm doing it right.

48 Replies

GIGA BRAIN•2y ago

Product Inventory Project - Create an application which manages an inventory of products. Create a product class which has a price, id, and quantity on hand. Then create an inventory class which keeps track of various products and can sum up the inventory value.

Angius•2y ago

Uh Price and quantity are already properties You can already access them

GIGA BRAIN•2y ago

oh so i don't even need a list in the first place

Angius•2y ago

Yeah

GIGA BRAIN•2y ago

jeez haha i'll look into how to implement that then i guess in this case it would be good to have a list just so i can have all the products and their properties as a history reference?

Angius•2y ago

Loop over the products in the inventory, then add product.Price * product.Quantity to some total Lastly, return the total You mean, historical quantities and prices of the product?

GIGA BRAIN•2y ago

I mean everything about the product price, ID, and quantity

Angius•2y ago

Why? To be able to compare that the product used to cost 7.99 and now it costs 8.78?

GIGA BRAIN•2y ago

just so there is an option for the user to see everything at once

Angius•2y ago

So.... not historical data You just want to place properties, that are already public, inside of some public list... to do what, exactly?

GIGA BRAIN•2y ago

ah now that you say it like that it doesn't make sense to do that i just want to have a nice interface when the user runs the program

Angius•2y ago

Console.WriteLine($"Product with id {product.Id} costs {product.Price} and there's {product.Quantity} in stock.");

Console.WriteLine($"Product with id {product.Id} costs {product.Price} and there's {product.Quantity} in stock.");

GIGA BRAIN•2y ago

GIGA BRAIN•2y ago

yeah this is what i have in my program.cs

Angius•2y ago

Ah, you're storing each product in a separate variable?

GIGA BRAIN•2y ago

But i feel like its redundant

Angius•2y ago

Why not a list of products?

GIGA BRAIN•2y ago

exactly i think thats what i was trying to get at i want to add the product and its information to a list that has all the products

Angius•2y ago

var products = List<Product>(){
    new Product(...),
    new Product(...),
    // ...
};

var products = List<Product>(){
    new Product(...),
    new Product(...),
    // ...
};

Then

foreach (var product in products)
{
    Console.WriteLine($"Product with id {product.Id} costs {product.Price} and there's {product.Quantity} in stock.");
}

foreach (var product in products)
{
    Console.WriteLine($"Product with id {product.Id} costs {product.Price} and there's {product.Quantity} in stock.");
}

to display all of them

GIGA BRAIN•2y ago

perfect

Angius•2y ago

Lists and loops

GIGA BRAIN•2y ago

awesome gonna try and implement that now thanks

Accord•2y ago

Was this issue resolved? If so, run /close - otherwise I will mark this as stale and this post will be archived until there is new activity.

GIGA BRAIN•2y ago

could someone point me in the right direction on how to calculate the total sum of my inventory?

GIGA BRAIN•2y ago

GIGA BRAIN•2y ago

GIGA BRAIN•2y ago

GIGA BRAIN•2y ago

earlier in the post i was told to loop over each product in the inventory and then calculate the total fromthere but im not sure how I would do that since I already initialize a list of the products in my program.cs

GIGA BRAIN•2y ago

i tried looking into how to access a list from a different class in c# but i dont think it would work in this case? https://stackoverflow.com/questions/3717028/access-list-from-another-class#:~:text=To%20make%20it%20accessible%20to,public%20property%20which%20exposes%20it.&text=To%20access%20the%20list%20from,MyList%20to%20access%20the%20list.

Stack Overflow

Access List from another class

can anyone tell me how to create a list in one class and access it from another?

GIGA BRAIN•2y ago

Would i create a list in product.cs and then somehow add each initialization of a product object to that list?

Angius•2y ago

Keep the list of products in the inventory Not in the top level

GIGA BRAIN•2y ago

namespace OOP2
{
    public class Inventory
    {
        // FIELDS:
        public int NumberOfProducts { get; set; }
        public List<Product> ProductList = new List<Product>
        {
            new Product("0001", "Macbook Pro", 100, 10000),
            new Product("0002", "Iphone 13", 50, 100)
        };

        // CONSTRUCTOR:
        public Inventory(int numProducts)
        {
            NumberOfProducts = numProducts;
        }

        // METHODS:
        public decimal getInventorySum()
        {
            decimal totalSum = 0;
            foreach (var product in ProductList)
            {
                totalSum += (product.Quantity * product.Price);
            }
            return totalSum;
        }

        public string getInventoryDetails()
        {
            var report = "";
            foreach (var product in ProductList)
            {
                report = ($"Product #{product.Identification} costs ${product.Price} per unit. There are {product.Quantity} units available.");
            }
            return report;
        }
    }
}

namespace OOP2
{
    public class Inventory
    {
        // FIELDS:
        public int NumberOfProducts { get; set; }
        public List<Product> ProductList = new List<Product>
        {
            new Product("0001", "Macbook Pro", 100, 10000),
            new Product("0002", "Iphone 13", 50, 100)
        };

        // CONSTRUCTOR:
        public Inventory(int numProducts)
        {
            NumberOfProducts = numProducts;
        }

        // METHODS:
        public decimal getInventorySum()
        {
            decimal totalSum = 0;
            foreach (var product in ProductList)
            {
                totalSum += (product.Quantity * product.Price);
            }
            return totalSum;
        }

        public string getInventoryDetails()
        {
            var report = "";
            foreach (var product in ProductList)
            {
                report = ($"Product #{product.Identification} costs ${product.Price} per unit. There are {product.Quantity} units available.");
            }
            return report;
        }
    }
}

this makes a lot more sense and i implemented it, but now i'm confused on how I would call this

Angius•2y ago

Instantiate Inventory And work with that instance Also, you never seem to use NumberOfProducts

GIGA BRAIN•2y ago

yeah i was just gonna talk about that i need to think on that one more actually but yes exactly i forgot to instantiate it thanks ok ive been struggling on how to implement NumberOfProducts in the code and i need some clarification please

GIGA BRAIN•2y ago

could I just do something like this?

GIGA BRAIN•2y ago

because from what i had originally i would have to manually say how many products there are and that doesn't really make sense im also not sure how I would add the list information to the constructor, do I even need to do that in the first place?

Angius•2y ago

ProductList.Length List<T> has a Length property

GIGA BRAIN•2y ago

oooh ok i'll try and implement that how would I put that into the constructor? and did you mean the count function?

GIGA BRAIN•2y ago

Angius•2y ago

Why do you need it in the constructor?

GIGA BRAIN•2y ago

to be honest im not even sure if i need a constructor i thought you did for oop

Angius•2y ago

Every class has at least a default constructor, that's what allows you to do var f = new Foo(); But a constructor doesn't need to have any parameters

GIGA BRAIN•2y ago

oh ok so i can just leave the constructor blank?

Angius•2y ago

Or you can just not have it A blank constructor is implicitly there

GIGA BRAIN•2y ago

gotcha yeah it works perfectly now 😄 well with the exception of number of products not being referenced i'm thinking I should just make it a method, what do you think?

Angius•2y ago

You can make a pass-through property Or reference the list of items, and its .Length property, directly inventory.ProductsList.Length Or make a public int ProductCount => ProductsList.Length; property in the Inventory and use it with inventory.ProductCount

GIGA BRAIN•2y ago

ok trying to process the info i'll try these out

using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;

namespace OOP2
{
    public class Inventory
    {
        // FIELDS:
        public List<Product> ProductList = new List<Product>
        {
            new Product("0001", "Macbook Pro", 100, 10000),
            new Product("0002", "Iphone 13", 50, 100)
        };

        // When not using a constructor, you don't even need to have one because there is already a blank one implicitly there
        // // CONSTRUCTOR:
        // public Inventory()
        // {
        // }

        // METHODS:
        public decimal getInventorySum()
        {
            decimal totalSum = 0;
            foreach (var product in ProductList)
            {
                totalSum += (product.Quantity * product.Price);
            }
            return totalSum;
        }

        public string getInventoryDetails()
        {
            var report = new System.Text.StringBuilder();
            foreach (var product in ProductList)
            {
                report.AppendLine($"Product #{product.Identification} costs ${product.Price} per unit. There are {product.Quantity} units available.");
            }
            return report.ToString();
        }
    }
}

using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;

namespace OOP2
{
    public class Inventory
    {
        // FIELDS:
        public List<Product> ProductList = new List<Product>
        {
            new Product("0001", "Macbook Pro", 100, 10000),
            new Product("0002", "Iphone 13", 50, 100)
        };

        // When not using a constructor, you don't even need to have one because there is already a blank one implicitly there
        // // CONSTRUCTOR:
        // public Inventory()
        // {
        // }

        // METHODS:
        public decimal getInventorySum()
        {
            decimal totalSum = 0;
            foreach (var product in ProductList)
            {
                totalSum += (product.Quantity * product.Price);
            }
            return totalSum;
        }

        public string getInventoryDetails()
        {
            var report = new System.Text.StringBuilder();
            foreach (var product in ProductList)
            {
                report.AppendLine($"Product #{product.Identification} costs ${product.Price} per unit. There are {product.Quantity} units available.");
            }
            return report.ToString();
        }
    }
}

i think i just like mentioning the property directly in the program.cs but other than that, i think the program is finished thanks as usual z, youre a real lifesaver

Accord•2y ago

Was this issue resolved? If so, run /close - otherwise I will mark this as stale and this post will be archived until there is new activity.

We are a programming server aimed at coders discussing everything related to C# (CSharp) and .NET.

52KMembers

View on Discord

Want results from more Discord servers?

Add your server

More Posts

❔ when i try to turn off gravity on rb it just doesnt happenmy code is 200+ lines so, but i can show u where i try to turn off gravity (c#)❔ when i try to turn off gravity on rb it just doesnt happenmy code is 200+ lines so, but i can show u where i try to turn off gravity (c#)❔ Asp.net API - the most reasonable in memory storage implementationHey guys, if you had to write asp.net web api with in memory storage - some list with items, what wo ❔ Ignoring Main?just starting out with c# and have a pretty basic question. Trying to just make a random array of in ❔ Is there a better way to insert a configuration source before the appsettings files?I have an AspNetCore web API where I am using the appsettings.json and appsettings.Development.json,❔ Most effective way to learn c#?Hello everyone, I'm brand-new to C#. I'm curious about the best technique to learn C# and maintain m ❔ Trying to make a bit of code to display all values of fibonacci until the nth term inputHave got this so far (i am a beginner) int n = Convert.ToInt32(Console.ReadLine()); Console.Wri ❔ Claim type of JwtRegisteredClaimNames.Name unrecognized as ClaimTypes.NameHello, I'm working with an API that returns a token with below structure: ```{ "sub": "someGuid",❔ How to properly redirect in Razor Pages?I'm redirecting w/ variables using: 1. `<a asp-page="Stats" asp-route-qurl="@j">button</a>` redirec ❔ Data ParsingHello,I'm trying to built a simulation program by using windows forms application and I have a quest