2012-03-31 4 views
-1

私が組み立てたコードに関するコメントが必要です。私はライブサイトでそれを使用しようとしているので効率的かどうかを知りたい。私はプログラマーの人ではないので、私は試行錯誤の方法を使ってこのコードを作ったので、私の最初の試みである私に笑わないでください。 :)Jqueryコードのコメント

入力時間は入力フォームです。

ここにある:http://jsfiddle.net/Ace6h/

あなたが何か提案を持っている場合は、それらを共有してください。ありがとう!

+1

のですか? http://codereview.stackexchange.com/ –

+0

おそらくCodeReviewで改善する可能性がある_what_についての回答を得て、そのようなことを実行するために_how_を見つけられない場合は、ここでヘルプを入手してください。 –

+0

提案をありがとう、私はそこにそれを掲示するでしょう。 – m1r0

答えて

0

ワウ。非常に最適化されていません。

また、ラベルが属するスパンを肝炎やチェックボックスIDがここ

一意ではありませんが、これは、より良いコードレビューに尋ねていない可能性があります開始Fiddle

$(this).append('<div id="label">' + day + ': </div>') 
     .append('<select id="' + day + 'FromH" name="' + day + 'FromH" class="hour from"></select>') 
     .append('<select id="' + day + 'FromM" name="' + day + 'FromM" class="min from"></select>') 
     .append('<select id="' + day + 'FromAP" name="' + day + 'FromAP" class="ampm from"></select>') 
     .append(' to <select id="' + day + 'ToH" name="' + day + 'ToH" class="hour to"></select>') 
     .append('<select id="' + day + 'ToM" name="' + day + 'ToM" class="min to"></select>') 
     .append('<select id="' + day + 'ToAP" name="' + day + 'ToAP" class="ampm to"></select>') 
     .append(' <input type="checkbox" id="closed_'+day+'" name="closed" value="closed" class="closed" /> <label for="closed_'+day+'">Closed</label>'); 

}); 
+0

AM/PMから24時間に変更する必要があります.24時間、AM/PM – mplungjan

+0

mil/am/pmの場合は – mplungjan

+0

となります。再度、感謝します。 – m1r0

関連する問題