mangler en floor funktion

er tallet en tekstreng??

#include <math.h>

double floor(double x);

Og hvis du undrer dig over at de returnerer double, så er det fordi
at hvis du skal have en int så assigner du bare (eller eksplicit
type caster).

int i = x;

int i = (int)x;

arne_v -> kan du ikke prøve at sætte det ind i det her:

#include <stdio.h>

int main() {
  int tid = 321324;

  int minutter = floor(tid/60);
return 1;
}

Hvorfor overhovedet bruge decimal tal eller floor ?

#include <stdio.h>

int main() {
  int tid = 321324;
  int minutter = tid/60;
  printf("%d\n",minutter);
  return 0;
}

virker det ikke ?

321324/60 = 5355,4 ? det er da et decimal tal

321324/60 = 5355 i C, C++, Java, Fortran
321324/60 = 5355.4 i Pascal (og måske i JavaScript men det kender jeg ikke)

Hvis du i C har en int / en int, så får du en int (som er det
"ægte" resultat nedrundet).

nu har jeg så prøvet mig lidt frem og kommet til følgende..

det giver ingen fejl når jeg compiler, men når jeg kører det, giver det en Bus Error..

#include <stdio.h>

char *str_uptime(int a) {
  char *output;
  int tid = a/1000;
  int dage = tid/(24*60*60);
  int timer = (tid%(40*60*60))/(60*60);
  int minutter = (tid%(60*60))/60;
  int sekunder = tid%60;
  sprintf(output, "%d dage, %d timer, %d minutter og %d sekunder\n",dage, timer, minutter, sekunder);

  return output;
}

int main() {
  char tiks[5000];

  tiks = str_uptime(20908374);

  printf("%s\n", tiks);
  return 0;
}

Ja.

Du bruger en uinitialiseret pointer !

Jeg prøver lige at lave et virkende eksempel.

#include <stdio.h>
#include <string.h>

char output[100];

char *str_uptime(int a) {
 
  int tid = a/1000;
  int dage = tid/(24*60*60);
  int timer = (tid%(40*60*60))/(60*60);
  int minutter = (tid%(60*60))/60;
  int sekunder = tid%60;
  sprintf(output, "%d dage, %d timer, %d minutter og %d sekunder\n",dage, timer, minutter, sekunder);
  return output;
}

int main() {
  char tiks[100];
  strcpy(tiks,str_uptime(20908374));
  printf("%s\n", tiks);
  return 0;
}

giver:

0 dage, 5 timer, 48 minutter og 28 sekunder

2 ændringer:

* output er nu et array i.s.f. en pointer

* jeg bruger strcpy til assignment nede i main

Arne må jeg spørge dig om dit design choise her?
hvorfor er str_uptime lavet non reentrant? (bruger global)
hvorfor har du valg at lave den ekstra copy i main istedet for at overføre en pointer til target? (str_uptime(int uptime, char *dest))

Ikke mit design choice.

Det var sådan den oprindelige funktion så ud. Jeg lavede
en funktion der virkede med præcis samme signatur.

Man kan udmærker argumentere for bedre måder at lave det på
(der er dog en del af C's standard funktioner der har
de samme problemer).

#include <stdio.h>
#include <string.h>

char *str_uptime(int a, char *output) {
  int tid = a/1000;
  int dage = tid/(24*60*60);
  int timer = (tid%(24*60*60))/(60*60);  // 40 her skulle vist være 24
  int minutter = (tid%(60*60))/60;
  int sekunder = tid%60;
  sprintf(output, "%d dage, %d timer, %d minutter og %d sekunder\n",dage, timer, minutter, sekunder);
  return output;
}

int main() {
  char tiks[100];
  strcpy(tiks,str_uptime(20908374, tiks));
  printf("%s\n", tiks);
  return 0;
}