ソースプログラムのエラー

質問

自分で四則演算のプログラムを組んだのですが
エラーが出てお手上げ状態です。
どこが間違っているのかどなたか教えていただけませんか？？
ちなみに足し算の場合はいくつ足しても計算できるという
プログラムです☆

#include<stdio.h>
int main(int argc, char *argv[]){

int i,a=0,b=0,x=0;
int m=0;
if(argv[1]!="+"||argv[1]!="-"||argv[1]!="*"||argv[1]!="/"||){
printf("指定された文字ではありませんよ\n");
return -1;
}
a = atoi(argv[2]);
b = atoi(argv[3]);

if(argv[1]=="-"){
m=a-b;
}

if(argv[1]=="*"){
m=a*b;
}
if(argv[1]=="/"){
m=a/b;
}
if(argv[1]=="+"){
for(i=2;i<argc;i++){
x=atoi(argv[i]);
m+=x;

}

printf("%d\n",m);

return 0;
}

回答No.2

yosi_yosi · ベストアンサー

if(argv[1]!="+"||argv[1]!="-"||argv[1]!="*"||argv[1]!="/"||){
や
if(argv[1]=="-"){
などが間違っています。
文字列を比較する場合には、strcmp()もしくはstrncmp()を使いましょう。
なぜ間違っているか...それはargv[1]が何を示しているかを考えればおのずと見えてくるはずです。

補足

ちなみに実行時は
>ex10 * 2 3 と入力したいんです。
学習した範囲での関数しか使用できないので
strcmp()は使えないです(゜O゜;

回答No.5

KenKen07

アドバイス程度で申し訳ないのですが・・・

if文ですが文字列「""」との比較は関数を使わなければなりません。
ですが、文字「''」比較では問題なくできます。
文字で比較する場合の注意事項としては
main(int argc, char *argv[])
での引数*argv[]がポインタと配列が使われていることです。
現時点で使われているargv[1]はargv配列1番目のアドレスを比較しています。
ポインタの中の要素を取り出すにはどうすればいいのか、調べてみましょう。

if(argv[1]!="+"||argv[1]!="-"||argv[1]!="*"||argv[1]!="/"||)
なのですが、もう一度見直してください。
何を入力しても全てif文の中に入りreturn-1が返ってしまうと思います。（一番最後の「||」も余分ですｗ）

atoi関数ですが、必要なヘッダがincludeされていないため使用できない可能性があります。

以上の点を変更すれば動作すると思います。

参考URL：

http://www9.plala.or.jp/sgwr-t/

回答No.4

BLUEPIXY

argv[1]!="+"
とかしている部分を
*argv[1]!='+'
とします。

回答No.3

yosi_yosi

#2です。
strcmpを使えないのであれば、自作の関数を作るか、それとも別の方法を取るかです。
渡される引数が1文字のみと分かっている場合でしたら、
char op;
op = argv[1][0];
if( op=='+' || op=='-' || op=='*' || op=='/' ) {
.
.
.
で対応することが出来ます。

回答No.1

nin2-y

どんなエラーでしょうか？

if(*argv[1]!='+'&&*argv[1]!='-'&&*argv[1]!='*'&&*argv[1]!='/'){

忘れているなぁ、C、あっているんだろうか？