Java - Basic use of arrays

Question

This is a basic question but I do need some help.

Given two arrays of ints, a and b, return true if they have the same first element or they have the same last element. Both arrays will be length 1 or more.

commonEnd({1, 2, 3}, {7, 3}) → true

commonEnd({1, 2, 3}, {7, 3, 2}) → false

commonEnd({1, 2, 3}, {1, 3}) → true

I have the following code but it wont compile:

public boolean commonEnd(int[] a, int[] b) {
    if(a[0] == b[0] || a[a.length-1] ==b[b.length-1])
        return true;
}

Please always include compiler error messages when code does not compile. — Thilo
– Thilo, Commented Nov 16, 2012 at 4:53
"it is simple basic question but could u help me?" I hope in your next question you can put an upper-case letter for the start of each sentence, and spell words like 'you' properly. Till then.. — Andrew Thompson
– Andrew Thompson, Commented Nov 16, 2012 at 4:53
@AndrewThompson: Why not just...edit the question instead of nitpicking? — Makoto
– Makoto, Commented Nov 16, 2012 at 4:55
@Makoto I edit many. It is always a judgment call. This one did not make the cut. — Andrew Thompson
– Andrew Thompson, Commented Nov 16, 2012 at 4:56
If one of the answers got it to work for you, please mark that answer as accepted. (Click on the check mark to the left of the answer.) — Ted Hopp
– Ted Hopp, Commented Nov 16, 2012 at 5:03

Thilo · Accepted Answer · 2012-11-16 04:50:43Z

4

You have missing right paren for the if.
You need to return something (false) in the "else" part.

You should have gotten a compiler error to tell you this.

Personally, I'd get rid of the if altogether and do

return  a[0] == b[0] || a[a.length-1] ==b[b.length-1];

(but this may be considered hard to read)

answered Nov 16, 2012 at 4:50

Thilo

264k107 gold badges527 silver badges674 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Makoto Over a year ago

I see nothing hard to read about this. Solid answer here. :)

Sean F · Accepted Answer · 2012-11-16 04:56:22Z

0

you had no return type if, if is untrue

  public boolean commonEnd(int[] a, int[] b) {
  if(a[0] == b[0] || a[a.length-1] ==b[b.length-1])
      return true;
  return false;
  }

answered Nov 16, 2012 at 4:56

Sean F

2,3884 gold badges29 silver badges41 bronze badges

Comments

Vatine · Accepted Answer · 2015-11-10 16:00:15Z

0

public boolean commonEnd(int[] a, int[] b) {
  if((a[0] == b[0]) || (a[a.length-1] ==b[b.length-1]))
      return true;
  return false;
}

You have missed the return statement(if the if statement no true), that's why it didn't compile.

edited Nov 10, 2015 at 16:00

Vatine

21.4k4 gold badges57 silver badges70 bronze badges

answered Nov 16, 2012 at 5:01

Rakhita Wickramatunge

4,5031 gold badge18 silver badges15 bronze badges

Comments

ritesh_ratn · Accepted Answer · 2020-06-20 09:15:02Z

0

This code works fine:-

public boolean commonEnd(int[] a, int[] b) {
  int length1=a.length;
  int length2=b.length;
  if(a[0]==b[0]||a[length1-1]==b[length2-1]){
    return true;
  }
  return false;
}

answered Jun 20, 2020 at 9:15

ritesh_ratn

15710 bronze badges

Comments

thar45 · Accepted Answer · 2012-11-16 04:58:02Z

-1

Expects return Statements for the function

public boolean commonEnd(int[] a, int[] b) {
    if(a[0] == b[0] || a[a.length-1] ==b[b.length-1]) //Missed `)`
          {
          return true;
          }
        return false;
     }

answered Nov 16, 2012 at 4:58

thar45

3,5705 gold badges35 silver badges49 bronze badges

1 Comment

thar45 Over a year ago

Actually the extra brace has occurred from workout for the probs sorry, updated the answer ...@SeanF

Collectives™ on Stack Overflow

Java - Basic use of arrays

5 Answers 5

1 Comment

Comments

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

1 Comment

Comments

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related