つづきです
/*求めた最大公約数で約分*/
if(*flag == 1){
*d = *m1 / *i;
*e = *n1 / *i;
}
else{
printf("約分できません。\n");
*d = *m1;
*e = *n1;
}
return 0;
}
int yakubun2(int *m2,int *n2,int *min2,int *flag,int *i,int *f,int *g)
{
/*最大公約数を見つける*/
if(*m2 < *n2){
*min2 = *m2;
}
else{
*min2 = *n2;
}
*flag = 0;
for(*i = min2; *i > 0; *i--){
if(*m2 % *i == 0){
if(*n2 % *i == 0){
*flag = 1;
break;
}
}
}
/*求めた最大公約数で約分*/
if(*flag == 1){
*f = *m2 / *i;
*g = *n2 / *i;
}
else{
printf("約分できません。\n");
*f = *m2;
*g = *n2;
}
return 0;
}
/*因数分解の結果を表示*/
int output(int *d,int *e,int *f,int *g)
{
printf("(%dχ-%d)(%dχ-%d)",*d,*e,*f,*g);
return 0;
}
関連URL:http://www.okweb.ne.jp/kotaeru.php3?q=474597
No.6ベストアンサー
- 回答日時:
>それから、変数iとflag以外にも
あとは、min1 と min2 だけでしょうか。
課題という事なので、プログラムが正しく動いているかどうかは試していませんが、全体を見て気になった事を・・・
yakubun1 と yakubun2 は同じ内容なので1つにまとめた方がいいです。これについては、bunkai1 と bunkai2 にも同じことが言えます。
1つにまとめると、試行錯誤している段階での修正量が減り、修正ミスも減ります。そして、C 言語の関数への理解度が高い事をアピールできます。
私の場合、ポインタを使って値を返す時、その変数を前に持ってくる事が多いですが、決まりというわけではないです。
yakubun(&d, &e, m1, n1);
yakubun(&f, &g, m2, n2);
void yakubun(int *rm, int *rn, int m, int n)
{
int i;
int flag;
int min;
/* 以下省略 */
}
No.5
- 回答日時:
前回は、特に指摘しなかったのですが、
普通、6X^2+28X+16を因数分解すると、
2(X+4)(3X+2) ですよね?
一般に、aX^2+bX+cがもし因数分解できれば、
p(qX-r)(sX-t) の形になると思います。
つまり、最後の printf は、
printf("%d(%dX-(%d))(%dX-(%d))", ...);
だと思いますよ。
この回答への補足
jmhさん、アドバイスありがとうございます。
確かにおっしゃられた通りです。
直したプログラムで6X^2+28X+16を因数分解したら、
(X+4)(3X+2) になってしまいました。
pは、どのようにプログラムで記述すればいいのですか?
今一、ピンときません。
よろしかったら、ご教授ください。お願いします。
No.4
- 回答日時:
#1, #2 の方の回答とも関連しますが、
プロトタイプ宣言→ int judge(int *,int *,int *);
呼び出し→ judge(&a,&b,&c);
関数の定義→ int judge(int *a,int *b,int *c,float *D)
になっているので、コンパイルが通ってしまったとしても、D が不定になります。
#3 の方の回答への補足ですが、
>for(*i = *min2; *i > 0; *i--){
*i-- は演算子の優先度から、(*i)-- としないと目的の結果にはならないので、このあたりは注意が必要です。
ポインタがよくわからないということですが、必要のないポインタを乱用していることが原因です。
関数 judge を例に、普通の呼び出し方を考えてみましょう。
/*因数分解できるか判断する関数*/
float judge(int a,int b,int c)
{
float D;
D = b * b - 4 * a * c;
if(D > 0 || D ==0){
printf("判別式は条件を満たしました。\n因数分解を行います。\n");
}
else if(D < 0){
printf("判別式はD < 0であるため、因数分解できません。\n");
exit(1);
}
return D;
}
int main(void)
{
...
D = judge(a, b, c);
...
}
ポインタは必要のないときには使わない方が誰にとっても見やすいです。
関数 main の中で全ての変数を定義するようなやりかたは良くないので、その関数だけで必要な変数は、その関数で定義するようにします。
関数 input などは、入力された値3つを main に返す必要があるから、今のままでいいのですが、特に返す値の無い関数は、戻り値の型を void としたほうが良いでしょう。
この回答への補足
皆さん、アドバイスどうもありがとうございます。
inthefloiさんのアドバイスを参考にして
前よりはプログラムが良くなりました。
しかし、まだ不具合がある感じです。
それから、変数iとflag以外にも
ローカル変数でもプログラムに支障がない変数
(ポインタである必要がない変数)を
教えては頂けないでしょうか?
お願いします
No.3
- 回答日時:
アルゴリズムは見ていませんが、ポインタを見たところ...
(2)の
for(*i = min1; *i > 0; *i--){
は
for(*i = *min1; *i > 0; *i--){
(3)の
for(*i = min2; *i > 0; *i--){
は
for(*i = *min2; *i > 0; *i--){
だと思います。
ちなみに素因数分解のプログラムは
http://www.okweb.ne.jp/kotaeru.php3?q=429203
の#4にあります。こちらも参考にしてはどうでしょう
No.1
- 回答日時:
if(D > 0 || D ==0){
printf("判別式は条件を満たしました。\n因数分解を行います。\n");
}
else if(D < 0){
ここは、
*D = (float)((*b) * (*b)) - (4 * (*a) * (*c));
if(*D > 0 || *D ==0){
printf("判別式は条件を満たしました。\n因数分解を行います。\n");
}
else if(*D < 0){
ではないでしょうか?
printf
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
似たような質問が見つかりました
- C言語・C++・C# c言語の問題です 3 2023/01/10 16:15
- C言語・C++・C# C言語のエラーについて 2 2022/07/11 13:56
- C言語・C++・C# プログラミング c言語 4 2023/03/07 01:05
- C言語・C++・C# c言語 プログラムのエラー 1 2023/02/11 20:31
- C言語・C++・C# c言語の問題の説明、各所ごとに 5 2023/07/26 11:03
- C言語・C++・C# プログラムの時、フローチャートはどうなりますか?図でお願いします。 int main(void) { 1 2022/10/01 22:45
- C言語・C++・C# C言語 3 2022/11/09 13:27
- C言語・C++・C# C 言語の Gauss Jordan 法について 2 2022/12/28 11:16
- C言語・C++・C# C言語 プログラミング 4 2022/05/22 11:53
- C言語・C++・C# プログラミングのペーパーテスト 実行結果を表示せよ #include <stdio.h> int h 1 2022/07/09 15:27
このQ&Aを見た人はこんなQ&Aも見ています
関連するカテゴリからQ&Aを探す
このQ&Aを見た人がよく見るQ&A
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
#define _CRT_SECURE_NO_WARNIN...
-
C言語での引数の省略方法
-
【C++】関数ポインタの使い方
-
C言語 エラーの原因がわからな...
-
「指定されたキャストは有効で...
-
(int *)の意味
-
16進数の計算の仕方を教えて下さい
-
任意の文字列のアルファベット...
-
毎回違う乱数を生成するにはど...
-
PowerShellがうまくいかない
-
構造体の勉強中です 合計点の高...
-
ポストの対応問題
-
数字列を3桁ごとにカンマで区切...
-
C言語の基礎 . 2乗値の差につ...
-
if と配列の組み合わせ
-
ラップ関数とはどんなものですか?
-
C言語
-
C言語のサイコロシミュレート
-
キーボードから自然数を入力し...
-
最早開始時間と最遅完了時刻を...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
「指定されたキャストは有効で...
-
C言語での引数の省略方法
-
#define _CRT_SECURE_NO_WARNIN...
-
複数桁10進数の*桁目だけを抽出...
-
へんな現象
-
【C++】関数ポインタの使い方
-
C言語 エラーの原因がわからな...
-
if と配列の組み合わせ
-
C言語での奇数の和
-
C言語 配列と関数の練習問題
-
ラップ関数とはどんなものですか?
-
(int *)の意味
-
C言語
-
実数の整数部,小数部の取得
-
足して100になるような乱数のア...
-
卒業研究でよく分からないとこ...
-
数字列を3桁ごとにカンマで区切...
-
c言語
-
std::set<int> で、ある値が何...
-
比較回数と交換回数表示について
おすすめ情報