Is this correct?

[nobiggieBIG]

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            int nos;
            int student;
            int result;
            int result2;

            nos = 100;
            student = 19;
            result = nos / student;
            result2 = 100 - (result * 19);

            Console.WriteLine(result2);

            Console.ReadKey();
            Console.WriteLine("Any key to end.");

        }
    }
}

A teacher has bought a packet of 100 sweets that she is going to share out equally between her 19 students. Because a single sweet cannot be shared, the teacher will keep what is not given to the children. The teacher will keep the minimum number of sweets possible.

Write a program to determine and output the number of sweets each child will receive. As a single sweet cannot be shared between pupils, your program must calculate the number of sweets per child and the number that the teacher keeps for herself.

 

Console outputs 5

[alexhelvetica]

Let's assume there are 20 kids in the class, each child would get 5 lollis (5*20=100, or alternatively 100/20=5)

 

Now if there is 19 students in the class, each student would still only get 5 lollies (19*5=95), since 5 (100-95=5) is not enough to give each student an extra lolli, each student will still get 5 lollies.

 

I don't like what you have name the lolli varible, lolli or sweet would be much clearer, I'd also recomend you make a comment giving a brief discription of what everything does, you are likely going to be marked on that too, not just the formular.

[nobiggieBIG]

Just now, Comic_Sans_MS said:

I don't like what you have name the lolli varible, lolli or sweet would be much clearer, I'd also recomend you make a comment giving a brief discription of what everything does, you are likely going to be marked on that too, not just the formular.

Yeah, i did that for the section in which the user imputs the number of students and number of sweets;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            int sweets;
            int student;
            int result;
            int result2;

            //User imputs number of sweets
            Console.WriteLine("Enter number of sweets.");
            sweets = Convert.ToInt32(Console.ReadLine());

            //User imputs the amount of students in the class
            Console.WriteLine("Enter number of students");
            student = Convert.ToInt32(Console.ReadLine());

            //The results
            result = sweets / student;
            result2 = 100 - (result * 19);

            //Amount of sweets the teacher will recieve
            Console.WriteLine("The teacher will get {0} sweets.", result2);

            //Ends string
            Console.WriteLine("Any key to end.");
            Console.ReadKey();

        }
    }
}

 

[straight_stewie]

4 hours ago, nobiggieBIG said:

A teacher has bought a packet of 100 sweets that she is going to share out equally between her 19 students. Because a single sweet cannot be shared, the teacher will keep what is not given to the children. The teacher will keep the minimum number of sweets possible.

Write a program to determine and output the number of sweets each child will receive. As a single sweet cannot be shared between pupils, your program must calculate the number of sweets per child and the number that the teacher keeps for herself.

What you're really looking for, to solve the general case, is Modulo. Modulo is division that returns the remainder of a division.

So the algorithm for calculating how much sweets each one gets then becomes:

// Setting up variables
int sweet = get_num_sweets();
int student = get_num_students();
int teacher_sweets;
int student_sweets;

// Calculating results
teacher_sweets = sweet % student;
student_sweets = sweet - teacher_sweets;
student_sweets /= student;

The algorithm that you have assumes that each student gets 19 candies and that there are less than 100 candies in total. Which brings me to the lesson of this thread: Never, ever, use "magic numbers" in your programs.

[rahul2141999]

6 hours ago, nobiggieBIG said:

//The results

result = sweets / student;

result2 = 100 - (result * 19);

it should be 

result2 = 100 - (result * student);

but code used by straight_stewie above is much more efficient.

[nobiggieBIG]

4 hours ago, straight_stewie said:

What you're really looking for, to solve the general case, is Modulo. Modulo is division that returns the remainder of a division.

So the algorithm for calculating how much sweets each one gets then becomes:

// Setting up variables
int sweet = get_num_sweets();
int student = get_num_students();
int teacher_sweets;
int student_sweets;

// Calculating results
teacher_sweets = sweet % student;
student_sweets = sweet - teacher_sweets;
student_sweets /= student;

The algorithm that you have assumes that each student gets 19 candies and that there are less than 100 candies in total. Which brings me to the lesson of this thread: Never, ever, use "magic numbers" in your programs.

Yeah, my lecturer knows c# but i dont think well enough, im beginner so its all a massive head ***