【C】配列を一つ左にずらしたいがバスエラーになってしまう

Question

該当コード（一部抜粋）

関数に渡された文字列に空白やマイナス、プラス記号があった場合左にずらしてこれらの記号を消したいのですがバスエラーがでてしまいうまく動作しません。どこが原因なのでしょうか？

※2連続で記号が来た場合jを初期化しないとダメですがそこは一回保留で1つだけ記号がある場合を考えてます

   char str[] = "-1234" //example
   i = 0;
   j = 0;
	while (str[i] == '-' || str[i] == '+' || str[i] == ' ') // check minus plus sings
	{
		while (digit > 1) //digitはstrの桁数
		{
			str[j] = str[j + 1];
			j++;
			digit--;
		} //2連続で記号が来た場合jを初期化しないとダメですがそこは一回保留で1つだけ記号がある場合を考えてます
		i++;
	}

dititを求める関数

int ft_len(char *base) //to check base
{
	int len;

	len = 0;
	while (base[len] != '\0')
		len++;
	return (len);
}

該当コード全て

#include <stdio.h>
int	base_check(char *base)
{
	int	i;
	int	j;

	i = 0;
	j = i + 1;
	while (base[i] == '\0')
	{
		if (base[i] == '+' || base[i] == '-')
			return (0);
		while (base[j] != '\0')
		{
			if (base[i] == base[j])
				return (0);
			j++;
		}
		i++;
	}
	return (1);
}

int	ft_atoi_base(char *str, char *base)
{
	int	num[256];
	int	base_s;
	int sum;
	int minus;
	int i;
	int j;

	i = 0;
	j = 0;
	minus = 1;
	if (!(base_check(base)))
		return (0);
	while (str[j] == '-' || str[j] == '+' || str[j] == ' ')
	{
		if (str[j] == '-')
			minus *= -1;
		j++;
	}
	while ((str[i++] = str[j++]) != '\0');
	printf("%s ", str);

	return (0);
}

int main(void)
{
	int num;
	num = ft_atoi_base("-1234", "0123456789"); // A798BF(16) → 10983615(10)
	printf("%d", num);
	return 0;
}

Comments

[itagagaki]

digitが初期化されてないようですね。
あとstrの終端'\0'以降にアクセスしないようなコードにしないといけないような気もします。

[maisumakun]

「digitはstrの桁数」を割り出すコードはどのように書いていますか？

[退会済みユーザー]

digit = ft_len(str)で値は入っている状態です

[退会済みユーザー]

コード追加しました

[退会済みユーザー]

wsl2上のubuntuでgcc 9.3.0でビルドした限りでは、少なくともご提示のコードで落ちることはありませんでした。
エラーを起こしてしまうのはどういった環境で動作させていますでしょうか。

[退会済みユーザー]

あ、すみません、自分が知らないだけかもしれませんが、ft_lenってなんでしょう

[退会済みユーザー]

MacOS
Unix環境です

[退会済みユーザー]

自分は今Mac触れる環境にないのでこれ以上はお役に立てなさそうです。ごめんなさい。

ちなみに終端文字の処理が気になる所ではありますが、落ちないです。
(動作させると12344が出力されます)

[ozwk]

ご自身でこの質問のコードを実行して問題が再現しますか?
再現できたら再現に使ったコードを貼ってください。
現状ではコピペで実行できるようになっていないので色々付け加えないとならず、
回答者側で問題を再現させるのが難しくなっています。

Answer 1

考え方を変えれば、 ft_len 関数や"記号が複数の場合等を考える"必要はありません。
先頭から記号等をスキップして最初の数字の位置を求め、その位置から文字列の終端までを先頭に移動させます。

#include <stdio.h>

int main(void){
  char str[10] = "-1234"; //example

  //最初の数字の位置
  int j = 0;
  while(str[j] == '-' || str[j] == '+' || str[j] == ' ') j++;

  if(j > 0) {
    //j の位置から ’\0’ までを先頭にコピー
    for(int i=0; (str[i++]=str[j++]) != '\0'; );
  }

  printf(str);
}

Comments

[退会済みユーザー]

このやり方を採用してみたのですがやはりバスエラーでした違うところに原因があるみたいです

[退会済みユーザー]

違うファイルを作ってやったらなぜかできました。これはファイルが損傷してるとかそういう次元のお話なのでしょうか？

[jimbe]

> 違うファイルを作ってやったらなぜかできました。

二つのファイルの違いを出してくれるツールなどを使って違う箇所を探してみるというのはどうでしょう。
[diff(https://atmarkit.itmedia.co.jp/ait/spv/1704/13/news021.html) とか。

[退会済みユーザー]

やはり関数を書き足していくとbusエラーになります。

[退会済みユーザー]

該当コード全て載せたのでお願いします

[退会済みユーザー]

charの配列に一旦処理したい文字列をコピーし、その配列を渡せば通ります。

char arr[10];
strcpy(arr, "-1234");
num = ft_atoi_base(arr, "0123456789");

[退会済みユーザー]

文字列リテラルを書き換えようとして落ちてます。

[退会済みユーザー]

なるほどそういうことなのですね。ありがとうございます。

[jimbe]

char str[] = "-1234"
で書き換え可能な領域に6バイトの値が保存されそのアドレスが str となるはずですが、 shuta_202141 さんの環境ではそうならないということになりそうです。
その可能性があるため、私のコードではサイズを指定しました。

STR30-C. 文字列リテラルを変更しない
https://www.jpcert.or.jp/sc-rules/c-str30-c.html

STR11-C. 文字列リテラルで初期化される文字配列のサイズを指定しない
https://www.jpcert.or.jp/sc-rules/c-str11-c.html

Answer 2

一応自分が確認に使ったコードを貼っておきます。

#include <stdio.h>

int ft_len(char *base) //to check base
{
	int len;

	len = 0;
	while (base[len] != '\0')
		len++;
	return (len);
}

int main() {
  char str[] = "-1234"; //example
  int i = 0;
  int j = 0;
  int digit = ft_len(str);
	while (str[i] == '-' || str[i] == '+' || str[i] == ' ') // check minus plus sings
	{
		while (digit > 1) //digitはstrの桁数
		{
			str[j] = str[j + 1];
			j++;
			digit--;
		} //2連続で記号が来た場合jを初期化しないとダメですがそこは一回保留で1つだけ記号がある場合を考えてます
		i++;
  }
  printf("%s\n", str);
  return 0;
}

これを以下のコマンドでコンパイルしています。

> gcc -o test test.c

実行すると「12344」が出力されます。

※補足1：文字列長の算出は標準でstrlenという関数が用意されているので、特別な事情がないのであればそちらを使うのをお勧めします。

※補足2：今回の質問からは外れますが、このコードだと「先頭に記号が書かれている文字列」以外は最初のwhile文の条件ではじかれてしまうので、エラーが解消されても望んでいる挙動にはならないのではないかと思います。

Answer 3

文字列終端で処理を打ち切る処理が入ってませんね
そのため、配列の範囲を超えてアクセスしてしまってるんでしょう