C de la programmation. Le programme FizzBuzz

J'ai eu un quiz et j'ai écrit ce code:

Impression Fizz si elle est divisible par 3 et il imprime Buzz si c'est
divisible par 5. Il imprime FizzBuss si elle est
divisible par deux. Sinon, il affichera les nombres entre 1 et 100.

Mais après je suis arrivé à la maison, je me demandais si ont pouvait
écrit avec moins de code. Cependant, je ne pouvais pas sortir
avec un code plus court.
Puis-je le faire avec un code plus court? Merci.

C'est ce que j'ai écrit et je pense qu'il fonctionne bien. Mais puis-je l'avoir fait
avec moins de code.

#include <stdio.h>

int main(void)
{
    int i;
    for(i=1; i<=100; i++)
    {
        if(((i%3)||(i%5))== 0)
            printf("number= %d FizzBuzz\n", i);
        else if((i%3)==0)
            printf("number= %d Fizz\n", i);
        else if((i%5)==0)
            printf("number= %d Buzz\n", i);
        else
            printf("number= %d\n",i);

    }

    return 0;
}

Est-ce devoirs? (BTW, avez-vous vraiment l'intention d'optimiser la première instruction if que beaucoup? 🙁
pile de change a un codegolf site je crois
Cette question tend à vaincre ceux qui essaient d'être intelligent. Vous pouvez faire plus court, mais il est alors facile de se tromper. Ce que vous avez fait est le moyen le plus simple pour obtenir ce droit.
Non, ce n'est pas un problème du tout. Je n'étais tout simplement essayer de savoir si je pourrait avoir fait dans le plus court chemin. Et j'ai appris quelque chose ici de l'utilisateur paxdiable, où si vous avez besoin de tester les deux conditions, parfois, vous pourriez réduire à ces deux conditions pour être juste et, par conséquent, votre code sera plus courte. Quand j'ai écrit mon code, je ne pense pas de cette façon. Maintenant je vais utiliser sa technique à partir de maintenant.
if(((i%3)||(i%5))== 0) N'essayez pas d'être trop intelligent, c'est juste obscurcit ce que votre code est destiné à faire. Aussi, il doit imprimer Fizz/Buzz au lieu de le nombre, alors méfiez-vous que certaines des solutions fournis dans les réponses sont fausses.
Je vais voter pour fermer cette question hors-sujet, parce que l'examen de code fonctionnel est hors sujet pour Stack Overflow. Votre question peut être sur le sujet de la Revue de Code.
Totalement bizarre que c'était fermé à mon humble avis. Comment est-il pas peut-être pas une question de programmation?

InformationsquelleAutor leocod | 2012-02-27

c fizzbuzz

13

Vous pouvez aussi faire:
```
#include <stdio.h>

int main(void)
{
    int i;
    for(i=1; i<=100; ++i)
    {
        if (i % 3 == 0)
            printf("Fizz");
        if (i % 5 == 0)
            printf("Buzz");
        if ((i % 3 != 0) && (i % 5 != 0))
            printf("number=%d", i);
        printf("\n");
    }

    return 0;
}
```
Quelques lignes plus courtes et beaucoup plus facile à lire.
- Merci. Il semble plus courte.
- Si vous avez toujours wan pas le numéro que vous pouvez également supprimer une ligne en faisant printf("nombre=%d ", i) en haut et en supprimant le dernier si.
- if ((i % 3 != 0) && (i % 5 != 0)) pouvez en fait juste être écrit comme if (i % 3 && i % 5). i % 3 et i % 5 dans ce cas sera de retour non nulle numéros, qui évalue à true.
- Parfois, il n'est pas facile pour les gens à garder les choses simples. Vous l'avez fait. +1.
InformationsquelleAutor martiert
8

Si un nombre est divisible par 3 et 5, alors il est divisible par 15, donc:
```
for each number 1 to 100:
    if number % 15 == 0:
        print number, "fizzbuzz"
    else if number % 5 == 0:
        print number, "buzz"
    else if number % 3 == 0:
        print number, "fizz"
    else:
        print number
```
Autre que cela, vous n'aurez probablement pas obtenir beaucoup plus courte, au moins dans un classique des langages comme le C (et je suis en supposant que vous ne voulez pas que la normale code-golf modifications de style qui font de votre code illisible).

Vous pouvez aussi obtenir le tout en deux lignes, si vous avez emballé l'ensemble de la main fonction sur une seule grande ligne, mais j'espère que vous ne seriez pas après ce genre de supercherie soit.

Vous pouvez éventuellement obtenir plus rapide (bien que vous devriez vérifier toutes les allégations de performance pour vous-même) avec quelque chose comme:
```
static const char *xyzzy[] = {
    "",     "",     "fizz", "",     "buzz",
    "fizz", "",     "",     "fizz", "buzz",
    "",     "fizz", "",     "buzz", "fizzbuzz",
    //Duplicate those last three lines to have seven copies (7x15=105).
};
for (int i = 1; i <= 100; i++)
    printf ("%d %s\n", i, xyzzy[i-1]);
```
En aparté, ce tableau de pointeurs de char sera probablement moins d'espace-cher que vous le pensez, grâce à l'action permanente de la fusion - en d'autres termes, il sera probable qu'il y aura seulement un de chaque C de la chaîne.

Comme je l'ai dit, si c'est plus rapide devrait être testé. En outre, l'original de votre specs que le plus court code de sorte qu'il peut être sans importance.
- Merci. C'est vrai. Pour le premier test que je pouvais penser en terme de seulement un numéro. Je vais le prendre en compte avec des exercices futurs. Merci.
InformationsquelleAutor paxdiablo
7

Je ne suis pas sûr de vous lorsque vous vous feriez commencer à appeler illisible, mais il y a ce.
```
#include <stdio.h>

int main(void)
{
   int i = 1;
   for (; i<=100; ++i) {
      printf("number= %d %s%s\n", i, i%3?"":"Fizz", i%5?"":"Buzz");
   }
   return 0;
}
```
- Il fonctionne bien, et c'est plus court. Merci.
- Je suis sûr que paxdiablo la solution est plus rapide si, et même pas beaucoup plus de temps.
- Eh bien, en toute honnêteté, la question ne demandez plus court plutôt que plus rapide 🙂 Si vous avez déclaré i dans le for déclaration et abandonné les accolades, il serait encore plus courte, sans pour autant sacrifier la lisibilité. +1.
- J'ai pu passer avec les accolades, mais je ne suis pas sûr de la je, depuis l'OP ne mentionne pas si c'était C99 ou pas. Il n'y a pas d'astuces dans le code.
- Il ne devrait pas imprimer le nombre lors de l'impression Fizz ou Buzz.
- Pas de la solution réelle. Vous ne pouvez pas imprimer le numéro devant chaque ligne.
- Je sais, mais je moulée ma réponse sur l'OP du code plutôt que sur la description de ce que le programme doit faire. La question était de savoir comment rendre son code plus court, pas sur la façon de mettre en œuvre FizzBuzz correctement!
InformationsquelleAutor Mr Lister

Je dirais que modulo est cher alors que les comparaisons ne sont pas cher, donc uniquement effectuer le modulo une fois. Que donnerait quelque chose comme ça.

int i;
for( i = 0; i!=100; ++i ) {
    bool bModThree = !(i % 3);
    bool bModFive = !(i % 5);

    if( bModThree || bModFive ) {
        if( bModThree ) {
            printf( "Fizz" );
        }
        if( bModFive ) {
            printf( "Buzz" );
        }
    } else {
        printf( "%d", i );
    }

    printf( "\n" );
}

Vous n'êtes pas censé toujours imprimer le nombre.
Oh putain. Vous avez raison. J'ai mis à jour la réponse avec un meilleur exemple.

InformationsquelleAutor Kenneth

1

je voudrais écrire quelque chose comme ça
```
    main(){
  if (i % 3 == 0){
  cout<<"Fizz";
  }
  if (i % 5 == 0){
  cout<<"Buzz";
  }
  //So if both are true, it will print “FizzBuzz” and augment the two strings
    }
```
- Vous n'êtes pas l'impression de le nombre.
- Cette question a été sur la programmation en C, pas en C++ programmation, de sorte que cette solution ne fonctionnera pas.
InformationsquelleAutor user889030

Cela on évite la répétition de code, mais nécessite une variable temporaire char t

void FizzBuzz( ) {
    char t = 0;
    for (unsigned char i = 1; i <= 100; ++i, t = 2) {
        (i % 3) ? --t : printf("Fizz");
        (i % 5) ? --t : printf("Buzz");
        if (!t) printf("%d", i);
        printf("\n");   
    }
}

InformationsquelleAutor Michele

#include <stdio.h>

char const * template[] = {
  "%i",
  "Buzz",
  "Fizz",
  "FizzBuzz"
};
const int __donotuseme3[] = { 2, 0, 0 };
const int __donotuseme5[] = { 1, 0, 0, 0, 0 };
#define TEMPLATE(x) (template[__donotuseme3[(x) % 3] | __donotuseme5[(x) % 5]])

int
main(void) {
  int i;
  for (i = 1; i <= 100; i++) {
    printf(TEMPLATE(i), i);
    putchar('\n');
  }
  return 0;
}

InformationsquelleAutor Domenico Shura

void main()
{
    int i = 0;
    char h[4];

    while (++i <= 100)
    {
        sprintf(h, "%d", i);
        printf("%s%s%s\n", i%3 ? "" : "fizz", i%5 ? "" : "buzz", (i%3 && i%5) ? h: "");
    }
}

Alors que ce code peut répondre à la question, en fournissant plus de contexte sur comment et/ou pourquoi il résout le problème serait d'améliorer la réponse à long terme de valeur.

InformationsquelleAutor user7652645

0

Vous pouvez le faire en utilisant une Chaîne de caractères:
```
String s="";
if(num%3==0)
   s+="fizz";
if(num%5==0)
   s+="buzz";
if(s.length()==0)
   s+=num+"";
```
- Merci d'éditer ton post afin que votre texte n'est pas formaté comme code.
- Dans la plaine, C il n'existe pas de type "String". Il y a std::string (minuscule!) en C++, mais cette question est balisé C.
- Je veux juste me concentrer sur l'algo . J'espère que vous pouvez obtenir !
- Il réduit la 4 si d'autre condition à tout juste 3 . le temps pris pour 100 numéros dans la réponse qui est donnée est de 12 milli sec et le mien est de 6 milli secondes . C'est la moitié du temps pris
InformationsquelleAutor Arpit Gupta

J'irais avec une fonction d'aide 🙂

#include <stdio.h>

int fbindex(int n) {
    int i = 0;
    if (n % 3 == 0) i += 1;
    if (n % 5 == 0) i += 2;
    return i;
}

int main(void) {                                 
    const char *fb[] = {"%d\n", "Fizz\n", "Buzz\n", "FizzBuzz\n"};
    for (int i = 1; i <= 100; i++) printf(fb[fbindex(i)], i);                
}

InformationsquelleAutor pmg

-3

Obscurci forme de M. Lister, en réponse

main(int i){while(i++<100){printf("number= %d %s%s",i,i%3?"":"Fizz",i%5?"":"Buzz");}}
- Ce n'est pas obscurci (enlever les sauts de ligne n'est guère de dissimulation), c'est mal.
InformationsquelleAutor N20084753

Vous devez vous connecter pour publier un commentaire.