Discussion :

[Fused]

Bonjour !

Sur mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
boolean containsForbiddenFormat (String url) {
	return (
		url.endsWith(".pdf") ||
		url.endsWith(".exe") ||
		url.endsWith(".vb")
	);
}

Seulement je veux l'appliquer à une bonne trentaine d'extensions, il y a un moyen plus pratique que d'écrire 30 lignes ?

Sinon en fait je veux récupérer des URL, mais à cause du rewriting d'URL qui se fait de plus en plus, si je me contente des ".html, .php, ..." je vais rater tous les trus du genre www.site.com/categorie/vacances-a-la-plage
Si vous avez une idée sur ça sinon, c'est encore mieux !

Merci

[gifffftane]

S'il y a une trentaine d'extensions, 30 lignes n'est pas idiot Nous sommes en Java, que diable il parait qu'il y a des langages qui sont plus efficaces... vas-y !

Sinon tu peux mettre tes extensions dans un tableau et vérifier si l'une d'elles convient dans une boucle, sinon tu peux aussi utiliser des regexp.

[tchize_]

sinon tu as çà, coté rapide:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
HashMap<String> extensions = new HashMap<String>();
extensions.addAll(Arrays.asList(new String[]{".pdf",".html",".php"}));
//.....
int i = filename.lastIndexOf(".");
String extension = filename.subString(i).toLowerCase();
if ( (i>=0) && extensions.contains(extension))
  // On a une extension interdite

avantage, comme la recherche se fait par hash, on ne passe pas sur toutes la liste d'extensions.

[Fused]

Pas trop mal comme solution, merci.

Si quelqu'un reprends ça, je crois qu'en java subString s'écrit "substring".

[tchize_]

et il faut faire le substring dans le if, sinon, quand il y a pas de . ca foire

et donc décomposer le if en deux (if i>=0 puis l'extraction de l'extension, puis le contains)