C言語 関数に関して

Question

関数プロトタイプが以下の通り
int point(int data[],int n);であり
point の引数は整数型配列データdataと中に入っているデータの数nであります。
pointの処理は、data内の先頭からn個のデータのうち奇数のみ合計し、その結果を整数で返すものであります。しかし、nが負の場合は0を返す。
これを条件分岐と繰り返し処理を使用し、求めるソースコードを教えてください！

#include <stdio.h>

int point(int data[], int n) {
int ans, i;

for (i = 1; i <= n; i++) {
	if (i % 2 == 1) 
		
		ans = ans + i;
		return ans;
	

	else(n < 0) 

		return 0;
	}

}

これで正解か否かすらも分からないのでどなたか教えてください。
また、間違っている部分があれば指摘してほしいです。

Comments

[kei344]

回答が付いた質問の編集は慎重に行ってください。タイプミス程度なら修正する事もありますが、本文を削除するのはやめましょう。

[kei344]

ご自身の環境で実行され、エラーは出ましたか？その場合はエラーの内容も書いてください。また、コードはコードブロックで囲んでいただけませんか？ ```（バッククオート3つ）で囲み、前後に改行をいれるか、コードを選択して「</>」ボタンを押すとコードブロックになります。

[archiver]

一つ確認ですが、point()の処理は「data内の先頭からn個のデータのうち奇数のみ合計」ですよね？「dataの奇数番目の要素の合計」ではないですよね？ここの解釈の違いで指摘内容が変わります。

[teratail398]

data内の先頭からn個のデータのうち奇数のみ合計です。

[PineMatsu]

プログラムソースを追記したのですね。コードは```で囲うと見やすくなります。具体的には</>ボタンをクリックするとコードブロックのテンプレが出るのでそこにソースを流し込めばOKです。

Answer 1

こんばんは、

丸投げと言われてしまっていますね ^_^；
初めての質問なのにもう少し優しくしてくれればいいのにと思います。

推奨していない質問は、読んでると思いますが、読んでいなければ読んでみてください。

ざっくり言えば、人に全部考えてもらおうという感じの質問は嫌がられます。

自分ではこう考えて、このようにプログラムしてみたけど、どうしても分からないエラーが出てしまいます。という感じでソースコードを質問に載せれば皆さん進んで回答してくれますので、そのようにしてみてはいかがですか？

私もそのような質問には、自分がわからないことでも検索したりして答えたりすることもあります。
めんどくさいなと思うのであれば、teratailの考え方とは合わないので、無回答のまま忘れ去られるだけです。
なるほどと思ったら改めて質問を編集してみてくださいね ^^

Comments

[Mr_Roboto]

なんか悲しい気持ちになりました (TOT)

[earnest_gay]

自分も最初こんな感じでマイナス評価ついてしまいました。（どーん）
削除できると思ってた←

[Mr_Roboto]

コメントありがとうございます。
私が未熟なばかりに残念なことになってしまいました。

いきなりマイナスを付ける人もどうかと思いますので、
丁寧に説明したつもりだったんですが、
こういう反応をする質問者様もいるのかと勉強になりました。

今回は、誰も幸せになりませんでしたね。とても 残念

残しておくのも皆さん嫌な気持ちになると思うので、
削除の依頼は、一応出しておきました。

Answer 2

タグがC++だったのでふざけてみた:

#include <iostream>
#include <valarray>
using namespace std;

int point(int data[], int n) {
  return valarray<int>(valarray<int>(data, n)[slice(1,n/2,2)]).sum(); // これでいいかな?
}
int main() {
  const int n = 10;
  int data[] = { 0, 1, 2, 3, 4, 5, 6 ,7 ,8, 9 };
  cout << point(data,n) << endl;
}

Comments

[archiver]

ほぉ、C++ではこう書きますか。これは思わぬ収穫です。

[episteme]

いやこう書くってゆーか、「条件分岐と繰り返し処理を使用し」をはぐらかしてみたかったんでwww

[mugicya]

質問者は意味わからないでしょうねｗ

[archiver]

epistemeさん、「はぐらかしてみたかった」というのは一目見てわかりました(^ ^)
それを踏まえて、こういう書き方があるんだなと感心した次第です。

Answer 3

