get me out of this i m tried a lot but confused

Question

0.00/5 (No votes)

See more:

C#

class Program
    {
        string MobileName;
        bool Nokia5233;
        bool Nokia2690;
        bool SonyP910i;
        int Price;
        public void ListOfMobiles()
        {
           
            
            Console.WriteLine("The first Model Is Nokia5233");
            
            Console.WriteLine("The second Model Is SonyP910i");
            
            Console.WriteLine("The third Model is Nokia2690");

            if (Nokia2690)
            {
                Console.WriteLine("The price is 4900");
                //Price = Convert.ToInt32(Console.ReadLine());
            }
            else
            {

            }
            if (Nokia5233)
            {
                Console.WriteLine("The Price is 7000");
                //Price = Convert.ToInt32(Console.ReadLine());
            }
            else
            {


            }
            
        }
        
        public void ChooseMobile()
        {
            Console.WriteLine("Enter Mobile Name");
            //Console.WriteLine("/n The Model Number is: ");
            MobileName = Console.ReadLine();
        }
        public void DisplayMobileDetails()
        {
            Console.WriteLine("The Mobile name is {0}", MobileName);
            Console.WriteLine("The price is {0}", Price );
        }
        static void Main(string[] args)
        {
            Program prm = new Program();
            prm.ListOfMobiles();
            prm.ChooseMobile();
            
            prm.DisplayMobileDetails();
            Console.ReadLine();
        }
    }
}

i wrote this code nd want the result but the desire result didn't came out i want when i enter mobile name it show mobile name as well as its price

Posted 6-Nov-11 21:26pm

Ankur Kaushal

Updated 6-Nov-11 21:31pm

P.Salini

v2

Add a Solution

3 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Pranit Kothari · Answer 1 · 2011-11-06T21:39:00

C#

if (Nokia2690)
           {
               Console.WriteLine("The price is 4900");
               //Price = Convert.ToInt32(Console.ReadLine());
           }
           else
           {

           }
           if (Nokia5233)
           {
               Console.WriteLine("The Price is 7000");
               //Price = Convert.ToInt32(Console.ReadLine());
           }
           else
           {


           }

Who is actually setting your Nokia2690,Nokia5233 bool values. And you do not have anything in else part.So no output.

Regards

OriginalGriff · Answer 2 · 2011-11-06T21:39:00

You need to check the entered details against you list of known items. In your case, it is probably easiest to hardwire it, but in the real world we wouldn't - for beginners though, it will give you an idea.

C#

public void ChooseMobile()
    {
    bool selected = false;
    do
        {
        Console.WriteLine("Enter Mobile Name");
        //Console.WriteLine("/n The Model Number is: ");
        MobileName = Console.ReadLine();
        switch (MobileName)
            {
            case "Nokia5233":
                Nokia5233 = true;
                selected = true;
                break;
            case Nokia2690:
                Nokia2690 = true;
                selected = true;
                break;
            case SonyP910i:
                SonyP910i = true;
                selected = true;
                break;
            }
        } while (!selected);
    }

Timberbird · Answer 3 · 2011-11-06T21:44:00

If the code you provided is the complete code, it won't. Never. You see, you've declared members

C#

bool Nokia5233;
bool Nokia2690;
bool SonyP910i;

but you dont' set their values. So, they retain thair default values, which are false. So no if is gonna trigger in ListOfMobiles(). And Price value is never set as well - so what do you expect?
What you need here is to set corresponding boolean value (e.g. Nokia2690) in ChooseMobile() method depending on user input. Use switch for that. One more - and probably better - option is to turn your MobileName into property with switch in its set part that would tune your object (set corresponding bool value and price).

That'll help your solution work, no more. There are certanly other improvements that can be done, for example removing bool variables completely or replacing them by enum member

get me out of this i m tried a lot but confused

3 solutions

Solution 1

Solution 2

Solution 3

Add your solution here

Preview 0