Feature/adriann elementary #1

Goodwitch68 · 2019-10-12T06:07:19Z

No description provided.

pjohn3

A következetes indentációra figyelj.
A 3-as feladat megoldása még nem jó.
Részleteket lásd a kommentekben.

Feladatok:

✔️ 1. Write a program that prints ‘Hello World’ to the screen.

✔️ 2. Write a program that asks the user for their name and greets them with their name.

❌ 3. Modify the previous program such that only the users Alice and Bob are greeted with their names.

✔️ 4. Write a program that asks the user for a number n and prints the sum of the numbers 1 to n

✔️ 5. Modify the previous program such that only multiples of three or five are considered in the sum, e.g. 3, 5, 6, 9, 10, 12, 15 for n=17

❔ 6. Write a program that asks the user for a number n and gives them the possibility to choose between computing the sum and computing the product of 1,…,n.

❔ 7. Write a program that prints a multiplication table for numbers up to 12.

❔ 8. Write a program that prints all prime numbers. (Note: if your programming language does not support arbitrary size numbers, printing all primes up to the largest number you can easily represent is fine too.)

❔ 9. Write a guessing game where the user has to guess a secret number. After every guess the program tells the user whether their number was too large or too small. At the end the number of tries needed should be printed. It counts only as one try if they input the same number multiple times consecutively.

❔ 10. Write a program that prints the next 20 leap years.

pjohn3 · 2019-10-12T13:43:50Z

helloworld/elementary_2_3.c

+
+int main() {
+
+    char name[100] ;/* define name */


Ez a sor 4 szóközzel van indentálva

pjohn3 · 2019-10-12T13:44:06Z

helloworld/elementary_2_3.c

+
+    char name[100] ;/* define name */
+
+  	setbuf(stdout, NULL);


Ez két szóközzel, és egy tab-bal

pjohn3 · 2019-10-12T13:51:57Z

helloworld/elementary_2_3.c

+
+  	setbuf(stdout, NULL);
+
+	printf("Hogy hívnak? ");    


Innentől meg minden egy vagy két tabbal, ha jól látom... Ez legyen kozisztens, mert más felületek más méretűnek mutatják a tab-ot. Pl. a Notepad++-ban 1 tab == 4 space, ezért ott nem látszik, hogy bármi fura lenne, de itt GitHub-on pl 1 tab kb 5-6 space méretű.

Vagy space-eket, vagy tab-okat használj. A Notepad++ alapértelmezésben tab-okat használ, ha te nem nyomsz mást. Szóval mikor manuálisan indentálsz, használj te is tab-ot, ne a space-t nyomkodd.

Notepad++-ban van egy olyan menüpont, hogy View > Show Symbol > Show White Space and TAB. Ha erre rákattintasz, halványan mutatja, hogy mi a space és mi a tab, és akkor látod. Ha zavar, akkor újra rákattintva kikapcsolhatod.

Ha van egy fájlod, amiben összevissza van keverve a tab és a space, akkor arra is van egy opció, hogy automatikusan lecseréljen minden 4 space-t tab-ra: Edit > Blank Operations > Space to TAB.

A (Leading) verzió csak a soreleji space-eket cseréli le TAB-okra. az (All) mindet.

OK, elolvastam, használom :-)

pjohn3 · 2019-10-12T14:00:50Z

helloworld/elementary_2_3.c

