Calculator γραμμένο σε C

[UnKnown96]

Για σας παιδιά!
Λιπόν μετά από αρκετό διάβασμα και εξάσκηση στη C αποφάσησα να φτιάξω ένα απλό Calculator.
Το ονόμασα CL-Calc (Command Line Calculator).
Λοιπόν εδώ είναι ο κώδικας:

Spoiler: show

Κώδικας: Επιλογή όλων


#include <stdio.h>
#include <math.h>

void main (void) {

double x, y;
char c, e=0;

puts ("Usage:\n======\n\nAddition:\t\t+ Addendum1 Addendum2\nSubtraction:\t\t- Reducer Subtrahend\nMultiplication:\t\t* Factor1 Factor2\nDivision:\t\t/ Divisor Divident\nPercentage:\t\t% Number\nPower:\t\t\t^ Base Exponent\nSquare root:\t\tr Number\nQuit:\t\t\tx");

do {

printf ("\n\n>>> ");
scanf ("%s", &c);

switch (c) {
case '+' :
scanf ("%lf %lf", &x, &y);
printf ("Total: %g", x + y);
break;

case '-' :
scanf ("%lf %lf", &x, &y);
printf ("Difference: %g", x - y);
break;

case '*' :
scanf ("%lf %lf", &x, &y);
printf ("Product: %g", x * y);
break;

case '/' :
scanf ("%lf %lf", &x, &y);
if (y == 0) printf ("Invalid divisor...");
else printf ("Quotient: %g | Remainder: %d", x / y, (int) x % (int) y);
break;

case '%' :
scanf ("%lf", &x);
printf ("Percentage: %g", x / 100);
break;

case '^' :
scanf ("%lf %lf", &x, &y);
printf ("Result: %g", pow (x, y) );
break;

case 'r' :
scanf ("%lf", &x);
printf ("Result: %g", sqrt (x) );
break;

case 'x' :
e=1;
break;

default:
printf ("Wrong option chosen...");
break;
}
}

while (!e);
}


Και ένα από τον MigF1.

Spoiler: show

Κώδικας: Επιλογή όλων


Ελέξτε εδώ:
http://ideone.com/Ce88n


Κάντε το compile με την εντολή gcc -lm <ονομα αρχείου>.c

[UnKnown96]

Παιδιά για κάποιο λόγο δεν κάνει σωστά τα space στα code tags ...
Ελπίζω να σας αρέσει όπως είναι γιατί δν μπορώ να το πιράξω.
Προσπάθισα και χωρίς αυτά αλλά τίποτα (συγνώμι QData.. )

[konnn]

1) Όταν γράψεις ένα γράμμα κολλάει και αρχίζει να βγάζει ασταμάτητο output.
και

Μπορείς εδώ να προσθέσεις έναν έλεγχο.

Στις επιλογές 1-5 δεν μπορεί να βγάλει αποτέλεσμα δεκαδικό

Τι εννοείς ακριβώς.

[UnKnown96]

Μπορείς να μου δώσεις και ένα παράδιγμα από κώδικα για τον έλεγχο γιατί τώρα άρχισα να μαθαίνω;
Εννοώ ότι ας πούμε ότι διαλέγεις την επιλογή 4 (διέρεση) και δώσεις 9/2 αντί να βγάλει 3,5 βγάζει 4...

[Qdata]

Δεν μπορώ να το κάνω compile.Μου βγάζει αυτό:

Κώδικας: Επιλογή όλων

$ gcc calc.c -o calc
calc.c: In function ‘main’:
calc.c:78:48: warning: incompatible implicit declaration of built-in function ‘pow’
calc.c:85:53: warning: incompatible implicit declaration of built-in function ‘sqrt’
/tmp/ccsCNbO4.o: In function `main':
calc.c:(.text+0x288): undefined reference to `pow'
calc.c:(.text+0x2e2): undefined reference to `sqrt'
calc.c:(.text+0x360): undefined reference to `getch'
collect2: ld returned 1 exit status


[UnKnown96]

Για κάποιο λόγο στο Linux δεν δέχετε τις εντολές pow sqrt και getch δοκίμασε κάτω από το #include <stdio.h> να βάλεις και αυτό : #include <math.h>
και πες μου αν δουλεύει.

edit: και αντικατέστησε το getch(); με getchar();

[Qdata]

Για κάποιο λόγο στο Linux δεν δέχετε τις εντολές pow sqrt και getch δοκίμασε κάτω από το #include <stdio.h> να βάλεις και αυτό : #include <math.h>
και πες μου αν δουλεύει.

edit: και αντικατέστησε το getch(); με getchar();

