selecteer de eerste index in de verzameling waar het leuk is

Question

selecteer de eerste index in de verzameling waar het leuk is

Ik voel soms dat ik het wiel opnieuw uitvind.

Ik vraag me af of er utiliteitsmethoden zijn in java/jakarta commons/guava/?, die dieper gaan in de verzameling en iets doen (testen, wijzigen, verwijderen) met de elementen.

Ik heb deze methode geschreven en nu voel ik dat er een one-liner is die dit kan.

/**
 * Find index of first line that contains search string.
 */
public static int findIdx(List list, String search) {
  for (int i = 0, n = list.size(); i < n; i++)
    if (list.get(i).contains(search))
      return i;
  return -1;
}

3

toegevoegd 27 oktober 2011 op 08:24 de auteur adfgvx

Bekeken: 2

de bron

Waarom doe je voor (int i = 0, n = list.size (); i in plaats van voor (int i = 0, i ?



toegevoegd 27 oktober 2011 op 08:33, 
de auteur Bhesh Gurung, 

de bron



@Bhesh Het ziet eruit als een optimalisatie in het geval dat de bewerking size() duur is. Maar het gaat er ook van uit dat de lijstgrootte tijdens het lussen blijft staan, wat mij gevaarlijk lijkt.

toegevoegd 27 oktober 2011 op 08:46, 
de auteur Kevin K, 

de bron




Dit is mijn stijl. Ik heb niet gecontroleerd hoe duur de methode call to size() is.

toegevoegd 27 oktober 2011 op 08:58, 
de auteur adfgvx, 

de bron


2 antwoord


Guava heeft  Iterables.indexOf  met een predicaat:
int index = Iterables.indexOf(list, new Predicate {
        @Override public boolean apply(String input) {
            return input.contains(search);
        }
    });

Niet veel beter, weliswaar - en je moet definitief  zoeken  maken. Maar in ieder geval met Java 8 kun je iets schrijven als:
int index = Iterables.indexOf(list, input => input.contains(search));

(Of tenminste iets  zoals  dat. En mogelijk in syntaxis van een uitbreidingsmethode ...)


5

toegevoegd 27 oktober 2011 op 08:30
de auteur Jon Skeet



de bron









Heel elegant, geen lijn maar mooi. Ik speelde eigenlijk met commons CollectionUtils.find, maar die methode geeft geen index ...

toegevoegd 27 oktober 2011 op 08:51, 
de auteur adfgvx, 

de bron






Guava heeft wat u zoekt in   Iterables.indexOf()  , hoewel ik niet precies zou beweren dat uw code leesbaarder wordt:
public static int findIdx(List list, final String search) {
  return Iterables. indexOf(list, new Predicate() {
    public boolean apply(String s) {
      return s.contains(search);
    }
  });
}



2

toegevoegd 27 oktober 2011 op 08:34
de auteur mergeconflict



de bron



Ja ... Java en functioneel programmeren spelen niet leuk. Zoveel boilerplate instantiating anonymous  Functie / Predicate  classes :(

toegevoegd 27 oktober 2011 op 09:50, 
de auteur mergeconflict, 

de bron




TNX voor uw gebruiksklare functie. Guava is erg krachtig, maar het is nog steeds veel typen.

toegevoegd 27 oktober 2011 op 09:10, 
de auteur adfgvx, 

de bron

de auteur Jon Skeet · Answer 1 · 2011-10-27T08:30Z

Guava heeft Iterables.indexOf met een predicaat:

int index = Iterables.indexOf(list, new Predicate {
        @Override public boolean apply(String input) {
            return input.contains(search);
        }
    });

Niet veel beter, weliswaar - en je moet definitief zoeken maken. Maar in ieder geval met Java 8 kun je iets schrijven als:

int index = Iterables.indexOf(list, input => input.contains(search));

(Of tenminste iets zoals dat. En mogelijk in syntaxis van een uitbreidingsmethode ...)

de auteur mergeconflict · Answer 2 · 2011-10-27T08:34Z

Guava heeft wat u zoekt in Iterables.indexOf() , hoewel ik niet precies zou beweren dat uw code leesbaarder wordt:

public static int findIdx(List list, final String search) {
  return Iterables. indexOf(list, new Predicate() {
    public boolean apply(String s) {
      return s.contains(search);
    }
  });
}