Closed nobe4 closed 9 years ago
nobe4
commented
10 years ago nobe4
commented
10 years ago appui sur le bouton statistique (logo en forme de graphique) :
apparition d'un "menu" avec les choix ci dessous (dont les fonctions non implémentées ).
Choix immédiat d'un mode d'affichage (en volume ou en valeur) avec un bouton de sélection (utilité ?)
Menu peu clair et style à refaire
clic sur le bouton statistique des adhérents : aucun résultat il faut clicker sur le choix du type d'unité (très peu clair et contre productif)
attente de 15 secondes (très long) et affichage des graphiques suivants :
Année et mois marchent, affichent les données, avec unités et repères concordants.
Type/mode n'affichent rien, et les unités ne sont pas bonnes, la fonctionnalité n'a pas été implémentée.
Encore une fois peu de clarté sur les choix (est-ce un menu, est-ce un indicateur ?)
idem que précédemment, le menu est peu clair, les fonctions mois et années sont présentes mais pas dons/offre
dons/offre : nombre de dons par offre ou choix dons et choix offre ?
manque de clarté et de simplicité
pas de résultat, les fonctionnalités n'ont pas l'air implémentées...
Séparation dans le menu, pourquoi ? c'est moche et ça n'a aucun intérêt ...
pas de résultat, les fonctionnalités n'ont pas l'air implémentées...
Click sur le bouton correspondant, pas d'affichage Choix de la campangne et clic sur "sélectionner cette campagne" (bouton qui a alors remplacé l'ancien "sélectionner cet affichage")
Erreur de base de données pour chaque campagne :
nobe4
commented
10 years ago Le fichier contenant l'erreur a 140 lignes pour faire des queries sur la bdd de dons... Je pense qu'il y a moyen de faire une classe mère puisque toutes les autres classes sont relativement similaires.
Sinon oui il y a pire comme code, je ne sais pas encore si ça a été généré automatiquement ou non ...
Le fichier controllers/stat.php (à renommer ?) me fait peur, il y a approx 1000 lignes avec toutes les fonctions de stats écrites dans l'ordre, sans séparation, sans classe, sans trop de commentaires. En mode 'je met tout dans le main parce que j'ai la flemme de faire mieux'. Il y a de la redondance dans le code (https://github.com/statox/caramel/blob/master/application/controllers/stat.php#L140-L159).
Les algos sont pas commentés, c'est limite étonnant que le rendu soit fait correctement ...
Quand il y a des commentaires on rigole (https://github.com/statox/caramel/blob/master/application/controllers/stat.php#L556-L569)
Du code à mettre à la poubelle (maybe, vu que c'est pas commenté on en sait rien) (https://github.com/statox/caramel/blob/master/application/controllers/stat.php#L739-L741)
Du gros travail de pro donc (https://github.com/statox/caramel/blob/master/application/controllers/stat.php#L978-L979)
Pour aller plus vite je fais un petit commentaire par fichier :
Réception et traitement de formulaire, je pense redondant et peu performant.
Code plus modeste (à peine 400 lignes !), tout un tas de vérification et opérations sur la bdd
200 lignes, on touche à un fichier avec une taille sympa. tout un tas de vérification et opérations sur la bdd
Bon il y a trop de fichier, du coup en parsant rapidement voilà ce que j'ai trouvé de rigolo (pour redonner le moral ! ) :
https://github.com/statox/caramel/blob/master/application/controllers/document.php#L18-L20 https://github.com/statox/caramel/blob/master/application/controllers/don.php#L6 https://github.com/statox/caramel/blob/master/application/controllers/don.php#L456-L479 https://github.com/statox/caramel/blob/master/application/controllers/message.php#L5-L19 https://github.com/statox/caramel/blob/master/application/controllers/offre.php#L391-L394 https://github.com/statox/caramel/blob/master/application/controllers/segment.php#L48 https://github.com/statox/caramel/blob/master/application/controllers/validationController.php#L1-L21 https://github.com/statox/caramel/blob/master/application/models/A_Lire_(CRUD).txt#L3 https://github.com/statox/caramel/blob/master/application/models/campagne_model.php#L21-L22 https://github.com/statox/caramel/blob/master/application/models/cible_model.php#L18-L20 https://github.com/statox/caramel/blob/master/application/models/pagination_model.php#L7-L11 https://github.com/statox/caramel/blob/master/application/models/user_model.php#L8 https://github.com/statox/caramel/blob/master/application/models/user_model.php#L37-L46 https://github.com/statox/caramel/blob/master/application/views/base/header.php#L7-L12 https://github.com/statox/caramel/blob/master/application/views/base/navigation.php#L35 https://github.com/statox/caramel/blob/master/application/views/campagne/create.php#L12 https://github.com/statox/caramel/blob/master/application/views/cible/list.php#L5-L7 https://github.com/statox/caramel/blob/master/application/views/contact/dedoublonnage_manuel.php#L60-L74 https://github.com/statox/caramel/blob/master/application/views/contact/menu.php#L13 https://github.com/statox/caramel/blob/master/application/views/contact/restauration.php#L11-L21 https://github.com/statox/caramel/blob/master/application/views/document/edit_letter.php#L1 https://github.com/statox/caramel/blob/master/application/views/document/edit_letter.php#L80 https://github.com/statox/caramel/blob/master/application/views/document/main.php#L5 https://github.com/statox/caramel/blob/master/application/views/don/create.php#L13 https://github.com/statox/caramel/blob/master/application/views/don/recuFiscal.php#L1-L10 https://github.com/statox/caramel/blob/master/application/views/reglage/fusion_comparaison.php#L4-L6 https://github.com/statox/caramel/blob/master/application/views/reglage/fusion_id_selection.php#L39-L58 https://github.com/statox/caramel/blob/master/application/views/segment/list_potentiel.php#L20 https://github.com/statox/caramel/blob/master/application/views/segment/list_potentiel.php#L38-L46 https://github.com/statox/caramel/blob/master/application/views/segment/search.php#L18 https://github.com/statox/caramel/blob/master/application/views/stat/campagnes/dons_histo.php#L14-L46 https://github.com/statox/caramel/blob/master/application/views/stat/end_menu.php
Pour résumer, c'est pas ça : http://css-tricks.com/examples/CleanCode/Beautiful-HTML.png
statox
commented
9 years ago Ancienne version des sources. Issue non valable.
