Rechercher / Remplacer

[WinXaito]

Ajoute les fonctions Rechercher et Remplacer accessible par le raccourcis CTRL + F dans l'éditeur.

Issue de référence: #70 / #71

Remplace la PR: #37

N'est pas prêt à être merge

[WinXaito]

Je bug sur un petit point.

J'ai fais un système de WholeWord, Mot entier, pour ce faire j'ajoute simplement un espace au début et à la fin du mot recherché. Mais ça ne fonctionne pas s'il est en tout début de ligne, comment je peux corriger ceci ?

(Donc détecter si on se trouve en début de ligne ou non)

[WinXaito]

Hello, j'ai encore un autre petit problème.

Collection<String> css = new ArrayList<>();
css.add("-fx-background-color: red;");
sourceText.setStyle(0, 20, css);

J'aimerais mettre un Backouground-color ou quelque chose du genre sur une partie de texte, mais ça ne fais rien (ça ne fais que légèrement changer l'écriture).

J'ai essayé de mettre un CodeArea, toujours pareil, j'ai essayer de faire sourceText.setStyle("-fx-font-weight:bold"), mais même ça ne fais rien (Sauf comme dit faire un légère effet bizarre sur l'écriture.

[firm1]

(Donc détecter si on se trouve en début de ligne ou non)

Pour détecter si tu es en début de ligne, tu peux vérifier si le caractère précédent est un saut de ligne.

Après j'avoue que je préfère déjà un système de recherche basique avant de faire la version compliquée. ^^

Pour le background-color, je ne l'ai pas encore essayé sur le StyledTextArea, mais j'aurai tendance à faire une simple sélection du texte trouvé.

[WinXaito]

Je rentre maintenant, je pourrai corriger tous mes conflits, et te montrer ou j'en suis.

[WinXaito]

Bon j'ai galérer sur un deux points, mais la c'est plus ou moins fonctionnel.

Tu en penses quoi ? (La j'ai un soucis quand je remplace par un mot plus court ...)

[edit] Pour le highlight c'est bon :p

[WinXaito]

Y a moyen de me dire si je suis dans la bonne voie avant de continuer ?

[firm1]

Y a moyen de me dire si je suis dans la bonne voie avant de continuer ?

Coté structure ça m'a l'air bon. c'est la bonne voie.

Par contre coté code, tout se joue dans la fonction findReplace. Elle me parait un peu grosse tout de même, il y a moyen de factoriser un peu. Je ne sais pas si tu connais l'usage des lambda en Java 8 , mais c'est l'occasion de les utiliser ici. Au pire si tu y arrives pas, je peux te faire une PR quand tu auras terminé le reste.

Mais sinon, ça m'a l'air correct donc tu peux avancer (il ne manque plus grand chose de ce que je vois)

[WinXaito]

Effectivement, je ne l'ai connaissais pas directement. J'en ai vu, et parfois IntelliJ me propose de transformer mes fonctions en fonction lambda.

Je vais donc continuer sur cette voie là :)

Et effectivement, c'est fonctionnel (On peut chercher avec une recherche instantané, parcourir les résultat avec le bouton Recherche, remplacer les résultat un par un avec le bouton Remplacer et remplacer toutes les occurrences avec le bouton Remplacer tout).

[WinXaito]

Pour cette PR je suis bon, j'ai corrigé le bug du RemplacerTout qui posais problème avec des mots remplacé plus court.

Je ne sais pas trop comment m'y prendre avec les fonctions lambda, donc je te laisse voir ou alors me donner un exemple de ce que je dois faire.

Penses-tu avoir le temps de faire ça (Et mon autre PR) pour la 0.11.0 ?

[firm1]

Wow, c'est un super travail que tu as fais là.

J'ai noté quelques bugs :

• la recherche en whole word ne fonctionne pas lorsque le mot est est début de phrase.
• la recherche sur la selection ne semble pas marcher de mon coté.

Mais je serais d'avis de merger en l'état et de voir pour les corrections plus tard. Qu'en penses tu ?

[WinXaito]

Wow, c'est un super travail que tu as fais là.

ça fait toujours plaisir de l'entendre :)

Mais je serais d'avis de merger en l'état et de voir pour les corrections plus tard. Qu'en penses tu ?

Oui, car la pour la 0.11.0 on a vraiment pas grand chose.

[firm1]

Yay c'est mergé. merci encore. Tu pourras fermer les issues qui vont bien s'il te plait ?

[WinXaito]

C'est fait merci.