Develop Reference

How to make my Console.ReadLine() stop returning my backspace inputs?

First and foremost, I feel it necessary to mention that I am currently using replit for this program. The issue that I am currently having with my code is that it runs perfectly, the only issue is that when I try to backspace using the methods listed below and fix a number or letter that I have mistyped in my input while running the program, I get an error saying that the value must be positive and below the buffer height. I realized that the issue is some of my Console.ReadLine(); placements. They are causing my program to read backspace as an input and in turn, I am receiving an error because backspace is neither a letter or number input. Is there any way that I can move them around without having to rewrite the whole method line?
public static void AddItemToStore() {
Console.WriteLine("========= Adding Item to Store =========");
Console.WriteLine();
Console.WriteLine("You will need to enter an item description, price, and quantity on hand.");
Console.WriteLine();
Console.WriteLine("Please enter an item description:");
string input = Console.ReadLine();
double entry = GetPrice();
int value = GetQuantity();
itemDescription.Add(input);
itemPrice.Add(Convert.ToDouble(entry));
itemQoh.Add(value);
SaveData();
}
public static string GetCustomerName() {
Console.WriteLine();
Console.WriteLine("=========== List of Customers =========== ");
Console.WriteLine();
for (int i = 0; i < salesCustomerName.Count; i++) {
Console.WriteLine($"{i + 1}. {salesCustomerName[i]} ");
Console.WriteLine();
}
while (true) {
Console.WriteLine("Please choose a Customer name:");
Console.WriteLine();
return Console.ReadLine();
}
}
public static double GetPrice() {
while (true) {
Console.WriteLine();
Console.WriteLine("Enter the price of the item (x.xx):");
string entry = Console.ReadLine();
double value;
if (!double.TryParse(entry, out value) || value <= 0){
Console.WriteLine("Please enter a valid price value.");
} else {
return Convert.ToDouble(entry);
}
}
}
public static int GetQuantity() {
while (true){
Console.WriteLine();
Console.WriteLine("Please enter the item quantity:");
string entry = Console.ReadLine();
int value = 0;
if (Int32.TryParse(entry, out value) && Convert.ToInt32(entry) > 0){
return Convert.ToInt32(entry);
}
else{
Console.WriteLine("Please enter a valid value.");
}
}
}
Listed above are the methods that I am currently having an issue with. They work perfectly with inputting information but as soon as I press backspace, I get an error. Down below I will have my whole program listed as well.
using System;
using System.Collections.Generic;
using System.IO;
class Program {
public static List<string> itemDescription = new List<string>();
public static List<double> itemPrice = new List<double>();
public static List<int> itemQoh = new List<int>();
public static List<string> customerName = new List<string>();
public static List<string> salesCustomerName = new List<string>();
public static List<string> salesItemDescription = new List<string>();
public static List<double> salesItemPrice = new List<double>();
public static List<int> salesQuantity = new List<int>();
public static void Main (string[] args) {
ReadData();
bool keepGoing = true;
while (keepGoing) {
string entry = GetMenuOption();
if (entry.ToLower() == "q"){
Console.WriteLine("Thanks for shopping with us! Come back soon! :)");
}
switch (entry) {
case "1":
GoShopping();
break;
case "2":
Management();
break;
case "Q":
keepGoing = false;
break;
}
}
}
public static void AddItemToStore() {
Console.WriteLine("========= Adding Item to Store =========");
Console.WriteLine();
Console.WriteLine("You will need to enter an item description, price, and quantity on hand.");
Console.WriteLine();
Console.WriteLine("Please enter an item description:");
string input = Console.ReadLine();
double entry = GetPrice();
int value = GetQuantity();
itemDescription.Add(input);
itemPrice.Add(Convert.ToDouble(entry));
itemQoh.Add(value);
SaveData();
}
public static void AddItemToCart(string name, int item) {
Console.WriteLine();
Console.WriteLine($"There are {itemQoh[item]} {itemDescription[item]} available for {itemPrice[item]:C2} each.");
Console.WriteLine();
Console.WriteLine("Please choose the amount of items that you would like to purchase");
string val = Console.ReadLine();
int pur_qty =0;
int num = -1;
if (!int.TryParse(val, out num)) {
Console.WriteLine("Please enter a valid integer");
} else {
pur_qty = Convert.ToInt32(val);
if (pur_qty <= itemQoh[item] ){
itemQoh[item] = itemQoh[item]-pur_qty;
salesCustomerName.Add(name);
salesItemDescription.Add(itemDescription[item]);
salesItemPrice.Add(itemPrice[item]);
salesQuantity.Add(pur_qty);
SaveData();
}else{
Console.WriteLine();
Console.WriteLine("Please enter a valid value");
}
}
}
public static void ChangePrice() {
Console.WriteLine("========= Changing Price =========");
int entry = Convert.ToInt32(GetItemNumber());
int price = 0;
Console.WriteLine();
Console.WriteLine($"Updating price. {itemDescription[price]} currently costs {itemPrice[price]:C2}.");
double new_price = GetPrice();
itemPrice[entry]= new_price;
SaveItems();
SaveData();
}
public static void ChangeQoh() {
Console.WriteLine("========= Changing Quantity =========");
int entry = Convert.ToInt32(GetItemNumber());
int value = 0;
Console.WriteLine($"Updating quantity. There are currently {itemQoh[value]} {itemDescription[value]} available.");
Console.WriteLine();
int new_qty = GetQuantity();
itemQoh[entry]=new_qty;
SaveItems();
SaveData();
}
public static string GetCustomerName() {
Console.WriteLine();
Console.WriteLine("=========== List of Customers =========== ");
Console.WriteLine();
for (int i = 0; i < salesCustomerName.Count; i++) {
Console.WriteLine($"{i + 1}. {salesCustomerName[i]} ");
Console.WriteLine();
}
while (true) {
Console.WriteLine("Please choose a Customer name:");
Console.WriteLine();
return Console.ReadLine();
}
}
public static int GetItemNumber() {
ListAllItems();
int counter = itemDescription.Count;
string number;
int item = 1;
bool keepGoing = true;
while (keepGoing){
Console.WriteLine("Please enter an item number:");
number = Console.ReadLine();
item = Convert.ToInt32(number);
if (item < 1 || item > counter) {
Console.WriteLine();
Console.WriteLine($"Please enter a valid item number between 1 and {counter}.");
Console.WriteLine();
} else {
keepGoing = false;
}
}
return item -1;
}
public static string GetMenuOption() {
Console.WriteLine("=========== Main Menu ===========");
Console.WriteLine("1. Go Shopping");
Console.WriteLine("2. Management Menu");
Console.WriteLine("Q. Quit the program");
Console.WriteLine();
Console.Write("Please enter your choice: ");
return Console.ReadLine();
}
public static string GetManagementMenuOption() {
Console.WriteLine();
Console.WriteLine("=========== Management Menu =====");
Console.WriteLine("1. Browse sales receipts");
Console.WriteLine("2. View all items");
Console.WriteLine("3. Add an item");
Console.WriteLine("4. Change Quantity of Items on Hand");
Console.WriteLine("5. Change Prices of Items");
Console.WriteLine("Q. Return to Main Menu");
Console.WriteLine();
Console.Write("Please enter your choice: ");
return Console.ReadLine();
}
public static double GetPrice() {
while (true) {
Console.WriteLine();
Console.WriteLine("Enter the price of the item (x.xx):");
string entry = Console.ReadLine();
double value;
if (!double.TryParse(entry, out value) || value <= 0){
Console.WriteLine("Please enter a valid price value.");
} else {
return Convert.ToDouble(entry);
}
}
}
public static string GetShoppingOption() {
Console.WriteLine();
Console.WriteLine("=========== Shopping Menu ===========");
Console.WriteLine("1. Choose an item");
Console.WriteLine("Q. Checkout");
Console.WriteLine();
Console.Write("Please enter your choice: ");
return Console.ReadLine();
}
public static int GetQuantity() {
while (true){
Console.WriteLine();
Console.WriteLine("Please enter the item quantity:");
string entry = Console.ReadLine();
int value = 0;
if (Int32.TryParse(entry, out value) && Convert.ToInt32(entry) > 0){
return Convert.ToInt32(entry);
}
else{
Console.WriteLine("Please enter a valid value.");
}
}
}
private static void GoShopping() {
Console.WriteLine();
Console.WriteLine("Please enter the name of the shopper:");
while (true) {
string name = Console.ReadLine();
string entry = GetShoppingOption();
switch (entry) {
case "1" :
int item = GetItemNumber();
if (item != -1) {
AddItemToCart(name, item);
SaveData();
}
break;
case "Q" :
PrintTotalCart(name);
return;
}
}
}
public static void ListAllItems() {
Console.WriteLine();
Console.WriteLine("============== All Items ============= ");
for (int i = 0; i < itemDescription.Count; i++) {
Console.WriteLine($"{i + 1}. {itemDescription[i]} costs {itemPrice[i]:C2}. There are {itemQoh[i]} available.");
Console.WriteLine();
}
}
public static void Management() {
while (true) {
string entry = GetManagementMenuOption();
if (entry.ToLower() == "q") {
return;
} else {
switch (entry) {
case "1" :
string customer = GetCustomerName();
PrintTotalCart(customer);
break;
case "2" :
ListAllItems();
break;
case "3" :
AddItemToStore();
break;
case "4" :
ChangeQoh();
break;
case "5" :
ChangePrice();
break;
default:
Console.WriteLine("Please make a valid selection");
break;
}
}
}
}
public static void PrintTotalCart(string name) {
Console.WriteLine();
Console.WriteLine($"======= All Purchases for {name} =======" );
double total_cost= 0;
for (int i = 0; i < salesCustomerName.Count; i++) {
if(salesCustomerName[i] == name) {
total_cost += salesQuantity[i] *salesItemPrice[i];
Console.WriteLine($"{salesQuantity[i]} {salesItemDescription[i]} at the price of {salesItemPrice[i]:C2} each for a total of {salesQuantity[i] *salesItemPrice[i]:C2}.");
}
}
Console.WriteLine();
Console.WriteLine($"Total cost of the cart for {name} is {total_cost:C2}.");
}
public static void ReadData() {
ReadItems();
ReadSales();
}
public static void ReadItems() {
var filename = "items.csv";
if (File.Exists(filename)) {
using (var reader = new StreamReader(filename)) {
reader.ReadLine();
while (!reader.EndOfStream) {
var line = reader.ReadLine();
var values = line.Split(",");
itemDescription.Add(values[0]);
itemPrice.Add(Convert.ToDouble(values[1]));
itemQoh.Add(Convert.ToInt32(values[2]));
}
}
}else {
Console.WriteLine($"{filename} not found");
}
}
public static void ReadSales() {
var filename = "sales.csv";
if (File.Exists(filename)) {
using (var reader = new StreamReader(filename)) {
reader.ReadLine();
while (!reader.EndOfStream) {
var line = reader.ReadLine();
var values = line.Split(",");
salesCustomerName.Add(values[0]);
salesItemDescription.Add(values[1]);
salesItemPrice.Add(Convert.ToDouble(values[2]));
salesQuantity.Add(Convert.ToInt32(values[3]));
}
}
}else {
Console.WriteLine($"{filename} not found");
}
}
public static void SaveData() {
SaveItems();
SaveSales();
}
public static void SaveItems() {
string filename = "items.csv";
using (var writer = new StreamWriter(filename)) {
writer.WriteLine("Item_Description,Item_Price,Qoh");
for (int i = 0; i < itemDescription.Count; i++) {
writer.WriteLine($"{itemDescription[i]},{itemPrice[i]},{itemQoh[i]}");
}
}
}
public static void SaveSales() {
string filename = "sales.csv";
using (var writer = new StreamWriter(filename)) {
writer.WriteLine("Customer_Name,Item_Description,Item_Price,Sales_Qty");
for (int i = 0; i < salesCustomerName.Count; i++) {
writer.WriteLine($"{salesCustomerName[i]},{salesItemDescription[i]},{salesItemPrice[i]},{salesQuantity[i]}");
}
}
}
}
This is the error that I receive each time that I enter the backspace.

