printf error

**Thor** · October 9th, 2012, 09:52 PM

Hello again, I finished my code and it works fine (it's far from elegant), but when I tried to format the printing of a few values, I started getting errors.

Here is the code (errors occur after the loop):

Scanner in = new Scanner (System.in);
 
			int w = 0;
			int d = 0;
			int p = 0;
			double total = 0;
			double overall = 0;
			int n = 0;
			double max = 0;
			double min = 0;
			String maxTroop = "";
			String minTroop = "";
 
			System.out.println("Cookie Sales Report for Young Scouts of America");
			System.out.println("Enter the troop name (or exit to quit): ");
 
			String t = in.nextLine();
 
			while (!t.equals("exit"))
				{
				System.out.println("Enter number of boxes of Chocolate Mint Wafers: ");
				w = in.nextInt();
				in.nextLine();
				System.out.println("Enter number of boxes of Peanut Delights: ");
				d = in.nextInt();
				in.nextLine();
				System.out.println("Enter number of boxes of Coconut Plops: ");
				p = in.nextInt();
				in.nextLine();
				total = w*3+d*3.5+p*2.25;
				System.out.printf("The total collected by Troop " + t + " is $%7.2f\n", total);
				overall += total;
				n++;
				if (1 == n)
				{
					max = total;
					min = total;
					maxTroop = t;
					minTroop = t;
				}
				else if (total > max)
				{
					max = total;
					maxTroop = t;
				}
				else if (total < min)
				{
					min = total;
					minTroop = t;
				}
				System.out.println("Enter the troop name (or exit to quit): ");
				t = in.nextLine();
				}
			double avg = overall/n;
			System.out.printf("The total collected overall is $%7.2f\n", overall + ".");
			if (n > 0)
			{
			System.out.printf("The average collected per troop is $%7.2f\n", avg);
			System.out.printf("The minimum was collected by Troop " + minTroop + " and is $%7.2f\n", min + ".");
			System.out.printf("The maximum was collected by Troop " + maxTroop + " and is $%7.2f\n", max + ".");
			}
		}

I get this error message:

The total collected overall is $Exception in thread "main" java.util.IllegalFormatConversionException: f != java.lang.String
	at java.util.Formatter$FormatSpecifier.failConversion(Unknown Source)
	at java.util.Formatter$FormatSpecifier.printFloat(Unknown Source)
	at java.util.Formatter$FormatSpecifier.print(Unknown Source)
	at java.util.Formatter.format(Unknown Source)
	at java.io.PrintStream.format(Unknown Source)
	at java.io.PrintStream.printf(Unknown Source)
	at tjf1218_lab5.Scouts.main(Scouts.java:66)

Why is this happening? Does it have something to do with the variable declarations earlier in the program?

Any help would be greatly appreciated. Thanks!

**Zaphod_b** · October 9th, 2012, 11:26 PM

Originally Posted by Thor

...
Why is this happening? ...

When you have an expression that adds a numerical variable to a String, Java converts the number to a String and concatenates it with the other String. It decides how to format the number in a generic manner.

The compiler is complaining that the stuff it sees after the comma is a String, but the format specifies %f.

Bottom line: If you want to print a number with formatting, and you want to put a '.' character after the number, put the '.' inside the formatting string.

The safest way is to keep the formatting string all together and put the comma-separated list of variables after the format.

If n is an int or a long, and mean is a float or a double, it goes like this:

System.out.printf("There are %d numbers and their average is %7.2f.\n", n, mean);

Java takes the "%" things inside the format String and applies them to the variables in the same order.

Cheers!

Z

**pbrockway2** · October 9th, 2012, 11:40 PM

System.out.printf("The total collected overall is $%7.2f\n", overall + ".");

I'm guessing line 66 of Scouts.java is this one. (It's a good idea to say)

What the runtime message is saying is that the format for what is printed includes %7.2f but the next argument (overall+".") is a string. The format specifier - in his case f for floating point value - must match the argument type.

There are two ways to go. You could change the format string so that it uses s (for string) instead of f. But the other way is better: put the dot inside the format string the same as you do for the dollar sign.

System.out.printf("The total collected overall is $%7.2f.\n", overall);

Now the f will agree with the type of overall which is a double.

In general I think it looks neater to put as much of the text as possible into the format string and have the rest of the arguments just expressions to be evaluated (including variables which are simple expressions).

//System.out.printf("The minimum was collected by Troop " + minTroop + " and is $%7.2f\n", min + ".");
System.out.printf("The minimum was collected by Troop %s and is $%7.2f.%n", minTroop, min);

Notice how s (for string) and f (for floating point) match the following arguments (string then double). And how the text is all gathered together in the format string and what follows are variables to be substituted. printf() calls can become long and I would format this as

System.out.printf(
        "The minimum was collected by Troop %s and is $%7.2f.%n", 
        minTroop, min);

Typically twice the normal indentation is used when breaking long lines.

Consult the API documentation for these format strings. But don't be overwhelmed by it. Just figure out what you want to use and remember the location so that you can look up what you want when you want it. The same (or similar) syntax is used in a number of languages so it isn't wasted effort to become familiar with it.

**Thor** · October 10th, 2012, 10:05 PM

Thanks a lot! That makes too much sense for me to have thought of it haha

Thread: printf error

LinkBack

Thread Tools

Display

printf error

Related threads:

Re: printf error

Re: printf error

Re: printf error

Similar Threads

Please! Help me to this error "ERROR CANNOT FIND SYMBOL"

[SOLVED] Setting field width System.out.printf

How to align text? Printf?

about printf please help having errors

The printf() method explanation needed