[jaj2]

I have to write a function that prints the sum of all the integers from 1 to 1000 that don't contain the digit 7 (e.g. 48933 does not contain a 7, but 443973 does).
can anyone help
thanks

[Advertisements]

try this:

private void sum() {
  int sum  = 0;
  String s = "";
  for( int i = 1;i < 1000;i++ ) {
	sum += i;
	s = " " + sum;
	if( s.indexOf( '7' ) == -1 ) {
	  System.out.print( s );
	}
}

note: s.indexOf( '7' ) return -1 if '7' is not found

[bdlt]

try this:

private void sum() {
  int sum  = 0;
  String s = "";
  for( int i = 1;i < 1000;i++ ) {
	sum += i;
	s = " " + sum;
	if( s.indexOf( '7' ) == -1 ) {
	  System.out.print( s );
	}
}

note: s.indexOf( '7' ) return -1 if '7' is not found

[jaj2]

Thank you for the help, i really appreciate it.
cheers,

[bdlt]

you're welcome.

the for loop needs to be modified to include 1000

for( int i = 1;i <= 1000;i++ ) {

[destin]

try this:

private void sum() {
  int sum  = 0;
  String s = "";
  for( int i = 1;i < 1000;i++ ) {
	sum += i;
	s = " " + sum;
	if( s.indexOf( '7' ) == -1 ) {
	  System.out.print( s );
	}
}

note: s.indexOf( '7' ) return -1 if '7' is not found

I would like to point out that constantly rebuilding a String is not a good habit. Strings are immutable. Everytime you append to a String or reset the value of a String, new memory will be allocated for that String. It would be much better to use StringBuffer or StringBuilder (which are mutable classes). In this case, the best way to go about printing out the sum is simply System.out.print(" " + sum);. This saves us a variable, and saves us tons of memory allocation (the way you're doing, memory will be allocated for s 1000 times! [actually 999, but it was fixed in your other post]).

So, rewritten:

public void sum() {
	int sum = 0;
	for (int i = 1; i <= 1000; i++) {
		sum += i;
		if (String.valueOf(sum).indexOf('7') == -1) {
			System.out.print(sum + " ");
		}
	}
}

Hope this helps. (oh... and you [bdlt] were missing a closing bracket too )

Edited by destin, 04 February 2006 - 11:32 AM.

[bdlt]

I would like to point out that constantly rebuilding a String is not a good habit.

destin has made a very good point regarding Strings. reassigning a new value to a String 1000 times is not sound java programming. thanks for pointing out acceptable alternatives.