Τώρα μου βγάζει:

Κώδικας: Επιλογή όλων

$ gcc calc.c -o calc
/tmp/ccUJ2S1N.o: In function `main':
calc.c:(.text+0x288): undefined reference to `pow'
calc.c:(.text+0x2e2): undefined reference to `sqrt'
collect2: ld returned 1 exit status


Οπότε δεν δέχεται τις εντολές pow και sqrt.

[M.Jackson]

Σε βαθμολογώ με ένα 3.
Αυτά που πρέπει να διορθώσεις είναι τα εξής
1) Καλύτερα να φτιάξεις functions για την κάθε πράξη παρά να χρησιμοποιήσεις πράξεις μέσα στην case
2) Να διαβάζεις του αριθμούς μια φορά και όχι για κάθε πράξη.
3) Καλύτερα τον αριθμό για τις πράξεις να τον διαβάζεις μέσα σε ένα while έτσι ώστε να απομονώσεις την είσοδο λανθασμένου αριθμού για την πράξη. αποφεύγεις πολλά else κτλ..
4) Στην διαίρεση καλύτερα να ελέγχεις (μέσα στο function τις διαίρεσης για τον αριθμό 0 => 5/0 = αδύνατη πράξη)
5) Καλύτερα να έχεις μια γενική μεταβλητή DOUBLE/FLOAT για αν αποθηκεύεις το αποτέλεσμα για κάθε πράξη έτσι ώστε να μην έχει πρόβλημα με τους δυαδικούς/πραγματικούς αριθμούς.
Αυτά από εμένα
Καλή προσπάθεια πάντως!

[evolgen]

Για κάποιο λόγο στο Linux δεν δέχετε τις εντολές pow sqrt και getch δοκίμασε κάτω από το #include <stdio.h> να βάλεις και αυτό : #include <math.h>
και πες μου αν δουλεύει.

edit: και αντικατέστησε το getch(); με getchar();

Τώρα μου βγάζει:

Κώδικας: Επιλογή όλων

$ gcc calc.c -o calc
/tmp/ccUJ2S1N.o: In function `main':
calc.c:(.text+0x288): undefined reference to `pow'
calc.c:(.text+0x2e2): undefined reference to `sqrt'
collect2: ld returned 1 exit status


Οπότε δεν δέχεται τις εντολές pow και sqrt.

Μήπως αν δοκίμαζες και

Κώδικας: Επιλογή όλων

gcc -lm calc.c -o calc

[UnKnown96]

Για κάποιο λόγο στο Linux δεν δέχετε τις εντολές pow sqrt και getch δοκίμασε κάτω από το #include <stdio.h> να βάλεις και αυτό : #include <math.h>
και πες μου αν δουλεύει.

edit: και αντικατέστησε το getch(); με getchar();

Τώρα μου βγάζει:

Κώδικας: Επιλογή όλων

$ gcc calc.c -o calc
/tmp/ccUJ2S1N.o: In function `main':
calc.c:(.text+0x288): undefined reference to `pow'
calc.c:(.text+0x2e2): undefined reference to `sqrt'
collect2: ld returned 1 exit status


Οπότε δεν δέχεται τις εντολές pow και sqrt.

Στα Windows γιατί δουλέυει?
Έλεος...

Σε βαθμολογώ με ένα 3.
Αυτά που πρέπει να διορθώσεις είναι τα εξής
1) Καλύτερα να φτιάξεις functions για την κάθε πράξη παρά να χρησιμοποιήσεις πράξεις μέσα στην case
2) Να διαβάζεις του αριθμούς μια φορά και όχι για κάθε πράξη.
3) Καλύτερα τον αριθμό για τις πράξεις να τον διαβάζεις μέσα σε ένα while έτσι ώστε να απομονώσεις την είσοδο λανθασμένου αριθμού για την πράξη. αποφεύγεις πολλά else κτλ..
4) Στην διαίρεση καλύτερα να ελέγχεις (μέσα στο function τις διαίρεσης για τον αριθμό 0 => 5/0 = αδύνατη πράξη)
5) Καλύτερα να έχεις μια γενική μεταβλητή DOUBLE/FLOAT για αν αποθηκεύεις το αποτέλεσμα για κάθε πράξη έτσι ώστε να μην έχει πρόβλημα με τους δυαδικούς/πραγματικούς αριθμούς.
Αυτά από εμένα
Καλή προσπάθεια πάντως!

Σε ευχαρίστω πολύ!

Να σε ρωτίσω κάτι...
Εσένα σου δουλεύει;
Δεν καταλαβένω γιατί δεν δέχετε τις εντολές στον Qdata...