+	printf("Hogy hívnak? ");    
+	scanf("%s", &name);
+
+	if (name[0] == 'A' || name[0] == 'B') {


❗️ a feladatban az szerepelt, hogy "only the users Alice and Bob are greeted with their names".
Ez itt viszont Antoniettát és Barbarellát is köszönteni fogja, szóval még nem teljesítettük a feladat követelményét.

Ezen még dolgozom, egyelőre abba futottam bele, hogy "Bob" == "Bob" értéke 0 :-(

pjohn3 · 2019-10-12T14:01:26Z

helloworld/elementary_4.c

+
+int main() {
+
+    int szam;


Ez a sor is tab-okkal legyen indentálva, mint a többi. Most space-ekkel van.

pjohn3 · 2019-10-12T14:05:55Z

helloworld/elementary_4.c

+	printf("Írj egy számot: ");    
+	scanf("%d", &szam);
+
+	osszeg = (1+szam)*szam/2; // persze lehetett volna nagyon idétlen ciklussal is megoldani, de ez elegánsabb megoldásnak tűnt :-)


👍 Ez felvet egy izgalmas témát az algoritmusokról, amivel még találkozni fogsz. Van az ún. "big O notation", ami azt fejezi ki, hogy egy algoritmus által igénybevett idő miként függ a bemenetek számától, méretétől, stb.

Ha kétszer annyi elemmel valami kétszer annyi idejig is tart, magyarán az idő n-nel egyenesen arányok, akkor az algoritmusra azt mondják, hogy O(n) komplexitású. Ha a szükséges idő négyzetesen nő a bemenettel, akkor az O(n²). Ha úgy nő, mint a faktoriális, akkor O(n!).

Itt ha ciklust használtál volna, akkor O(n) komplexitású algoritmust írtál volna, mert minél nagyobb számot ír be az ember, annál többször kell lefutnia a ciklusnak. Ez a megoldás viszont O(1) komplexitású, aminek örülünk, mert ugyanolyan gyors bármely számra.

legközelebb osszeg = (1 + szam)*szam / 2;

pjohn3 · 2019-10-12T14:16:17Z

helloworld/elementary_4_5.c

+	int i;
+	for (i=1; i <= szam; i++)
+	{
+		if (i % 3 == 0 || i % 5 == 0) {
+
+			osszeg = osszeg + i;
+
+		}
+	}


Ha a <= operátor köré raksz space-t, akkor a = operátor köré is rakj:
for (i = 1; i <= szam; i++)

Ha az int main() { és az if (i % 3 == 0 || i % 5 == 0) {-nél ugyanarra a sorra raktad a nyitó kapcsos zárójelet, akkor for-nál ne kezdd új sorba.

osszeg = osszeg + i; helyett írhattad volna azt, hogy osszeg += i;

A megoldás jó, és szerintem ezt a választ várta aki kitalálta, de ha már az előzőt meg tudtad oldani jobban, akkor megsúgom, hogy erre is van egy jobb megoldás. A for loop minden egyes számon végigmegy, és minden alkalommal meg kell állapítani if-fel, hogy osztható-e az adott szám 3-al, vagy 5-tel, nem pazarlás ez?

Nem lehetne gyorsabban valahogy két for loop-pal csak a 3-al és az 5-tel osztható számokon végigmenni? és akkor még az if se kéne. Van vele valami bökkenő?

Szóval, lehetne azt csinálni, hogy végig megyek a 3-mmal osztható számokon úgy, hogy 3-mmal inkrementálok, aztán az öttel oszthatókon, úgy hogy 5-tel inkrementálok, aztán a 15-tel osztható számokat meg kivonom egy 3. looppal, amit 15-ösével inkrementálok :-). Gyorsabb lesz, de megírni lassabb. Szeretnéd, hogy megcsináljam?

pjohn3 · 2019-10-25T16:39:52Z

helloworld/elementary_2_3.c

 	scanf("%s", &name);

-	if (name[0] == 'A' || name[0] == 'B') {
+	if ( name == "Alice" || name == "Bob") {


A name elé nem kell space.

Arról, hogy miért nem működik a ==-val történő hasonlítás majd beszéljük.

Egyébként nem azt mondtad, hogy ezt egyszer már megoldottad hogy csak Alice-el és Bob-bal működjön, csak "továbbfejlesztetted", hogy megmutasd, hogy kezdőbetűk szerint is tudod?

pjohn3 · 2019-10-25T16:41:52Z

helloworld/elementary_10.c

+#include <time.h>
+int main() {
+
+	time_t s, val = 1;


Ezeket külön sorban deklaráld, az olvashatóbb.
Az is kicsit fura, hogy az egyiknek egyből adsz értéket, a másiknak meg csak két sorral lejebb.

Én így csinálnám:

time_t s = time(NULL); time_t val = 1;

pjohn3 · 2019-10-25T16:42:48Z

helloworld/elementary_10.c

+int main() {
+
+	time_t s, val = 1;
+	struct tm* current_time;


struct tm* current_time = localtime(&s);

pjohn3 · 2019-10-25T16:47:26Z

helloworld/elementary_10.c

+	current_time = localtime(&s);
+	printf("Current year = %d\n",(current_time->tm_year + 1900));
+	int current_year = current_time->tm_year + 1900;
+	int next_leap_year = 4 - current_year % 4;


❌ félrevezető változónév: a next_leap_year alapján arra gondol az ember, hogy ez a következő szökőév évszáma. Én valami olyan nevet adnék neki, hogy years_until_next_leap_year

pjohn3 · 2019-10-25T16:49:52Z

helloworld/elementary_10.c

+	int next_leap_year = 4 - current_year % 4;
+	int leap_year = current_year + next_leap_year;
+
+	for (int i = 0; i < 20; i++){


A kapcsos zárójel előtt legyen szóköz:
for (int i = 0; i < 20; i++) {

pjohn3 · 2019-10-25T17:59:26Z

helloworld/elementary_9.c

+	printf(" %d", a);
+	printf("\nGondoltam egy számra 1 és 100 között, találd ki melyikre!  \n");
+	scanf(" %d", &number);
+	trials++;


Ez miért kell?

pjohn3 · 2019-10-25T17:59:51Z

helloworld/elementary_9.c

+			trials--;
+			printf("ezt a számot épp most írtad, figyelj jobban!\n");
+		}
+		if (number < a){


space a { előtt

pjohn3 · 2019-10-25T18:00:13Z

helloworld/elementary_9.c

+		}
+		else {


pjohn3 · 2019-10-25T18:00:59Z

helloworld/elementary_9.c

+	printf("Szuper! Eltaláltad! %d-t találgattál.\n", trials);
+	return 0;
+}
+//most jó


Ezt ne ide írd, hanem a commit message-be (fájlnévvel együtt)

pjohn3 · 2019-10-25T18:01:41Z

helloworld/kukutyin.c

+#include<stdio.h>
+
+int main(){
+//definiáljuk a szükséges változókat
+	int ellen;
+	int adossag;
+
+	setbuf(stdout, NULL);
+
+	printf("\nHány ellenség van?");
+	scanf("%d", &ellen);
+	printf("\nHány tallér az adósság?");
+	scanf("%d", &adossag);
+
+	if ((ellen == 1 || ellen == 2 ) && adossag > 100){
+
+			printf("\nSKANDALUM");}
+
+	if (ellen == 0 && adossag > 0) {
+
+			printf("\nIMBECIL");
+		}
+
+
+
+return 0;
+}


Ez nem erre a branch-re való

Goodwitch68 added 5 commits October 12, 2019 07:52

finalization

8ca0a27

final version

ebe0fdf

final version

0610b58

az elemetary 4 és 5 feladat kész

29c851b

change indentation

a950431

pjohn3 requested changes Oct 12, 2019

View reviewed changes

Goodwitch68 added 24 commits October 13, 2019 09:36

Committing everything before reinstalling Acer.

f22c8ef

ez volt a megabyte megoldása ugyanerre a kérdésre

a7fb8c2

jól dönti el, hogy prim-e, de nem jól ír ki

726d085

itt nem ír már ki semmit, pedig csak egy if-et tettem bele

97147f0

Elementary_7 working, before adding is_prime().

5f65aef

Elementary_6 working with bug in printf

1859aeb

Elementary_8 working

f235aa4

Elementary_6 properly working ver

febeedf

Elementary_9 working, without eliminating of consecutiv repeatition

dce717a

elementary_9 proper working

8b35926

elementary_9 better

c5a2e5f

elementary_10 proper working

5d747d0

Interesting problem to discuss

b2c70df

Elementary-11 proper working

98cd6a5

Elementary_6 done, committed earlier

b4911af

Elementary_7 done, committed earlier

f335068

Elementary_8 done, committed earlier

c73e876

Elementary_2_3 Bob == Bob return 0, why?

d2d9c4b

elementary_2_3.c intended proper way

b6cb2f0

elementary_4.c intended proper way

4c30325

elementary_4_5.c intended proper way

f222325

elementary_6 intended proper way

03ef10d

elementary_7 intended proper way

d0cfd51

elementary_8 intended proper way

e561815

Goodwitch68 added 3 commits October 25, 2019 12:16

elementary_9 intended proper way

9a1d9b1

elementary_10 intended proper way

95a5c87

elementary_11 intended proper way

1404159

pjohn3 requested changes Oct 25, 2019

View reviewed changes

Feature/adriann elementary #1

Are you sure you want to change the base?

Feature/adriann elementary #1

Uh oh!

Conversation

Goodwitch68 commented Oct 12, 2019

Uh oh!

pjohn3 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants