文字列の連結について

Question

#include<stdio.h>
#include<ctype.h>
#include<string.h>

int main(){
    char str[4][N];
    char s[N];
    int i;
    for (i=0; i<=3; i++) {
        scanf("%s",str[i]);
    }
    for (i=0; i<=3; i++) {
        strcat(s, toupper(str[i]);
    }
        printf("%s\n",s);
　　  return 0;
    }

二次元配列に4つの文字列を入力した後、その文字列を全て大文字にした後に一次元配列に連結させたいのですが、コンパイルエラーが出てしまいます。どうしたら良いか教えて欲しいです。

Comments

[maisumakun]

> コンパイルエラーが出てしまいます。

どのようなエラーが出たのか「正確に」ご提示ください。

[ozwk]

> どうしたら良いか

エラーメッセージを読んだ上でここがわからないということを書いてください

[y_waiwai]

エラーが出たなら、エラーメッセージを提示しましょう
エラーメッセージは、よけいな省略翻訳しないで出たそのママをコピペで提示してください

Answer 1

とりあえず、修正しておきます。
toupperが変換できるのは１文字だけです。

#include<stdio.h>
#include<ctype.h>
#include<string.h>
#define N   (128)
int main(){
    char str[4][N];
    char s[N] = "";
    int i,j;
    for (i=0; i<=3; i++) {
        scanf("%s",str[i]);
    }
    for (i=0; i<=3; i++) {
        for (j=0; j < N; j++){
            str[i][j] = toupper(str[i][j]);
        }
    }
    for (i=0; i<=3; i++) {
        strcat(s, str[i]);
    }
    printf("%s\n",s);
    return 0;
}

Answer 2

とりあえず、N が何も設定されていないので、
エラーにはなるでしょう。
#define
で N を定義するとか、
N を宣言して値を設定するとかが必要です。
まあ、普通は #define ですかね。

Answer 3

toupperの引数はintとなっていて、1文字だけを変換する関数です（man）。文字列ポインタを投げ込んでも意図通りに動きません。