Aller au contenu

Ce code présente t-il de graves anomalies ?

steph13

Par steph13
dans Le salon de Webmaster Hub

 Partager

Sujets conseillés

steph13

steph13

Posté
Posté (modifié)

Bonjour le Hub.

J'ai besoin de l'avis des puristes sur ce code.

Visiblement, cette page ci-après, présente de sérieux problèmes, mais je ne vois pas les quelles....

CODE
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" &quot;http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

<html xmlns=&quot;http://www.w3.org/1999/xhtml">

<!-- InstanceBegin template="/Templates/page_modele.dwt.php" codeOutsideHTMLIsLocked="false" -->

<head>

<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />

<!--

Pour chaque page, donner un titre, une description et mots clef différent,

mais toujours en rapport avec le contenu de la page en question.

-->

<!-- InstanceBeginEditable name="balise" -->

<title>Telephonie IP teambox</title>

<meta name="Description" content="Maquette du site" />

<meta name="Keywords" content="mot, clé"/>

<!-- InstanceEndEditable -->

<!-- -->

<!--[if !IE]> <-->

<link href="css/portail.css" rel="stylesheet" type="text/css" media="screen" />

<!-->

<![endif]-->

<!--[if gte IE 7]>

<link href="css/portail.css" rel="stylesheet" type="text/css" media="screen" />

<![endif]-->

<!--[if lte IE 6]>

<link href="portail/css/portail-IE6.css" rel="stylesheet" type="text/css" media="screen" />

<![endif]-->

<!--Appel Jquery 1.1.3.1 et les fonctions associées-->

<script type="text/javascript" src="utils/jquery.js"></script>

<script type="text/javascript" src="utils/jquery.imageMenu.js"></script>

<!--la gestion des effets du menu-->

<script language="javascript" type="text/javascript">

$(function(){

var menu = new imageMenu();

menu.build('#menu','.menuOption',50,'.menuOptionHover');

});

</script>

<!-- -->

</head>

<body>

<div id="g_site">

<div id="global">

<div id="navigation">

<div id="top">

<h1>Titre de la page</h1>

</div>

<ul id="menu">

<!-- #BeginLibraryItem "/Library/menu_site.lbi" -->

<li id="menu-1" class="menuOption"><a href="/index.php"><span class="menuOptionHover">Accueil</span></a></li>

<li id="menu-3" class="menuOption"><a href="/produits.php"><span class="menuOptionHover">Produit</span></a></li>

<li id="menu-4" class="menuOption"><a href="/services.php"><span class="menuOptionHover">Service</span></a></li>

<li id="menu-5" class="menuOption"><a href="/contact.php"><span class="menuOptionHover">Contact</span></a></li>

<li id="menu-2" class="menuOption"><a href="#"><span class="menuOptionHover">Client</span></a></li>

<!-- #EndLibraryItem -->

</ul>

</div>

<div id="actuhome">

<!-- InstanceBeginEditable name="Centre" -->

<div id="lecture">

<!--Contient le texte centrale de chaque page-->

<h1>Titre de la page</h1>

<p>lorem ipsum.....</p>

</div>

<div id="voir_aussi">

<!--Placer ici des pdf à télécharger, des breves, en rapport avec le contenu de la page-->

<h1 class="titreRubrique">Télécharger</h1>

<p>Pdf à telécharger</p>

<a href="pdf/fichier.pdf"><img src="css/images/pdf.gif" border="0"/></a>

<p> </p>

</div>

<!-- InstanceEndEditable -->

</div>

<div id="accordion">

<!-- InstanceBeginEditable name="Gauche" -->

<h2>Bon à savoir</h2>

<ul>

<li><a href="#">Item</a></li>

<li><a href="#">Item</a></li>

<li><a href="#">Item</a></li>

<li><a href="#">Item</a></li>

<li><a href="#">Item</a></li>

<li><a href="#">Item</a></li>

</ul>

<!-- InstanceEndEditable -->

</div>

<div id="copy">

<p><a href="mentions.php">Mention Legales</a> - © 2007-2008 Société Machin SARL</p>

</div>

</div>

<div id="close"></div>

</div>

