C de bir program için yardım lütfen

quote:

Orijinalden alıntı: skoylu

saxahan, eğer 101 sayı varsa?

ve dahasi:

average = (double)sum / count;

Eğer hiç 7 ile bolunen sayi girilmemişse, bu artık ne acayip şeyler yapar, kim bilir. Aslında sisteme gore bellidir, SIGFP (UNIX sinyali) veya muadili bir sinyalle programınız vahşice sonlandırılır sistem tarafından.

Bu işin iki yolu olabilir. Birisi, baştan kaç sayı gerektiğini öğrenmek. Diğeri ise, sayıların bittiğini gösteren bir bilgi girilmesini istemek.

Her iki yolla yapalım:

#include <stdio.h> 
  #include <stdlib.h> 
  #include <ctype.h> 
   
  int sayi_oku(int *sayi) { 
  	char okunan[16]; 
  	int inx = 0, i; 
  	while (read(0, & okunan[inx], 1) == 1) { 
  		if (okunan[inx] == '\n') { 
  			okunan[inx] = 0; 
  			if (strcmp(okunan, "end") == 0) return 1; 
  			for (i = 0; i < inx; i++) { 
  				if (i == 0)  
  					if (okunan[i] == '-') continue; 
  				if (!isdigit(okunan[i])) 
  					return -1; 
  			} 
  			*sayi = atoi(okunan); 
  			return 0; 
  		} 
  		inx++; 
  		if (inx > 15) { 
  			printf("Cok fazla karakter girdiniz. Lutfen bir sayı veya bitirmek için 'end' giriniz..\n"); 
  			inx = 0; 
  		} 
  	} 
  } 
  int main(char **argv, int argc) { 
  	int sayi_adedi, sayi, ret, *bolunen_sayilar, bs_adet, islenen; 
  	int toplam = 0, i; 
  	float ortalama; 
  	printf("Kac sayi girilecek:\n"); 
  	ret = sayi_oku(&sayi_adedi); 
  	if (ret) { 
  		printf("Hatali giris..\n"); 
  		return 1; 
  	} 
  	bolunen_sayilar = (int *)malloc(sayi_adedi * sizeof(int)); 
  	islenen = 0; 
  	bs_adet = 0; 
  	while (1) { 
  		ret = sayi_oku(&sayi); 
  		if (ret) { 
  			printf("Gecersiz sayi..\n"); 
  			continue; 
  		} 
  		if ((sayi % 7) == 0) {  
  			/* 7 ile tam bolunuyor.. */ 
  			bolunen_sayilar[bs_adet++] = sayi; 
  			toplam += sayi; 
  		} 
  		islenen++; 
  		if (islenen == sayi_adedi) break; 
  	} 
  	if (bs_adet == 0) { 
  		printf("Hic 7 ile bolunen sayi bulunamadi..\n"); 
  		return 0; 
  	} 
  	ortalama = toplam / (float) bs_adet; 
  	printf("7 ile tam bolunen sayilar: %d adet, ortalama: %.2f\n", bs_adet, ortalama); 
  	printf("Sayilar: %d", bolunen_sayilar[0]); 
  	for (i = 1; i < bs_adet; i++) printf(" %d", bolunen_sayilar[i]); 
  	printf("\n"); 
  	return 0; 
  }

Bu da kullanıcının sayı kalmadığını söylemesi ile çalışan versiyon:

int main(char **argv, int argc) { 
  	int sayi, ret, *bolunen_sayilar, bs_adet; 
  	int toplam = 0, i; 
  	float ortalama; 
  	bolunen_sayilar = NULL; 
  	bs_adet = 0; 
  	while (1) { 
  		ret = sayi_oku(&sayi); 
  		if (ret) { 
  			if (ret == 1) // 'end' girildi.. 
  				break; 
  			printf("Gecersiz sayi..\n"); 
  			continue; 
  		} 
  		if ((sayi % 7) == 0) {  
  			/* 7 ile tam bolunuyor.. */ 
  			bolunen_sayilar = realloc(bolunen_sayilar, sizeof(int) * (bs_adet + 1)); 
  			bolunen_sayilar[bs_adet++] = sayi; 
  			toplam += sayi; 
  		} 
  	} 
  	if (bs_adet == 0) { 
  		printf("Hic 7 ile bolunen sayi bulunamadi..\n"); 
  		return 0; 
  	} 
  	ortalama = toplam / (float) bs_adet; 
  	printf("7 ile tam bolunen sayilar: %d adet, ortalama: %.2f\n", bs_adet, ortalama); 
  	printf("Sayilar: %d", bolunen_sayilar[0]); 
  	for (i = 1; i < bs_adet; i++) printf(" %d", bolunen_sayilar[i]); 
  	printf("\n"); 
  	return 0; 
  } 
  

