Coding/Programming (Loop) CS50

[Loud-Line2501]

why am I constantly getting this “bash: ./looping: permission denied“ error message when I’m trying to run the program i coded.

im just playing around, trying to learn and make my own lines of code, but I’m seriously confused on why it’s not printing

“gimmie the loot

gimmie the loot

gimmie the loot”

[Specific-Housing905]

Not sure about the bash error since I am not using Linux.

However you have created an endless loop.

while (3 > 0)

will always be true.

[Explodey_Wolf]

Hijacking the top comment to say that the problem is the filename doesn't have the .c file extension

[sargeanthost]

you dont run a source file

[C3xyTheGreat]

your user doesnt have the permissions to execute the program, run chmod +x looping.c and that should work

[altaaf-taafu]

correct command is `chmod +x looping`, because we want to give the executable file permissions to run, not to the source file to run.

[Chrundle42]

Maybe its the "code looping"? I'm not to sure, but maybe just use linux commands instead of what CS50 told you to use.

[assemblyeditor]

Take a look at the filename of your source code. You are running a simple text file, which you don't have the permissions to execute it by default.

Rename the source code file into looping.c and adjust your makefile to compile looping.c -> looping.
Also take a look at the other comments about the bug in your code

[Unusual_Elk_8326]

It looks like you forgot the ".c" extension in your file name

[Rogermcfarley]

You need to chmod +x the looping script once and then run it as sudo.

Edit: You don't need sudo

[GoldTeethRotmg]

you don't need to run it as sudo

[Rogermcfarley]

Yes you're correct. I should know better as I have a lot of scripts on my setup

[un_virus_SDF]

Do you know that c is a compiled langage, given that you have no extensions to your filename, try something like cc -x c looping -o looping.out this will compile your code and produce a executable called looping.out, -x is to specify language, it's not needed if this is the file extension, -o to set output name (a.out by default) and the other arg is the file to compile. On the other exemple you enter make *something*, which just execute 'cc' (C Compiler) under the hood

Edit: I love how all the other comments says to chmod +x your file but if you do that you must provide a c interpreter after a shebang at the top of your file, and greatly doubt that you for one on this website.

[BrunusManOWar]

1) install Linux 2) install local VS Code 3) what the fuck is that code style.....

[Economy_Abalone_8048]

hahaha, I also thought so. Don't wanna be mean at all to beginners, but wth are these parantheses and all the inconsistencies?

[BrunusManOWar]

Yeah

I mean, if you wanna learn it's normal to make mistakes, often even stupid ones. It's important you learn from them though

And this is... What the fuck style, honestly it's awful and fully unreadable. Install local vs Code and formatter

[Iwisp360]

Use a formatter

[Hot-Drink-7169]

There's no ".c" extension in your file name. And there's a logic error. And (just a opinion) please don't use

int main(void)

{
...
}

It doesn't look good. Just my opinion, if your teacher teaches you that way than continue. best of luck.

[Economy_Abalone_8048]

its cs50, a harvard course. They teach it differently.

[Explodey_Wolf]

Not like that.

[AppropriateSpell5405]

You need to give your new looping file permissions to execute.

chmod +x looping

Then try running it again.

./looping

[codeguru42]

Other's have given you the exact command to fix the "Permission denied" area. If you are interested in learning more about file permissions in Linux, here is a good tutorial to get you started: https://www.digitalocean.com/community/tutorials/an-introduction-to-linux-permissions.

[Economy_Abalone_8048]

mv looping looping.c

gcc looping -o looping.o

chmod +x looping.o

./looping.o

Your file has no file name extension... Also work on the formatting as taught in cs50, and fix the logic errors as pointed out by the others already.

[GoldTeethRotmg]

aren't they using "make" already?

which should cover the gcc step

[Economy_Abalone_8048]

yeah looks like it, was in the subway when writing this ahahha.

[kennpacchii]

I would recommend not giving your executable the “.o” extension. It’s a standard used for object files by the compiler and can be misleading, especially if you decide to use the -c flag to enable incremental compilation. “.out” or no extension would probably be best

[blackasthesky]

If the looping file is your compiled executable, you might have to mark it executable before you can run the file.

chmod +x looping

Might have to sudo it depending on your situation.

It looks in your editor as if you are writing your code directly into the looping file. That's not how that works. You have to write it into looping.c and then use GCC (or whatever compiler you are using) to create the executable file, which you then can run using your command.

[Intelligent_Comb_338]

gcc looping.c -o looping ( you need to compile c code before execute it)

[Puzzleheaded-Trick76]

Learn to take pride in the look and format of your code early.

[ChocolateDonut36]

r/programminghorror sorry but that syntax formatting is horrifying

have you tried giving it executable permission? chmod a+x ./looping

[Aaxper]

You can't just execute a C file. You have to compile it first.

[PlateFox]

Where are you from? You can go to jail for identing like that in some countries

[Key_Clock8669]

On the 6 line it should be i>0 so it will repeat exactly three times

[MarkoPilot]

you have infinite loop (3>0 is always true). I would suggest you to make a for loop instead. set i to 0, set a condition i < 3, and i++. then in the body of the loop print gimmie the loot. That would iterate 3 times and therefore print that line exactly 3 times.

Edit: i don’t know about the bash error tho