プログラムをすっきり整然と書くコツを教えて下さい。

こういう見た目のじゃなくて、中身の話ですかね？？

オブジェクト指向でしょうか。

ＰＨＰに関しては 二番目のＵＲＬが参考になると思います。

いきなりすっきりは書けないと思います。試行錯誤しましょう。

最低限、やるべきことは

１．名前でわかる関数・変数名の役割

　１文字の a() なんてのはダメです。

　短い≠良いプログラム。

　getUserInfoById($id)、とかなら何をしたいのか一目瞭然。

２．積極的に連想配列を使いましょう。

　$user = getUserInfoById($id);

　echo($user[0]); ←これじゃわからない。

　echo($user[’name’]);←これなら、わかる。

３．定石を使え。

　プログラミングには定石があります。

　奇をてらった書き方は、自己満足以外の何者でもありません。

PHP に限らず、プログラミング全般における定石というのであれば、一般的によく使われるアルゴリズムをまとめた本などを、一度読まれてみるのをオススメします。

実際教育の場で、プログラミング技法をとりあげる場合は、よく使われる定石として、広く知られたアルゴリズムについての学習は欠かせないものとなっています。

URLはアルゴリズムをまとめた図書の一例で、私も持っていますが、C言語は PHP とも割と文法が似ていますので、掲載されているコードも流用しやすいかと思います。

スクリプト言語はどうしてもごちゃごちゃしてきて人の人のソースをチェックしていると泣きそうになることがあります。

すっきりの反対の悪例をあげるので参考にしてください。この点に気をつけていれば相当すっきりしたプログラムができる予感。

共通関数があるのに利用していない。

やたらとフラグをつかっている。

変数の命名規約などの基本的なコーディング規約にちっとも即していない。

同じような関数で微妙に違うのがいくつもある。

そもそも関数なんてない。

ページレイアウトの為のHTMLと実行の為のスクリプトが複雑に入り組んでいる。

ネストが長い。

ひとつの処理が２スクロール以上もある…

外部で定義されているスタイルシートを利用していない。

外部で定義されているページヘッダー情報などを読み込んでいない。

同じ事が永遠と書き連ねてある。

条件判断のための変数が無意味に多く事態を無意味に複雑化させている。

普通にクライアントスクリプトにサーバーサイドスクリプトが盛り込まれている。

仕様として可変的な固定値(数値等)がコードに直接記述されている。

固定値等を宣言したincをそもそも用意していない。

そんなところですかねぇ〜。

これに気をつけていれば相当すっきりしますよ。

どの言語でも通用するであろう基本的な事が書かれています。

個々の言語で具体的にどうするかに関しては、「コーディング標準」というキーワードで検索すると良いかも。

でも一番大切なのは、コーディングする人自身がやりたい事をしっかりと把握する事ですね。

そのコードで実現したい事を本人が理解していなければ、技術的な知識がいくらあっても結局雑然としてしまいますから。

プログラムをすっきり書く第一歩は、

「明確な仕様書を作成する」です。これ絶対！

プログラムを書き出してから想像力を働かせると

２度手間３度手間になり処理も複雑化してしまいがち

です。

変数リストやフローチャートも必須です。

プログラマ泣かせはクライアントによる思いつきの

仕様変更ですからｗ

> ちなみに現在、PHPでプログラムを作成しています。

徹底したコードリーディング、PEAR や優れたオープンソースアプリケーションを読む。海外のコードで、権威のある所を数こなして優れたコードに出会いましょう。

コメントについては PEAR を見れば PhpDocumentor を意識したコメントが為されているのでそれでいいでしょう。

他の方がコーディングポリシーについて触れているようですが、それなら まるごと PHP の上鍵さんの章によくまとまっています。

アンテナを高く上げ、優れたコード、優れた手法を常に探求し出会いを求める、これに尽きると思います。