</body>

<!-- InstanceEnd -->

</html>

D'après vous, ce code est-il propre ?

Est-ce du travail bâclé ?

Comment est-il perfectible (en restant dans de l'HTML pur) ?

Merci de vos avis éclairés.

++

Stef'

Modifié par Dan
Dan : merci d'utiliser la balise codebox pour les longs codes
Lien vers le commentaire
Partager sur d’autres sites
steph13

steph13

Posté
  • Auteur
Posté

Ca affiche tout simplement une page web, avec un header, une colonne, et un footer.

Un petit menu en JS (géré par Jquery), rien de bien sorcier en fait.

Mais j'ai besoin de savoir si il y a des choses infamantes dans ce code, des erreurs, des choses perfectibles, ect...

Je vous dirais le pourquoi du comment ensuite.

Merci.

Stef'

Lien vers le commentaire
Partager sur d’autres sites
petit-ourson

petit-ourson

Posté
Posté

En HTML pur, oui je pense qu'on peut dire que c'est un bon code.

Il manque un alt, il y a des h1 multiples mais à part cela c'est propre. Peut être un peut trop de div inutile mais bon, on a vu pire.

Reste à savoir ce que font les css, et si le fait de mettre 3 css pour Internet Explorer est une bonne idée.

Lien vers le commentaire
Partager sur d’autres sites
WebmasterL

WebmasterL

Posté
Posté

Salut,

Ben en fait je crois que ton code a été réalisé avec un éditeur comme frontpage ou autres.

J'ai rencontré un site avec le même style de code.

<--- if et endif --->

Je pense que c'est ton problème majeur euh sauf si je me trompe :smartass:

Avec quoi as tu réalisé ta page ?

Lien vers le commentaire
Partager sur d’autres sites
steph13

steph13

Posté
  • Auteur
Posté

Ce bout de code est "masqué" via le css.

				<div id="top">
					<h1>Titre de la page</h1>
				</div>

Il n'apparait donc pas sur la page présentée à l'internaute.

Les 3 css, j'y suis arrivé pour m'éviter d'avoir des hacks de partout. Choix discutable et longuement débattu un peu partout sur le net.

Je crois que c'est selon les "goûts" de chacun. Auparavant, j'utilisait les hacks, chose que je ne fais plus. Je trouve cela plus lisible.

Le code est en effet écrit avec DW. Pas le choix sur ce coup la, je devait livrer une page "template" utilisable sous DW pour le client final.

Pour vous éclairer un peu sur ma demande originelle, cette page a donc été écrite pour un client.

Or celui-souhaite "m'attaquer" sur le fait que ce code n'est ni fait, ni à faire.

De mon point de vue, je ne vois pas bien le problème. Hormis les divs inutile comme le souligne petit-ourson.

Lien vers le commentaire
Partager sur d’autres sites
pluriels

pluriels

Posté
Posté (modifié)

Je n'aime pas trop les commentaires conditionnels pour IE dans les feuilles de style.

Par contre sans le résultat final, on ne peut pas vraiment dire.

D'un point de vu code, je ne crois pas qu'on puisse dire quoi que ce soit.

Question optimisation, peut-être passer le div du contenu texte avant celui de la navigation.

Dernière chose, il faudrait le tester sur Dreamweaver en tant que template.

Il y a peut-être des interactions avec les balises commentaires DW ?

Mais je n'ai pas Dreamweaver. :P

Modifié par pluriels
Lien vers le commentaire
Partager sur d’autres sites
Dadou

Dadou

Posté
Posté

quelques maladresses dans le code, par exemple :

				
<div id="top">
	<h1>Titre de la page</h1>
</div>

Mettre directement :

<h1 id="top">Titre de la page</h1>

et :

<ul id="menu">
	<li id="menu-1" class="menuOption"><a href="/index.php"><span class="menuOptionHover">Accueil</span></a></li>
	<li id="menu-3" class="menuOption"><a href="/produits.php"><span class="menuOptionHover">Produit</span></a></li>
	<li id="menu-4" class="menuOption"><a href="/services.php"><span class="menuOptionHover">Service</span></a></li>
	<li id="menu-5" class="menuOption"><a href="/contact.php"><span class="menuOptionHover">Contact</span></a></li>
	<li id="menu-2" class="menuOption"><a href="#"><span class="menuOptionHover">Client</span></a></li>
</ul>

Plutôt : 

<ul id="menu">
	<li id="menu-1"><a href="/index.php"><span>Accueil</span></a></li>
	<li id="menu-3"><a href="/produits.php"><span>Produit</span></a></li>
	<li id="menu-4"><a href="/services.php"><span>Service</span></a></li>
	<li id="menu-5"><a href="/contact.php"><span>Contact</span></a></li>
	<li id="menu-2"><a href="#"><span>Client</span></a></li>
</ul>

et tu appelles ton javascript comme ça :

$(document).ready(function()
{
		var menu = new imageMenu();
		menu.build('#menu','li',50,'span');

});

en oubliant pas dans ton CSS de remplacer .menuOptionHover par span

Lien vers le commentaire
Partager sur d’autres sites
Dadou

Dadou

Posté
Posté
Pour vous éclairer un peu sur ma demande originelle, cette page a donc été écrite pour un client.

Or celui-souhaite "m'attaquer" sur le fait que ce code n'est ni fait, ni à faire.

T'attaquer comment ? qu'est ce qui lui faire dire que ce code n'est ni fait ni à faire?

Si tu as un zip de tous le dossier, je peux y jeter un oeil complet pour voir plus en détails

Lien vers le commentaire
Partager sur d’autres sites
chtipepere

chtipepere

Posté
Posté

Pour ma part :

- XHTML transitional, "pouak"

- on ne cache d'élément en CSS, c'est pas gentil

- pas de majuscule dans les noms des Meta

- jQuery pas à jour

- même css appelé si >IE7 ou !IE, donc pour moi une condition HTML en trop

- fonction javascript dans la page, et pas dans un fichier extérieur

- effectivement beaucoup de divs superflux

Mais si tous les sites de net étaient codés comme ton exemple, on serait dans un monde presque parfait, car mes remarques sont minimes (voire des améliorations), mais je ne vois pas pourquoi l'on pourrait t'attaquer sur ce genre de code, sauf si ton code ne respecte pas ce qui avait été demandé dans le cahier des charges, mais comme on ne connaît pas le contenu du cahier des charges...

Bon courage en tous cas.

Lien vers le commentaire
Partager sur d’autres sites
steph13

steph13

Posté
  • Auteur
Posté

Ah tiens, y'a des petits trucs que je ne savais pas dans ce que tu viens de m'écrire la.

M'attaquer comment ? Comme suit :

Il me doivent encore une partie du devis signé.

Leur deal :

A- Tu t'assois sur tes euros, car ton boulot est à chier et on te fou la paix.

B- Si tu ne veux pas, avocat, procédure et on demande à ce que tu nous rembourse les somme versées.

Sur le code par lui même, aucune info. J'attend le courrier qui devrait m'en dire plus.

Tout ce que je sais c'est ce que je viens de dire. A savoir "boulot ni fait, ni à faire".

Lien vers le commentaire
Partager sur d’autres sites
steph13

steph13

Posté
  • Auteur
Posté
Pour ma part :

- XHTML transitional, "pouak"

- on ne cache d'élément en CSS, c'est pas gentil

- pas de majuscule dans les noms des Meta

- jQuery pas à jour

- même css appelé si >IE7 ou !IE, donc pour moi une condition HTML en trop

- fonction javascript dans la page, et pas dans un fichier extérieur

- effectivement beaucoup de divs superflux

Mais si tous les sites de net étaient codés comme ton exemple, on serait dans un monde presque parfait, car mes remarques sont minimes (voire des améliorations), mais je ne vois pas pourquoi l'on pourrait t'attaquer sur ce genre de code, sauf si ton code ne respecte pas ce qui avait été demandé dans le cahier des charges, mais comme on ne connaît pas le contenu du cahier des charges...

Bon courage en tous cas.

Ok pour le css caché , mais demande client ;)