Array won't output anything

Doing a school activity but I hit a roadblock. I can't seem to make the program output the arrays I've entered. The output works when the variables inside the PlayerCharacter() and Mage() were all regular variables. Then I turned them into arrays because that was what was required of us, but then it started not showing anything during output.
using System;
namespace Summative
{
class PlayerCharacter
{
string[] name = new string[10];
int[] life = new int[10];
char[] gender = new char[10];
public int i = 0;
public int x = 0;
public PlayerCharacter()
{
Console.Write("Enter character name: ");
this.Name = Console.ReadLine();
Console.Write("Enter Life: ");
this.Life = int.Parse(Console.ReadLine());
Console.Write("Enter Gender (M/F): ");
this.Gender = char.Parse(Console.ReadLine());
i++;
}
public string Name
{
get
{
return name[i];
}
set
{
name[i] = value;
}
}
public int Life
{
get
{
return life[i];
}
set
{
life[i] = value;
}
}
public char Gender
{
get
{
return gender[i];
}
set
{
if (value == 'M' || value == 'F')
{
gender[i] = value;
}
else
{
Console.WriteLine("Invalid Gender!");
}
}
}
public virtual void Output()
{
Console.WriteLine("Name: " + string.Join(",", Name));
Console.WriteLine("Life: " + string.Join(",", Life));
Console.WriteLine("Gender: " + string.Join(",", Gender));
}
}
class Mage : PlayerCharacter
{
string[] element = new string[10];
int[] mana = new int[10];
public Mage() : base()
{
Console.Write("Enter element: ");
this.Elements = Console.ReadLine();
Console.Write("Enter mana: ");
this.Mana = int.Parse(Console.ReadLine());
}
public string Elements
{
get
{
return element[x];
}
set
{
element[x] = value;
}
}
public int Mana
{
get
{
return mana[x];
}
set
{
mana[x] = value;
}
}
public override void Output()
{
base.Output();
Console.WriteLine("Element: " + string.Join(",", Elements));
Console.WriteLine("Mana: " + string.Join(",", Mana));
x++;
}
}
class Rogue : PlayerCharacter
{
string faction;
float energy;
public Rogue() : base()
{
Console.Write("Enter faction name: ");
this.Faction = Console.ReadLine();
Console.Write("Enter energy: ");
this.Energy = float.Parse(Console.ReadLine());
}
public string Faction
{
get
{
return faction;
}
set
{
faction = value;
}
}
public float Energy
{
get
{
return energy;
}
set
{
energy = value;
}
}
public override void Output()
{
base.Output();
Console.WriteLine("Faction: " + Faction);
Console.WriteLine("Energy: " + Energy);
}
}
class Fighter : PlayerCharacter
{
string weapon;
double strength;
public Fighter() : base()
{
Console.Write("Enter weapon name: ");
this.Weapon = Console.ReadLine();
Console.Write("Enter strength: ");
this.Strength = double.Parse(Console.ReadLine());
}
public string Weapon
{
get
{
return weapon;
}
set
{
weapon = value;
}
}
public double Strength
{
get
{
return strength;
}
set
{
strength = value;
}
}
public override void Output()
{
base.Output();
Console.WriteLine("Weapon: " + Weapon);
Console.WriteLine("Strength: " + Strength);
}
}
class TestPlayer
{
static void Main(string[] args)
{
PlayerCharacter character;
CharacterCreator:
int switchchoice = 0;
Console.WriteLine("Select a class");
Console.WriteLine("1. Mage");
Console.WriteLine("2. Rogue");
Console.WriteLine("3. Fighter");
Console.WriteLine("0. Exit");
Console.Write("Enter Choice: ");
start:
try
{
switchchoice = int.Parse(Console.ReadLine());
}
catch
{
Console.Clear();
Console.WriteLine("Invalid Input! Please Try Again.");
goto CharacterCreator;
}
switch (switchchoice)
{
case 1:
Console.Clear();
character = new Mage();
character.Output();
break;
case 2:
Console.Clear();
character = new Rogue();
character.Output();
break;
case 3:
Console.Clear();
character = new Fighter();
character.Output();
break;
case 0:
Console.Clear();
goto start;
default:
Console.Clear();
Console.WriteLine("Invalid Input! Please Try Again.");
goto CharacterCreator;
}
int choice;
int y = 0;
int z;
int i = character.i;
int x = character.x;
Console.Clear();
Console.WriteLine("Player Character Designer");
Console.WriteLine("1. Create");
Console.WriteLine("2. View");
Console.Write("Enter Choice: ");
choice = int.Parse(Console.ReadLine());
if (choice == 1)
{
goto CharacterCreator;
}
else if (choice == 2)
{
z = i;
i = 0;
x = 0;
do
{
character.Output();
Console.WriteLine(" ");
y++;
i++;
x++;
} while (y != z);
}
}
}
}

