自作CGIを評価するスレ
自分の作ったCGIをみんなに評価してもらうスレだよ。言語は問わないよ。 よほどプログラムが短くないかぎりはアプロダにでもアップしてね。
>>491 titleタグはmetaの後に書いてくらはい >>492 いま検索して調べたら metaタグの後にtitleタグを書いてました。 >>491 あのね、見た目とかを評価してもらうスレじゃないの ソースを公開してちょうだい > サーバーサイドで動いているのでCGIです。 CGIは動いたりしない。 >もっと勉強してください。 悪代官が黄門様に無礼者と言ってるのを思い出しちゃったw >>496 CGIとはWEBサーバーがWEBブラウザからの要求に応じてプログラム を起動するための仕組みです。 ”CGIは動いたりしない。” 馬鹿か? CGIはシステムでも仕掛けでもない。 インターフィエスの一仕様。 モジュールからプロセス作ることもできるが、 あれもCGI? >>502 どうせperl=cgiとか まだ思ってるアフォだろ? unlessはほかの言語と混同するからやめれ とか書いてある本があった気がする ほかの言語を使わなければいいだけの話なのに・・・ >>504 マジ? それって、if ははほかの言語と混同するからやめれというのと同じことだよね 汎用的なプログラミングの作法を身につける為なんじゃないかな?if だと他の言語でも大抵応用が効く。 同じような理由でperlの$_(暗黙の代入)はなるべく使うなとか書いてる本もあったよ。 >>506 なんだそりゃ?(藁 じゃあ混同しやすい elsif は使わないほうがいいってことか? >>508 差別はイクナイ! 何国ってのがどこのことだか知らんが,何国の人に失礼だよ(藁 >>509 ハァ? あたまのよわいのがなんにんか・・ って書いてあるが、どう読んだら何国の人になるんだ? 502,504,506,508が(ry >>510 「なにじんか」って読んだんだろ。>>511 の言うように洒落だろうがつまらん。 表計算ができるランキングCGIを作ってみたので評価お願いします。 http://ssr.xrea.jp けっこういい出来に仕上がったと思うけど、初めての人は操作にとまどうかも >>513 落としたYo とりあえず使えそう・面白そうな物だったら、朴らせてもらうわ(w ところで、ちゃんと評価してるのって最初のほうだけだね。 >>521 ・ここは見た目じゃなくコードを評価するスレだ。 ・use strict; しろ。 ・perlstyle 読め。 ・デカいの持ってきても面倒だから読まんぞ。 …のどれかに該当していたら、まともなレスは返ってこない。 DirectoryIndexの設定をしているtoktokも問題かと思うが。 Forbiddenにしたみたいだけど、 せめてファイル名変えなよ。 誰も居ない……ケチつけるなら今のうち。 if-elsifの書き方、 if{ .. }elsif{ .. } より if{ .. } elsif{ .. } の方が俺にとっては読み易いんだコンチクショー。 >>522 perlstyleなんて知らなかった…ありがとう、勉強になったよ。 例えば既成の掲示板のCGIとかから require 'axlog.pl'; AxLog::getlog('ログファイル名','ログ最大記録数'); の用に呼び出して使うアクセス解析。 何となく作ったんですが評価お願いします。 アクセス解析 http://torihamlog.s55.xrea.com/search/axlog.pl.txt ログビューワー http://torihamlog.s55.xrea.com/search/axlogv.cgi.txt ここにログビューワを設置して、それにアクセス解析しこんでます。 リモホとかIPとか晒す事になるので困る人は見ないで下さい。 パスワードは 1234 です。 http://torihamlog.s55.xrea.com/search/axlogv.cgi 念のため…コレ呼び出すとログを削除します。 http://torihamlog.s55.xrea.com/search/dellog.cgi >>529 とりあえず axlog.pl をザッと見た感じ。 > 39: my $logfile = ($_[0]) ? $_[0] : "log.dat"; my $logfile = $_[0] || "log.dat"; か my $logfile = defined $_[0] && length $_[0] ? $_[0] : "log.dat"; > 48: $ENV{'TZ'} = "JST-9"; local するか、呼び出し元と条件を揃える (設定しない) 。 > 58 〜 62 my($path) = $ENV{'SCRIPT_NAME'} =~ m|^(.*/)|; ログに ASCII 以外の文字を吐くのはあまりお勧めしない。 あと axlog.pl.txt や axlog.cgi.txt でも「.pl」や「.cgi」は含まれて いるので CGI プログラムとして動く。Apache のマニュアル参照。 >>529 axlogv.cgi 。 > 6: use vars qw/$pw $logfile $gzip_send $gzip/; 不要。 > 47 〜 53: my $gzip = $^O =~ /Win/ ? "gzip.exe -1 -c" : "/bin/gzip -1 -c" ; > 69: if ($mode eq "css") { Style_Sheets(); } わざわざ CGI で吐かせる理由は? > 74: else { print "Status: 403 Forbidden\n\n"; exit; } エラーメッセージを吐いた方が親切。 > 256: Content-type: text/plain; charset=Shift_JIS text/css >>530-531 評価ありがとうございます。 色々参考になります。 cssについてはファイルを一つにまとめたかったからです。 やっぱあんまり良くないですか? >> 74: else { print "Status: 403 Forbidden\n\n"; exit; } >エラーメッセージを吐いた方が親切。 if ($mode eq "css") { Style_Sheets(); } elsif ($pass ne $pw) { Error('パスワードが違います。') } elsif (($pass eq $pw) && ($mode eq "view") && ($ENV{'REQUEST_METHOD'} eq "POST")) { View(); } else { Pass_form(); } こうゆう風にしようと思っています。 >>532 ・CSS を変更する為にスクリプトを編集する危険を冒す必要がある。 ・外部に CSS を持つのに対し、常に二倍の回数実行される。 ・CSS を使い回す時に悩む事になる。 これらのデメリットを考慮の上で、なお一つにまとめるメリットの方が 勝ると判断したならいいんじゃないか? >>532 自己レス すげー馬鹿なこと書いていた if ($mode eq "css") { Style_Sheets(); } elsif (($pass) && ($pass ne $pw)) { Error('パスワードが違います。') } elsif (($pass eq $pw) && ($mode eq "view") && ($ENV{'REQUEST_METHOD'} eq "POST")) { View(); } else { Pass_form(); } たぶんコレでいいかな… 作った動機はなんとなくなんだけど、実用性とかどうでせう? 一応CGIの設置とかできて、ほんのちょっとだけいじれる程度人向けのつもりです。 いろいろわからんことだらけで作りましたが 添削して下しさい。 【スクリプト】http://www.tcn.ne.jp/ ~majima/dog/quiz.txt 【動作例】http://www.tcn.ne.jp/ ~majima/dog/quiz/quiz.cgi >>535 とりあえず画像のファイル名を変えた方が良いよw 20点頂きました >>535 もじゅーるは使えない鯖なんでつか? 使えるんなら使ったほうがいいでつよ。 Image::Magick、CGI、Jcode等々 何に使うのかは自分で調べてね。 EUCで書いたり表示したりしたほうがいいと思うけど、Shift_JISでやりたいのなら反対しないさ。 所々myを付けているが、どうせなら全部に付けちゃいな。 strictしろってこった。 ついでにHTML的にも4.01strict すれば? タイトル::時刻::内容\n というようなデータが数行あるような場合 それらを読んできて切り分けるって処理を してます。 これの評価お願いします。 sub split_data{ my @ref,@tmp,%hush; @tmp = @_; for (@tmp){ chomp; ($hush{title},$hush{time},$hush{contents}) = split /::/; push (@ref,{%hush}); } return(@ref); } @tmp = ("a::b::c\n","d::e::f\n","g::h::i\n"); @ref = split_data(@tmp) for(0..$#ref){ print "title:".$ref[$_]->{title}; print " time:".$ref[$_]->{time}; print " contents:".$ref[$_]->{contents}."\n"; } >>540 myで複数宣言するときは()がいる hushじゃなくてたぶんhashね. 連想配列っていみなら ハッシュのスライスを使うとエレガントかも @hash{qw/title time contents/} = split /::/; foreachつかえよ for(...$#ref) 541> >ハッシュのスライスを使うとエレガントかも >@hash{qw/title time contents/} = split /::/; ありがと、参考になりました。 my は複数でも()いりませんよ。バージョンによるだろうけど 最後に、今更foreach使ってる人いるんですか?w >>542 #--- test.p --- use strict; my $foo, $bar, $baz; __END__ > /usr/local/bin/perl -v This is perl, v5.8.5 built for i386-freebsd-64int <snip> > /usr/local/bin/perl -wc test.p Parentheses missing around "my" list at test.p line 2. Global symbol "$bar" requires explicit package name at test.p line 2. Global symbol "$baz" requires explicit package name at test.p line 2. test.p had compilation errors. > /usr/bin/perl -v This is perl, version 5.005_03 built for i386-freebsd <snip> > /usr/bin/perl -wc test.p Parens missing around "my" list at test.p line 2. Global symbol "$bar" requires explicit package name at test.p line 2. Global symbol "$baz" requires explicit package name at test.p line 2. test.p had compilation errors. >>541 >foreachつかえよ >>542 >今更foreach使ってる人いるんですか?w for と foreach は単なるシノニムなので、その表現は正しくないな。 foreach (LIST) BLOCK を for (LIST) BLOCK と略せるのと同様に、 for (EXPR; EXPR; EXPR) BLOCK を foreach (EXPR; EXPR; EXPR) BLOCK と書いてもエラーにならない。 foreach ループを意味するところで foreach と「書け」という >>541 の アドヴァイスは論拠の確かな正しいものだ。しかし Perl では C 風の for ループを使う機会が foreach ループに比べて圧倒的に少ない上、 人間の目からも for ループと foreach ループの区別は容易なので、 砕けた流暢な Perl を目指すならこだわる必要のない部分だと思うな。 >>540 %hush(多分hash?)の宣言の位置。 GCのコストを考慮した場合に、変数の再利用をするはあるけど...、 ループないで初期化してないとフィールドの数が少ない時に困るよ。 split/::/の代わりに正規表現を使ってデータを切り出すと、 データのチェックとsplitを同時にできて便利。 chompはデータ格納前に済ませておいた方が良いんじゃないかな。chomp(@tmp) もしくは、大抵はファイルから読み込むだろうからその時に。 >>544 どうもありがとう。foreachは普通に使えるんだろうけど、 大抵forで済ませてしまうのです。言い方悪かったです、スイマセン。 >>545 ありがとうございます。 >ループないで初期化してないとフィールドの数が少ない時に困るよ。 というのは、forのループ内で undef(%hush); (ほんとはhashでしたw) でいいですか? あと、フィールドの数が少ない時というのはどういうことでしょうか? >split/::/の代わりに正規表現を使ってデータを切り出すと どう書いたらいいのか思いつきません。宜しかったらご教授下さい。 >chompはデータ格納前に済ませておいた方が良いんじゃないかな。 そうですね。 open(FH,"file_name"); @tmp = <FH>; chomp(@tmp); close(FH); 言いたかったのはforeach (@ref)にしろよってことね 別にfor (@ref)でもいいかもしれんが あと>>543 読んだか? myは複数宣言するときは()いる >>546 >ループ内で初期化 for(hoge){ my %hash; statement; } こういう事だと思う my () つけなくてもエラー出てないです。 という事は 省略も可って事ではないでしょうか? >>548 エラーにならないのは strict 'vars' 宣言が無いからだ。 my $foo, $bar, $baz; という文自体は文法的には間違いじゃないが、 my は , より優先順位が高いので、my($foo), $bar, $baz; と書くのと 等価になる。 #--- test.p --- my $foo, $bar, $baz; print defined $main::{$_} ? "'$_' is global.\n" : "'$_' is rexical.\n" for qw(foo bar baz); __END__ > perl -w test.p Parentheses missing around "my" list at test.p line 1. Useless use of a variable in void context at test.p line 1. Useless use of a variable in void context at test.p line 1. Name "main::bar" used only once: possible typo at test.p line 1. Name "main::baz" used only once: possible typo at test.p line 1. 'foo' is rexical. 'bar' is global. 'baz' is global. >>546 > あと、フィールドの数が少ない時というのはどういうことでしょうか? これは、今回のケースでは問題なかったです。 でも、変数を再利用する時は気を付けないと、前の値が残っていたり、 意図しない場所の値を書き換えてしまったりするので注意。 例えば、pushの所を \%hash とりファレンスを使うように変更した場合。 my (@ref, %hash); foreach (@_) { # @refの中の前の値を上書きすることになる # {%hash}とした場合は、@refにはいってる値はコピーなので影響ない @hash[qw/title time contents/] = split/::/; push(@ref, \%hash); } my (@ref); foreach (@_) { my %hash; @hash[qw/title time contents/] = split/::/; # {%hash} でも期待通りに動作するが、余分なコピーを生成 push(@ref, \%hash); } 速度とかを重視するなら >>540 が効率良さそうだけど、(多分) バグとかみつかった場合に変数のスコープが広いと特定するのが困難になる。 >>542 > my は複数でも()いりませんよ。バージョンによるだろうけど 他の方に指摘のある通り。 最低限 use strict; use warnings;(もしくは -w) を宣言してないとPerlは教えてくれない。 ここってものすごいでかいスクリプトでも細かい所まで見てくれるの? >>550 何かおかしな日本語だった。訂正 s/他の方に/他の方からも/ あと、正規表現を使った方法を書き忘れてたのでついでに、、 例えば time が整数の場合の例 if (/^(.+)::(\d+)::(.+)$/) { @hash{qw/title time contents/} = ($1, $2, $3) } >>551 読むに耐えうる書き方&内容ならな。 KENTレベルなら却下 >>553 オブジェクト指向とか頑張ってやってみたけど中途半端になったようなのでも大丈夫ですか? 何書いたって叩かれるんだから、結城出していいと思う ほんとにこんなに色々な方が、教えてくださるとは思ってませんでした。 どうもありがとうございます。 特にエレガントな方法を教えてくださった542氏 ハイレベルな(少なくとも今の私には)指摘をしてくれた 545=550=552氏には感謝します。 その他あのようなケチなコードに意見を下さった方々ありがとう。 またお世話になるかもしれません。m(_ _)m > my は複数でも()いりませんよ。バージョンによるだろうけど > 最後に、今更foreach使ってる人いるんですか?w こんなこと言うやつはもう来なくていい 誘導されてきました。 評価というか不具合の問題です 初めてperl書いてみました。 簡易的な計算のプログラムを作ってみたんですがうまく動作しません。 ちょっと見て頂き、不具合等御指摘下さい 特定の計算式で、入力(1項目だけセレクトボックスにより選択)した数値を計算するものです。 ソース http://tool-ya.ddo.jp/2ch/trash-box/contents.jsp?file=20050123003423509.txt URL http://sr-cafe.chu.jp/kok/calc.html >>563 ソースはマンドクサイのでちゃんと読んでないけど、 文字コードと改行コードの問題の様な希ガス。 >>563 calc.html × <SELECT name="set" name="s1"> ○ <SELECT name="s1"> >>563 ReadFormData(*FORM); → %FORM=&ReadFormData; print <<END_OF_DATA; → print <<"END_OF_DATA"; Content-type: text/html; → Content-type: text/html\n\n 以下 sub ReadFormDataの修正 local(*FORM) = @_; → my %FORM=(); local($buf, $tmp, $name, $value); → my($buf, $tmp, $name, $value); return %FORM; ← 最後に追加 >>566 改行コード2つとダブルクォーテーション必要無くね? \n\nはいらんね。 ヒアドキュメントのダブルクォートはどちらかというとあったほうが・・・ ありがとうございます。 >>566 サンの通りにしたり、色々試してるんですがまだ動かないです… printが閉まってなくてフォームの値を処理出来てないんでしょうか… >>569 うちのサーバでスクリプト試したんだけどちゃんと動くぜ。 おまい、サーバに嫌われてるんだ。きっと。 問題は print <<END_OF_DATA 〜 END_OF_DATA までか、 まさかと思うがパーミッションか、perlの指定。 一度 print <<END_OF_DATA 〜 END_OF_DATA を たいした行数じゃないんだし、print "〜\n"; に置き換えてみ。 それからコメント行やめろ。余計見づらくなる。 >>569 今試したら566の変更だけで動いたわけだが。 Content-typeの行はそのままでも良いみたい。 動かないとしたらCGIの設置方法の問題。 まさかとは思うがローカルでテストしてるよね? つーか563のソースのままでも、ちゃんと結果出るじゃん。 問題はコードではない予感。 >>573 経験で言うとさ、ヒアドキュメントはサーバの機嫌で、最初はうまく動作して くれないことがある。Cookieなんか入れるとどうも変なんだよな。 そういうとき俺はメンドーでも print "〜\n" に置換えてやる。すると、あとは 「ヒアドキュメントでも読込んでやるかぁ」ってことでサーバは折り合ってくれる。 いや、あるんだって。コード書く時も、こうした方が処理が楽だろ、とか ムリしなくっていいから、とか気を使ってやると機嫌よくなる。ほんとだぜ。 577はコード丸写しで自作CGIと称してるだろw。 多かれ少なかれ576のような感じはあるな。 皆さん本当スイマセンでした! パーミッションの所為だった… 元のままでも566のコードでもどちらでも動きました。 >>571-572 そのとおりです。 ローカルでテスト出来るという事すら知りませんでした。 勉強します。 ねるねるねるねは Ψ(`∀´)Ψイッヒッヒッヒッヒ _▲_ ねればねるほど ((φ(..。) 色が変わって _▲_ こうやってつけて р(゚Д゚ ) *_▲_* ウマイ━━━*━ヘ(゚∀゚)ヘ━*━━━━!!!! テ-レッテレ-♪ ttp://c.hibi.info/Apple-1.zip どなたか評価(清書?)お願いします_ _ 指定されたスレの中から前日から当日までの既定の時間にあるレスを抽出して、 そのレスから株価コードを抜き出し株価の前日比、前日比率でソートしランキングを作成した後スレに書き込むツールです。 2chへの書き込み時のCookiee処理に曖昧な点があるのと、処理の最適化をして貰えると嬉しいです。 宜しくお願いします_ _ 他のスレから来ました。 Perlで他のサイトを取得して表示したいと思っています。 (著作権侵害とかそういうことをしたいのではないです) お時間のある方にお願いしますが、次のコードで@niftyのトップページを取得してみたら どうなるか教えていただけませんでしょうか? print "Content-Type: text/html\n"; print "\n"; use LWP::Simple; print get( 'http://www.nifty.com/' ); 私のところだと、ブラウザ上での表示がぐちゃぐちゃになるんですが…。 簡易予定表のようなものを組んでみたんですが、なんだかモッサリしてる気がします。 もっと簡潔に書けるような気がするんですが自分ではこれが限界でした。 あと、ファイルロックのやり方が合っているのか自信がありません。 一応動いていることは動いているんですが、問題はないでしょうか。 改良すべき点、根本的にこういう書き方に直した方が良い、とかあればアドバイスお願いします。 大まかな内容は以下のようなものです。 予定として保存する内容は「予定日(月と日)」と 「予定内容(長くても200バイト程度)」の2つでフォームから渡します。 予定を保存するファイルは月単位で作成します。 例えば2006年6月の予定の場合なら「200606.txt」というファイル名です。 保存するファイルがないときはその都度作成し、 6月1日 6月2日 6月3日 . . . という風に日付だけを先に入れておきます。 保存するファイルがあるときは、 フォームから渡された「予定日」とマッチした日付の次の行に「予定内容」を書き込みます。 内容は以上です。次レスにコードを晒します。 # 保存するファイル名 $a = sprintf("%04d%02d",$YEAR,$MON) $file = "$a.txt"; # 月から日数を出す if($MON == 1|3|5|7|8|10|12){ $m = 31; }elsif($MON == 4|6|9|11){ $m = 30; }else{ if($YEAR % 4 ==0){ $m = 29; }else{ $m = 28; } } for($i=1;$i<=$m;$i++){ $date .= "$MON月$i日\r\n\r\n"; } # 該当する月のファイルがなければ作成 if(!open(IN,"$file")){ open(OUT,">$file") flock(OUT,2); print OUT $date; close(OUT); } open(IN,"$file"); flock(IN,2); @A = <IN>; close(IN); # 「予定日」とファイル内の「日付」をマッチングさせて該当日付に「予定内容」を書き込む $loop = @A; $i = 0; while($loop > 0){ if($A[$i] eq $plan_day){# $plan_day = "フォームからの情報(予定日)"; $A[$i] .= "$plan_value\r\n";# $plan_value = "フォームからの情報(予定内容)"; } $loop --; $i ++; } if(!open(OUT, "+< $file")){&error;}; flock(OUT, 2); truncate(OUT, 0); seek(OUT, 0, 0); print OUT @A; close(OUT); >>589 まだちゃんと読んでないが…。 >if($MON == 1|3|5|7|8|10|12){ >}elsif($MON == 4|6|9|11){ これって比べてるのは、両方とも $MON == 15 だよね。 どっちの条件にも入らないと思うけど…正しく動作してるの? read.cgi ver 07.5.5 2024/06/08 Walang Kapalit ★ | Donguri System Team 5ちゃんねる