Solved

int in the range of 30 to 50 challenge

Posted on 2014-10-08
19
175 Views
Last Modified: 2014-10-09
Hi,

I was working on the int in the range of 30 to 50 challenge


http://codingbat.com/prob/p132134


I wrote code like below
public boolean in3050(int a, int b) {

if((Math.abs(a-30)>=10 &&Math.abs(b-30)>=10)  || (Math.abs(a-40)>=10&& Math.abs(b-40)>=10))
{
return true;
}

if((Math.abs(a-40)>=10 && Math.abs(b-40)>=10) || (Math.abs(a-50)>=10 && Math.abs(b-50)>=10))
{
return true;
}
  return false;
}

Open in new window

I have couple of test cases failing as below

Expected      Run            
in3050(30, 31) → true      true      OK         
in3050(30, 41) → false      false      OK         
in3050(40, 50) → true      true      OK         
in3050(40, 51) → false      true      X         
in3050(39, 50) → false      false      OK         
in3050(50, 39) → false      false      OK         
in3050(40, 39) → true      true      OK         
in3050(49, 48) → true      true      OK         
in3050(50, 40) → true      true      OK         
in3050(50, 51) → false      true      X         
in3050(35, 36) → true      true      OK         
in3050(35, 45) → false      false      OK         
Correct for more than half the tests


How to fix and improve my program. Please advise
0
Comment
Question by:gudii9
  • 8
  • 7
  • 2
  • +1
19 Comments
 
LVL 84

Accepted Solution

by:
ozo earned 300 total points
ID: 40368987
Look at how your code is handling the not OK cases, and see if you can figure out why it is getting the wrong answer.

You may also want to think about how you are doing the range 30..40 inclusive and range 40..50 inclusive tests
0
 
LVL 7

Author Comment

by:gudii9
ID: 40368999
public boolean in3050(int a, int b) {

if((Math.abs(a-30)>=10 &&Math.abs(b-30)>=10)  || (Math.abs(a-40)>=10&& Math.abs(b-40)>=10))
{
return true;
}

if((Math.abs(a-40)>=10 && Math.abs(b-40)>=10) || (Math.abs(a-50)>=10 && Math.abs(b-50)>=10))
{
return true;
}

if((a-50)>=1 && (b-50)>=1)
{
return false;
}
return false;

}

Open in new window


I think i improved my code bit still 2 failing

Expected      Run            
in3050(30, 31) → true      true      OK         
in3050(30, 41) → false      false      OK         
in3050(40, 50) → true      true      OK         
in3050(40, 51) → false      true      X         
in3050(39, 50) → false      false      OK         
in3050(50, 39) → false      false      OK         
in3050(40, 39) → true      true      OK         
in3050(49, 48) → true      true      OK         
in3050(50, 40) → true      true      OK         
in3050(50, 51) → false      true      X         
in3050(35, 36) → true      true      OK         
in3050(35, 45) → false      false      OK
0
 
LVL 84

Expert Comment

by:ozo
ID: 40369006
Think more carefully about what in the range 30..40 inclusive means, and what Math.abs(a-30)>=10 means
0
How Do You Stack Up Against Your Peers?

With today’s modern enterprise so dependent on digital infrastructures, the impact of major incidents has increased dramatically. Grab the report now to gain insight into how your organization ranks against your peers and learn best-in-class strategies to resolve incidents.

 
LVL 27

Assisted Solution

by:dpearson
dpearson earned 200 total points
ID: 40369068
Along the same lines as ozo is saying,

What does:

Math.abs(a-30)>=10

return for "a = 25"?

And is this what you want?

Doug
0
 
LVL 7

Author Comment

by:gudii9
ID: 40369099
thinking. Why thinking is takes time. why it is not spontaneous yet with java language unlike english language?
0
 
LVL 7

Author Comment

by:gudii9
ID: 40369102
in the mind i have answer but to teach computer to understand is bit hard for me
0
 
LVL 27

Expert Comment

by:dpearson
ID: 40369110
Don't worry that it takes time.  It will all get faster with practice.

Everyone is slow when they start out.

Doug
0
 
LVL 7

Author Comment

by:gudii9
ID: 40369822
public class Test9 {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		in3050(40,51);
	}

	
	public static boolean in3050(int a, int b) {

		if((Math.abs(a-30)<=10 &&Math.abs(b-30)<=10)  || (Math.abs(a-40)<=10&& Math.abs(b-40)<=10))
		{
		return true;
		}

		if((Math.abs(a-40)<=10 && Math.abs(b-40)<=10) || (Math.abs(a-50)<=10 && Math.abs(b-50)<=10))
		{
		return true;
		}

		if((a-50)>0 && (b-50)>0)
		{
		return false;
		}
		return false;

		}
}

Open in new window


tried as above but still not working. WIll modify more
0
 
LVL 84

Expert Comment