Answer: Use a list
The main issue is that you should add the created characters in a list rather than keeping arrays of attributes inside characters. This lets you loop over each character and print all the information in turn. I.e.
var myCharacters = new List<PlayerCharacter>();
...
myCharacters.Add(myCharacter);
Other Issues
Constructors & object design
Inject parameters in constructors whenever possible, and use auto-properties when applicable. This helps avoid complex logic in the constructors, and reduces the size of the classes. Let the creator of the characters be responsible for reading the necessary parameters. I would also prefer to separate the construction of the information-string and the output, that way you can output the same information to a log file or whatever, and not just to the console. I.e:
class PlayerCharacter
{
public string Name { get; }
public int Life { get; }
public string Gender { get; }
public PlayerCharacter(string name, int life, string gender)
{
Name = name;
Life = life;
Gender = gender;
}
public override string ToString()
{
return $"Name: {Name}, Life {Life}, Gender: {Gender}";
}
}
Control flow
Use loops instead of goto. While I think there are cases where goto is the best solution, they are rare, and the general recommendation is to use loops for control flow i.e. something like this pseudocode
MyOptions option;
var myCharacters = new List<PlayerCharacter>();
do{
myCharacters.Add(ReadCharacter());
option = ReadOption();
}
while(option != MyOptions.ViewCharacters);
PrintCharacters(myCharacters);
Split code into methods
Move code into logical functions, especially for repeated code. This helps to make it easier to get a overview of the program. For example, for reading numbers it is better to use the TryParse function than trying to catch exceptions, and put it in a method to allow it to be reused:
int ReadInteger()
{
int value;
while (!int.TryParse(Console.ReadLine(), out value))
{
Console.WriteLine("Invalid Input! Please Try Again.");
}
return value;
}

