CODEWITHSUNDEEP
java
Becka
Becka

Reputation: 21

Count elements in array

I am trying to count an element in an array of objects.

              long number = Stream.of(jobTitle).count();
              System.out.println("There are " + number + " employees.");

What happens is that it will print out the message as many times as many employees have the same job title. Yet "number" stays always 1.

Any guiding would be much appreciated.

Upvotes: 0

Views: 104

Answers (2)

Kavitha Karunakaran
Kavitha Karunakaran

Reputation: 1400

You need to change the stream of command to define a proper Predicate for filter option.

Stream.of(employees).filter(e -> e.getJobTitle().equals(jobTitle)).count();

Upvotes: 0

GhostCat
GhostCat

Reputation: 140457

long number = Stream.of(jobTitle).count();

Counts the elements in a stream that contains one element.

It is not surprising that this operation always ends up with the exact same result.

Your code is equivalent to:

List<Whatever> titels = new ArrayList<>();
titels.put(oneEntry);
... print titels.size()

Long story short: that statement is nonsensical. What you probably meant was:

if (arbetstitel.equalsIgnoreCase(jobCount)){
  g++;

or something alikw. Of course g is a rather bad name for a counter.

But the real answer here is: step back. Think what the problem is you intend to solve, and what the elements are you need to look at. The code you are showing here is simply not making (much) sense. I can't tell you how to fix it, because, as said: it is not clear what you try to achieve here.

A streamish way of counting:

long usersWithMatchingTitle = Arrays.stream(employees).filter(e -> e.getJobTitle().equalsIgnoreCase(jobTitleFromUser)).count();

Meaning: instead of manually iterating your array, you can turn the whole array into a stream, and then filter/count whatever you want to.

Please note: your code seems to only care about the first 30 elements in that array. If that is really what you want, you will need ...stream(employees).limit(30)...

Upvotes: 4

Related Questions