by:ozo
ID: 40369835
Think more carefully about what in the range 30..40 inclusive means, and what Math.abs(a-40)<=10 means
Think about what happens when a is 29, 30, 31, 39, 40, 41

Another hint:  How would you test for in the range -10..10?
0
 
LVL 37

Expert Comment

by:zzynx
ID: 40370144
>> why it is not spontaneous yet with java language unlike english language?
I think that's because you use the method Math.abs()
I wonder why you use it for this challenge?

Let's do it in English:
Checking if an int x is in the range 30...40 is just as simple as
1) checking that x is 30 or more
but ALSO
2) checking that x is no more than 40, thus 40 or less

In java
1) x >= 30
AND
2) x <= 40

Hence, checking if an int x is in the range 30...40 is just as simple as:

if ( x>=30 && x<=40) {
}

You have to check if both a and b are in the range 30..40, so that just doubling what you already had above:
if ( (a>=30 && a<=40) && (b>=30 && b<=40) ) {
}

And you have to check it for another range: 40...50, so that's replacing 30 by 40 and 40 by 50:
if ( (a>=40 && a<=50) && (b>=40 && b<=50) ) {
}

So the eventual result is:

public boolean in3050(int a, int b) {
      if ( (a>=30 && a<=40) && (b>=30 && b<=40) ) { // a and b in the range 30...40?
         return true; // Yes!
      }

      if ( (a>=40 && a<=50) && (b>=40 && b<=50) ) { // a and b in the range 40...50?
         return true; // Yes!
      }

      return false; // no succes (otherwhise, we shouldn't have come till here)
}

Open in new window

0
 
LVL 7

Author Comment

by:gudii9
ID: 40370836
Above solution works great.. in my mind i was struck with one direction of Math.abs for this challenge. How to keep mind working  thinking on all different directions and possibilities.
0
 
LVL 84

Expert Comment

by:ozo
ID: 40370843
For what range of numbers  is Math.abs(a-40)<=10 true?
0
 
LVL 7

Author Comment

by:gudii9
ID: 40371117
both sides of 40 ie 30-50 since -10 also considered as 10 with abs
0
 
LVL 84

Expert Comment

by:ozo
ID: 40371128
Yes.  So can you see how to modify 30-50 to 30-40?
0
 
LVL 7

Author Comment

by:gudii9
ID: 40371137
public class Test8 {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
    boolean b=in3050(30, 51);
    System.out.println("b is-->"+b);
	}

	
	public static boolean in3050(int a, int b) {
	      if ( a>=30 && a<=40 && b>=30 && b<=40 ) { // a and b in the range 30...40?
	         return true; // Yes!
	      }

	      if ( a>=40 && a<=50 && b>=40 && b<=50 ) { // a and b in the range 40...50?
	         return true; // Yes!
	      }

	      return false; // no succes (otherwhise, we shouldn't have come till here)
	}
}

Open in new window


Even if i remove the inner brackets I still get same results.

Instead of

  if ( (a>=30 && a<=40) &&( b>=30 && b<=40) )
changed as below

  if ( a>=30 && a<=40 && b>=30 && b<=40 )

So no need of inner extra brackets right?
Please advise
0
 
LVL 84

Expert Comment

by:ozo
ID: 40371185
In java, comparisons have precedence over boolean operators,

but I was asking whether you could figure out if
30-50 corresponds to Math.abs(a-40)<=10
then
30-40 corresponds to what?
0
 
LVL 7

Author Comment

by:gudii9
ID: 40371200
(a-35)<=5
which adds 5 on both sides of 35
0
 
LVL 84

Expert Comment

by:ozo
ID: 40371238
Correct, Math.abs(a-35)<=5 would test for the range 30..40
0
 
LVL 37

Expert Comment

by:zzynx
ID: 40372346
>> Above solution works great..
Then why didn't you give it any points?
0

Featured Post

Active Directory Webinar

We all know we need to protect and secure our privileges, but where to start? Join Experts Exchange and ManageEngine on Tuesday, April 11, 2017 10:00 AM PDT to learn how to track and secure privileged users in Active Directory.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
eclipse with Angularjs2.0 3 57
jboss 7.1 start up error 1 42
Select statement in @Query (JPA Repository) - SpringMVC 2 19
running on tomcat not jboss eap 7.0 3 19
Having just graduated from college and entered the workforce, I don’t find myself always using the tools and programs I grew accustomed to over the past four years. However, there is one program I continually find myself reverting back to…R.   So …
If you haven’t already, I encourage you to read the first article (http://www.experts-exchange.com/articles/18680/An-Introduction-to-R-Programming-and-R-Studio.html) in my series to gain a basic foundation of R and R Studio.  You will also find the …
Viewers will learn how to properly install Eclipse with the necessary JDK, and will take a look at an introductory Java program. Download Eclipse installation zip file: Extract files from zip file: Download and install JDK 8: Open Eclipse and …
The viewer will learn additional member functions of the vector class. Specifically, the capacity and swap member functions will be introduced.

820 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question