Arithmetic bug?

Authoring Inform 7
1

I might have found a bug in one of the arithmetic routines. When I try to convert a decimal fractions to an integer, it seems to repeat the last digit; e.g. 0.7 becomes 77 instead of 7.
Here is the code and the result.

To decide what number is silverv :
	let G be floor of money of player to the nearest whole number;     [convert to integer]
	say "[line break]gold = [G]";
	let Sr be money of player - G;
	say "[line break]spr = [Sr]";
	let S be (Sr * 10) to the nearest whole number;
	say "[line break]sp = [S]";
	decide on S;

Screen Shot 2024-09-26 at 2.55.08 PM
Screen Shot 2024-09-26 at 2.55.08 PM796Ă—152 4.96 KB

Any help here? Should I report it?

1 Like
2

Hmm. If you break up (Sr * 10) to the nearest whole number into two lines, it works, apparently. Not sure why. I do know that, yes, arithmetic is messed up in Inform 7.

Ha! I figured it out I think. There’s no full stop at the end of your line

say "[line break]sp = [S]";

so you’re printing 7 twice right after itself.

4 Likes
3

OK, but why? Why does it repeat?

4

Whatever is calling silverv must be printing the value as well.

5

I think part of the problem is that the arithmetic functions are not as expected. The int() function actually ROUNDS to the nearest integer, instead of truncating as most int() functions do (every one I’ve found, actually). What I need was the floor() function, which removes the decimal fraction and converts from a real number to a number.

I made my on trunc() function. I was able to get what I wanted with this code. The phrase “to the nearest whole number” converts from a real number to an integer number.

To decide what number is silverv :
	let G be goldv;
	let Sr be money of player - G;
	let S be (Sr * 10) to the nearest whole number; 
	decide on S;
6

Could you post a bit more context, specifically the part that is requesting silverv?

Phil is suggesting that you are calling silverv, and immediately printing out the returned value, and the two are juxtaposed. Try something like

to X:
  let temp be silverv;
  say "[line break]and temp is [temp].";

to make it clearer.

1 Like
7

That’s all I have: the To decide phrase to produce silverv. This Decide rule works.
For the problem with the arithmetic function, see the topmost post when I opened the topic.

8

But what’s calling for this decision? Is whatever is asking for the decision also printing something? It must because where else is that “final sp” coming from in your output?

Like Bosh suggested, you could try adding either a period or a line break to the end of

say "[line break]sp = [S]";

then test again to see if you get the same output or not. It seems likely the break will happen between the two 7s