Suche und behebe den Fehler

[Michael Langmaack (Gast)]

Guten Tag,
ich hoffe ich bin im richtigen Forum und habe somit auch Erfolg mit 
meiner Frage.
Ich habe ein C Programm gegeben und soll nun den Fehler finden und es so 
umschreiben das es das tut was es soll.
_____________________________________________________________________
Programm ist als Bild im Anhang, konnte die nicht sauber hier ins 
Textfeld schreiben.
_____________________________________________________________________
Aufgabenstellung:
c)
Geben sie an, was das Programm ausgibt, wenn sie es mit 5 als Argument
aufrufen. Begründen sie ihre aussage.

d)
Korrigieren sie das Programm, so dass es korrekt funktioniert.


zu c) ich habe das Programm mal in einen online Compiler gehackt aber da 
kam nichts bei raus auch wenn ich z=5 eingegeben habe ... -.-

zu d) ich glaube das einfachste wäre eine math.h Bibliothek einzubinden 
und das Programm dann umzuschreiben, aber glaube nicht das es das ziel 
der aufgaben Stellung ist das so zu machen, da ja die Funktion _sqrt 
klar definiert wurde ... :/

Gibt es hier Ideen, Anmerkungen oder Lösungen ? Bin da gerade etwas 
hilflos ...

[Lillifee (Gast)]

Michael Langmaack schrieb:
> Bin da gerade etwas hilflos ...

Studiere bitte etwas anderes ...

[Eugen Marmorstein (Gast)]

Probiers mal mit

static void _sqrt(int &x)

Und außerdem berechnet Deine Funktion da keine Quadratwurzel sondern ein 
Quadrat!

[Norbert (Gast)]

Michael Langmaack schrieb:
> Gibt es hier Ideen, Anmerkungen oder Lösungen ? Bin da gerade etwas
> hilflos ...

Ausdrucken.
Dann auf ein hölzernes Küchenbrettchen kleben (die mit dem Griff)
Demjenigen der das Gelumpe fabriziert hat, nun dieses Brettchen rechts 
und links kräftig in die Fre…

[This is no Rätselforum (Gast)]

Michael Langmaack schrieb:
> ich hoffe ich bin im richtigen Forum und habe somit auch Erfolg mit
> meiner Frage.

> Gibt es hier Ideen, Anmerkungen oder Lösungen ? Bin da gerade etwas
> hilflos ...

Suche Dir ein Rätselforum.

[Sebastian (Gast)]

Michael Langmaack schrieb:
> zu c) ich habe das Programm mal in einen online Compiler gehackt aber da
> kam nichts bei raus auch wenn ich z=5 eingegeben habe ... -.-

Da hätte 0 bei rauskommen sollen, oder?

LG, Sebastian

[J. S. (jojos)]

Ikebana oder bei handwerklichem Geschick was mit Holz.

Ich mag solche Antworten auch nicht, aber wenn es kein Troll ist fällt 
mir nichts anderes ein.

[Sebastian (Gast)]

Michael Langmaack schrieb:
> zu c) ich habe das Programm mal in einen online Compiler gehackt aber da
> kam nichts bei raus auch wenn ich z=5 eingegeben habe ... -.-

Da hätte 0 bei rauskommen sollen, oder?

LG, Sebastian

Michael Langmaack schrieb:
> zu d) ich glaube das einfachste wäre eine math.h Bibliothek einzubinden
> und das Programm dann umzuschreiben, aber glaube nicht das es das ziel
> der aufgaben Stellung ist das so zu machen,

Ich sehe nicht wieso math.h hier hilfreich wäre ...

LG, Sebastian

[Heiner (Gast)]

Eugen Marmorstein schrieb:
> Probiers mal mit
>

static void _sqrt(int &x)

>


Na Kollege, das stimmt ja wohl nicht.
Nochmal probieren!!

[Daniel A. (daniel-a)]

Michael Langmaack schrieb:
> Programm ist als Bild im Anhang, konnte die nicht sauber hier ins
> Textfeld schreiben.

Steht doch direkt im Absendeformular oben. [c ]code[/c]
Übrigens, Hausaufgaben werden im Forum generell nicht gerne gesehen.

Michael Langmaack schrieb:
> ich habe das Programm mal in einen online Compiler gehackt aber da
> kam nichts bei raus auch wenn ich z=5 eingegeben habe ... -.-

argc ist die Anzahl an Argumenten. Mit dem if wird der ganze Teil mit 
der Abfrage nur ausgeführt, wenn argc 2 ist. Ohne Argumente ist argc 1, 
da das erste Argument (argv[0]), in der Regel der Programmname ist 
(obwohl es diverse Möglichkeiten gibt, auch argv[0] zu ändern). Damit 
argc 2 ist, müsste also exakt 1 Parameter übergeben werden. Da sollst du 
anscheinend 5 mitgeben. Also als "./program 5" oder so aufrufen, sonnst 
passiert gar nichts.

Michael Langmaack schrieb:
> zu d) ich glaube das einfachste wäre eine math.h Bibliothek einzubinden
> und das Programm dann umzuschreiben, aber glaube nicht das es das ziel
> der aufgaben Stellung ist das so zu machen, da ja die Funktion _sqrt
> klar definiert wurde ... :/

Das wäre sicher die Sauberste Lösung.

Das Programm oben hat 2 Fehler:
1) In C hat man immer pass by value. Wenn du der Funktion _sqrt einen 
Parameter übergibst, und dessen Wert darin änderst, hatt das ausserhalb 
der Funktion keinen Einfluss. Du könntest folgende 2 Sachen machen:
  1.a) Entweder das Resultat zurück geben (wie in der stdlib)
  1.b) Einen Pointer auf die Variable mit dem Wert übergeben, und den 
Wert auf den dieser Zeigt auslesen und dann überschreiben. Vermutlich 
will er das.
2) Vermutlich wurde _sqrt gewählt, weil sqrt bereits von der standard 
Library definiert wird. Die diversen C Standards legen aber fest, dass 
Namen die mit _ anfangen für die Implementierung reserviert sind. Die 
Funktion _sqrt zu nennen ist deshalb unportabl und implementation 
defined, man sollte soetwas nicht machen.

Zu
> d) Korrigieren sie das Programm, so dass es korrekt funktioniert.

Würde ich, wenn ich die Aufgabe bekommen hätte, zusätzlich zur Lösung 
noch anmerken, dass nicht definiert wurde, was das Programm denn tun 
solle. Abgesehen vom _ bei _sqrt, ist es ein gültiges C Program, und 
daher in keiner möglichen Definition tatsächlich falsch. Ohne die Info 
was das Programm tun soll, ist formell gesehen davon auszugehen, dass es 
korrekt funktioniert.

[J. S. (jojos)]

Online Compiler war dann doch eine gute Idee. Beim onlinegdb kann man 
auch im single step durch das Programm, das ist hilfreicher als fertige 
Lösungen.

[Heiner (Gast)]

Hallo Michael Langmaack

1.) die Funktion zur Berechnung muss einen Wert zurückgeben:

=> int _sqrt(int x)

dieser Wert muss ausgegeben werden mit printf:

=> printf("%d\n", _sqrt(z));

somit wird bei auf Aufruf:

sqrt 5 der Wert 25 angezeigt

[Eugen Marmorstein (Gast)]

Heiner schrieb:
> Na Kollege, das stimmt ja wohl nicht.
> Nochmal probieren!!

Hast Recht.

static void _sqrt(int *x)

[Forist (Gast)]

Michael Langmaack schrieb:
> Programm ist als Bild im Anhang, konnte die nicht sauber hier ins
> Textfeld schreiben.

Gelesen?

Formatierung (mehr Informationen...)

C-Code

Oder brauchst du auch noch jemand, der dir das vorliest?

[Heiner (Gast)]

Eugen Marmorstein schrieb:
> Heiner schrieb:
>> Na Kollege, das stimmt ja wohl nicht.
>> Nochmal probieren!!
>
> Hast Recht.
>
>

static void _sqrt(int *x)

Kollege!!

Hast Du Deinen Mist mal compiliert und getestet oder nur hirnlos hier 
reingeschrieben????

[Eugen Marmorstein (Gast)]

Heiner schrieb:
> Hast Du Deinen Mist mal compiliert und getestet oder nur hirnlos hier
> reingeschrieben????

Zweiteres, sonst hätt ich das ja abschreiben müssen.

[Heiner (Gast)]

