negating an entry in a list

[Cody Biggs]

So  I have a method withdraw that just adds the amount of the withdraw to the list. So if then enter 400 to withdraw I just add that to the list. However when I print the list I want the withdraw to be negative. So the user enters a positive number, and when it prints I want it to be negative. This is what I have

Got more to do, but right now thats my focus

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

namespace Project0
{
    class MainClass
    {

        static void Main(string[] args)
        {
            Console.WriteLine("What is the starting balance? ");
            String startingBalance = Console.ReadLine();
            double d = Double.Parse(startingBalance);
            Account acc = new Account(d);

            Console.WriteLine("Please make a Selection: ");
            Console.WriteLine("(D)eposit ");
            Console.WriteLine("(W)ithdraw ");
            Console.WriteLine("(C)alculate Interest ");
            Console.WriteLine("(S)how Balance ");
            Console.WriteLine("(Q)uit ");

            Char menuSelection = Console.ReadLine()[0];

            while (menuSelection != 'Q')
            {

                if(menuSelection == 'D' || menuSelection =='d')
                {
                    Console.WriteLine("Please enter the amount you would like to deposit ");
                    String deposit = Console.ReadLine();
                    double dep = Double.Parse(deposit);

                    acc.Deposit(dep);
                    Console.WriteLine("Please make a Selection: (D)eposit, (W)ithdraw, (C)alculateInterest, (S)howBalance, (Q)uit ");
                    menuSelection = Console.ReadLine()[0];



                }

                else if(menuSelection == 'W')
                {
                    Console.WriteLine("Please enter the amount you would like to withdraw ");
                    String withdraw = Console.ReadLine();
                    double with = Double.Parse(withdraw);

                    acc.Deposit(with);
                    Console.WriteLine("Please make a Selection: (D)eposit, (W)ithdraw, (C)alculateInterest, (S)howBalance, (Q)uit ");
                    menuSelection = Console.ReadLine()[0];

                }

                else if (menuSelection == 'C')
                {
                    acc.AddInterest();

                    Console.WriteLine("Please make a Selection: (D)eposit, (W)ithdraw, (C)alculateInterest, (S)howBalance, (Q)uit ");
                    menuSelection = Console.ReadLine()[0];

                }

                else if (menuSelection == 'S')
                {
                    acc.GetBalance();

                    Console.WriteLine("Please make a Selection: (D)eposit, (W)ithdraw, (C)alculateInterest, (S)howBalance, (Q)uit ");
                    menuSelection = Console.ReadLine()[0];
                }

                 if(menuSelection == 'Q'){
                    acc.ShowTransaction();
                    Console.Read();
                    
                }
            }//end while loop

            


        }
    }
}

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

namespace Project0
{
    class Account
    {
        private double startingBalance;
        private int transactions = 0;
        private double interest;
        List<double> transaction = new List<double>();

        public Account(double startingBalance)
        {
            this.startingBalance = startingBalance;
            transaction.Add(startingBalance);
        }

        public void Deposit(double dep)
        {
            transaction.Add(dep);
            transactions++;
            

        }

        public void Withdraw(double with)
        {
            transaction.Add(with);
            transactions++;
        }

        public void AddInterest()
        {
            interest = GetBalance() * 0.05;
            transaction.Add(interest);
            transactions++;
        }

        public double GetBalance()
        {
            transactions++;
            return 0;
        }

        public void ShowTransaction()
        {
            foreach (var i in transaction)
            {
                Console.WriteLine(i);
            }

        }







    }
}

[Piet Souris]

hi Cody,

what about a simple

else if(menuSelection == 'W')
                {
                    (...)
 
                    acc.Deposit(-with);

                    (...)

What about creating a Transaction class, with fields TransactionType (that could be an enum), the amount involved and the date and time of the Transaction.
That would enable to do some interest calculations, given a list of periods and the interest rate in that period.

[Cody Biggs]

Yea, I figured that out last night. The first thing my mind wants to do is research for some kind of special method out there instead of just looking at it and take a simple approach. I do have have a menu in my Main class that allows for withdraw, deposit, check the balance, and calculate interest. As far as creating the other classes the instructor just wanted something simple, this is the first assignment of the class so its more of just to get us transitioning from java to c#

[Cody Biggs]

I am screwing myself right now though. So I want to add my transactions together to get the balance, and add that to my list. I think I just confused myself so bad. So here is my method

So before I mess myself up anymore this is my current get balance

public double GetBalance()
        {
            double balance = startingBalance;
            foreach ( var i in transaction)
            {
               
                balance += i;
                Console.WriteLine(balance);
            }
            transaction.Add(balance);
            transactions++;
            return balance;
        }

I want to add all the transactions in the list to the balance variable. So thats adding i to balance. so obviously += is wrong because that will add the starting balance twice, and because im returning balance that will return balance + balance, but im printing two number somehow. I get the balance + balance, but then it also prints my deposit to that balance + balance. I have been testing it with just a deposit from my menu

[Cody Biggs]

Fixed it. Nothing to do with balance