では、一つずつ。まずforループですが、このループの中でdata[]の中を見ていこうとしているのだと思います。が、ループカウンタであるiの初期値が1となっていますね。これでは、dataの２番目以降の要素しか見なくなります。なぜならば、C言語の配列の添字（[]の中の数値）は"0"始まりだからです。したがって、data配列の先頭要素から見るのであれば、iの初期値は0でなくてはなりません。ということで、ループの終了条件も…（ここは自分で考えてみてください）。
二つ目。forループ内の条件文ですが、

if (i % 2 == 1)

としていますね。iがdata配列の添字になるとすれば、この判定では「data配列の添字が奇数か」という判定になってしまいます。問題とは違ってきますね。「data配列の要素が奇数であれば」というのが問題で意図しているものなので、ここの判定文も変わります。

ロジックでの明らかな間違いとして、forループの中で

ans = ans + i;
return ans; 

としていますが、これでは最初の要素だけ判定して、残りは見ないで処理を抜けますよ。それと、ansの中身も期待しているものではないです。

それとは別に文法的な間違いの指摘ですが、else節の後には条件式は付きません。さらに条件判断するのであればelse if(条件式)となります（コンパイルしてないのが見え見えですね。これはコンパイラでエラーにします）。ということで、コンパイルもかけましょう。文法間違いはそこでわかります。ここのelse節にはもう一つ言いたいことがありますが、動作上問題にはならないので、特に指摘には挙げません。気になるようでしたら、考えてみてください。

Answer 4

data内の先頭からn個のデータのうち奇数のみ合計し
これを条件分岐と繰り返し処理を使用し、

この文は理解できてますか？つまり、for文などを用いて配列の先頭からn個分のデータを走査することが繰り返し処理です。そして、奇数データかどうかを判断するのが条件分岐（if文）です。
合計はわかりますよね？
for文とif文、習ってますよね？それを使ってプログラムを書くわけです。

こんなプログラム、ここに出入りしている回答者なら簡単すぎる問題です。
初めてプログラミングに触れてどうして良いのかわからない、という状態かもしれません。そもそもプログラムって何？とかというレベルなのかもしれません。

たぶん学校の課題だと思うのですが、ここは宿題の解答を教える場ではなく、プログラム中に行き詰まってどうしてもわからない場合にどうしたら良いか教えてもらう場所です。なので、自分の書いたプログラムを載せて、エラーが出るのならエラー・メッセージを一緒に載せて、それについてコメントを貰う。そういう風にしないと「問題丸投げ」の指摘を受けることになるし、回答者も答えてやろうという気が起きません。

どこかで答えを教えてもらってそれをコピペして、などと「ええかっこ」はせず、宿題ができないのなら、できませんでしたと正直に先生に言って、もっと一から教えてもらうような態度を先生に示すべきです。それが、学習に対する正しい態度だと私は思います。

Comments

[Mr_Roboto]

回答ご苦労さまです。

人として成長中のかたのようです、私がなぜこんなコメントを書いているのか
右上の編集履歴も見てあげてください。

2016/07/01 00:08　投稿
2016/07/01 11:54　編集 ← ここ

いろいろあると思いますが、頑張って欲しいと思いますね。

[PineMatsu]

何ですか？この「うよゃ。なたまひかたま、さかだやひかたよなあまやらはさなやわりなま」って？暗号文？

[archiver]

運営さんが元に戻したみたいですね。でも質問者もあんな事するくらいだから、見返すことはしないんじゃないかなぁ。
epistemeさんみたいな思わぬ別解が出てきて面白いですけど。

Answer 5

何が正しくて何がまちがっているかはっきりさせるためにテストデータを用意しましょう。
分野によっては、
テストケースの作成と、テストの実行の方に
コードを書くよりも時間/手間をかけることもあります。

充分なテストデータを使って、調べればコードが間違っているかどうか一目だと思います。
充分なテストデータとは、
一般的なデータだけではなく、
変則的な正しいデータ、たとえばデータの個数が0、
間違ったデータ、ポインタが配列をさしていないもの、データの個数が負
などを使う必要があります。

Answer 6

ひねくれた回答してみる。動作未確認

int point(int data[], int n)
{
return 0 < n ? (data[0] * (data[0] & 1) + point(&data[1], n - 1) : 0);
}

Answer 7

偶数奇数の判定は簡単にできます。入力がｎとして、

if(n & 1) で trueなら奇数です。

この辺りがわからないのであれば、２進数をちゃんと見て考えましょう。