c言語 配列について

Question

コード
#include <stdio.h>
#include <string.h>
#define A 10
int main (void){
	double ring_buff[A];
	int head=0; /* データの先頭を表す変数 */ 
	int tail=0; /* データの最後を表す変数 */ 
	int counter=0;
	int i=0,j;
	char s[5];


	while(1){
		printf("操作を選択してください(enq/deq/end):");
		scanf("%s",s);
		//enq
		if(strcmp(s,"enq")==0){
			if(counter==A-1){
				printf("enqできません,deqするか\n");
			}
			else {
				printf("追加する値を入力してください==>");
				if(tail==A){
					tail=0;
				}
				scanf("%lf",&ring_buff[tail]);
				printf("tail=%d\n",tail);
				tail++;

				counter++;
			} 

		}
		//deq
		else if(strcmp(s,"deq")==0){
			if(counter!=0){

				printf("%2.0fはキューから削除されました\n",ring_buff[head]);
				printf("head=%d\n",head );
				head++;
				if(head==A-1){
					head=0;
				}

			}
			else{
				printf("deqできません.\nプログラムを終了します");

				break;
			}

			counter--;
		}
		//end
		else if(strcmp(s,"end")==0){
			printf("プログラムを終了します\n");
			break;
		}//end end

		//中身表示
		if(head<tail){
			for(i=0;i<tail-head;i++){
				printf("->%2.0f",ring_buff[head+i]);
			}
			printf("\n");
		}

		else if(head>tail){
			for(i=head;i<A-1;i++){	
				printf("->%2.0f",ring_buff[i]);
			}

			for(i=0;i<tail;i++){
				printf("->%2.0f",ring_buff[i]);
			}
			printf("\n");
		}

		else if(counter==0){
			printf("データは空です\n");
		}
		printf("counter=%d\n",counter );


	}

	return 0;
}

リングバッファで待ち行列を作ろうとしているのですが
実行でenqを9回(1,2,...,9)
そのあとdeqを6回
またenqを6回(10,11,..,15)
最後にdeqを6回したとき、理想は(13,14,15)と出るようにしたいのですが
9をdeqした後もう一度deqをすると10がdeqされず11がdeqされます

Comments

[y_waiwai]

提示のコードではどういう動作になるんでしょうか

[cateye]

インデントを揃えましょう。インデントの揃っていない、１００行も有るソースは見たくありません。
また、マクロ名 (#define aaa)は（必須ではないですが）大文字にしましょう。
これ→**ボールドテキスト**・・・・はなに？・・・修正（削除）しましょう。

[Daregada]

ええと、「実行でenqを9回」の時点で配列の長さ(5)が足りないんですが、これは意図的なもの(仕様の一部)なんでしょうか。提示された実装だと、途中で「enqできません(略)」になりそうだけど。

[MCV]

配列は10のつもりで書いていました
見にくい文章すいません修正します

Answer 1

-                 if(head==A-1){
+                 if(head==A){

-             for(i=head;i<A-1;i++){    
+             for(i=head;i<A;i++){    

Comments

[MCV]

理想の実行結果になりました
ありがとうございます

Answer 2

回答ではないですが、、

if(counter==aaa-1){

if(tail==aaa){

これは、
if(counter>=aaa-1){
こうしましょう。
なんかの具合でcounterのほうが大きくなってしまうと破綻します

＃aaaってマクロだった