Elbette sadece main() tarafı. Bu küçücük yerde kod yazmak zor, tam bir zulüm. Bu yüzden bir kontrol edin atlanmış vs. bir şey olabilir. Derlerken, isdigit() için -lc eklemek gerekebilir. Umarım çalışır.

Ama çalışmasada, bu gibi bir şeyde bakılması gereken şu şeylere bir örnek vermek için makul olduğunu sanıyorum.

* Kullanıcıyı başıboş bırakmayın. Girdiği değeri kontrol altında tutun.
* sonradan belli olacak değerler için, baştan yer vs. ayırmayın.
* Her olası durumu göz önüne alın.

Umarım faydası olur.. Eğer kullanacaksanız münhal yerlere bir kaç printf() koyup ilgili mesajları ekleyin derim..

Ha soruda biraz acayiplik olduğu akılda tutulmalı. Zira, 0, 7 ila tam bölünür. 0 Hariç denmediği için, 0'ıda bir sayı olarak kabul ettik.

quote:

Orijinalden alıntı: skoylu

Sevgili saxahan 100 tane sayı girmesi zor filan değil :)

$ gcc -o saxahan saxahan.c
$ python -c "print 101 * '75\n'"|saxahan
Bummm...

Binde yapsan, 5 binde yapsan, bu iş yaş olur. Ve sen büyüttükçe iş daha yaş olmaya başlar. Herşeyden önce, 100 yaptın, 5 sayı giriliyor. O 95 sayı için neden kaynak harcadın? Müsrifliğin azı çoğu olmaz. Müsrif kod yazmaya alışmamak gerekir.

Olur o kadar, çalışıyorsa gene iyi. Ayaküstü, benzer bir şey nasıl olabilir, giriş seviyesi bir örnek. Forumda her gün bir scanf görmekten gına geldi.

Nasıl bir işse, ömür boyu hiç kullanmayacağınız ve kullanmamanız gereken bir şeyi, yeni başlayanın eline tutuşturmak hangi akla hizmettir, anlamış değilim. Ha, bize de böyle olmuştu. Elimize bir "input a%" verdiler, o yüzden boşu boşuna debelenip durduk senelerce.

quote:

Orijinalden alıntı: skoylu

İki gözüm, şu huyunuzu bir bırakın: Faraziye ile kod yazmak..

Nedne 100 olmasın? Neden 100 tane olsun? Böyle faraziyelerle kod yazılmaz. Yol basit:

printf("Kac sayi girilecek:");
scanf("%d", &SIZE);

int *array = malloc(SIZE)

yada

int *array = alloca(SIZE);

herneyse..

Size lazım olmayan kaynağı kullanmayın. Müsriflik yapmayın. Cimrilik iyi bir şeydir, elbette abartmamak kaydıyla.

Bu işin en zor tarafı bilgisayar gibi düşünebilmektir. Onunda yolu bu şekilde düşünmekten geçer. 100 adet INT için kafadan stackta yer ayırmak, abesle iştigaldir. Kafayı karıştırır, neden 100, neden daha fazla değil vs gibi bir sürü soruya gebedir. Ve KISS iyi bir kaidedir, uyulması gerekir.

quote:

Orijinalden alıntı: AlperXp

Kafam örneğe takıldı, son durumda, bellekteki boş veya çöplük bir sayı vb. herneyse kodla dolu olan bir bellek adresini gösterir p2 bildiğim kadarıyla, bunu da gayet 2-3 kitap okuyarak anlayabiliyorsunuz. Aslında bakarsak önemli olan istemek, sonra gerisi geliyor =)

quote:

max performans için bu yapıyı çok iyi değerlendirmek zorunda.
ama ilk amca işini doğru yaptı ise, C++ programcısı bunu düşünmemeli.