解決済みの質問なので大丈夫です

Question

ポインタの使い方やループについての質問をしましたが、予期せぬ指摘を受けたのでコードを削除させていただきました。

Comments

[mjk]

引用元の問題があるならサイトへのリンクを明記すべきです。
問題を提供しているサイトや問題を作った方への敬意が欠けていると思います。
問題がオンラインのものであれば解説や模範解答や提出済みのコードがあるはずです。
それらを見たのか見ていないのかや見ても解決出来ないのかどうかも質問内容に書けるはずです。

[退会済みユーザー]

サイトの問題ではないです模範解答も解説もないです。
指摘する前にそれ以外の可能性も考えるべきではないのしょうか。

[mjk]

>>問題がオンラインのものであれば
そういうことを言うかもしれないと思い但し書きもしています。
オンラインでなければ学校の宿題なのか著作物なら著者や著書名を明記すべきです。
どちらにしても問題の出自を伏せてることは明白なので印象が悪いことに変わりはありません。
社会人の方でしたらそれくらい常識だと思います。
学生の方だとしてもそれくらい常識だということに注意したほうが良いと思います。

[退会済みユーザー]

友人から出された問題ですと明記する必要が感じられませんでしたので明記しませんでした。但し書きをしたとしても～すべきです。敬意が欠けていると思います。などこちらからも印象がとても悪いので返信したまでです。常識だの語られる筋合いはありません。どうぞこれからもいろんな可能性を考慮して指摘し続けてください。

[thkana]

友人、問題出しまくっているみたい。
https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q14232887284

Answer 1

すでに指摘されている部分はいいとして
(私だったら、for( int s=i+1; s<n; s++)としそうですが、大した違いじゃないですね)

*k = fabs(t[0] - t[1]); に対応して
*a=0;
*b=1;
を設定しておかないと、最初の2つの要素の間が最小距離だったときに正しい答えが得られないことがあります。

また、mainも、悪いとは言いませんがポインタを変数に取らなくても

int main(void) {
    int n;
    scanf("%d",&n);
    double a[n];
    for (int i = 0; i < n; i++) {
        scanf("%lf", &a[i]);
    }//ここまで配列の作成と入力
    int p1, p2;//最初距離のペアの配列番号を記憶する変数
    double min;//最小距離
    cal(&p1,&p2,&min, a, n);
    printf("最も近いペア：(a[%d],a[%d]), 距離：%f\n", p1, p2, min);
    return 0;
}

で十分に思います。

さらに余計なことを言えば、fabs()が不安なら、二乗した値で比較する、という手もあります。

            double tmp=t[i]-t[s];
            if (tmp*tmp < (*k)*(*k)) { 
                *k = tmp<0?-tmp:tmp; //最小距離の記憶

とかなんとか。

Comments

[退会済みユーザー]

最初に最小距離だけでなく、*a=0;*b=1;も書き加える必要があるというのは完全に頭に無かったです。稚拙なコードですがアドバイス頂けてとても嬉しいです。

[退会済みユーザー]

すみません。メインでポインタを使わなかった場合、返り値として最小値など受け取る必要がありますよね。ポインタ無しで済む方法が思い浮かびません。

Answer 2

            count++;
            s = count;

の位置が間違っていると思います。

void cal(int* a, int* b, double* k, double *t, int n){
    int s = 1, count = 1;
    *k = fabs(t[0] - t[1]);
    for (int i=0; i < n - 1; i++) {
        for (s; s < n; s++) {
            if (fabs(t[i] - t[s]) < (*k)) { 
                *k = fabs(t[i] - t[s]); //最小距離の記憶
                *a = i;
                *b = s;
            }
        }
        count++;
        s = count;
    }
}

Comments

[thkana]

> count
> の位置が間違っていると思います。

それを指摘するなら、
int count=1;
for(int i=0;i<n-1;i++){
count++;
}
と使われるcountの値ってほぼi+1と同じだっていうことの方が先じゃないかしら。(つまりcount変数の存在意義って...?)

[Gak]

`s = i + 1;`にすればよい。というのは前の方の回答にあったので、
あくまで提示されたプログラムの間違いを回答しました。

[thkana]

「前の方の回答」でいうなら「それは無意味だ」と言われているわけで、敢えてその「無意味」を活かすのもどうでしょう...まぁ、そういう考え方での回答なら「そうですか」としか言いようがないですけれど。

[Gak]

考え方の違いですね。
`s = i + 1;`はプログラムの間違いの指摘ではなく、冗長な処理の指摘であり改善案の提示と考えたので、そもそも間違っていた点は指摘した方が良いと思いました。「そうですか」と流してください。

[退会済みユーザー]

お二方本当にありがとうございます。わざわざcountを使うのではなくiを使えばいいですよね、、自分のプログラムに合わせてアドバイス頂けてとても嬉しいです。

Answer 3

関数 cal()内の

            count++;
            s = count;

は何をしたいのでょうか?
多分、ここが間違いで、

    for (int i=0; i < n - 1; i++) {

の次に

        s = i + 1;

としたら、どうでしょう?

また、最初から、数値を入れてますが、毎回入力するのは大変なので、テストの時は、

    int n = 5;
    double a[] = {-4, 4.5, -1.1, 4.2, 10.4};

として、初期値とした方が、楽と思います。