Eugen Marmorstein schrieb:
> Heiner schrieb:
>> Hast Du Deinen Mist mal compiliert und getestet oder nur hirnlos hier
>> reingeschrieben????
>
> Zweiteres, sonst hätt ich das ja abschreiben müssen.

Dann halte Dich besser in Zukunft raus, da Du ja eh' keine Ahnung hast

[Heiner (Gast)]

Ok, um's kurz zu machen:

#include<stdio.h>

static int _sqrt(int x) {
  return x = x*x;
}
int main(int argc, char *argv[]) {
  if(2 == argc) {
    int z;
    sscanf(argv[1], "%d", &z);
    _sqrt(z);
    printf("%d\n", _sqrt(z));
  }
  return 0;
}

[Eugen Marmorstein (Gast)]

Heiner schrieb:
> Dann halte Dich besser in Zukunft raus, da Du ja eh' keine Ahnung hast

Ach lass mich doch auf meinem Mount Stupid in Frieden.

[Ingolf G. (frosch)]

Heiner schrieb:
> Ok, um's kurz zu machen:
> ...
>     _sqrt(z);
>     printf("%d\n", _sqrt(z));
>
Warum zwei mal
der Aufruf _sqrt(z) ?

C ist schon lustig  :D

[Norbert (Gast)]

Heiner schrieb:
> Ok, um's kurz zu machen:
> #include<stdio.h>
> static int _sqrt(int x) {
>   return x = x*x;
> }
> int main(int argc, char *argv[]) {
>   if(2 == argc) {
>     int z;
>     sscanf(argv[1], "%d", &z);
>10     _sqrt(z);
>11     printf("%d\n", _sqrt(z));
>   }
>   return 0;
> }

Ist das _sqrt(z) in Zeile 10 zum Anwärmen, damit's in Zeile 11 besser 
läuft? ;-)

[Heiner (Gast)]

Ingolf G. schrieb:
> Heiner schrieb:
>> Ok, um's kurz zu machen:
>> ...
>>     _sqrt(z);
>>     printf("%d\n", _sqrt(z));
>>
> Warum zwei mal
> der Aufruf _sqrt(z) ?
>
> C ist schon lustig  :D

Prima! Ein aufmerksamer Leser!! Hast den Test bestanden! Freu Dich!
Also:
- einmal dient's dazu, den Leser "wachzurütteln"
- einmal dient's dazu, das Ergebnis anzuzeigen

[Heiner (Gast)]

Norbert schrieb:
> Heiner schrieb:
>> Ok, um's kurz zu machen:

> Ist das _sqrt(z) in Zeile 10 zum Anwärmen, damit's in Zeile 11 besser
> läuft? ;-)

Auch Du hast den Test bestanden!!
Gratuliere!

[Norbert (Gast)]

Heiner schrieb:
> Gratuliere!

Gut verkauft! ;-)

[K.Thaler (Gast)]

Kürzer:
return x*x;

[Eugen Marmorstein (Gast)]

Danke fürs tippen Heiner.

#include<stdio.h>
static void _sqrt(int *x) {
  *x = *x * *x;
}
int main(int argc, char *argv[]) {
  if(2 == argc) {
    int z;
    sscanf(argv[1], "%d", &z);
    _sqrt(&z);
    printf("%d\n", z);
  }
  return 0;
}

[J. S. (jojos)]

K.Thaler schrieb:
> Kürzer:
> return x*x;

Kein Wunder das die Raketen nicht mehr zum Mond kommen.

[Daniel A. (daniel-a)]

Hier noch eine Version mit Pointer. Und die Funktion hab ich auch noch 
umbenannt.

#include<stdio.h>

void quadrat(int* x) {
  *x = *x**x;
}

int main(int argc, char *argv[]){
  if(2 != argc){
    printf("Usage: %s number", *argv);
    return 1;
  }
  int z;
  sscanf("5", "%d", &z);
  quadrat(&z);
  printf("%d\n", z);
  return 0;
}

[Paul (Gast)]

J. S. schrieb:
> K.Thaler schrieb:
>
>> Kürzer:
>> return x*x;
>
> Kein Wunder das die Raketen nicht mehr zum Mond kommen.

was ist daran falsch?

[Daniel A. (daniel-a)]

sqrt steht für Wurzel. Aber x*x ist das Gegenteil davon, das Quadrat.

