C программирование. Программа FizzBuzz

Question

У меня была викторина, и я писал этот код:

печати Fizz, если оно делится на 3 и печатает Buzz, если она делится на 5. Он печатает FizzBuss если делится на обоих. В противном случае он напечатает цифры от 1 до 100.

Но после того как я приехал домой, я подумал, может ли записать его с меньшим количеством кода. Однако я не смог выйти с более коротким кодом. Могу ли я сделать это с более коротким кодом? Благодарю.

Это то, что я написал, и я думаю, что он работает хорошо. Но могу ли я сделать это с меньшим количеством кода.

#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; 
} 

Answer 1

Если число делится на оба 3 и 5, то это делится на 15, так:

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 

Кроме этого, вы, вероятно, не получите намного короче, по крайней мере, обычный язык, такой как C (и я предполагаю, что вы не хотите, чтобы обычные модификации стиля кода и гольфа делали ваш код нечитаемым).

Вы также можете получить все это в две строки, если вы упаковали всю функцию main на одну большую линию, но я надеюсь, что вы не будете после этого обмана.

Возможно, вы можете получить его быстрее (хотя вы должны проверить все требования производительности для себя) с чем-то вроде:

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]); 

Как и в сторону, что массив символьных указателей, вероятно, будет менее пространственно дорогой, чем вы думаете, благодаря постоянному объединению - другими словами, вероятно, что будет только один каждой строки C.

Как я уже сказал, нужно ли тестировать быстрее. Кроме того, в ваших оригинальных спецификациях вызывается только кратчайший код, поэтому он может быть неактуальным.

Answer 2

Вы также можете сделать:

#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; 
} 

Несколько строк короче и намного легче читать.

Answer 3

Я бы сказал, что modulo стоит дорого, в то время как сравнение дешево, поэтому выполняйте только один модуль. Это даст что-то подобное.

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"); 
} 

Answer 4

Я не уверен, когда вы начнете называть его нечитаемым, но это так.

#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; 
} 

Answer 5

Запутанный форма ответа г Листера

main(int i){while(i++<100){printf("number= %d %s%s",i,i%3?"":"Fizz",i%5?"":"Buzz");}}

Answer 6

я бы написать что-то вроде этого

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 
    } 

Answer 7

#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; 
} 

Answer 8

Это один избегает некоторые повторения кода, но требует временного переменная 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"); 
    } 
} 

Answer 9

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: ""); 
    } 
} 

Answer 10

Вы можете сделать это с помощью строки:

String s=""; 
if(num%3==0) 
    s+="fizz"; 
if(num%5==0) 
    s+="buzz"; 
if(s.length()==0) 
    s+=num+""; 

Answer 11

Я бы с вспомогательной функцией :-)

#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);     
}