Value of class member List<> not updating in C#

i have following code, there are 2 classes, 'Account' and 'Program'. What i am trying to do here is just simulating a simple bank management console system where user can create instances of 'Account' class ( which are being stored in List<Account> accounts member variable ( Data structure ). I'am confused about why the account list is not updating when i add the Account object and access the accounts list in other methods ( like Login() ).
Please guide me. Thanks
namespace Banking_System
{
class Account
{
public Account(string f, string l, string id, string ph, string t, int bal, int no)
{
fname = f;
lname = l;
cnic = id;
phone = ph;
amount = bal;
acc_no = no;
}
public string fname { get; set; }
public string lname { get; set; }
public string cnic { get; set; }
public string phone { get; set; }
public string type { get; set; }
public int amount { get; set; }
public int acc_no { get; set; }
}
class Program
{
//private Program bank_instance;
private List<Account> accounts {get;set;}
static void Main(string[] args)
{
while (true)
{
Console.Clear();
Console.WriteLine("Welcome to the bank");
Console.WriteLine("====================\n");
Console.WriteLine("Please choose an option from the list below");
Console.WriteLine("============================================");
Console.WriteLine("e ñ Existing account");
Console.WriteLine("n ñ New account");
Console.WriteLine("c ñ close");
string input = Console.ReadLine();
Account acc;
Program bank_instance = new Program();
bank_instance.accounts = new List<Account>();
if (input == "n" || input == "N")
{
bank_instance.NewAccount();
int x = bank_instance.accounts.Count;
}
else if (input == "e" || input == "E")
{
acc = bank_instance.login();
if (acc == null)
{
Console.WriteLine("Account does not exists !");
continue;
}
bank_instance.operate(acc);
}
else if (input == "c" || input == "C")
{
Console.WriteLine("Goodbye !");
break;
}
else
{
Console.WriteLine("You entered the wrong character \nPlease enter the correct one");
}
}
}
private void operate(Account acc)
{
string input = null;
while (true)
{
Console.Clear();
Console.WriteLine("Choose an operation:");
Console.WriteLine("1) Check balance");
Console.WriteLine("2) Deposit cash");
Console.WriteLine("3) Withdraw cash");
Console.WriteLine("4) Close");
input = Console.ReadLine();
if (input == "1")
{
check_balance(acc.acc_no);
}
else if (input == "2")
{
deposite_cash(acc);
}
else if (input == "3")
{
withdraw(acc);
}
else if (input == "4")
{
Console.WriteLine("Goodbye !");
break;
}
else
{
Console.WriteLine("You entered the wrong character \nPlease enter the correct one");
}
}
}
private void check_balance(int inp)
{
foreach (Account a in accounts)
{
if (a.acc_no.Equals(inp))
{
Console.Write("Your Balance is : " + a.amount);
}
}
}
private void deposite_cash(Account acc)
{
int bal = 0;
Console.WriteLine("Enter the amount");
bal = int.Parse(Console.ReadLine());
if (bal <= 0)
{
Console.WriteLine("Please enter valid amount");
}
else
{
acc.amount += bal;
Console.WriteLine("Amount has been deposited successfully!");
}
}
private void withdraw(Account acc)
{
int bal = 0;
Console.WriteLine("Enter the amount");
bal = int.Parse(Console.ReadLine());
if (bal <= 0)
{
Console.WriteLine("Please enter valid amount");
}
else
{
acc.amount -= bal;
Console.WriteLine("Amount has been Withdrawn successfully!");
}
}
private void NewAccount()
{
Console.WriteLine("================================================================");
Console.WriteLine("Please fill in the following information to create a New Account");
Console.WriteLine("================================================================");
Console.WriteLine("1) First name");
string f_name = Console.ReadLine();
Console.WriteLine("2) Last Name");
string l_name = Console.ReadLine();
int check_length;
int id_card_no;
do
{
Console.WriteLine("3) National ID card Number (xxxxxxxxxxxxx)");
check_length = Console.ReadLine().Length;
id_card_no = Convert.ToInt32(check_length);
}
while (check_length != 13);
Console.WriteLine("4) Contact Number");
System.Int64 contact_no = Convert.ToInt64(Console.ReadLine());
Console.WriteLine("5) Account type - (Current , Saving) ");
string account_type = Console.ReadLine();
Console.WriteLine("6) Initial amount to deposit (Minimum 50,000) ");
int ini_amount = int.Parse(Console.ReadLine());
if (ini_amount <= 50000)
{
Console.WriteLine(ini_amount);
}
else
{
Console.WriteLine("teri mehrbani");
}
// Generate a random number
Random rand = new Random();
Account acc = new Account(f_name, l_name, id_card_no.ToString(), contact_no.ToString(), account_type, ini_amount, rand.Next(00000, 55555));
accounts.Add(acc);
Console.WriteLine("Account created ! you Account number is : " + acc.acc_no + "\n Press any key to continue");
Console.Read();
}
private Account login()
{
int inp = 0;
Console.WriteLine("Enter your account number");
inp = int.Parse(Console.ReadLine());
foreach (Account a in accounts)
{
if (a.acc_no.Equals(inp))
{
Console.Write("Welcome !");
return a;
}
}
return null;
}
}
}

Every time you run your while loop, you're creating a new instance of bank_instance, and you initialize the bank_instance.accounts to a new List, which effectively erases all the data you've collected.
Instead, try initializing these things outside your loop, so they exist the next time through.
For example:
static void Main(string[] args)
{
Program bank_instance = new Program();
bank_instance.accounts = new List<Account>();
while (true)
{

I need some help running a C# menu for customers

I have completed most of my project but I need help calling a function. On line 38 where it says DisplayCustomerMenu(customer);, I failed to call out the function. What is the correct way to call out this function?
using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
namespace LabMenu {
public static class Program
{
public static PreferredCustomer[] preferredCustomers;
public static Customer customer;
public static string firstName { get; set; }
public static string lastName { get; set; }
public static double flashlight { get; private set; }
public static double iphone { get; private set; }
public static double printer { get; private set; }
public static double laptop { get; private set; }
public static double playstation { get; private set; }
static void Main(string[] args)
{
DisplayMainMenu();
}
public static void DisplayMainMenu()
{
do
{
Console.Clear();
Console.WriteLine("Welcome to the Menu!");
Console.WriteLine("0. Quit");
Console.WriteLine("1. Make An Order");
Console.WriteLine("2. Manage Customers");
switch (ConsoleHelper.ReadInt32(0, 1))
{
case 0: return;
case 1: DisplayCustomersMenu(); break;
};
} while (true);
}
public static void DisplayCustomersMenu()
{
do
{
Console.Clear();
Console.WriteLine("Make An Order");
Console.WriteLine($"What is your first name?");
firstName = Console.ReadLine();
Console.WriteLine($"What is your last name?");
lastName = Console.ReadLine();
string strTarget = String.Format("Hello, {0} {1}.", firstName, lastName);
Console.WriteLine("{0}{1}{0}",
Environment.NewLine, strTarget);
Console.WriteLine("1) Select Customer");
Console.WriteLine("0) Return to Manage Customers");
switch (ConsoleHelper.ReadInt32(0, 3))
{
case 0: return;
case 1:
{
string enteredID = "";
do
{
Console.WriteLine("Enter the customer ID: ");
enteredID = Console.ReadLine();
//var id = ConsoleHelper.ReadInt32("Invalid customer ID", 0, Int32.MaxValue);
foreach (var id in preferredCustomers)
{
if (id.CustomerID == enteredID)
{
DisplayCustomerMenu(customer); //Is this correct?
}
else
{
Console.WriteLine("You are not in the database");
}
}
} while (true);
//var customer = SelectCustomer();
//if (customer != null)
// DisplayCustomerMenu(customer);
//else
// Console.WriteLine("Customer not found");
//break;
};
};
} while (true);
}
public static void DisplayCustomerMenu(Customer customer)
{
do
{
Console.Clear();
Console.WriteLine("Manage Customers");
Console.WriteLine($"Custumer name, ID and current order");
Console.WriteLine("1) Add to Order");
Console.WriteLine("2) Remove from Order");
Console.WriteLine("3) Finalize Order");
Console.WriteLine("0) Return to Manage Customers");
switch (ConsoleHelper.ReadInt32(0, 3))
{
case 0: return;
case 1: AddToOrder(customer); break;
case 2: RemoveFromOrder(customer); break;
case 3: FinalizeOrder(customer); break;
};
} while (true);
}
public static void AddToOrder(Customer id)
{
Console.Clear();
int numberOfInputForFlashlight = 0;
int numberOfInputForIphone = 0;
int numberOfInputForPrinter = 0;
int numberOfInputForLaptop = 0;
int numberOfInputForPlaystation = 0;
int myint = -1;
while (myint != 0)
{
string group;
Console.WriteLine("Add To Order");
Console.WriteLine("1) Flashlight");
Console.WriteLine("2) iPhone 7");
Console.WriteLine("3) Printer");
Console.WriteLine("4) Dell Laptop");
Console.WriteLine("5) Playstation 4");
Console.WriteLine("6) View Total");
Console.WriteLine("[Press 0 to quit]");
group = Console.ReadLine();
myint = Int32.Parse(group);
switch (myint)
{
case 0:
break;
case 1:
double input1;
string inputString1;
Console.WriteLine("How many flashlights do you want?");
inputString1 = Console.ReadLine();
input1 = Convert.ToDouble(inputString1);
flashlight += input1;
numberOfInputForFlashlight++;
break;
case 2:
double input2;
string inputString2;
Console.WriteLine("How many iPhone 7 do you want?");
inputString2 = Console.ReadLine();
input2 = Convert.ToDouble(inputString2);
iphone += input2;
numberOfInputForIphone++;
break;
case 3:
double input3;
string inputString3;
Console.WriteLine("How many Printers do you want?");
inputString3 = Console.ReadLine();
input3 = Convert.ToDouble(inputString3);
printer += input3;
numberOfInputForPrinter++;
break;
case 4:
double input4;
string inputString4;
Console.WriteLine("How many Dell Laptops do you want?");
inputString4 = Console.ReadLine();
input4 = Convert.ToDouble(inputString4);
laptop += input4;
numberOfInputForLaptop++;
break;
case 5:
double input5;
string inputString5;
Console.WriteLine("How many Playstion 4 do you want?");
inputString5 = Console.ReadLine();
input5 = Convert.ToDouble(inputString5);
playstation += input5;
numberOfInputForPlaystation++;
break;
case 6:
Console.WriteLine("Flashlist quantity is {0}", flashlight.ToString("F"));
Console.WriteLine("iPhone 7 quantity is {0}", iphone.ToString("F"));
Console.WriteLine("Printer quantity is {0}", printer.ToString("F"));
Console.WriteLine("Dell Laptop quantity is {0}", laptop.ToString("F"));
Console.WriteLine("Playstation 4 quantity is {0}", playstation.ToString("F"));
Console.WriteLine("Flashlight total is {0}", (flashlight * 15.00).ToString("C"));
Console.WriteLine("iPhone 7 total is {0}", (iphone * 700.00).ToString("C"));
Console.WriteLine("Printer total is {0}", (printer * 80.00).ToString("C"));
Console.WriteLine("Dell Laptop total is {0}", (laptop * 500.00).ToString("C"));
Console.WriteLine("Playstation 4 total is {0}", (playstation * 380.00).ToString("C"));
break;
default:
Console.WriteLine("Incorrect input", myint);
break;
}
}
}
public static void FinalizeOrder(Customer customer)
{
do
{
Console.Clear();
} while (true);
}
static void RemoveFromOrder(Customer customer)
{
do
{
Console.Clear();
Console.WriteLine("Remove From Order:");
Console.WriteLine("1) Flashlight total is {0}", (flashlight * 15.00).ToString("C"));
Console.WriteLine("2) iPhone 7 total is {0}", (iphone * 700.00).ToString("C"));
Console.WriteLine("3) Printer total is {0}", (printer * 80.00).ToString("C"));
Console.WriteLine("4) Dell Laptop total is {0}", (laptop * 500.00).ToString("C"));
Console.WriteLine("5) Playstation 4 total is {0}", (playstation * 380.00).ToString("C"));
switch (ConsoleHelper.ReadInt32(0, 5))
{
case 0: break;
case 1:
double input1;
string inputString1;
Console.WriteLine("How many flashlights do you want to remove?");
inputString1 = Console.ReadLine();
input1 = Convert.ToDouble(inputString1);
flashlight -= input1;
break;
case 2:
double input2;
string inputString2;
Console.WriteLine("How many iPhone 7 do you remove?");
inputString2 = Console.ReadLine();
input2 = Convert.ToDouble(inputString2);
iphone -= input2;
break;
case 3:
double input3;
string inputString3;
Console.WriteLine("How many Printers do you remove?");
inputString3 = Console.ReadLine();
input3 = Convert.ToDouble(inputString3);
printer -= input3;
break;
case 4:
double input4;
string inputString4;
Console.WriteLine("How many Dell Laptops do you remove?");
inputString4 = Console.ReadLine();
input4 = Convert.ToDouble(inputString4);
laptop += input4;
break;
case 5:
double input5;
string inputString5;
Console.WriteLine("How many Playstion 4 do you remove?");
inputString5 = Console.ReadLine();
input5 = Convert.ToDouble(inputString5);
playstation += input5;
break;
};
} while (true);
}
public static void GetData()
{
using (var reader = new StreamReader("CustomerInfo.txt"))
{
var index = 0;
preferredCustomers = new PreferredCustomer[5];
while (!reader.EndOfStream)
{
var line = reader.ReadLine().Split(':');
var name = line[0];
var address = line[1];
var phone = line[2];
var id = line[3];
var email = line[4];
var spentAmount = Convert.ToInt32(line[5]);
var onEmailList = Convert.ToBoolean(line[6]);
preferredCustomers[index] = new PreferredCustomer(name, address, phone, id, email, spentAmount,
onEmailList);
index++;
}
}
}
public static void UpdateData()
{
using (var writer = new StreamWriter("CustomerInfo2.txt"))
{
for (var i = 0; i < 5; i++)
{
var name = preferredCustomers[i].CustomerName;
var address = preferredCustomers[i].Address;
var phone = preferredCustomers[i].Phone;
var id = preferredCustomers[i].CustomerID;
var email = preferredCustomers[i].CustomerID;
var updatedSpentAmount = preferredCustomers[i].CalcAmount();
var onEmailList = preferredCustomers[i].OnEmailList;
writer.WriteLine("{0}:{1}:{2}:{3}:{4}:{5}:{6}",
name, address, phone, id, email, updatedSpentAmount, onEmailList == true ? "true" : "false");
}
}
}
public static string UserChoice()
{
var userChoice = Console.ReadLine();
while (userChoice !="1" && userChoice != "2")
{
Console.WriteLine("Wrong Entry. Try Again.");
DisplayMenu();
userChoice = Console.ReadLine();
}
return userChoice;
}
public static void DisplayMenu()
{
Console.WriteLine("1. Display Customer Info");
Console.WriteLine("2. Update Customer Info");
Console.Write("Please enter your choice: ");
}
}
}
Also here is 'Customer.cs'
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
namespace LabMenu
{
public class Customer : Person
{
public Customer(string name, string address, string phone, string id, string email, int spentAmount,
bool onEmailList)
: base(name, address, phone)
{
CustomerID = id;
CustomerEmail = email;
SpentAmount = spentAmount;
OnEmailList = onEmailList;
}
public string CustomerID { get; set; }
public string CustomerEmail { get; set; }
public int SpentAmount { get; set; }
public bool OnEmailList { get; set; }
public virtual double CalcAmount()
{
return SpentAmount;
}
}
}
Person.cs
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
namespace LabMenu
{
public class Person
{
public Person(string name, string address, string phone)
{
CustomerName = name;
Address = address;
Phone = phone;
}
PreferredCustomers.cs
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
namespace LabMenu
{
public class PreferredCustomer : Customer
{
public PreferredCustomer(string name, string address, string phone, string id,
string email, int spentAmount, bool onEmailList)
: base (name, address, phone, id, email, spentAmount, onEmailList)
{
DiscountLevel = SetDiscountLevel();
}
public readonly decimal DiscountLevel;
public decimal SetDiscountLevel()
{
int range = SpentAmount / 500;
switch (range)
{
case 0:
return 0;
case 1:
return 0.05m;
case 2:
return 0.05m;
case 3:
return 0.08m;
default:
return 0.1m;
}
}
public double GetDiscount()
{
return SpentAmount * (double)DiscountLevel;
}
public override double CalcAmount()
{
return base.CalcAmount() - GetDiscount();
}
public override string ToString()
{
return
String.Format(
"CustomerID: {0}\nCustomer Name: {1}\nCustomerAddress: {2}\n" +
"Customer Phone: {3} \nCustomer Email: {4}" +
"Customer Spending: {5:C2}\nCustomer On Email List: {6}",
CustomerID, CustomerName, Address, Phone, CustomerEmail, SpentAmount, OnEmailList
);
}
}
}
public string CustomerName { get; set; }
public string Address { get; set; }
public string Phone { get; set; }
}
}
CustomerInfo.txt
John Candy:123 10th st, Allen, TX 78714:972-555-0000:A0000001:email#gmail.com:2500:true
Robert Smith:456 15th st, Austin, TX 78504:512-456-1000:A0000002:bobsmith#gmail.com:2500:true
Bill Clinton:2004 44th st, Washington, DC 20001:202-456-2222:A0000003:williamc#gmail.com:495:false
Patricia Garner:123 16th st, Washington, DC 20002:202-555-6666:A0000004:garnerp#gmail.com:1200:true
Pablo Corbin:777 2th st, Houston, TX 77002:832-100-2000:A0000005:pcorzbin#yahoo.com:1750:false

Delete Function for a Menu Driven Program

I'm having trouble implementing the delete function, any help would be appreciated. I have a list called memberlist that stores the persons name and number. I have everything working except the delete function. Any help would be appreciated.
public class Person
{
private string name;
private string phoneNumber;
public string Name
{
set { name = value; }
get { return name; }
}
public string PhoneNumber
{
set { phoneNumber = value; }
get { return phoneNumber; }
}
public void PrintInfo()
{
Console.WriteLine();
Console.WriteLine(" Name: {0}", name);
Console.WriteLine(" Phone Number: {0}", phoneNumber);
Console.WriteLine();
}
public void SaveASCII(ref StreamWriter output)
{
output.WriteLine(name);
output.WriteLine(phoneNumber);
}
public void LoadASCII(ref StreamReader input)
{
name = input.ReadLine();
phoneNumber = input.ReadLine();
}
}
public class membershipList
{
public Person[] ML = null;
public void AddMember(Person p)
{
if (ML == null)
{
ML = new Person[1];
ML[0] = p;
}
else
{
Person[] temp = ML;
ML = new Person[temp.Length + 1];
for (int i = 0; i < temp.Length; ++i)
{
ML[i] = new Person();
ML[i] = temp[i];
}
ML[temp.Length] = new Person();
ML[temp.Length] = p;
temp = null;
}
}
public void DeleteMember(string name)
{
if (ML == null)
{
Console.WriteLine(name + " had not been added before.");
}
else
{
DeleteMember(name);
}
}
public void PrintAll()
{
if (ML != null)
foreach (Person pers in ML)
pers.PrintInfo();
else Console.WriteLine("Then list is empty");
}
public void Search(string p)
{
if (ML != null)
{
foreach (Person pers in ML)
{
if (pers.Name.ToLower().CompareTo(p.ToLower()) == 0)
{
Console.WriteLine("1 Record Found:");
pers.PrintInfo();
break;
}
}
Console.WriteLine("Record not found.");
}
else Console.WriteLine("Then list is empty.");
}
public void ReadASCIIFile()
{
StreamReader input = new StreamReader("memberlist.dat"); ;
try
{
int num = Convert.ToInt32(input.ReadLine());
ML = new Person[num];
for (int i = 0; i < num; ++i)
{
ML[i] = new Person();
ML[i].LoadASCII(ref input);
}
input.Close();
}
catch (FormatException e)
{
Console.WriteLine(e.Message);
input.Close();
}
}
public void SaveASCIIFile()
{
StreamWriter output = new StreamWriter("memberlist.dat");
output.WriteLine(ML.Length);
foreach (Person pers in ML)
{
pers.SaveASCII(ref output);
}
output.Close();
}
}
class Program
{
static void Main(string[] args)
{
membershipList ML = new membershipList();
ML.ReadASCIIFile();
string option;
do
{
// Console.Clear();
Console.WriteLine();
Console.WriteLine();
Console.WriteLine("MemberShip List MENU");
Console.WriteLine();
Console.WriteLine(" a. Add");
Console.WriteLine(" b. Seach");
Console.WriteLine(" c. Delete");
Console.WriteLine(" d. Print All");
Console.WriteLine(" e. Exit");
Console.WriteLine();
Console.Write("option: ");
option = Console.ReadLine().ToLower();
switch (option)
{
case "a":
Person np = new Person();
Console.Write("Enter Name: ");
np.Name = Console.ReadLine();
Console.Write("Enter PhoneNumber: ");
np.PhoneNumber = Console.ReadLine();
ML.AddMember(np);
break;
case "b":
Console.Write("Enter Name: ");
string name = Console.ReadLine();
ML.Search(name);
break;
case "c":
Console.Write("Enter Name to be Deleted:");
ML.DeleteMember(name);
break;
case "d":
ML.PrintAll();
break;
case "e":
ML.SaveASCIIFile();
Console.WriteLine("BYE...... ");
break;
default:
Console.WriteLine("Invalid Option");
break;
}
} while (option.ToLower() != "d");
}
}

If you change Person[] to List as suggested by #Wiktor Zychla, this code should work well:
public static List<Person> ML = new List<Person>();
public static void DeleteMember(string name)
{
var deleteMe = ML.Find(p => p.Name == name);
if (deleteMe == null)
{
Console.WriteLine(name + " had not been added before.");
}
else
{
ML.Remove(deleteMe);
}
}

Deleting is similar to what you have for adding. The suggestion to change to using List is a good one, but it CAN be localized to your DeleteMember() routine.
Try something like this (edited):
public void DeleteMember(string name)
{
if (ML == null)
{
Console.WriteLine(name + " had not been added before.");
}
else
{
int memberIndex = ML.ToList().FindIndex(p => p.Name == name);
if (memberIndex == -1)
{
Console.WriteLine(name + " had not been added before.");
return;
}
List<Person> tmp = new List<Person>(ML);
tmp.RemoveAt(memberIndex);
ML = tmp.ToArray();
}
}