Jquery pas à jour > Yes, mais la dernière version ne fonctionne pas avec le menu en question... :(

Css !IE et IE7 >> Ah, coquille... !

JS dans la page >> Yes, mais le client veut retoucher le fichier lui même, et bon...

Divs superflus >> Si tu savais combien de fois j'ai fait et défait ce site... A m'arracher les cheveux.

Mais tout ceci est formateur ! Je suis dans la boucle depuis peu, et la, ce cas précis, il va m'en apprendre pas mal je crois...

Je suis tout à fait d'accord sur le fait qu'il y ait des améliorations à apporter. On apprend tous les jours :)

Lien vers le commentaire
Partager sur d’autres sites
Dadou

Dadou

Posté
Posté
- XHTML transitional, "pouak"

Ah non pas d'accord transitional reste un bon choix, il ne faut pas choisir strict pour faire du strict!

A mon avis c'est plus une tentative d'intimidation pour ne pas payer, tu es tombé sur des mauvais payeurs. Soit ils émettent des preuves sur la mauvaise qualité du code (donc ils devaient avoir quelqu'un qui aurait du être à même de faire ta prestation, et je ne vois pas forcement l'utilité de passer par un presta), soit ils paient !

Mais je réitère ma demande sur le code, j'aime pas trop les façons de faire.

