Prise indépendamment les lignes de codes sont relativement propres.
La critique viens plutôt AHMA d'un problème de factorisation.
Exemple cette combo box qui est copiée 3 fois en dur dans le script:
<code>
<combobox>
<variable>mgenre</variable>
<item>Blues</item>
#snip 150 lignes
<item>Trip-Hop</item>
<item>Vocal</item>
</combobox>
</code>
Ou encore ceci:
<code>
if [ "$search" = "Deezer" ]; then
{
#Conversion des espaces en url html et autres charactères spéciaux
mquery=$(echo $query | sed -e 's/ /%20/g' | sed -e 's/[éèêë]/e/g' | sed -e 's/à/a/g' | sed -e 's/[ôö]/o/g' | sed -e 's/ù/u/g' | sed -e 's/ç/c/g')
##Lancement de firefox
firefox http://www.deezer.com/#music/result/all/$mquery 2>/dev/null &
}
elif [ "$search" = "Imeem" ]; then
{
##Recherche à effectuer sur imeem
#Conversion des espaces en url html et autres charactères spéciaux
mquery=$(echo $query | sed -e 's/ /%20/g' | sed -e 's/[éèêë]/e/g' | sed -e 's/à/a/g' | sed -e 's/[ôö]/o/g' | sed -e 's/ù/u/g' | sed -e 's/ç/c/g')
##Lancement de firefox
firefox http://www.imeem.com/tag/$mquery/music/ 2>/dev/null &
}
elif [ "$search" = "Les 2" ]; then
{
#Conversion des espaces en url html et autres charactères spéciaux
mquery=$(echo $query | sed -e 's/ /%20/g' | sed -e 's/[éèêë]/e/g' | sed -e 's/à/a/g' | sed -e 's/[ôö]/o/g' | sed -e 's/ù/u/g' | sed -e 's/ç/c/g')
##Lancement de firefox
firefox http://www.deezer.com/#music/result/all/$mquery 2>/dev/null &
firefox http://www.imeem.com/tag/$mquery/music/ 2>/dev/null &
}
fi
</code>
Ne pense tu pas que répéter 3 fois mquery est un peut dommage ? Ne serait il pas mieux placé avant le case ? Ou alors en faire une fonction "urlecode" par exemple.
Enfin n'importe quoi mais éviter ces copier collé.
Rien d'horrible mais je pense que les 1K lignes de ce script pourrait fondre comme neige au soleil, et surtout que la partie interface mériterait d'être séparée ce qui rendrait le tout bien plus maintenable.
[^] # Re: Pourquoi le cas des CC ?
Posté par Jean B . En réponse au journal Deezer et législation.. Évalué à 3.
La critique viens plutôt AHMA d'un problème de factorisation.
Exemple cette combo box qui est copiée 3 fois en dur dans le script:
<code>
<combobox>
<variable>mgenre</variable>
<item>Blues</item>
#snip 150 lignes
<item>Trip-Hop</item>
<item>Vocal</item>
</combobox>
</code>
Ou encore ceci:
<code>
if [ "$search" = "Deezer" ]; then
{
#Conversion des espaces en url html et autres charactères spéciaux
mquery=$(echo $query | sed -e 's/ /%20/g' | sed -e 's/[éèêë]/e/g' | sed -e 's/à/a/g' | sed -e 's/[ôö]/o/g' | sed -e 's/ù/u/g' | sed -e 's/ç/c/g')
##Lancement de firefox
firefox http://www.deezer.com/#music/result/all/$mquery 2>/dev/null &
}
elif [ "$search" = "Imeem" ]; then
{
##Recherche à effectuer sur imeem
#Conversion des espaces en url html et autres charactères spéciaux
mquery=$(echo $query | sed -e 's/ /%20/g' | sed -e 's/[éèêë]/e/g' | sed -e 's/à/a/g' | sed -e 's/[ôö]/o/g' | sed -e 's/ù/u/g' | sed -e 's/ç/c/g')
##Lancement de firefox
firefox http://www.imeem.com/tag/$mquery/music/ 2>/dev/null &
}
elif [ "$search" = "Les 2" ]; then
{
#Conversion des espaces en url html et autres charactères spéciaux
mquery=$(echo $query | sed -e 's/ /%20/g' | sed -e 's/[éèêë]/e/g' | sed -e 's/à/a/g' | sed -e 's/[ôö]/o/g' | sed -e 's/ù/u/g' | sed -e 's/ç/c/g')
##Lancement de firefox
firefox http://www.deezer.com/#music/result/all/$mquery 2>/dev/null &
firefox http://www.imeem.com/tag/$mquery/music/ 2>/dev/null &
}
fi
</code>
Ne pense tu pas que répéter 3 fois mquery est un peut dommage ? Ne serait il pas mieux placé avant le case ? Ou alors en faire une fonction "urlecode" par exemple.
Enfin n'importe quoi mais éviter ces copier collé.
Rien d'horrible mais je pense que les 1K lignes de ce script pourrait fondre comme neige au soleil, et surtout que la partie interface mériterait d'être séparée ce qui rendrait le tout bien plus maintenable.