Çözüldü Yazılan kod nasıl geliştirilebilir?

SubhanXd · 14 Mayıs 2024

Merhaba. Soru: Dört pozitif tam sayı A, B, C, D verilmiştir. Verilen kenar uzunluklarıyla bir kare oluşturulabiliyorsa herhangi bir sayının karesini bulun. Aksi takdirde "no" yazdırın.
Yazdığım kod:

C++:

#include <iostream>
using namespace std;

int main() {
 int a, b, c, d;
 cin >> a >> b >> c >> d;
 if (a>0 && b >0 && c>0 && d >0) {
 if (a == b && b == c && c == d)
 {
 cout << a *a << endl;
 }
 else
 {
 cout << "No" << endl;
 }
 }
}

Kod doğru ama site nedense 90 puan veriyor. Kacırdığım veya hatamın olduğu kısım neresi?

brkdnmz · 14 Mayıs 2024

SubhanXd dedi:

Kod:

#include <iostream>
using namespace std;

int main() {
   int a, b, c, d;
   cin >> a >> b >> c >> d;
   if (a>0 && b >0 && c>0 && d >0) {
   if (a == b && b == c && c == d)
   {
       cout <<(long long)(a *a) << endl;
   }
   else
   {
       cout << "No" << endl;
   }
   }
}

(long long)(a*a) değil de tam gösterdiğim gibi (long long)a*a yazmanız gerekiyor. Sizin yazdığınız halinde ilk overflow meydana gelip (çarpımın sonucu int sınırını aşıyorsa) sonra long long'a dönüştürme yapılıyor. Parantezsizken ilk a long long'a dönüşüyor, bu sayede işlem güvenli oluyor.

TheAny · 14 Mayıs 2024

SubhanXd dedi:
Sizin sorunu görmeniz için sizi grupa davet etmem gerekiyor.

Problemin linki yok mu? Private problem ilk defa duyuyorum. Kayıt olayım siteye madem. vjudge.net sanırım?

Kullanıcı adım; theany.

SubhanXd · 14 Mayıs 2024

TheAny dedi:
Problemin linki yok mu? Private problem ilk defa duyuyorum. Kayıt olayım siteye madem. vjudge.net sanırım?

Hayır hocam grub private.Saat bir geç olduğundan hocaya sormak istemedim açıkçası.

TheAny · 14 Mayıs 2024

Sorunun ekran görüntüsünü paylaşır mısın? Soru tam olarak ne istiyor onu göreyim, gereksinimlerini.

bitwise · 14 Mayıs 2024

Kenar uzunlugu hepsi sifirdan buyuk degilse No yazdirmiyorsun. ( ilk if blogu )
Bir de soruda "no" demissin kodda "No" yaziyorsun ama bu sorun olsaydi 90 dan dusuk alirdin

SubhanXd · 14 Mayıs 2024

bitwise dedi:
Kenar uzunlugu birbirlerine esit degilse No yazdirmiyorsun.
Bir de soruda "no" demissin kodda "No" yaziyorsun ama bu sorun olsaydi 90 dan dusuk alirdin

Çeviride hata olmuş.

TheAny dedi:
Sorunun ekran görüntüsünü paylaşır mısın? Soru tam olarak ne istiyor onu göreyim, gereksinimlerini.

Paylaşamıyorum.Sunucu hatası oluşuyor.

bitwise dedi:
Kenar uzunlugu hepsi sifirdan buyuk degilse No yazdirmiyorsun. ( ilk if blogu )
Bir de soruda "no" demissin kodda "No" yaziyorsun ama bu sorun olsaydi 90 dan dusuk alirdin

Aslında o ilk if bloguna gerek yok.Bundan onceki sorularda da pozitif olmasini istiyordu ama ben oyle bir sart yazmiyordum.

brkdnmz · 14 Mayıs 2024

SubhanXd dedi:

Kod:

#include <iostream>
using namespace std;

int main() {
   int a, b, c, d;
   cin >> a >> b >> c >> d;
   if (a>0 && b >0 && c>0 && d >0) {
   if (a == b && b == c && c == d)
   {
       cout <<(long long)(a *a) << endl;
   }
   else
   {
       cout << "No" << endl;
   }
   }
}

(long long)(a*a) değil de tam gösterdiğim gibi (long long)a*a yazmanız gerekiyor. Sizin yazdığınız halinde ilk overflow meydana gelip (çarpımın sonucu int sınırını aşıyorsa) sonra long long'a dönüştürme yapılıyor. Parantezsizken ilk a long long'a dönüşüyor, bu sayede işlem güvenli oluyor.

SubhanXd · 14 Mayıs 2024

brkdnmz dedi:
(long long)(a*a) değil de tam gösterdiğim gibi (long long)a*a yazmanız gerekiyor. Sizin yazdığınız halinde ilk overflow meydana gelip (çarpımın sonucu int sınırını aşıyorsa) sonra long long'a dönüştürme yapılıyor. Parantezsizken ilk a long long'a dönüşüyor, bu sayede işlem güvenli oluyor.

Sorun çözüldü.Çok teşekkürler.

Çözüldü Yazılan kod nasıl geliştirilebilir?

Ayrıntılı düzenleme

SubhanXd

Centipat

brkdnmz

TheAny

Gigapat

SubhanXd

Centipat

TheAny

Gigapat

bitwise

Gigapat

SubhanXd

Centipat

brkdnmz

Megapat

SubhanXd

Centipat

Benzer konular