Level 4
You can't use the fixed keywords as your class names and all..try some other names rather than naming them same as library defined names.
No, I don't think it have to be plural, make it as simple as possible and singular.
2 likes
Feb 3, 2018
7
Level 6
class name class?
i use phpstorm as an IDE. i have a model named class that contains information about classes as in school classes. so i would think to use class as the name of the class. the IDE reports it as a possible error. before moving forward would it be an error?
// Class model
<?php
namespace App\Models;
use Illuminate\Database\Eloquent\Model;
class Class extends Model
{
protected $table = 'classes';
/**
* Class belongs to program type.
* @return mixed
*/
public function programType()
{
return $this->belongsTo(ProgramType::class);
}
}
// Program Types model
<?php
namespace App\Models;
use Illuminate\Database\Eloquent\Model;
class ProgramType extends Model
{
protected $table = 'program_types';
public $timestamps = false;
/**
* Program type has many classes.
* @return \Illuminate\Database\Eloquent\Relations\HasMany
*/
public function classes()
{
return $this->hasMany(Class::class);
}
}
i think i ran into something similar with a model called Event but it worked out alright.
if this doesn't work, should i use a different name - what name? or use the plural?
Level 88
Well some words are reserved in some languages. For example since PHP7.0 you can can't have a class that's called String anymore. As far as I know Class is not one of them!
Reserved words: http://php.net/manual/en/reserved.other-reserved-words.php
Level 88
Here is the full list of reserved things: http://php.net/manual/en/reserved.php
Level 6
@Shahrukh4 @bobbybouwmann thank you.
i'll keep it as is. i think the IDE is set to be sensitive by default. right now i'm just building models and seeders. i'm moving data from mssql to mysql as an experiment project.
i won't know if it becomes an issue until i write tests. if it does fail, i'll just refactor that model. luckily the relationship only exists in 2 other models. i just want to get the data over and play with it in laravel - cleaning things up in the process.
Level 88
Goodluck! Let me know if you have any other questions!
Level 4
Well may be it will not occur any issue sometimes but it is a good practice not to use the reserved keywords as your class names and methods rather make your own.
1 like
Level 67
I'd call it "Course". Using "Class" will come back to bite you and might as well avoid that from the get-go so you don't have to refactor it all later.
1 like
Please or to participate in this conversation.