Edit, j'avais pas vu ton mp, j'attend donc ton mail et vérifierait que le code est exploitable sur Dreamweaver au passage

Lien vers le commentaire
Partager sur d’autres sites
steph13

steph13

Posté
  • Auteur
Posté
Quel était le cahier des charges ? Dur de savoir si le client peut t'attaquer si on ne connait pas le cahier des charges.

Réduit à sa plus simple expréssion :

Livraison d'un site statique en XHTML et CSS, pas de php, aucun script server-side.

Dadou, je t'envoie tout cela de suite.

Merci du temps consacré.

Lien vers le commentaire
Partager sur d’autres sites
Dadou

Dadou

Posté
Posté

J'ai regardé le zip et mis sur mon serveur de test, effectué quelques modifs pour voir si le modèle est exploitable sous Dream, aucun problème de ce coté tout a fonctionné parfaitement. Sinon, c'est un choix d'avoir pour un site statique tout en .php??

A part quelques maladresses de divite aigue, et l'inclusion des les styles selon le navigateur (d'ailleurs pour quel bug? la je ne peut faire de test sous IE, je suis sur un linux) rien qui ne mérite l'appellation de "ni fait ni a faire", le code reste dans sa grande globalité correct.

Moi je relancerais le client pour qu'il m'apporte des éclaircissement pour "ni fait ni a faire"!

.

Lien vers le commentaire
Partager sur d’autres sites
steph13

steph13

Posté
  • Auteur
Posté
J'ai regardé le zip et mis sur mon serveur de test, effectué quelques modifs pour voir si le modèle est exploitable sous Dream, aucun problème de ce coté tout a fonctionné parfaitement. Sinon, c'est un choix d'avoir pour un site statique tout en .php??

A part quelques maladresses de divite aigue, et l'inclusion des les styles selon le navigateur (d'ailleurs pour quel bug? la je ne peut faire de test sous IE, je suis sur un linux) rien qui ne mérite l'appellation de "ni fait ni a faire", le code reste dans sa grande globalité correct.

Moi je relancerais le client pour qu'il m'apporte des éclaircissement pour "ni fait ni a faire"!

.

L'extension php, c'est la force de l'habitude...

Les css ie/ff c'est tout simplement pour des question de margin, de bordures, etc....

Quand à la relance client, je me demande si je dois demander des explications claires ou attendre leur fameux courrier.

Lien vers le commentaire
Partager sur d’autres sites
Dadou

Dadou

Posté
Posté

Ils doivent t'envoyer un courrier?

Pour ma part, j'aime pas laisser les problèmes en suspens, et le courrier il est possible que tu l'attende longtemps : ils font probablement preuve de mauvaise fois pour ne pas payer. Plus tu attendras, plus ce sera difficile d'avoir quelque chose.

Lien vers le commentaire
Partager sur d’autres sites
Anonymus

Anonymus

Posté
Posté

Après avoir lu tout ca, à ta place, je dirais à mon client :