[Noch was (Gast)]

> Aber x*x ist das Gegenteil davon

Falls dies der 2. Fehler ist, hätte das Programm noch einen 3. Fehler.
sqrt(5) ist keine Integer und das printf("%d") unterschlägt die 
Nachkommastellen.

[Heiner (Gast)]

Daniel A. schrieb:
> Hier noch eine Version mit Pointer. Und die Funktion hab ich auch noch
> umbenannt.


Du bist .... einfach genial!
Nur entspricht Dein Beitrag nicht dem Original (siehe Eingangspost des 
TO),

if(2 != argc){
>     printf("Usage: %s number", *argv);
>     return 1;
>   }

 und ich glaube nicht, dass der TO etwas mit

 *x = *x**x;

anfangen kann (dafür fehlt ihm das Wissen, sonst hätte er nicht die 
Fragen gestellt!!).

[Heiner (Gast)]

Daniel A. schrieb:
> sqrt steht für Wurzel. Aber x*x ist das Gegenteil davon, das Quadrat.

Wo steht, dass die Funktion die WURZEL (!!) berechnen soll? Lt Fkt soll 
das Quadrat von x berechnet werden, und nur die Berechnungsformel ist 
maßgebend, nicht der Name der Funktion!!!

Grundgütiger!

[Daniel A. (daniel-a)]

Heiner schrieb:
> Nur entspricht Dein Beitrag nicht dem Original (siehe Eingangspost des
> TO),

Ja natürlich nicht. Das Original ist ja anscheinend falsch, und es gibt 
keine Anforderungen, was es tun sollte, wenn es richtig wäre. Eigentlich 
kann man da machen, zu was auch immer man gerade Lust hätte. Es gibt ja 
keine Anforderungen.

[Daniel A. (daniel-a)]

Heiner schrieb:
> Wo steht, dass die Funktion die WURZEL (!!) berechnen soll?

sqrt steht eigentlich immer für Square root, also Wurzel, egal ob Mathe 
oder Programmieren. Darum ist es definitiv Sinvoll, zumindest die 
Funktion umzubennenen. (Denn wie ich auch schon darauf (mehrfach) 
hinwies, gibt es ja keine Anforderungen, was das Programm eigentlich tun 
soll).

[Heiner (Gast)]

Daniel A. schrieb:
> Heiner schrieb:
>> Nur entspricht Dein Beitrag nicht dem Original (siehe Eingangspost des
>> TO),
>
> Ja natürlich nicht. Das Original ist ja anscheinend falsch, und es gibt
> keine Anforderungen, was es tun sollte, wenn es richtig wäre. Eigentlich
> kann man da machen, zu was auch immer man gerade Lust hätte. Es gibt ja
> keine Anforderungen.

Der war gut!
Klar! Man hätte auch ein Kreissegment berechnen können oder eine 
verkettete Liste oder ... - es gibt ja keine Anforderungen.

ohje

[Daniel A. (daniel-a)]

Ja, warum nicht?

[Heiner (Gast)]

Daniel A. schrieb:
> Ja, warum nicht?

Off Topic

Hoffentlich bist Du NICHT(!!) in einem technischen Beruf tätig!!

[Daniel A. (daniel-a)]

Heiner schrieb:
> Hoffentlich bist Du NICHT(!!) in einem technischen Beruf tätig!!

Doch. Ist momentan etwas Applikationsentwicklung (grösstenteils 
Firmeninterne), diverse Systeme verwalten, und noch ein paar andere 
Sachen.

[Percy N. (vox_bovi)]

Daniel A. schrieb:
> Ja natürlich nicht. Das Original ist ja anscheinend falsch, und es gibt
> keine Anforderungen, was es tun sollte, wenn es richtig wäre. Eigentlich
> kann man da machen, zu was auch immer man gerade Lust hätte. Es gibt ja
> keine Anforderungen.

Vermutlich doch, nur hat leider der TO davon abgesehen, Punkte a) und b) 
seiner Aufgabenstellung mitzuteilen, die insoweit vielleicht 
aufschlussreich hätten sein können.

[A. S. (Gast)]

Michael Langmaack schrieb:
> auch wenn ich z=5 eingegeben habe ...

Wo???

