Member Avatar for lost_c++_dude

I keep on getting one error when I compile this. I can't figure out what is wrong with it. Please help.

//Write a program that asks the user to enter two integers.  
//The program should divide the first integer by the second and then display the resulting quotient and remainder.

#include <iostream.h>

//using namespace std;

int main()
{
	//Declare the variables

	int il, i2,  quotient, remainder;

// Obtain the data from the user

	cout<<"Input an integer followed by a return: ";
	cin>>i1;

	cout<<"Input an integer followed by a return: ";
	cin>>i2;
	
// Do the arithmetic

quotient= i1/i2;
remainder=i1 % i2;

// Output the results

    cout<<"\nThe quotient of " << i1 << " and "<< i2 <<"is"<< i1/i2;
	cin>>il;
	cout<<"The quotient of " << i1 << " and "<< i2 <<"is"<< i1/i2;
	cin>>i2;
	cout<<"\nThe remainder when "<<i1<<" is divided by "<<i2<<"is"<< remainder;


return 0;
}

error C2065: 'i1' : undeclared identifier
Error executing cl.exe.

  • 4 Contributors
  • 5 Replies
  • 136 Views
  • 16 Hours Discussion Span
  • Latest Post Latest Post by vegaseat

Dani AI

Generated

Quick diagnosis and the immediate fix: the compiler is complaining because the program uses a variable i1 that was never declared — the declaration in the original code used a lowercase letter l instead of the digit 1. and already pointed that out. Renaming the declared variable to match the one you read from, or better yet choosing clearer names, will remove that compile error.

A few practical improvements that go beyond the typo: 1) pick readable names (for example numerator and denominator) so 1 vs l confusion can’t happen; 2) remove any stray input statements left after output (the original code had extra input reads that look accidental and will make the program pause unexpectedly); 3) always check the denominator before dividing to avoid a run-time divide-by-zero; and 4) prefer modern headers (#include <iostream>) and either std:: qualification or a deliberate using declaration. ’s suggestion to use meaningful names is exactly the right idea here.

A minimal, safe pattern (illustrative) to follow:

#include <iostream>

int main() {
    int numerator = 0, denominator = 1;
    std::cout << "Enter numerator: ";
    if (!(std::cin >> numerator)) return 1;
    std::cout << "Enter denominator: ";
    if (!(std::cin >> denominator)) return 1;
    if (denominator == 0) {
        std::cout << "Error: division by zero\n";
        return 1;
    }
    int quot = numerator / denominator;
    int rem  = numerator % denominator;
    std::cout << "Quotient: " << quot << "\nRemainder: " << rem << '\n';
    return 0;
}

Final troubleshooting tips: enable compiler warnings (e.g., -Wall or /W4), use an editor/font that makes 1, l, and I distinct, and test with both normal and edge inputs (denominator zero). After renaming the variable and removing accidental extra cin lines, the compile error reported by should disappear.

Recommended Answers

All 5 Replies

Member Avatar for alc6379

Did you mean to have an il and and i1? I'm no C++ guru or anything, but it doesn't look like you ever declared i1 -- that's all.

Member Avatar for lost_c++_dude

how do i declare i1 ?

Member Avatar for kakilang 
#include <iostream.h>

//using namespace std;

int main()
{
	//Declare the variables
             //[B]error is here! u want delcare  "i1"  but here u declare it "il"     accidently[/B] 
	int [B]il[/B], i2,  quotient, remainder;

// Obtain the data from the user

	cout<<"Input an integer followed by a return: ";
	cin>>i1;

	cout<<"Input an integer followed by a return: ";
	cin>>i2;
	
// Do the arithmetic

quotient= i1/i2;
remainder=i1 % i2;

// Output the results

    cout<<"\nThe quotient of " << i1 << " and "<< i2 <<"is"<< i1/i2;
            //[B]here again error[/B]
	cin>>[B]il[/B];
	cout<<"The quotient of " << i1 << " and "<< i2 <<"is"<< i1/i2;
	cin>>i2;
	cout<<"\nThe remainder when "<<i1<<" is divided by "<<i2<<"is"<< remainder;


return 0;
}
Member Avatar for alc6379

how do i declare i1 ?

Wait a minnnit....

You don't know how to declare variables? What about this part in your code?

[b]//Declare the variables[/b]

	int il, i2,  quotient, remainder;

You declared some variables RIGHT THERE! How could you know to initialize variables there, but then forget? Did you copy this code, or something? Without stepping too far out on a ledge, I'll bet you could initialize i1 by

//Declare the variables

	int i1, i2,  quotient, remainder;

Like kakilang, you used il several times instead of i[/b]1[/b]

Member Avatar for vegaseat

Just a hint! Mixing up l and 1 are easy to do and hard to catch reading the code. I would use slightly more meaningful variable names like q and r. Also avoid using the often used i in loops and counters, it is another hard to read character, replace with k or such.

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.