Chiche ? Ne vous donnez pas la peine de m'attaquer, j'le fais à votre place ;)

Va au tribunal, et demandes le paiement de ton dû.

Avant de prouver que ton code est à chier, il risque d'en baver pas mal :D

Parce que ce que l'on te demande, c'est que ca marche. Pas que les majuscules des meta soient ou non respectées.

C'est clairement un mauvais payeur, et ne t'emmerdes pas avec lui. Si tu ne veux pas aller au tribunal, alors envoies lui une lettre de relance, où tu lui déclares explicitement qu'en cas de non paiement sous xxx jours, alors tu saisira le tribunal compétent.

Ne fais pas mention de ce qu'il dit dans cette lettre, et s'il t'en reparles au téléphone, tu l'envoies chier direct :>

> Mon code est à chier ? Mettez le moi par écrit.

Vraiment : Ne t'inquiètes pas.

Ils essaient de t'intimider, et à ce jeu là, c'est le plus couillu qui gagne :D

T'es un professionnel ? T'as un devis ?

Vraiment, aucun soucis à te faire ;)

Lien vers le commentaire
Partager sur d’autres sites
steph13

steph13

Posté
  • Auteur
Posté

J'ai pris mes infos de partout ces derniers jours.

Alors, d'une j'ai en ma possession les devis concurrents qu'ils ont fait.

Le commercial me les avaient laissés !! :)

Or, je suis le "moins-disant". De peu, mais quand même.

Et ça, devant un tribunal, pour eux, c'est mort.

Autre point : Il ne sont pas content, mais le site est en ligne sur leur serveur...

C'est un peu je ne suis pas content, mais j'utilise quand même.

Nous avons en effet, signé un devis pour la livraison d'un site XHTML/CSS.

Je n'ai aucun papier officiel de leur entreprise à propos de quelque sujet que cela soit.

Je ne vous parle même pas du mail de leur commercial où je me fait purement traiter de voleur....

Hier, j'ai envoyé un premier recommandé avec A/R.

Au vu de tous ces éléments, il est clair que s'il ne me verse pas ce qu'ils me doivent, ne serait ce que par principe, on va y allez !!!

Je suis peut être une petite boite, mais je n'entend pas me faire piétiner de la sorte.

Merci pour votre soutien :)

Edit: C'est vrai que quand cela te tombe dessus la première fois, ça fait tout drôle, même si tu te sais honnête.

Passé ce sentiment, ce sont les reflexes de grands groupes qui reviennent : écrire, écrire, envoyé en recommandé, etc....

Lien vers le commentaire
Partager sur d’autres sites
Dadou

Dadou

Posté
Posté

Cela te permettra aussi d'apprendre un principe de base : ne jamais livrer avant paiement total, pour ma part, le paiement s'échelonnait de la manière suivante :

1/3 à la signature,

1/3 à la moitié du temps de la prestation,

et le dernier tiers à la livraison.

En gros, si ils ne paient pas la totalité, ils ne doivent pas pouvoir profiter de ton travail, ce qu'ils font actuellement.

Lien vers le commentaire
Partager sur d’autres sites
steph13

steph13

Posté
  • Auteur
Posté

C'est ce qui était initialement prévue...

Mais j'ai voulu être serviable en leur mettant le site à dispo.

Surtout que jusque la, tout se passait bien.

Ca m'apprendra !

Lien vers le commentaire
Partager sur d’autres sites
Dadou

Dadou

Posté
Posté

Deuxième règle d'or, tu peux te permettre d'être serviable qu'avec les clients que tu connais bien et avec lesquels tu n'a jamais eu de soucis. Je ne compte plus le nombre de clients quand je me montrais serviable qui traînaient des pieds pour payer (enfin, comme je m'occupais de l'hébergement, une journée d'indispo de leur site me permettais d'avoir un moyen de pression)

Lien vers le commentaire
Partager sur d’autres sites

Veuillez vous connecter pour commenter

Vous pourrez laisser un commentaire après vous êtes connecté.



Connectez-vous maintenant
 Partager
Aller sur la liste des sujets
×
×
  • Créer...