C言語のコマンドライン引数の関数を使った値の受け渡しについて

Question

コマンドライン引数もそうなんですが、ポインタの値の受け渡しがよく分かっていないのだと思います。
プログラムは引数として入力した値から、最大値を出力するプログラムです。
コマンドライン引数はchar型になってしまうので、intにするためにatoi()を使っています。
saidai()の&data[i]にするとエラーは起きないのですが、アドレスをint型のmaxに渡すことになるのでやっぱりうまく動きません。
maxをポインタ型にすることもやってみましたが、うまくいきませんでした。
できる限り詳しく原因と解決方法をおしえてくださいm(_ _)m

#include<stdio.h>
#include<stdlib.h>

void saidai(int max,char* data,int n){
max=0;
int i;
for(i=1;i<n;i++){
if( atoi( &data[i])>max )max=atoi(&data[i]);
}
}

       int main(int argc,char* argv[]){
           int max=0;
           int i;
           for(i=1;i<argc;i++){
               printf("入力値%d=%s\n",i,argv[i]);
           }
           
           saidai(max,*argv,argc);
           
           printf("最大値=%d\n",max);
           
           
       }

Comments

[rubato6809]

gccをお使いだと思うけど、ポインタのサイズは何バイトですか？ printf("%ld\n", sizeof(char *)); が何を表示するか確かめてください。

Answer 1

まず、

saidai(max,*argv,argc);

max を引数にしてますが、これだと値を返せません。引数は基本的に値を渡すものです
値を返すには、返り値にしましょう

max = saidai(argv,argc);

また、*argv だとまずいです（argv[0]を指してしまう）
そして、
saidai関数は返り値を返すようにして、引数はmainのそれと同じにしとけば、同じように扱えます

int　saidai(char* data[],int n){

...
return max;

}

＃結局別回答と全くおなじになってしまっている

Comments

[cateye]

ありがとうございます。細かい事を書く事が出来ずに気になっていたのですが^_^

[panda_man]

ありがとうございます。てっきり、関数の値の受け渡しはreturnがなくてもできると思い込んでいました。
返信遅れてすいません。ありがとうございました。

Answer 2

こんなの？

usr~/test/c % cat ct0.c 
#include <stdio.h>
#include <stdlib.h>
//
int saidai(char *data[],int n)
{
    int max= 0;
    int a= 0;
    for(int i= 1; i < n ; i++ ){
        a= atoi(data[i]);
        if( max < a ){
            max= a;
        }
    }
    return max;
}

int main(int argc, char *argv[])
{
    if( argc < 2 ){
        return 1;
    }
    //
    for(int i= 1; i < argc ; i++ ){
        puts(argv[i]);
    }
    //
    int max= saidai(argv,argc);
    //
    printf("最大値=%d\n",max);
    //
    return 0;
}

usr~/test/c % 
usr~/test/c % ./a.out 11 21 31 41 5 12 31
11
21
31
41
5
12
31
最大値=41