Schreib lieber Zeile für Zeile, was Du verstanden hast. Oder frag was Du 
nicht verstehst.

[Heiner (Gast)]

Daniel A. schrieb:
> Heiner schrieb:
>> Hoffentlich bist Du NICHT(!!) in einem technischen Beruf tätig!!
>
> Doch. Ist momentan etwas Applikationsentwicklung (grösstenteils
> Firmeninterne), diverse Systeme verwalten, und noch ein paar andere
> Sachen.

Off Topic

Gottseidank firmenintern!! Da kann wenigstens für die Allgemeinheit kein 
Schaden entstehen, wenn Du Anforderungen nach Gusto interpretierst, das 
Einzige, was drohen könnte wäre die Insolvenz Deines Arbeitgebers ...

[Daniel A. (daniel-a)]

Unsere internen Kunden sind oft sehr Froh, wenn ich mit denke. (Bei 
Unklarheiten frage ich natürlich auch nach). Es ist selten, dass an 
alles gedacht wird, und dann ist es immer am besten zu sehen, was die da 
effektiv zu machen versuchen.

Klar, ich könnte auch nicht mitdenken, und alles exakt nach Vorgabe 
machen, auch wenn ich weiss, dass dann üble Sachen passieren können & 
werden, und Zeit mit unnötigen manuellen Schritten verschwendet werden 
wird. Aber das macht niemanden Froh.

[Michael Langmaack (Gast)]

Vielen Dank für die doch vielen hilfreichen Lösungen und 
Lösungsvorschläge...

für die die mir vorwerfen das man hier seine Hausaufgaben nicht machen 
soll, es ist ein Klausuraufgabe einer Altklausur in der ich festgehangen 
habe.

für die die sagen das x = x * x keine Wurzel ist, das ist mir bewusst, 
für die Aufgabenstellung kann ich leider nichts :/

und an alle die sich hier einen Kopf gemacht haben und das mal eben so 
runter getackert haben, danke für die Mühe :))

[Michael Langmaack (Gast)]

die aufgabenteile a) und b) beinhalten, beschreibe das programm Zeile 
für Zeile und zeichne ein nassi-shneidermann-diagramm zur main funktion

[Heiner (Gast)]

Michael Langmaack schrieb:
> Vielen Dank für die doch vielen hilfreichen Lösungen und
> Lösungsvorschläge...
>
> für die die mir vorwerfen das man hier seine Hausaufgaben nicht machen
> soll, es ist ein Klausuraufgabe einer Altklausur in der ich festgehangen
> habe.
>
> für die die sagen das x = x * x keine Wurzel ist, das ist mir bewusst,
> für die Aufgabenstellung kann ich leider nichts :/
>
> und an alle die sich hier einen Kopf gemacht haben und das mal eben so
> runter getackert haben, danke für die Mühe :))

+1 😉👍

[A. S. (Gast)]

Michael Langmaack schrieb:
> die aufgabenteile a) und b) beinhalten, beschreibe das programm Zeile
> für Zeile und zeichne ein nassi-shneidermann-diagramm zur main funktion

Wenn Du das gemacht hast, weisst Du doch, wo genau Dein Problem ist. Wir 
wissen das nicht. Ich habe sogar zufällig genau danach gefragt.

A. S. schrieb:
> Schreib lieber Zeile für Zeile, was Du verstanden hast. Oder frag was Du
> nicht verstehst.

[MaWin (Gast)]

Da die Fehlerinformation von sscanf nicht abgefragt wird und z nicht 
initialisiert wird, entsteht bei Eingabe von Nicht-Zahlen Undefined 
Behaviour.

[rbx (Gast)]

Potenzieren ist wohl auch einfacher als Wurzel ziehen, was? ;)

Beim (selbst programmierten) Wurzelziehen könnte man eine 
Annährungsformel einsetzen - die funktioniert aber nicht gut mit 
Integern, sondern viel besser mit Fließkommazahlen.

Beim Multiplizieren muss man auch wissen, wie weit man das treiben kann, 
und sollte außerdem wissen, wenn man schnelle Register nutzen möchte, 
wie man das in Assembler hinbekommt.
In Assembler sieht man die Grenzen und die Möglichkeiten der Umschiffung 
eher als bei einer Hochsprache, bei der erstmal nicht klar ist, wie weit 
man multiplizieren kann.